feat/aws-auditor: run tests

Author: hminaee-tc

.github/workflows/tests.yaml

@@ -0,0 +1,53 @@
+---
+name: Tests
+
+on:
+  push:
+    branches: [main, develop, feat/aws-auditor]
+  pull_request:
+
+jobs:
+  test:
+    name: Run Tests
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.x'
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -e ".[test]"
+
+      - name: Run tests
+        run: |
+          pytest cpk_lib_python_aws/tests/ -v
+
+
+  test-installation:
+    name: Test Package Installation
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: '3.x'
+
+      - name: Install package
+        run: |
+          python -m pip install --upgrade pip
+          pip install -e .
+
+      - name: Test CLI command
+        run: |
+          aws-sso-auditor --help

cpk_lib_python_aws/tests/__init__.py

@@ -0,0 +1,80 @@
+import os
+import pytest
+from cpk_lib_python_aws.aws_sso_auditor.config import Config
+from cpk_lib_python_aws.aws_sso_auditor.exceptions import ConfigurationError
+
+
+def test_default_config_values():
+    """Test that default configuration values are set correctly."""
+    config = Config()
+    assert config.aws_region == "us-east-1"
+    assert config.output_formats == ["json", "yaml"]
+    assert config.output_directory == "."
+    assert config.include_timestamp is True
+    assert config.debug is False
+    assert config.quiet is False
+    assert config.timeout == 30
+    assert config.aws_profile is None
+
+
+def test_config_validation_valid_formats():
+    """Test that valid output formats pass validation."""
+    config = Config(output_formats=["json"])
+    config.validate()  # Should not raise
+    
+    config = Config(output_formats=["yaml"])
+    config.validate()  # Should not raise
+    
+    config = Config(output_formats=["both"])
+    config.validate()  # Should not raise
+
+
+def test_config_validation_invalid_format():
+    """Test that invalid output formats raise ConfigurationError."""
+    config = Config(output_formats=["invalid"])
+    with pytest.raises(ConfigurationError, match="Invalid output format: invalid"):
+        config.validate()
+
+
+def test_config_validation_invalid_timeout():
+    """Test that invalid timeout raises ConfigurationError."""
+    config = Config(timeout=0)
+    with pytest.raises(ConfigurationError, match="Timeout must be greater than 0"):
+        config.validate()
+    
+    config = Config(timeout=-5)
+    with pytest.raises(ConfigurationError, match="Timeout must be greater than 0"):
+        config.validate()
+
+
+def test_environment_variable_override():
+    """Test that environment variables override default values."""
+    # Set environment variables
+    os.environ["AWS_REGION"] = "eu-west-1"
+    os.environ["AWS_SSO_AUDITOR_DEBUG"] = "true"
+    os.environ["AWS_SSO_AUDITOR_QUIET"] = "true"
+    
+    try:
+        config = Config()
+        assert config.aws_region == "eu-west-1"
+        assert config.debug is True
+        assert config.quiet is True
+    finally:
+        # Clean up environment variables
+        os.environ.pop("AWS_REGION", None)
+        os.environ.pop("AWS_SSO_AUDITOR_DEBUG", None)
+        os.environ.pop("AWS_SSO_AUDITOR_QUIET", None)
+
+
+def test_constructor_overrides():
+    """Test that constructor parameters override defaults."""
+    config = Config(
+        aws_region="ap-southeast-1",
+        output_directory="/tmp/test",
+        debug=True,
+        timeout=60
+    )
+    assert config.aws_region == "ap-southeast-1"
+    assert config.output_directory == "/tmp/test"
+    assert config.debug is True
+    assert config.timeout == 60

cpk_lib_python_aws/tests/aws_sso_auditor/__init__.py

@@ -21,6 +21,11 @@ dependencies = [
 ]
 
 [project.optional-dependencies]
+test = [
+    "pytest>=7.0.0",
+    "pytest-cov>=4.0.0",
+    "pytest-mock>=3.10.0",
+]
 dev = [
     "pytest>=7.0.0",
     "pytest-cov>=4.0.0",
@@ -94,3 +99,25 @@ ignored-modules = [
 [tool.pylint.imports]
 # Allow relative imports within the package
 allow-any-import-level = true
+
+
+[tool.pytest.ini_options]
+testpaths = [
+    "cpk_lib_python_aws/tests"
+]
+python_files = ["test_*.py"]
+python_classes = ["Test*"]
+python_functions = ["test_*"]
+addopts = [
+    "--strict-markers",
+    "--strict-config",
+    "--verbose",
+    "--cov=cpk_lib_python_aws",
+    "--cov-report=term-missing",
+    "--cov-report=html"
+]
+markers = [
+    "unit: Unit tests with mocked responses",
+    "integration: Integration tests with real AWS API calls",
+    "slow: Slow running tests"
+]