From 4ecd3d6d53b5f09692f693d000d58cd8270cb7a3 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jul 11 2018 14:32:08 +0000 Subject: Fix the RELATES regex as we fixed the FIXES ones Signed-off-by: Pierre-Yves Chibon --- diff --git a/pagure/lib/link.py b/pagure/lib/link.py index 04ca77b..a983f4e 100644 --- a/pagure/lib/link.py +++ b/pagure/lib/link.py @@ -37,7 +37,7 @@ RELATES = [ re.compile(r'(?:.*\s+)?relate[sd]?:?\s?#(\d+)', re.I), re.compile( r'(?:.*\s+)?relate[sd]?:?\s*?(?:to)?\s*?' - 'https?://.*/(\w+)/issue/(\d+)', re.I), + 'https?://.*/([a-zA-z0-9_][a-zA-Z0-9-_]*)/issue/(\d+)', re.I), ] diff --git a/tests/test_pagure_lib_link.py b/tests/test_pagure_lib_link.py index b6ec4fa..bb51d8e 100644 --- a/tests/test_pagure_lib_link.py +++ b/tests/test_pagure_lib_link.py @@ -223,6 +223,13 @@ class PagureLibLinktests(tests.Modeltests): else: self.assertEqual(regex.match(text), None) + text = 'relates https://localhost/SSSD/ding-libs/issue/31' + for index, regex in enumerate(pagure.lib.link.RELATES): + if index == 2: + self.assertNotEqual(regex.match(text), None) + else: + self.assertEqual(regex.match(text), None) + def test_fixes_regex(self): ''' Test the fixes regex present in pagure.lib.link. '''