rules: skip line-length rule for URLs and quoted lines (nodejs#30)

Fixes: nodejs#24

Author: addaleax

lib/rules/line-length.js

@@ -31,6 +31,12 @@ module.exports = {
     var failed = false
     for (let i = 0; i < parsed.body.length; i++) {
       const line = parsed.body[i]
+      // Skip quoted lines, e.g. for original commit messages of V8 backports.
+      if (line.startsWith('    '))
+        continue
+      // Skip lines with URLs.
+      if (/https?:\/\//.test(line))
+        continue
       if (line.length > len) {
         failed = true
         context.report({

test/rules/line-length.js

@@ -67,5 +67,69 @@ ${'aaa'.repeat(30)}`
     tt.end()
   })
 
+  t.test('quoted lines', (tt) => {
+    const v = new Validator()
+    const context = new Commit({
+      sha: 'e7c077c610afa371430180fbd447bfef60ebc5ea'
+    , author: {
+        name: 'Evan Lucas'
+      , email: '[email protected]'
+      , date: '2016-04-12T19:42:23Z'
+      }
+    , message: `src: make foo mor foo-ey
+
+Here’s the original code:
+
+    ${'aaa'.repeat(30)}
+
+That was the original code.
+`
+    }, v)
+
+    context.report = (opts) => {
+      tt.pass('called report')
+      tt.equal(opts.id, 'line-length', 'id')
+      tt.equal(opts.string, '', 'string')
+      tt.equal(opts.level, 'pass', 'level')
+    }
+
+    Rule.validate(context, {
+      options: {
+        length: 72
+      }
+    })
+    tt.end()
+  })
+
+  t.test('URLs', (tt) => {
+    const v = new Validator()
+    const context = new Commit({
+      sha: 'e7c077c610afa371430180fbd447bfef60ebc5ea'
+    , author: {
+        name: 'Evan Lucas'
+      , email: '[email protected]'
+      , date: '2016-04-12T19:42:23Z'
+      }
+    , message: `src: make foo mor foo-ey
+
+https://${'very-'.repeat(80)}-long-url.org/
+`
+    }, v)
+
+    context.report = (opts) => {
+      tt.pass('called report')
+      tt.equal(opts.id, 'line-length', 'id')
+      tt.equal(opts.string, '', 'string')
+      tt.equal(opts.level, 'pass', 'level')
+    }
+
+    Rule.validate(context, {
+      options: {
+        length: 72
+      }
+    })
+    tt.end()
+  })
+
   t.end()
 })

test/validator.js

@@ -205,11 +205,11 @@ test('Validator - real commits', (t) => {
       const filtered = msgs.filter((item) => {
         return item.level === 'fail'
       })
-      tt.equal(filtered.length, 3, 'messages.length')
+      tt.equal(filtered.length, 2, 'messages.length')
       const ids = filtered.map((item) => {
         return item.id
       })
-      const exp = ['line-length', 'line-length', 'title-length']
+      const exp = ['line-length', 'title-length']
       tt.deepEqual(ids.sort(), exp.sort(), 'message ids')
       tt.end()
     })