feat: update lint

Author: colyerdeng

tests/components/test_github_actions.py

@@ -37,7 +37,7 @@ def test_github_actions_component_exists(github_actions_component):
 
 def test_github_actions_dependencies(github_actions_component):
     """Test that GitHub Actions has correct dependencies."""
-    expected_dependencies = {'task_automation'}
+    expected_dependencies = {"task_automation"}
     assert set(github_actions_component.dependencies) == expected_dependencies
 
 
@@ -90,9 +90,6 @@ def test_ci_tests_workflow_content(components_dir):
     assert "./docker/Dockerfile" not in content
 
 
-
-
-
 def test_version_bump_workflow_content(components_dir):
     """Test version bump workflow content."""
     workflow_path = components_dir / "github_actions" / "files" / ".github" / "workflows" / "version-bump.yaml.j2"

tests/components/test_pre_commit.py

@@ -24,21 +24,21 @@ def component_manager(components_dir):
 def pre_commit_component(component_manager):
     """Get the Pre-commit component."""
     components = component_manager._discover_components()
-    return components.get('pre_commit')
+    return components.get("pre_commit")
 
 
 def test_pre_commit_component_exists(pre_commit_component):
     """Test that Pre-commit component exists and has correct metadata."""
     assert pre_commit_component is not None
-    assert pre_commit_component.name == 'pre_commit'
-    assert pre_commit_component.display_name == 'Pre-commit Hooks'
-    assert 'Pre-commit hooks' in pre_commit_component.description
-    assert pre_commit_component.category == 'quality'
+    assert pre_commit_component.name == "pre_commit"
+    assert pre_commit_component.display_name == "Pre-commit Hooks"
+    assert "Pre-commit hooks" in pre_commit_component.description
+    assert pre_commit_component.category == "quality"
 
 
 def test_pre_commit_dependencies(pre_commit_component):
     """Test that Pre-commit has correct dependencies."""
-    expected_dependencies = {'task_automation', 'github_actions'}
+    expected_dependencies = {"task_automation", "github_actions"}
     assert set(pre_commit_component.dependencies) == expected_dependencies
 
 
@@ -49,9 +49,7 @@ def test_pre_commit_no_conflicts(pre_commit_component):
 
 def test_pre_commit_cookiecutter_vars(pre_commit_component):
     """Test that Pre-commit sets correct cookiecutter variables."""
-    expected_vars = {
-        'use_pre_commit': True
-    }
+    expected_vars = {"use_pre_commit": True}
 
     for key, expected_value in expected_vars.items():
         assert key in pre_commit_component.cookiecutter_vars
@@ -60,11 +58,9 @@ def test_pre_commit_cookiecutter_vars(pre_commit_component):
 
 def test_pre_commit_files(pre_commit_component):
     """Test that Pre-commit component has correct files."""
-    expected_files = {
-        '.pre-commit-config.yaml'
-    }
+    expected_files = {".pre-commit-config.yaml"}
 
-    actual_files = {file_config['dest'] for file_config in pre_commit_component.files}
+    actual_files = {file_config["dest"] for file_config in pre_commit_component.files}
     assert actual_files == expected_files
 
 
@@ -73,108 +69,108 @@ def test_pre_commit_config_content(components_dir):
     config_path = components_dir / "pre_commit" / "files" / ".pre-commit-config.yaml.j2"
     assert config_path.exists()
 
-    content = config_path.read_text(encoding='utf-8')
+    content = config_path.read_text(encoding="utf-8")
 
     # Check for conditional uv usage
-    assert 'use_uv' in content
-    assert 'uv-pre-commit' in content
-    assert 'uv-lock' in content
+    assert "use_uv" in content
+    assert "uv-pre-commit" in content
+    assert "uv-lock" in content
 
     # Check for conditional GitHub Actions integration
-    assert 'use_github_actions' in content
-    assert 'check-github-workflows' in content
-    assert 'actionlint' in content
+    assert "use_github_actions" in content
+    assert "check-github-workflows" in content
+    assert "actionlint" in content
 
     # Check for YAML formatting
-    assert 'yamlfmt' in content
+    assert "yamlfmt" in content
 
 
 def test_component_yaml_syntax(components_dir):
     """Test that component.yaml has valid syntax."""
     component_yaml = components_dir / "pre_commit" / "component.yaml"
     assert component_yaml.exists()
 
