@pennyfarthing/core 11.3.3 → 11.3.5

package/{pennyfarthing_scripts → pennyfarthing-dist/pf}/tests/test_healthscore.py

@@ -23,15 +23,15 @@ from unittest.mock import patch
 
 from click.testing import CliRunner
 
-from pennyfarthing_scripts.healthscore.analyze import (
+from pf.healthscore.analyze import (
     analyze_healthscore,
     compute_composite_score,
     get_cache_path,
     read_cached_score,
     write_cached_score,
 )
-from pennyfarthing_scripts.healthscore.cli import healthscore
-from pennyfarthing_scripts.healthscore.models import (
+from pf.healthscore.cli import healthscore
+from pf.healthscore.models import (
     DEFAULT_WEIGHTS,
     DimensionScore,
     HealthscoreResult,
@@ -42,35 +42,35 @@ from pennyfarthing_scripts.healthscore.models import (
 # ---------------------------------------------------------------------------
 
 class TestModuleStructure:
-    """AC1: New module at pennyfarthing_scripts/healthscore/ with standard files."""
+    """AC1: New module at pf/healthscore/ with standard files."""
 
     def test_package_has_init(self):
         """Module must be importable as a package."""
-        import pennyfarthing_scripts.healthscore
-        assert hasattr(pennyfarthing_scripts.healthscore, "HealthscoreResult")
-        assert hasattr(pennyfarthing_scripts.healthscore, "DimensionScore")
-        assert hasattr(pennyfarthing_scripts.healthscore, "DEFAULT_WEIGHTS")
-        assert hasattr(pennyfarthing_scripts.healthscore, "analyze_healthscore")
+        import pf.healthscore
+        assert hasattr(pf.healthscore, "HealthscoreResult")
+        assert hasattr(pf.healthscore, "DimensionScore")
+        assert hasattr(pf.healthscore, "DEFAULT_WEIGHTS")
+        assert hasattr(pf.healthscore, "analyze_healthscore")
 
     def test_models_module_exists(self):
         """models.py must define DimensionScore and HealthscoreResult."""
-        from pennyfarthing_scripts.healthscore import models
+        from pf.healthscore import models
         assert hasattr(models, "DimensionScore")
         assert hasattr(models, "HealthscoreResult")
 
     def test_analyze_module_exists(self):
         """analyze.py must define analyze_healthscore."""
-        from pennyfarthing_scripts.healthscore import analyze
+        from pf.healthscore import analyze
         assert hasattr(analyze, "analyze_healthscore")
 
     def test_cli_module_exists(self):
         """cli.py must define the click group."""
-        from pennyfarthing_scripts.healthscore import cli
+        from pf.healthscore import cli
         assert hasattr(cli, "healthscore")
 
     def test_formatters_module_exists(self):
         """formatters.py must define table/json/csv formatters."""
-        from pennyfarthing_scripts.healthscore import formatters
+        from pf.healthscore import formatters
         assert hasattr(formatters, "format_table")
         assert hasattr(formatters, "export_json")
         assert hasattr(formatters, "export_csv")
@@ -239,7 +239,7 @@ class TestCLI:
             ],
         )
         with patch(
-            "pennyfarthing_scripts.healthscore.cli._run_analysis",
+            "pf.healthscore.cli._run_analysis",
             return_value=mock_result,
         ):
             runner = CliRunner()
@@ -261,7 +261,7 @@ class TestCLI:
             ],
         )
         with patch(
-            "pennyfarthing_scripts.healthscore.cli._run_analysis",
+            "pf.healthscore.cli._run_analysis",
             return_value=mock_result,
         ):
             runner = CliRunner()
@@ -280,7 +280,7 @@ class TestCLI:
             ],
         )
         with patch(
-            "pennyfarthing_scripts.healthscore.cli._run_analysis",
+            "pf.healthscore.cli._run_analysis",
             return_value=mock_result,
         ):
             runner = CliRunner()
@@ -299,7 +299,7 @@ class TestCLI:
         )
         output_file = tmp_path / "result.json"
         with patch(
-            "pennyfarthing_scripts.healthscore.cli._run_analysis",
+            "pf.healthscore.cli._run_analysis",
             return_value=mock_result,
         ):
             runner = CliRunner()
