Add min tests

Author: mrunalhirve128

tests/runtime/test_environment_utils.py

@@ -5,7 +5,6 @@
 
 import os
 import unittest
-from unittest.mock import patch
 
 from microsoft_agents_a365.runtime.environment_utils import (
     DEVELOPMENT_ENVIRONMENT_NAME,
@@ -18,14 +17,15 @@
 
 
 class TestEnvironmentUtils(unittest.TestCase):
-    """Test cases for environment utility functions."""
+    """Test cases for environment utility functions.
+    
+    Tests: Constants validation, observability scope retrieval, and environment detection logic.
+    """
 
     def tearDown(self):
         """Clean up environment variables after each test."""
-        env_vars = ["PYTHON_ENVIRONMENT"]
-        for var in env_vars:
-            if var in os.environ:
-                del os.environ[var]
+        if "PYTHON_ENVIRONMENT" in os.environ:
+            del os.environ["PYTHON_ENVIRONMENT"]
 
     def test_constants(self):
         """Test that environment constants have expected values."""
@@ -44,7 +44,6 @@ def test_get_observability_authentication_scope_returns_prod_scope(self):
 
     def test_is_development_environment_with_no_env_var(self):
         """Test is_development_environment returns False when no environment variable is set."""
-        # Ensure environment variables are not set
         if "PYTHON_ENVIRONMENT" in os.environ:
             del os.environ["PYTHON_ENVIRONMENT"]
 
@@ -60,16 +59,6 @@ def test_is_development_environment_with_development_env_var(self):
 
         self.assertTrue(result)
 
-    def test_is_development_environment_case_insensitive(self):
-        """Test is_development_environment is case-insensitive."""
-        test_cases = ["development", "DEVELOPMENT", "DeveLoPMenT", "Development"]
-
-        for env_value in test_cases:
-            with self.subTest(env_value=env_value):
-                os.environ["PYTHON_ENVIRONMENT"] = env_value
-                result = is_development_environment()
-                self.assertTrue(result)
-
     def test_is_development_environment_with_production_env_var(self):
         """Test is_development_environment returns False when PYTHON_ENVIRONMENT is 'production'."""
         os.environ["PYTHON_ENVIRONMENT"] = "production"
@@ -78,86 +67,6 @@ def test_is_development_environment_with_production_env_var(self):
 
         self.assertFalse(result)
 
-    def test_is_development_environment_with_other_env_var(self):
-        """Test is_development_environment returns False for other environment values."""
-        test_cases = ["staging", "test", "preprod", "custom"]
-
-        for env_value in test_cases:
-            with self.subTest(env_value=env_value):
-                os.environ["PYTHON_ENVIRONMENT"] = env_value
-                result = is_development_environment()
-                self.assertFalse(result)
-
-    def test_is_development_environment_with_empty_env_var(self):
-        """Test is_development_environment returns False when PYTHON_ENVIRONMENT is empty."""
-        os.environ["PYTHON_ENVIRONMENT"] = ""
-
-        result = is_development_environment()
-
-        self.assertFalse(result)
-
-    def test_is_development_environment_with_whitespace_env_var(self):
-        """Test is_development_environment returns False when PYTHON_ENVIRONMENT is whitespace."""
-        os.environ["PYTHON_ENVIRONMENT"] = "   "
-
-        result = is_development_environment()
-
-        self.assertFalse(result)
-
-    @patch.dict(os.environ, {"PYTHON_ENVIRONMENT": "Development"}, clear=False)
-    def test_python_environment_precedence(self):
-        """Test that PYTHON_ENVIRONMENT takes precedence."""
-        result = is_development_environment()
-
-        self.assertTrue(result)
-
-    def test_default_environment_is_production(self):
-        """Test that the default environment is production when no env vars are set."""
-        # Ensure no environment variables are set
-        if "PYTHON_ENVIRONMENT" in os.environ:
-            del os.environ["PYTHON_ENVIRONMENT"]
-
-        # The _get_current_environment function should default to PRODUCTION_ENVIRONMENT_NAME
-        result = is_development_environment()
-
-        self.assertFalse(result)
-
-
-class TestObservabilityAuthenticationScope(unittest.TestCase):
-    """Test cases for observability authentication scope."""
-
-    def test_scope_is_list(self):
-        """Test that the scope is returned as a list."""
-        result = get_observability_authentication_scope()
-
-        self.assertIsInstance(result, list)
-
-    def test_scope_contains_single_value(self):
-        """Test that the scope list contains exactly one value."""
-        result = get_observability_authentication_scope()
-
-        self.assertEqual(len(result), 1)
-
-    def test_scope_value_is_string(self):
-        """Test that the scope value is a string."""
-        result = get_observability_authentication_scope()
-
-        self.assertIsInstance(result[0], str)
-
-    def test_scope_value_format(self):
-        """Test that the scope value has the correct format."""
-        result = get_observability_authentication_scope()
-
-        self.assertTrue(result[0].startswith("https://"))
-        self.assertTrue(result[0].endswith(".default"))
-
-    def test_scope_consistency(self):
-        """Test that multiple calls return the same scope."""
-        result1 = get_observability_authentication_scope()
-        result2 = get_observability_authentication_scope()
-
-        self.assertEqual(result1, result2)
-
 
 if __name__ == "__main__":
     unittest.main()

tests/runtime/test_version_utils.py

@@ -11,24 +11,18 @@
 
 
 class TestVersionUtils(unittest.TestCase):
-    """Test cases for version utility functions."""
+    """Test cases for version utility functions.
+    
+    Tests: Default version behavior, environment variable usage, and deprecation warning.
+    """
 
     def tearDown(self):
         """Clean up environment variables after each test."""
         if "AGENT365_PYTHON_SDK_PACKAGE_VERSION" in os.environ:
             del os.environ["AGENT365_PYTHON_SDK_PACKAGE_VERSION"]
 
-    def test_build_version_returns_string(self):
-        """Test that build_version returns a string."""
-        with warnings.catch_warnings():
-            warnings.simplefilter("ignore", DeprecationWarning)
-            result = build_version()
-
-        self.assertIsInstance(result, str)
-
     def test_build_version_default_value(self):
         """Test build_version returns default version when no env var is set."""
-        # Ensure environment variable is not set
         if "AGENT365_PYTHON_SDK_PACKAGE_VERSION" in os.environ:
             del os.environ["AGENT365_PYTHON_SDK_PACKAGE_VERSION"]
 
@@ -49,36 +43,6 @@ def test_build_version_with_env_var(self):
 
         self.assertEqual(result, test_version)
 
-    def test_build_version_with_complex_version(self):
-        """Test build_version with complex version strings."""
-        test_cases = [
-            "1.0.0-alpha",
-            "2.3.4-beta.1",
-            "3.0.0-rc.2",
-            "1.2.3+build.456",
-            "2.0.0-alpha.1+build.789",
-        ]
-
-        for version in test_cases:
-            with self.subTest(version=version):
-                os.environ["AGENT365_PYTHON_SDK_PACKAGE_VERSION"] = version
-
-                with warnings.catch_warnings():
-                    warnings.simplefilter("ignore", DeprecationWarning)
-                    result = build_version()
-
-                self.assertEqual(result, version)
-
-    def test_build_version_with_empty_env_var(self):
-        """Test build_version with empty environment variable."""
-        os.environ["AGENT365_PYTHON_SDK_PACKAGE_VERSION"] = ""
-
-        with warnings.catch_warnings():
-            warnings.simplefilter("ignore", DeprecationWarning)
-            result = build_version()
-
-        self.assertEqual(result, "")
-
     def test_build_version_deprecation_warning(self):
         """Test that build_version raises DeprecationWarning."""
         with self.assertWarns(DeprecationWarning) as cm:
@@ -88,64 +52,6 @@ def test_build_version_deprecation_warning(self):
         self.assertIn("deprecated", warning_message.lower())
         self.assertIn("setuptools-git-versioning", warning_message)
 
-    def test_build_version_deprecation_warning_message(self):
-        """Test that deprecation warning contains the correct message."""
-        with warnings.catch_warnings(record=True) as w:
-            warnings.simplefilter("always")
-            build_version()
-
-            # Check that exactly one warning was raised
-            self.assertEqual(len(w), 1)
-
-            # Check that it's a DeprecationWarning
-            self.assertTrue(issubclass(w[0].category, DeprecationWarning))
-
-            # Check the message content
-            message = str(w[0].message)
-            self.assertIn("build_version() is deprecated", message)
-            self.assertIn("setuptools-git-versioning", message)
-
-    def test_build_version_stacklevel(self):
-        """Test that deprecation warning has correct stack level."""
-        with warnings.catch_warnings(record=True) as w:
-            warnings.simplefilter("always")
-            build_version()
-
-            # The warning should point to the caller of build_version
-            # not to the function itself (stacklevel=2)
-            self.assertEqual(len(w), 1)
-            warning = w[0]
-
-            # Verify the warning was captured
-            self.assertIsNotNone(warning.filename)
-            self.assertGreater(warning.lineno, 0)
-
-
-class TestVersionUtilsDocstring(unittest.TestCase):
-    """Test cases for version_utils module documentation."""
-
-    def test_module_has_docstring(self):
-        """Test that the module has a docstring."""
-        import microsoft_agents_a365.runtime.version_utils as version_utils
-
-        self.assertIsNotNone(version_utils.__doc__)
-        self.assertGreater(len(version_utils.__doc__), 0)
-
-    def test_module_docstring_mentions_deprecation(self):
-        """Test that module docstring mentions deprecation."""
-        import microsoft_agents_a365.runtime.version_utils as version_utils
-
-        self.assertIn("deprecated", version_utils.__doc__.lower())
-
-    def test_function_has_docstring(self):
-        """Test that build_version function has a docstring."""
-        self.assertIsNotNone(build_version.__doc__)
-        self.assertGreater(len(build_version.__doc__), 0)
-
-    def test_function_docstring_mentions_deprecation(self):
-        """Test that function docstring mentions deprecation."""
-        self.assertIn("DEPRECATED", build_version.__doc__)
-
 
 if __name__ == "__main__":
     unittest.main()