Scalastyle: capture messages with no line number

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## master (unreleased)
 
+* Fix `Scalastyle` pre-commit hook to capture messages with no line number
 * Fix `CoffeeLint` pre-commit hook detection of modified lines
 
 ## 0.33.0

lib/overcommit/hook/pre_commit/scalastyle.rb

@@ -6,8 +6,8 @@ class Scalastyle < Base
     MESSAGE_REGEX = /
       ^(?<type>error|warning)\s
       file=(?<file>(?:\w:)?.+)\s
-      message=.+\s
-      line=(?<line>\d+)
+      message=.+\s*
+      (line=(?<line>\d+))?
     /x
 
     def run

spec/overcommit/hook/pre_commit/scalastyle_spec.rb

@@ -41,7 +41,21 @@
         OUT
       end
 
-      it { should warn }
+      it { should warn(/Use : Unit = for procedures/) }
+    end
+
+    context 'and it reports a warning with no line' do
+      before do
+        result.stub(stderr: '', stdout: normalize_indent(<<-OUT))
+          warning file=file1.scala message=File must end with newline character
+          Processed 1 file(s)
+          Found 0 errors
+          Found 1 warnings
+          Finished in 490 ms
+        OUT
+      end
+
+      it { should warn(/File must end with newline character/) }
     end
   end
 
@@ -64,7 +78,21 @@
         OUT
       end
 
-      it { should fail_hook }
+      it { should fail_hook(/Use : Unit = for procedures/) }
+    end
+
+    context 'and it reports an error with no line' do
+      before do
+        result.stub(stderr: '', stdout: normalize_indent(<<-OUT))
+          error file=file1.scala message=File must end with newline character
+          Processed 1 file(s)
+          Found 1 errors
+          Found 0 warnings
+          Finished in 490 ms
+        OUT
+      end
+
+      it { should fail_hook(/File must end with newline character/) }
     end
 
     context 'with a usage message' do
@@ -83,7 +111,7 @@
         OUT
       end
 
-      it { should fail_hook }
+      it { should fail_hook(/Usage/) }
     end
 
     context 'with a runtime error' do
@@ -104,7 +132,7 @@
         ERR
       end
 
-      it { should fail_hook }
+      it { should fail_hook(/Exception/) }
     end
   end
 end