From 3313b4f9c3c5d6a579588e77068ca3ae3edffe2b Mon Sep 17 00:00:00 2001
From: Daniel Baumann <daniel.baumann@progress-linux.org>
Date: Mon, 25 Jan 2021 14:26:08 +0100
Subject: Adding upstream version 0.15.0.

Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
---
 gitlint/tests/config/test_config_builder.py | 80 ++++++++++++++---------------
 1 file changed, 40 insertions(+), 40 deletions(-)

(limited to 'gitlint/tests/config/test_config_builder.py')

diff --git a/gitlint/tests/config/test_config_builder.py b/gitlint/tests/config/test_config_builder.py
index 5a28c9f..e0d7f9b 100644
--- a/gitlint/tests/config/test_config_builder.py
+++ b/gitlint/tests/config/test_config_builder.py
@@ -42,30 +42,30 @@ class LintConfigBuilderTests(BaseTestCase):
         config_builder = LintConfigBuilder()
 
         # nothing gitlint
-        config_builder.set_config_from_commit(self.gitcommit(u"tëst\ngitlint\nfoo"))
+        config_builder.set_config_from_commit(self.gitcommit("tëst\ngitlint\nfoo"))
         config = config_builder.build()
         self.assertSequenceEqual(config.rules, original_rules)
         self.assertListEqual(config.ignore, [])
 
         # ignore all rules
-        config_builder.set_config_from_commit(self.gitcommit(u"tëst\ngitlint-ignore: all\nfoo"))
+        config_builder.set_config_from_commit(self.gitcommit("tëst\ngitlint-ignore: all\nfoo"))
         config = config_builder.build()
         self.assertEqual(config.ignore, original_rule_ids)
 
         # ignore all rules, no space
-        config_builder.set_config_from_commit(self.gitcommit(u"tëst\ngitlint-ignore:all\nfoo"))
+        config_builder.set_config_from_commit(self.gitcommit("tëst\ngitlint-ignore:all\nfoo"))
         config = config_builder.build()
         self.assertEqual(config.ignore, original_rule_ids)
 
         # ignore all rules, more spacing
-        config_builder.set_config_from_commit(self.gitcommit(u"tëst\ngitlint-ignore: \t all\nfoo"))
+        config_builder.set_config_from_commit(self.gitcommit("tëst\ngitlint-ignore: \t all\nfoo"))
         config = config_builder.build()
         self.assertEqual(config.ignore, original_rule_ids)
 
     def test_set_from_commit_ignore_specific(self):
         # ignore specific rules
         config_builder = LintConfigBuilder()
-        config_builder.set_config_from_commit(self.gitcommit(u"tëst\ngitlint-ignore: T1, body-hard-tab"))
+        config_builder.set_config_from_commit(self.gitcommit("tëst\ngitlint-ignore: T1, body-hard-tab"))
         config = config_builder.build()
         self.assertEqual(config.ignore, ["T1", "body-hard-tab"])
 
@@ -89,14 +89,14 @@ class LintConfigBuilderTests(BaseTestCase):
         config_builder = LintConfigBuilder()
 
         # bad config file load
-        foo_path = self.get_sample_path(u"föo")
-        expected_error_msg = u"Invalid file path: {0}".format(foo_path)
+        foo_path = self.get_sample_path("föo")
+        expected_error_msg = f"Invalid file path: {foo_path}"
         with self.assertRaisesMessage(LintConfigError, expected_error_msg):
             config_builder.set_from_config_file(foo_path)
 
         # error during file parsing
         path = self.get_sample_path("config/no-sections")
-        expected_error_msg = u"File contains no section headers."
+        expected_error_msg = "File contains no section headers."
         # We only match the start of the message here, since the exact message can vary depending on platform
         with self.assertRaisesRegex(LintConfigError, expected_error_msg):
             config_builder.set_from_config_file(path)
@@ -105,7 +105,7 @@ class LintConfigBuilderTests(BaseTestCase):
         path = self.get_sample_path("config/nonexisting-rule")
         config_builder = LintConfigBuilder()
         config_builder.set_from_config_file(path)
-        expected_error_msg = u"No such rule 'föobar'"
+        expected_error_msg = "No such rule 'föobar'"
         with self.assertRaisesMessage(LintConfigError, expected_error_msg):
             config_builder.build()
 
@@ -113,7 +113,7 @@ class LintConfigBuilderTests(BaseTestCase):
         path = self.get_sample_path("config/nonexisting-general-option")
         config_builder = LintConfigBuilder()
         config_builder.set_from_config_file(path)
