fix unit tests

Author: tora-kozic

src/code42cli/config.py

@@ -145,6 +145,7 @@ def _create_profile_section(self, name):
         self.parser[name][self.USERNAME_KEY] = self.DEFAULT_VALUE
         self.parser[name][self.IGNORE_SSL_ERRORS_KEY] = str(False)
         self.parser[name][self.USE_V2_FILE_EVENTS_KEY] = str(False)
+        self.parser[name][self.API_CLIENT_AUTH_KEY] = str(False)
 
     def _save(self):
         with open(self.path, "w+", encoding="utf-8") as file:

src/code42cli/profile.py

@@ -35,7 +35,7 @@ def use_v2_file_events(self):
     @property
     def api_client_auth(self):
         val = self._profile.get(ConfigAccessor.API_CLIENT_AUTH_KEY)
-        return val if val is not None else False
+        return val if val is not None else "False"
 
     @property
     def has_stored_password(self):

tests/conftest.py

@@ -87,14 +87,18 @@ def alert_namespace():
 
 
 def create_profile_values_dict(
-    authority=None, username=None, ignore_ssl=False, use_v2_file_events=False
+    authority=None,
+    username=None,
+    ignore_ssl=False,
+    use_v2_file_events=False,
+    api_client_auth="False",
 ):
     return {
         ConfigAccessor.AUTHORITY_KEY: "example.com",
         ConfigAccessor.USERNAME_KEY: "foo",
-        ConfigAccessor.IGNORE_SSL_ERRORS_KEY: True,
-        ConfigAccessor.USE_V2_FILE_EVENTS_KEY: False,
-        ConfigAccessor.API_CLIENT_AUTH_KEY: False,
+        ConfigAccessor.IGNORE_SSL_ERRORS_KEY: "True",
+        ConfigAccessor.USE_V2_FILE_EVENTS_KEY: "False",
+        ConfigAccessor.API_CLIENT_AUTH_KEY: "False",
     }
 
 

tests/test_config.py

@@ -153,7 +153,7 @@ def test_create_profile_when_given_default_name_does_not_create(
         accessor = ConfigAccessor(config_parser_for_create)
         with pytest.raises(Exception):
             accessor.create_profile(
-                ConfigAccessor.DEFAULT_VALUE, "foo", "bar", False, False
+                ConfigAccessor.DEFAULT_VALUE, "foo", "bar", False, False, False
             )
 
     def test_create_profile_when_no_default_profile_sets_default(
@@ -165,7 +165,9 @@ def test_create_profile_when_no_default_profile_sets_default(
         accessor = ConfigAccessor(config_parser_for_create)
         accessor.switch_default_profile = mocker.MagicMock()
 
-        accessor.create_profile(_TEST_PROFILE_NAME, "example.com", "bar", None, None)
+        accessor.create_profile(
+            _TEST_PROFILE_NAME, "example.com", "bar", None, None, None
+        )
         assert accessor.switch_default_profile.call_count == 1
 
     def test_create_profile_when_has_default_profile_does_not_set_default(
@@ -177,7 +179,9 @@ def test_create_profile_when_has_default_profile_does_not_set_default(
         accessor = ConfigAccessor(config_parser_for_create)
         accessor.switch_default_profile = mocker.MagicMock()
 
-        accessor.create_profile(_TEST_PROFILE_NAME, "example.com", "bar", None, None)
+        accessor.create_profile(
+            _TEST_PROFILE_NAME, "example.com", "bar", None, None, None
+        )
         assert not accessor.switch_default_profile.call_count
 
     def test_create_profile_when_not_existing_saves(
@@ -188,7 +192,9 @@ def test_create_profile_when_not_existing_saves(
         setup_parser_one_profile(mock_internal, mock_internal, config_parser_for_create)
         accessor = ConfigAccessor(config_parser_for_create)
 
-        accessor.create_profile(_TEST_PROFILE_NAME, "example.com", "bar", None, None)
+        accessor.create_profile(
+            _TEST_PROFILE_NAME, "example.com", "bar", None, None, None
+        )
         assert mock_saver.call_count
 
     def test_update_profile_when_no_profile_exists_raises_exception(