-    with open(component_yaml, encoding='utf-8') as f:
+    with open(component_yaml, encoding="utf-8") as f:
         config = yaml.safe_load(f)
 
     # Validate required fields
-    required_fields = ['name', 'display_name', 'description', 'category', 'dependencies', 'files']
+    required_fields = ["name", "display_name", "description", "category", "dependencies", "files"]
     for field in required_fields:
         assert field in config
 
     # Validate dependencies
-    assert 'task_automation' in config['dependencies']
-    assert 'github_actions' in config['dependencies']
+    assert "task_automation" in config["dependencies"]
+    assert "github_actions" in config["dependencies"]
 
     # Validate files structure
-    for file_config in config['files']:
-        assert 'src' in file_config
-        assert 'dest' in file_config
+    for file_config in config["files"]:
+        assert "src" in file_config
+        assert "dest" in file_config
 
 
 def test_file_templates_exist(components_dir):
     """Test that all referenced template files exist."""
     component_yaml = components_dir / "pre_commit" / "component.yaml"
 
-    with open(component_yaml, encoding='utf-8') as f:
+    with open(component_yaml, encoding="utf-8") as f:
         config = yaml.safe_load(f)
 
     files_dir = components_dir / "pre_commit" / "files"
 
-    for file_config in config['files']:
-        src_file = files_dir / file_config['src']
+    for file_config in config["files"]:
+        src_file = files_dir / file_config["src"]
         assert src_file.exists(), f"Template file not found: {src_file}"
 
 
 def test_uv_integration(components_dir):
     """Test that pre-commit config integrates with uv."""
     config_path = components_dir / "pre_commit" / "files" / ".pre-commit-config.yaml.j2"
-    content = config_path.read_text(encoding='utf-8')
+    content = config_path.read_text(encoding="utf-8")
 
     # Should include uv-specific hooks
-    assert 'uv-pre-commit' in content
-    assert 'uv-lock' in content
+    assert "uv-pre-commit" in content
+    assert "uv-lock" in content
 
 
 def test_conditional_uv_usage(components_dir):
     """Test conditional uv usage in pre-commit hooks."""
     config_path = components_dir / "pre_commit" / "files" / ".pre-commit-config.yaml.j2"
-    content = config_path.read_text(encoding='utf-8')
+    content = config_path.read_text(encoding="utf-8")
 
     # Should have conditional blocks for uv
-    assert 'if cookiecutter.use_uv' in content
-    assert 'uv-lock' in content
+    assert "if cookiecutter.use_uv" in content
+    assert "uv-lock" in content
 
     # Should have conditional ending
-    assert 'endif' in content
+    assert "endif" in content
 
 
 def test_github_actions_integration(components_dir):
     """Test GitHub Actions integration in pre-commit config."""
     config_path = components_dir / "pre_commit" / "files" / ".pre-commit-config.yaml.j2"
-    content = config_path.read_text(encoding='utf-8')
+    content = config_path.read_text(encoding="utf-8")
 
     # Should have conditional GitHub Actions specific hooks
-    assert 'if cookiecutter.use_github_actions' in content
-    assert 'check-github-workflows' in content
-    assert 'actionlint' in content
+    assert "if cookiecutter.use_github_actions" in content
+    assert "check-github-workflows" in content
+    assert "actionlint" in content
 
 
 def test_yamlfmt_integration(components_dir):
     """Test YAML formatting integration."""
     config_path = components_dir / "pre_commit" / "files" / ".pre-commit-config.yaml.j2"
-    content = config_path.read_text(encoding='utf-8')
+    content = config_path.read_text(encoding="utf-8")
 
     # Should include yamlfmt for YAML formatting
-    assert 'yamlfmt' in content
-    assert 'google/yamlfmt' in content
+    assert "yamlfmt" in content
+    assert "google/yamlfmt" in content
 
 
 def test_actionlint_integration(components_dir):
     """Test GitHub Actions linting integration."""
     config_path = components_dir / "pre_commit" / "files" / ".pre-commit-config.yaml.j2"
-    content = config_path.read_text(encoding='utf-8')
+    content = config_path.read_text(encoding="utf-8")
 
     # Should include actionlint for GitHub Actions validation
-    assert 'actionlint' in content
-    assert 'rhysd/actionlint' in content
+    assert "actionlint" in content
+    assert "rhysd/actionlint" in content