-        expected_error_msg = u"'foo' is not a valid gitlint option"
+        expected_error_msg = "'foo' is not a valid gitlint option"
         with self.assertRaisesMessage(LintConfigError, expected_error_msg):
             config_builder.build()
 
@@ -121,7 +121,7 @@ class LintConfigBuilderTests(BaseTestCase):
         path = self.get_sample_path("config/nonexisting-option")
         config_builder = LintConfigBuilder()
         config_builder.set_from_config_file(path)
-        expected_error_msg = u"Rule 'title-max-length' has no option 'föobar'"
+        expected_error_msg = "Rule 'title-max-length' has no option 'föobar'"
         with self.assertRaisesMessage(LintConfigError, expected_error_msg):
             config_builder.build()
 
@@ -129,8 +129,8 @@ class LintConfigBuilderTests(BaseTestCase):
         path = self.get_sample_path("config/invalid-option-value")
         config_builder = LintConfigBuilder()
         config_builder.set_from_config_file(path)
-        expected_error_msg = u"'föo' is not a valid value for option 'title-max-length.line-length'. " + \
-                             u"Option 'line-length' must be a positive integer (current value: 'föo')."
+        expected_error_msg = "'föo' is not a valid value for option 'title-max-length.line-length'. " + \
+                             "Option 'line-length' must be a positive integer (current value: 'föo')."
         with self.assertRaisesMessage(LintConfigError, expected_error_msg):
             config_builder.build()
 
@@ -141,39 +141,39 @@ class LintConfigBuilderTests(BaseTestCase):
         config_builder = LintConfigBuilder()
         config_builder.set_config_from_string_list(['general.verbosity=1', 'title-max-length.line-length=60',
                                                     'body-max-line-length.line-length=120',
-                                                    u"title-must-not-contain-word.words=håha"])
+                                                    "title-must-not-contain-word.words=håha"])
 
         config = config_builder.build()
         self.assertEqual(config.get_rule_option('title-max-length', 'line-length'), 60)
         self.assertEqual(config.get_rule_option('body-max-line-length', 'line-length'), 120)
-        self.assertListEqual(config.get_rule_option('title-must-not-contain-word', 'words'), [u"håha"])
+        self.assertListEqual(config.get_rule_option('title-must-not-contain-word', 'words'), ["håha"])
         self.assertEqual(config.verbosity, 1)
 
     def test_set_config_from_string_list_negative(self):
         config_builder = LintConfigBuilder()
 
         # assert error on incorrect rule - this happens at build time
-        config_builder.set_config_from_string_list([u"föo.bar=1"])
-        with self.assertRaisesMessage(LintConfigError, u"No such rule 'föo'"):
+        config_builder.set_config_from_string_list(["föo.bar=1"])
+        with self.assertRaisesMessage(LintConfigError, "No such rule 'föo'"):
             config_builder.build()
 
         # no equal sign
-        expected_msg = u"'föo.bar' is an invalid configuration option. Use '<rule>.<option>=<value>'"
+        expected_msg = "'föo.bar' is an invalid configuration option. Use '<rule>.<option>=<value>'"
         with self.assertRaisesMessage(LintConfigError, expected_msg):
-            config_builder.set_config_from_string_list([u"föo.bar"])
+            config_builder.set_config_from_string_list(["föo.bar"])
 
         # missing value
-        expected_msg = u"'föo.bar=' is an invalid configuration option. Use '<rule>.<option>=<value>'"
+        expected_msg = "'föo.bar=' is an invalid configuration option. Use '<rule>.<option>=<value>'"
         with self.assertRaisesMessage(LintConfigError, expected_msg):
-            config_builder.set_config_from_string_list([u"föo.bar="])
+            config_builder.set_config_from_string_list(["föo.bar="])
 
         # space instead of equal sign
-        expected_msg = u"'föo.bar 1' is an invalid configuration option. Use '<rule>.<option>=<value>'"
+        expected_msg = "'föo.bar 1' is an invalid configuration option. Use '<rule>.<option>=<value>'"
         with self.assertRaisesMessage(LintConfigError, expected_msg):
-            config_builder.set_config_from_string_list([u"föo.bar 1"])
+            config_builder.set_config_from_string_list(["föo.bar 1"])
 
         # no period between rule and option names
-        expected_msg = u"'föobar=1' is an invalid configuration option. Use '<rule>.<option>=<value>'"
+        expected_msg = "'föobar=1' is an invalid configuration option. Use '<rule>.<option>=<value>'"
         with self.assertRaisesMessage(LintConfigError, expected_msg):
             config_builder.set_config_from_string_list([u'föobar=1'])
 