@@ -363,7 +363,7 @@ class TestCaching:
         # This tests that the analyze function respects cache_ttl=0
         # (will fail until implementation — that's the point)
         with patch(
-            "pennyfarthing_scripts.healthscore.analyze.read_cached_score",
+            "pf.healthscore.analyze.read_cached_score",
             return_value=99.0,
         ) as mock_read:
             asyncio.run(
@@ -471,11 +471,11 @@ class TestADR0008Pattern:
 # ---------------------------------------------------------------------------
 
 class TestMainCLIRegistration:
-    """AC8: healthscore command registered in pennyfarthing_scripts/cli.py."""
+    """AC8: healthscore command registered in pf/cli.py."""
 
     def test_healthscore_registered_in_main_cli(self):
         """Main CLI must have a 'healthscore' command group."""
-        from pennyfarthing_scripts.cli import cli
+        from pf.cli import cli
         command_names = list(cli.commands)
         assert "healthscore" in command_names
 

package/{pennyfarthing_scripts → pennyfarthing-dist/pf}/tests/test_jira_package.py

@@ -1,6 +1,6 @@
 """Tests for jira/ library package.
 
-Story 63-9: Reorganize pennyfarthing_scripts into fan-out CLI pattern.
+Story 63-9: Reorganize pf into fan-out CLI pattern.
 
 These tests verify the jira/ package modules work correctly
 after reorganization from flat modules.
@@ -14,7 +14,7 @@ class TestJiraClient:
 
     def test_jira_client_initialization(self) -> None:
         """JiraClient should initialize with default or custom values."""
-        from pennyfarthing_scripts.jira.client import JiraClient
+        from pf.jira.client import JiraClient
 
         # Default initialization
         client = JiraClient()
@@ -33,7 +33,7 @@ class TestJiraClient:
 
     def test_get_auth_header_returns_basic_auth(self) -> None:
         """_get_auth_header should return Basic auth header."""
-        from pennyfarthing_scripts.jira.client import JiraClient
+        from pf.jira.client import JiraClient
 
         client = JiraClient(
             user="user@example.com",
@@ -46,7 +46,7 @@ class TestJiraClient:
 
     def test_get_auth_header_empty_without_token(self) -> None:
         """_get_auth_header should return empty dict without token."""
-        from pennyfarthing_scripts.jira.client import JiraClient
+        from pf.jira.client import JiraClient
 
         client = JiraClient(token="")
         headers = client._get_auth_header()
@@ -55,7 +55,7 @@ class TestJiraClient:
 
     def test_get_headers_includes_accept(self) -> None:
         """_get_headers should include Accept header."""
-        from pennyfarthing_scripts.jira.client import JiraClient
+        from pf.jira.client import JiraClient
 
         client = JiraClient(token="test")
         headers = client._get_headers()
@@ -65,7 +65,7 @@ class TestJiraClient:
 
     def test_get_headers_includes_content_type_when_requested(self) -> None:
         """_get_headers should include Content-Type when content_type=True."""
-        from pennyfarthing_scripts.jira.client import JiraClient
+        from pf.jira.client import JiraClient
 
         client = JiraClient(token="test")
         headers = client._get_headers(content_type=True)
@@ -79,7 +79,7 @@ class TestStatusMappings:
 
     def test_map_status_to_jira_known_status(self) -> None:
         """map_status_to_jira should map known statuses correctly."""
-        from pennyfarthing_scripts.jira.client import map_status_to_jira
+        from pf.jira.client import map_status_to_jira
 
         assert map_status_to_jira("backlog") == "To Do"
         assert map_status_to_jira("in-progress") == "In Progress"
@@ -89,14 +89,14 @@ class TestStatusMappings:
 
     def test_map_status_to_jira_unknown_defaults_to_todo(self) -> None:
         """map_status_to_jira should default to 'To Do' for unknown."""
-        from pennyfarthing_scripts.jira.client import map_status_to_jira
+        from pf.jira.client import map_status_to_jira
 
         assert map_status_to_jira("unknown_status") == "To Do"
         assert map_status_to_jira(None) == "To Do"
 
     def test_map_jira_to_status_known_status(self) -> None:
         """map_jira_to_status should map known Jira statuses correctly."""
-        from pennyfarthing_scripts.jira.client import map_jira_to_status
+        from pf.jira.client import map_jira_to_status
 
         assert map_jira_to_status("To Do") == "backlog"
         assert map_jira_to_status("In Progress") == "in_progress"
@@ -105,7 +105,7 @@ class TestStatusMappings:
 
     def test_map_jira_to_status_unknown_defaults_to_backlog(self) -> None:
         """map_jira_to_status should default to 'backlog' for unknown."""
-        from pennyfarthing_scripts.jira.client import map_jira_to_status
+        from pf.jira.client import map_jira_to_status
 
         assert map_jira_to_status("Unknown Status") == "backlog"
         assert map_jira_to_status(None) == "backlog"
@@ -116,20 +116,20 @@ class TestExtractJiraKey:
 
     def test_extract_jira_key_from_key(self) -> None:
         """extract_jira_key should return key as-is if already a key."""
-        from pennyfarthing_scripts.jira.client import extract_jira_key
+        from pf.jira.client import extract_jira_key
 
         assert extract_jira_key("MSSCI-12345") == "MSSCI-12345"
 
     def test_extract_jira_key_from_url(self) -> None:
         """extract_jira_key should extract key from URL."""
-        from pennyfarthing_scripts.jira.client import extract_jira_key
+        from pf.jira.client import extract_jira_key
 
         url = "https://1898andco.atlassian.net/browse/MSSCI-12345"
         assert extract_jira_key(url) == "MSSCI-12345"
 
     def test_extract_jira_key_returns_none_for_none(self) -> None:
         """extract_jira_key should return None for None input."""
-        from pennyfarthing_scripts.jira.client import extract_jira_key
+        from pf.jira.client import extract_jira_key
 
         assert extract_jira_key(None) is None
 
@@ -139,14 +139,14 @@ class TestGetJiraField:
 
     def test_get_jira_field_simple_path(self) -> None:
         """get_jira_field should extract simple field paths."""
-        from pennyfarthing_scripts.jira.client import get_jira_field
+        from pf.jira.client import get_jira_field
 
         issue = {"key": "MSSCI-123", "id": "10001"}
         assert get_jira_field(issue, "key") == "MSSCI-123"
 
     def test_get_jira_field_nested_path(self) -> None:
         """get_jira_field should extract nested field paths."""
-        from pennyfarthing_scripts.jira.client import get_jira_field
+        from pf.jira.client import get_jira_field
 
         issue = {
             "fields": {
@@ -159,7 +159,7 @@ class TestGetJiraField:
 
     def test_get_jira_field_returns_default_if_missing(self) -> None:
         """get_jira_field should return default for missing paths."""
-        from pennyfarthing_scripts.jira.client import get_jira_field
+        from pf.jira.client import get_jira_field
 
         issue = {"key": "MSSCI-123"}
         assert get_jira_field(issue, "fields.missing", "default") == "default"
@@ -167,7 +167,7 @@ class TestGetJiraField:
 
     def test_get_jira_field_handles_none_input(self) -> None:
         """get_jira_field should handle None issue input."""
-        from pennyfarthing_scripts.jira.client import get_jira_field
+        from pf.jira.client import get_jira_field
 
         assert get_jira_field(None, "key", "default") == "default"
 
@@ -177,7 +177,7 @@ class TestJiraSyncModule:
 
     def test_sync_result_dataclass(self) -> None:
         """SyncResult should be a valid dataclass."""
-        from pennyfarthing_scripts.jira.sync import SyncResult
+        from pf.jira.sync import SyncResult
 
         result = SyncResult(
             story_id="63-1",
@@ -193,7 +193,7 @@ class TestJiraSyncModule:
 
     def test_format_story_line(self) -> None:
         """format_story_line should format story for display."""
-        from pennyfarthing_scripts.jira.sync import format_story_line
+        from pf.jira.sync import format_story_line
 
         story = {"id": "63-1", "title": "Test Story", "status": "in_progress"}
         result = format_story_line(story)
@@ -204,7 +204,7 @@ class TestJiraSyncModule:
 
     def test_format_summary(self) -> None:
         """format_summary should format sync summary."""
-        from pennyfarthing_scripts.jira.sync import format_summary
+        from pf.jira.sync import format_summary
 
         result = format_summary(synced=5, skipped=2, errors=1)
 
@@ -218,7 +218,7 @@ class TestJiraBidirectionalModule:
 
     def test_sync_change_dataclass(self) -> None:
         """SyncChange should be a valid dataclass."""
-        from pennyfarthing_scripts.jira.bidirectional import SyncChange
+        from pf.jira.bidirectional import SyncChange
 
         change = SyncChange(
             key="MSSCI-12345",
@@ -233,7 +233,7 @@ class TestJiraBidirectionalModule:
 
     def test_sync_plan_dataclass(self) -> None:
         """SyncPlan should be a valid dataclass."""
-        from pennyfarthing_scripts.jira.bidirectional import SyncPlan
+        from pf.jira.bidirectional import SyncPlan
 
         plan = SyncPlan()
         assert plan.changes == []
@@ -243,7 +243,7 @@ class TestJiraBidirectionalModule:
 
     def test_generate_sync_plan_empty_inputs(self) -> None:
         """generate_sync_plan should handle empty inputs."""
-        from pennyfarthing_scripts.jira.bidirectional import generate_sync_plan
+        from pf.jira.bidirectional import generate_sync_plan
 
         plan = generate_sync_plan([], [], sync_status=True)
 
@@ -253,7 +253,7 @@ class TestJiraBidirectionalModule:
 
     def test_generate_sync_plan_identifies_yaml_only(self) -> None:
         """generate_sync_plan should identify stories only in YAML."""
-        from pennyfarthing_scripts.jira.bidirectional import generate_sync_plan
+        from pf.jira.bidirectional import generate_sync_plan
 
         yaml_stories = [{"id": "63-1", "jira": "MSSCI-12345", "status": "in_progress"}]
         jira_stories: list[dict[str, Any]] = []
@@ -264,7 +264,7 @@ class TestJiraBidirectionalModule:
 
     def test_generate_sync_plan_identifies_jira_only(self) -> None:
         """generate_sync_plan should identify stories only in Jira."""
-        from pennyfarthing_scripts.jira.bidirectional import generate_sync_plan
+        from pf.jira.bidirectional import generate_sync_plan
 
         yaml_stories: list[dict[str, Any]] = []
         jira_stories = [{"key": "MSSCI-12345", "fields": {"status": {"name": "To Do"}}}]
@@ -275,7 +275,7 @@ class TestJiraBidirectionalModule:
 
     def test_format_sync_plan(self) -> None:
         """format_sync_plan should return formatted string."""
-        from pennyfarthing_scripts.jira.bidirectional import (
+        from pf.jira.bidirectional import (
             SyncPlan,
             format_sync_plan,
         )
@@ -297,7 +297,7 @@ class TestJiraEpicModule:
 
     def test_build_epic_payload(self) -> None:
         """build_epic_payload should create valid Jira API payload."""
-        from pennyfarthing_scripts.jira.epic import build_epic_payload
+        from pf.jira.epic import build_epic_payload
 
         epic_data = {"title": "Test Epic", "description": "Epic description"}
         payload = build_epic_payload(epic_data)
@@ -309,7 +309,7 @@ class TestJiraEpicModule:
 
     def test_create_epic_dry_run(self) -> None:
         """create_epic with dry_run should not call API."""
-        from pennyfarthing_scripts.jira.epic import create_epic
+        from pf.jira.epic import create_epic
 
         result = create_epic("Test Epic", "Description", dry_run=True)
 
@@ -323,7 +323,7 @@ class TestJiraStoryModule:
 
     def test_sync_story_missing_story(self) -> None:
         """sync_story should return error for missing story."""
-        from pennyfarthing_scripts.jira.story import sync_story
+        from pf.jira.story import sync_story
 
         result = sync_story("nonexistent-99", do_transition=False, dry_run=True)