InvalidLinkBear: Multiple links

The InvalidLinkBear should analyze all the links
in a line instead of just analyzing the first one.

Fixes coala#1296

bears/general/InvalidLinkBear.py

@@ -81,9 +81,8 @@ def find_links_in_file(file, network_timeout, link_ignore_regex,
             (?<!\.)(?<!,)               # Exclude trailing `.` or `,` from URL
             """, re.VERBOSE)
         for line_number, line in enumerate(file):
-            match = regex.search(line)
-            if match:
-                link = match.group()
+            for match in re.findall(regex, line):
+                link = match[0]
                 if not (link_ignore_regex.search(link) or
                         fnmatch(link, link_ignore_list)):
                     if link.startswith(('hg+', 'bzr+', 'git+', 'svn+')):

tests/general/InvalidLinkBearTest.py

@@ -77,6 +77,15 @@ def assertResult(self, valid_file=None, invalid_file=None, settings={}):
                 self.assertNotEqual(out, [])
                 self.assertNotEqual(out, None)
 
+    def assertResultCount(self, test_file, expected_num):
+        with requests_mock.Mocker() as m:
+            InvalidLinkBear.check_prerequisites = lambda *args: True
+            uut = InvalidLinkBear(self.section, Queue())
+            m.add_matcher(custom_matcher)
+            for line, num in zip(test_file, expected_num):
+                outputs = list(uut.run('testline', [line]))
+                self.assertEqual(num, len(outputs))
+
     def test_run(self):
         # Valid Links
         valid_file = """
@@ -155,6 +164,20 @@ def test_redirect_threshold(self):
                           invalid_file=short_url_redirect,
                           settings={'follow_redirects': 'yeah'})
 
+    def test_multiple_results_per_line(self):
+
+        test_file = """
+        http://httpbin.org/status/410
+        http://httpbin.org/status/200
+        http://httpbin.org/status/404 http://httpbin.org/status/410
+        http://httpbin.org/status/200 http://httpbin.org/status/404
+        http://httpbin.org/status/200 http://httpbin.org/status/201
+        """.splitlines()
+
+        expected_num_results = [0, 1, 0, 2, 1, 0]
+
+        self.assertResultCount(test_file, expected_num_results)
+
     def test_pip_vcs_url(self):
         with_at = """
         git+http://httpbin.org/status/200@master