lib: add test for lowercase first word in title

Author: Trott

lib/rules/title-format.js

@@ -9,6 +9,7 @@ module.exports = {
   , recommended: true
   }
 , validate: (context, rule) => {
+    let pass = true
     if (/[\.\?\!]$/.test(context.title)) {
       context.report({
         id: id
@@ -18,14 +19,29 @@ module.exports = {
       , column: context.title.length
       , level: 'fail'
       })
-      return
+      pass = false
     }
 
-    context.report({
-      id: id
-    , message: 'Title is formatted correctly.'
-    , string: ''
-    , level: 'pass'
-    })
+    const result = /^(.+?): [A-Z]/.exec(context.title)
+    if (result) {
+      context.report({
+        id: id
+      , message: 'First word after subsystem(s) in title should be lowercase.'
+      , string: context.title
+      , line: 0
+      , column: result[1].length + 3
+      , level: 'fail'
+      })
+      pass = false
+    }
+
+    if (pass) {
+      context.report({
+        id: id
+      , message: 'Title is formatted correctly.'
+      , string: ''
+      , level: 'pass'
+      })
+    }
   }
 }

test/validator.js

@@ -113,6 +113,20 @@ Date:   Thu Mar 3 10:10:46 2016 -0600
     test: check memoryUsage properties.
 `
 
+const str8 = `commit 7d3a7ea0d7df9b6f11df723dec370f49f4f87e99
+Author: Wyatt Preul <[email protected]>
+Date:   Thu Mar 3 10:10:46 2016 -0600
+
+    test: Check memoryUsage properties
+`
+
+const str9 = `commit 7d3a7ea0d7df9b6f11df723dec370f49f4f87e99
+Author: Wyatt Preul <[email protected]>
+Date:   Thu Mar 3 10:10:46 2016 -0600
+
+    test: Check memoryUsage properties.
+`
+
 test('Validator - misc', (t) => {
   const v = new Validator()
 
@@ -278,7 +292,7 @@ test('Validator - real commits', (t) => {
     })
   })
 
-  t.test('trailing punctuation in first line', (tt) => {
+  t.test('trailing punctuation in title line', (tt) => {
     const v = new Validator({
       'validate-metadata': false
     })
@@ -296,5 +310,39 @@ test('Validator - real commits', (t) => {
     })
   })
 
+  t.test('first word is lowercase in title line', (tt) => {
+    const v = new Validator({
+      'validate-metadata': false
+    })
+    v.lint(str8)
+    v.on('commit', (data) => {
+      const msgs = data.messages
+      const filtered = msgs.filter((item) => {
+        return item.level === 'fail'
+      })
+      tt.equal(filtered.length, 1, 'messages.length')
+      tt.equal(filtered[0].message,
+               'First word after subsystem(s) in title should be lowercase.',
+               'message')
+      tt.equal(filtered[0].column, 7, 'column')
+      tt.end()
+    })
+  })
+
+  t.test('more than one formatting error in title line', (tt) => {
+    const v = new Validator({
+      'validate-metadata': false
+    })
+    v.lint(str9)
+    v.on('commit', (data) => {
+      const msgs = data.messages
+      const filtered = msgs.filter((item) => {
+        return item.level === 'fail'
+      })
+      tt.equal(filtered.length, 2, 'messages.length')
+      tt.end()
+    })
+  })
+
   t.end()
 })