adding tests

Author: kieranmanning

cloudsmith_cli/cli/commands/main.py

@@ -73,6 +73,7 @@ def main(ctx, opts, version, no_warn):
         opts.no_warn = True
 
     if version:
+        opts.no_warn = True
         print_version()
     elif ctx.invoked_subcommand is None:
         click.echo(ctx.get_help())

cloudsmith_cli/cli/tests/commands/test_main.py

@@ -6,17 +6,14 @@
 
 
 class TestMainCommand:
-    @pytest.mark.usefixtures("set_no_warn_env_var")
     @pytest.mark.parametrize("option", ["-V", "--version"])
     def test_main_version(self, runner, option):
         """Test the output of `cloudsmith --version`."""
         result = runner.invoke(main, [option])
         assert result.exit_code == 0
-        assert (
-            result.output == "Versions:\n"
-            "CLI Package Version: " + get_version() + "\n"
-            "API Package Version: " + get_api_version() + "\n"
-        )
+
+        assert "CLI Package Version: " + get_version() in result.output
+        assert "API Package Version: " + get_api_version() in result.output
 
     @pytest.mark.parametrize("option", ["-h", "--help"])
     def test_main_help(self, runner, option):

cloudsmith_cli/cli/tests/test_warnings.py

@@ -0,0 +1,26 @@
+from cloudsmith_cli.cli.warnings import (
+    ApiAuthenticationWarning,
+    CliWarnings,
+    ConfigLoadWarning,
+    ProfileNotFoundWarning,
+)
+
+
+class TestWarnings:
+    def test_warning_append(self):
+        """Test appending warnings to the CliWarnings."""
+
+        config_load_warning_1 = ConfigLoadWarning({"test_path_1": False})
+        config_load_warning_2 = ConfigLoadWarning({"test_path_2": True})
+        profile_load_warning = ProfileNotFoundWarning(
+            {"test_path_1": False}, "test_profile"
+        )
+        api_authentication_warning = ApiAuthenticationWarning("test.cloudsmith.io")
+        cli_warnings = CliWarnings()
+        cli_warnings.append(config_load_warning_1)
+        cli_warnings.append(config_load_warning_2)
+        cli_warnings.append(profile_load_warning)
+        cli_warnings.append(profile_load_warning)
+        cli_warnings.append(api_authentication_warning)
+        assert len(cli_warnings) == 5
+        assert len(cli_warnings.__dedupe__()) == 4