fix: pylint errors

Author: matthew-on-git

tests/__init__.py

@@ -1 +0,0 @@
- 

tests/test_api.py

@@ -1,13 +1,19 @@
 """Unit tests for Bitbucket API logic in manage_bitbucket_env.py."""
-from manage_bitbucket_env import get_environment_uuid, get_variables
 import unittest
 from unittest.mock import patch, Mock
-import sys
-import os
-sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+from typing import Optional
+
+from manage_bitbucket_env import get_environment_uuid, get_variables
 
 class TestAPI(unittest.TestCase):
     """Test API-related functions."""
+    # pylint: disable=attribute-defined-outside-init
+    logger: Optional[Mock] = None
+    auth: Optional[Mock] = None
+    workspace: Optional[str] = None
+    repo_slug: Optional[str] = None
+    deployment_name: Optional[str] = None
+    env_uuid: Optional[str] = None
 
     def setUp(self):
         """Set up test fixtures."""
@@ -65,4 +71,4 @@ def test_get_variables(self, mock_get):
         self.assertEqual(result_vars[0]["key"], "A")
 
 if __name__ == '__main__':
-    unittest.main() 
+    unittest.main()

tests/test_arg_parser.py

@@ -1,10 +1,9 @@
 """Unit tests for argument parsing in manage_bitbucket_env.py."""
-from manage_bitbucket_env import arg_parser
 import unittest
 from unittest.mock import patch
 import sys
-import os
-sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+
+from manage_bitbucket_env import arg_parser
 
 class TestArgParser(unittest.TestCase):
     """Test argument parsing functionality."""
@@ -40,4 +39,4 @@ def test_mutually_exclusive(self):
                 arg_parser()
 
 if __name__ == '__main__':
-    unittest.main() 
+    unittest.main()

tests/test_export_import.py

@@ -1,18 +1,25 @@
 """Unit tests for export/import logic in manage_bitbucket_env.py."""
 # pylint: disable=duplicate-code
-from manage_bitbucket_env import (
-    export_variables, export_all_variables, export_secure_keys, import_variables
-)
 import unittest
 from unittest.mock import patch, Mock
-import sys
 import os
 import tempfile
 import json
-sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+from typing import Optional
+
+from manage_bitbucket_env import (
+    export_variables, export_all_variables, export_secure_keys, import_variables
+)
 
 class TestExportImport(unittest.TestCase):
     """Test export and import functionality."""
+    # pylint: disable=attribute-defined-outside-init
+    logger: Optional[Mock] = None
+    auth: Optional[Mock] = None
+    workspace: Optional[str] = None
+    repo_slug: Optional[str] = None
+    deployment_name: Optional[str] = None
+    env_uuid: Optional[str] = None
 
     def setUp(self):
         """Set up test fixtures."""
@@ -125,4 +132,4 @@ def test_import_variables(self, mock_update, mock_get_vars, mock_get_uuid):
             os.unlink(fname)
 
 if __name__ == '__main__':
-    unittest.main() 
+    unittest.main()

tests/test_main.py

@@ -1,18 +1,19 @@
 """Unit tests for main function integration in manage_bitbucket_env.py."""
 # pylint: disable=unused-argument
-from manage_bitbucket_env import main
 import unittest
 from unittest.mock import patch, Mock
-import sys
-import os
-sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+from typing import Optional
+
+from manage_bitbucket_env import main
 
 class TestMainFunction(unittest.TestCase):
     """Test main function integration."""
+    # pylint: disable=attribute-defined-outside-init
+    mock_logger: Optional[Mock] = None
 
     def setUp(self):
         """Set up test fixtures."""
-        self.mock_logger: Mock = Mock()
+        self.mock_logger = Mock()
 
     @patch('manage_bitbucket_env.arg_parser')
     @patch('manage_bitbucket_env.load_dotenv')
@@ -59,4 +60,4 @@ def test_main_missing_credentials(self, mock_env_get, mock_load_dotenv, mock_arg
             main(self.mock_logger)
 
 if __name__ == '__main__':
-    unittest.main() 
+    unittest.main()

tests/test_update_vars.py

@@ -1,14 +1,19 @@
 """Unit tests for update_vars logic in manage_bitbucket_env.py."""
 # pylint: disable=duplicate-code
-from manage_bitbucket_env import update_vars
 import unittest
 from unittest.mock import patch, Mock
-import sys
-import os
-sys.path.insert(0, os.path.dirname(os.path.abspath(__file__)))
+from typing import Optional
+
+from manage_bitbucket_env import update_vars
 
 class TestUpdateVars(unittest.TestCase):
     """Test variable update functionality."""
+    # pylint: disable=attribute-defined-outside-init
+    logger: Optional[Mock] = None
+    auth: Optional[Mock] = None
+    workspace: Optional[str] = None
+    repo_slug: Optional[str] = None
+    env_uuid: Optional[str] = None
 
     def setUp(self):
         """Set up test fixtures."""
@@ -49,4 +54,4 @@ def test_update_new(self, mock_post):
         mock_post.assert_called_once()
 
 if __name__ == '__main__':
-    unittest.main() 
+    unittest.main()