@@ -216,15 +216,15 @@ class LintConfigBuilderTests(BaseTestCase):
         # Add a named rule by setting an option in the config builder that follows the named rule pattern
         # Assert that whitespace in the rule name is stripped
         rule_qualifiers = [u'T7:my-extra-rüle', u' T7 :   my-extra-rüle  ', u'\tT7:\tmy-extra-rüle\t',
-                           u'T7:\t\n  \tmy-extra-rüle\t\n\n', u"title-match-regex:my-extra-rüle"]
+                           u'T7:\t\n  \tmy-extra-rüle\t\n\n', "title-match-regex:my-extra-rüle"]
         for rule_qualifier in rule_qualifiers:
             config_builder = LintConfigBuilder()
-            config_builder.set_option(rule_qualifier, 'regex', u"föo")
+            config_builder.set_option(rule_qualifier, 'regex', "föo")
 
             expected_rules = copy.deepcopy(default_rules)
-            my_rule = rules.TitleRegexMatches({'regex': u"föo"})
-            my_rule.id = rules.TitleRegexMatches.id + u":my-extra-rüle"
-            my_rule.name = rules.TitleRegexMatches.name + u":my-extra-rüle"
+            my_rule = rules.TitleRegexMatches({'regex': "föo"})
+            my_rule.id = rules.TitleRegexMatches.id + ":my-extra-rüle"
+            my_rule.name = rules.TitleRegexMatches.name + ":my-extra-rüle"
             expected_rules._rules[u'T7:my-extra-rüle'] = my_rule
             self.assertEqual(config_builder.build().rules, expected_rules)
 
@@ -233,32 +233,32 @@ class LintConfigBuilderTests(BaseTestCase):
             # to the same rule
             for other_rule_qualifier in rule_qualifiers:
                 cb = config_builder.clone()
-                cb.set_option(other_rule_qualifier, 'regex', other_rule_qualifier + u"bōr")
+                cb.set_option(other_rule_qualifier, 'regex', other_rule_qualifier + "bōr")
                 # before setting the expected rule option value correctly, the RuleCollection should be different
                 self.assertNotEqual(cb.build().rules, expected_rules)
                 # after setting the option on the expected rule, it should be equal
-                my_rule.options['regex'].set(other_rule_qualifier + u"bōr")
+                my_rule.options['regex'].set(other_rule_qualifier + "bōr")
                 self.assertEqual(cb.build().rules, expected_rules)
-                my_rule.options['regex'].set(u"wrong")
+                my_rule.options['regex'].set("wrong")
 
     def test_named_rules_negative(self):
         # T7 = title-match-regex
         # Invalid rule name
-        for invalid_name in ["", " ", "    ", "\t", "\n", u"å b", u"å:b", u"åb:", u":åb"]:
+        for invalid_name in ["", " ", "    ", "\t", "\n", "å b", "å:b", "åb:", ":åb"]:
             config_builder = LintConfigBuilder()
-            config_builder.set_option(u"T7:{0}".format(invalid_name), 'regex', u"tëst")
-            expected_msg = u"The rule-name part in 'T7:{0}' cannot contain whitespace, colons or be empty"
-            with self.assertRaisesMessage(LintConfigError, expected_msg.format(invalid_name)):
+            config_builder.set_option(f"T7:{invalid_name}", 'regex', "tëst")
+            expected_msg = f"The rule-name part in 'T7:{invalid_name}' cannot contain whitespace, colons or be empty"
+            with self.assertRaisesMessage(LintConfigError, expected_msg):
                 config_builder.build()
 
         # Invalid parent rule name
         config_builder = LintConfigBuilder()
-        config_builder.set_option(u"Ž123:foöbar", u"fåke-option", u"fåke-value")
-        with self.assertRaisesMessage(LintConfigError, u"No such rule 'Ž123' (named rule: 'Ž123:foöbar')"):
+        config_builder.set_option("Ž123:foöbar", "fåke-option", "fåke-value")
+        with self.assertRaisesMessage(LintConfigError, "No such rule 'Ž123' (named rule: 'Ž123:foöbar')"):
             config_builder.build()
 
         # Invalid option name (this is the same as with regular rules)
         config_builder = LintConfigBuilder()
-        config_builder.set_option(u"T7:foöbar", u"blå", u"my-rëgex")
-        with self.assertRaisesMessage(LintConfigError, u"Rule 'T7:foöbar' has no option 'blå'"):
+        config_builder.set_option("T7:foöbar", "blå", "my-rëgex")
+        with self.assertRaisesMessage(LintConfigError, "Rule 'T7:foöbar' has no option 'blå'"):
             config_builder.build()
-- 
cgit v1.2.3