format fix

Author: mrunalhirve128

tests/runtime/test_environment_utils.py

@@ -37,7 +37,7 @@ def test_constants(self):
     def test_get_observability_authentication_scope_returns_prod_scope(self):
         """Test that get_observability_authentication_scope returns production scope."""
         result = get_observability_authentication_scope()
-        
+
         self.assertIsInstance(result, list)
         self.assertEqual(len(result), 1)
         self.assertEqual(result[0], PROD_OBSERVABILITY_SCOPE)
@@ -47,23 +47,23 @@ def test_is_development_environment_with_no_env_var(self):
         # Ensure environment variables are not set
         if "PYTHON_ENVIRONMENT" in os.environ:
             del os.environ["PYTHON_ENVIRONMENT"]
-        
+
         result = is_development_environment()
-        
+
         self.assertFalse(result)
 
     def test_is_development_environment_with_development_env_var(self):
         """Test is_development_environment returns True when PYTHON_ENVIRONMENT is 'Development'."""
         os.environ["PYTHON_ENVIRONMENT"] = "Development"
-        
+
         result = is_development_environment()
-        
+
         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
@@ -73,15 +73,15 @@ def test_is_development_environment_case_insensitive(self):
     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"
-        
+
         result = is_development_environment()
-        
+
         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
@@ -91,35 +91,35 @@ def test_is_development_environment_with_other_env_var(self):
     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)
 
 
@@ -129,33 +129,33 @@ class TestObservabilityAuthenticationScope(unittest.TestCase):
     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)
 
 

tests/runtime/test_version_utils.py

@@ -23,30 +23,30 @@ def test_build_version_returns_string(self):
         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"]
-        
+
         with warnings.catch_warnings():
             warnings.simplefilter("ignore", DeprecationWarning)
             result = build_version()
-        
+
         self.assertEqual(result, "0.0.0")
 
     def test_build_version_with_env_var(self):
         """Test build_version returns version from environment variable."""
         test_version = "1.2.3"
         os.environ["AGENT365_PYTHON_SDK_PACKAGE_VERSION"] = test_version
-        
+
         with warnings.catch_warnings():
             warnings.simplefilter("ignore", DeprecationWarning)
             result = build_version()
-        
+
         self.assertEqual(result, test_version)
 
     def test_build_version_with_complex_version(self):
@@ -58,32 +58,32 @@ def test_build_version_with_complex_version(self):
             "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:
             build_version()
-        
+
         warning_message = str(cm.warning)
         self.assertIn("deprecated", warning_message.lower())
         self.assertIn("setuptools-git-versioning", warning_message)
@@ -93,13 +93,13 @@ def test_build_version_deprecation_warning_message(self):
         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)
@@ -110,12 +110,12 @@ def test_build_version_stacklevel(self):
         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)
@@ -127,14 +127,14 @@ class TestVersionUtilsDocstring(unittest.TestCase):
     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):