Fix a few tests

testing/factories.py

@@ -4,11 +4,11 @@
 from detect_secrets.core.secrets_collection import SecretsCollection
 
 
-def potential_secret_factory(type_='type', filename='filename', lineno=1, secret='secret'):
+def potential_secret_factory(type_='type', filename='filename', secret='secret', lineno=1):
     """This is only marginally better than creating PotentialSecret objects directly,
     because of default values.
     """
-    return PotentialSecret(type_, filename, lineno, secret)
+    return PotentialSecret(type_, filename, secret, lineno)
 
 
 def secrets_collection_factory(secrets=None, plugins=(), exclude_regex=''):
@@ -51,7 +51,7 @@ def _add_secret(collection, type_='type', secret='secret', filename='filename',
     tmp_secret = potential_secret_factory(
         type_=type_,
         filename=filename,
-        lineno=lineno,
         secret=secret,
+        lineno=lineno,
     )
     collection.data[filename][tmp_secret] = tmp_secret

tests/core/potential_secret_test.py

@@ -9,7 +9,7 @@
 class TestPotentialSecret(object):
 
     @pytest.mark.parametrize(
-        'a,b,is_equal',
+        'a, b, is_equal',
         [
             (
                 potential_secret_factory(lineno=1),

tests/core/secrets_collection_test.py

@@ -93,7 +93,7 @@ def test_success_multiple_plugins(self):
         line_numbers = [entry.lineno for entry in logic.data['filename']]
         assert set(line_numbers) == set([1, 2, 3])
 
-    def test_removal_of_password_plugin_secrets_if_reported_already(self):
+    def test_reporting_of_password_plugin_secrets_if_reported_already(self):
         logic = secrets_collection_factory(
             secrets=[
                 {
@@ -110,9 +110,7 @@ def test_removal_of_password_plugin_secrets_if_reported_already(self):
         with mock_open('junk text here'):
             logic.scan_file('filename')
 
-        # One from only the MockPluginFileValue, and one from existing secret
-        # This is because the MockPasswordPluginValue was not placed into data
-        assert len(logic.data['filename']) == 2
+        assert len(logic.data['filename']) == 3
 
         line_numbers = [entry.lineno for entry in logic.data['filename']]
         assert set(line_numbers) == set([2, 3])