atdd 0.2.1__py3-none-any.whl

atdd/tester/validators/test_acceptance_urn_filename_mapping.py

@@ -0,0 +1,359 @@
+"""
+Platform validation: Test filename generation from acceptance URNs.
+
+Validates that actual test files in the repository follow the URN-based
+naming convention for all supported languages.
+
+Spec: SPEC-TESTER-CONV-0069 through SPEC-TESTER-CONV-0078
+URN: acc:coach:platform-validation.filename-mapping
+"""
+
+import os
+import re
+from pathlib import Path
+import pytest
+
+
+REPO_ROOT = Path(__file__).parent.parent.parent
+
+
+# SPEC-TESTER-CONV-0069: Dart URN to filename mapping
+def test_dart_urn_to_filename():
+    """Test Dart filename generation from URN."""
+    from atdd.tester.utils.filename import dart_filename
+
+    test_cases = [
+        ("acc:maintain-ux:C004-E2E-019-user-connection", "C004_E2E_019_user_connection_test.dart"),
+        ("acc:maintain-ux:C004-UNIT-001", "C004_UNIT_001_test.dart"),
+        ("acc:stage-characters:K005-WIDGET-012-avatar-display", "K005_WIDGET_012_avatar_display_test.dart"),
+    ]
+
+    for urn, expected in test_cases:
+        assert dart_filename(urn) == expected
+
+
+# SPEC-TESTER-CONV-0070: TypeScript URN to filename mapping
+def test_typescript_urn_to_filename():
+    """Test TypeScript filename generation from URN."""
+    from atdd.tester.utils.filename import typescript_filename
+
+    test_cases = [
+        ("acc:maintain-ux:C004-E2E-019-user-connection", "c004-e2e-019-user-connection.test.ts"),
+        ("acc:maintain-ux:C004-HTTP-001", "c004-http-001.test.ts"),
+        ("acc:resolve-dilemmas:R007-EVENT-005-choice-made", "r007-event-005-choice-made.test.ts"),
+    ]
+
+    for urn, expected in test_cases:
+        assert typescript_filename(urn) == expected
+
+
+# SPEC-TESTER-CONV-0071: Python URN to filename mapping
+def test_python_urn_to_filename():
+    """Test Python filename generation from URN."""
+    from atdd.tester.utils.filename import python_filename
+
+    test_cases = [
+        ("acc:maintain-ux:C004-E2E-019-user-connection", "test_c004_e2e_019_user_connection.py"),
+        ("acc:commit-state:D001-UNIT-042", "test_d001_unit_042.py"),
+        ("acc:construct-graph:M008-INTEGRATION-007-graph-build", "test_m008_integration_007_graph_build.py"),
+    ]
+
+    for urn, expected in test_cases:
+        assert python_filename(urn) == expected
+
+
+# SPEC-TESTER-CONV-0072: Go URN to filename mapping
+def test_go_urn_to_filename():
+    """Test Go filename generation from URN."""
+    from atdd.tester.utils.filename import go_filename
+
+    test_cases = [
+        ("acc:maintain-ux:C004-E2E-019-user-connection", "c004_e2e_019_user_connection_test.go"),
+        ("acc:commit-state:D001-UNIT-042", "d001_unit_042_test.go"),
+    ]
+
+    for urn, expected in test_cases:
+        assert go_filename(urn) == expected
+
+
+# SPEC-TESTER-CONV-0073: Java/Kotlin URN to classname mapping
+def test_java_urn_to_classname():
+    """Test Java/Kotlin classname generation from URN."""
+    from atdd.tester.utils.filename import java_classname
+
+    test_cases = [
+        ("acc:maintain-ux:C004-E2E-019-user-connection", "C004E2E019UserConnectionTest"),
+        ("acc:commit-state:D001-UNIT-042", "D001UNIT042Test"),
+        ("acc:stage-characters:K005-INTEGRATION-003-full-flow", "K005INTEGRATION003FullFlowTest"),
+    ]
+
+    for urn, expected in test_cases:
+        assert java_classname(urn) == expected
+
+
+# SPEC-TESTER-CONV-0074: Handle URNs without slug
+def test_no_slug_handling():
+    """Test filename generation for URNs without optional slug."""
+    from atdd.tester.utils.filename import dart_filename, typescript_filename, python_filename
+
+    urn = "acc:maintain-ux:C004-E2E-019"
+
+    assert dart_filename(urn) == "C004_E2E_019_test.dart"
+    assert typescript_filename(urn) == "c004-e2e-019.test.ts"
+    assert python_filename(urn) == "test_c004_e2e_019.py"
+
+
+def test_typescript_preact_urn_to_filename():
+    """Test Preact TypeScript filename generation from URN."""
+    from atdd.tester.utils.filename import typescript_preact_filename
+
+    test_cases = [
+        ("acc:maintain-ux:C004-E2E-019-user-connection", "C004_E2E_019_user_connection.test.ts"),
+        ("acc:maintain-ux:C004-HTTP-001", "C004_HTTP_001.test.ts"),
+        ("acc:maintain-ux:C001-WIDGET-001-button", "C001_WIDGET_001_button.test.tsx"),
+    ]
+
+    for urn, expected in test_cases:
+        assert typescript_preact_filename(urn) == expected
+
+
+# SPEC-TESTER-CONV-0076: URN pattern validation
+def test_urn_pattern_validation():
+    """Test URN regex pattern matches valid acceptance URNs."""
+    from atdd.tester.utils.filename import URN_PATTERN
+
+    valid_urns = [
+        "acc:maintain-ux:C004-E2E-019-user-connection",
+        "acc:pace-dilemmas:P003-UNIT-042",
+        "acc:predict-cascade:L009-HTTP-001-api-endpoint",
+        "acc:commit-state:D001-EVENT-005",
+    ]
+
+    for urn in valid_urns:
+        assert re.match(URN_PATTERN, urn), f"Should match: {urn}"
+
+    invalid_urns = [
+        "acc:maintain_ux:C004-E2E-019",  # Underscore not allowed
+        "wmbt:maintain-ux:C004-E2E-019",  # Wrong prefix
+        "acc:maintain-ux:C4-E2E-019",     # WMBT not zero-padded
+    ]
+
+    for urn in invalid_urns:
+        assert not re.match(URN_PATTERN, urn), f"Should not match: {urn}"
+
+
+# SPEC-TESTER-CONV-0078: Platform validation enforces filename convention compliance
+def test_dart_files_match_convention():
+    """Validate Dart test files follow URN-based naming convention."""
+    from atdd.tester.utils.filename import parse_acceptance_urn, dart_filename
+
+    # Pattern: {WMBT}_{HARNESS}_{NNN}[_{slug}]_test.dart
+    dart_pattern = re.compile(r'^([A-Z][0-9]{3})_([A-Z0-9]+)_([0-9]{3})(?:_([a-z0-9_]+))?_test\.dart$')
+
+    test_dirs = [
+        REPO_ROOT / "test",
+        REPO_ROOT / "integration_test",
+    ]
+
+    violations = []
+
+    for test_dir in test_dirs:
+        if not test_dir.exists():
+            continue
+
+        for dart_file in test_dir.rglob("*_test.dart"):
+            filename = dart_file.name
+
+            # Check pattern compliance
+            match = dart_pattern.match(filename)
+            if not match:
+                # Check if there's a URN comment to determine expected filename
+                content = dart_file.read_text()
+                urn_match = re.search(r'// urn: (acc:[a-z][a-z0-9-]*:[A-Z][0-9]{3}-[A-Z0-9]+-[0-9]{3}(?:-[a-z0-9-]+)?)', content)
+
+                if urn_match:
+                    urn = urn_match.group(1)
+                    expected = dart_filename(urn)
+                    violations.append({
+                        'file': str(dart_file.relative_to(REPO_ROOT)),
+                        'actual': filename,
+                        'expected': expected,
+                        'urn': urn
+                    })
+
+    # Allow some violations during migration, but report them
+    if violations:
+        report = "\n".join([
+            f"  {v['file']}: {v['actual']} → {v['expected']} (URN: {v['urn']})"
+            for v in violations[:5]  # Show first 5
+        ])
+        print(f"\nFilename convention violations found:\n{report}")
+
+    # For now, just log violations without failing
+    # In the future, this should assert len(violations) == 0
+    assert True, "Validation complete (violations logged)"
+
+
+def test_typescript_files_match_convention():
+    """Validate TypeScript test files follow URN-based naming convention."""
+    from atdd.tester.utils.filename import typescript_filename
+
+    # Pattern: {wmbt}-{harness}-{nnn}[-{slug}].test.ts
+    ts_pattern = re.compile(r'^([a-z][0-9]{3})-([a-z0-9]+)-([0-9]{3})(?:-([a-z0-9-]+))?\.test\.ts$')
+
+    test_dirs = [
+        REPO_ROOT / "supabase" / "functions",
+        REPO_ROOT / "e2e",
+    ]
+
+    violations = []
+
+    for test_dir in test_dirs:
+        if not test_dir.exists():
+            continue
+
+        for ts_file in test_dir.rglob("*.test.ts"):
+            filename = ts_file.name
+            match = ts_pattern.match(filename)
+
+            if not match:
+                content = ts_file.read_text()
+                urn_match = re.search(r'// urn: (acc:[a-z][a-z0-9-]*:[A-Z][0-9]{3}-[A-Z0-9]+-[0-9]{3}(?:-[a-z0-9-]+)?)', content)
+
+                if urn_match:
+                    urn = urn_match.group(1)
+                    expected = typescript_filename(urn)
+                    violations.append({
+                        'file': str(ts_file.relative_to(REPO_ROOT)),
+                        'actual': filename,
+                        'expected': expected,
+                        'urn': urn
+                    })
+
+    if violations:
+        report = "\n".join([
+            f"  {v['file']}: {v['actual']} → {v['expected']} (URN: {v['urn']})"
+            for v in violations[:5]
+        ])
+        print(f"\nFilename convention violations found:\n{report}")
+
+    assert True, "Validation complete (violations logged)"
+
+
+def test_typescript_preact_files_match_convention():
+    """Validate Preact TypeScript test files follow URN-based naming convention."""
+    from atdd.tester.utils.filename import typescript_preact_filename
+
+    ts_pattern = re.compile(r'^([A-Z][0-9]{3})_([A-Z0-9]+)_([0-9]{3})(?:_([a-z0-9_]+))?\.test\.ts(x)?$')
+
+    test_dirs = [
+        REPO_ROOT / "web" / "tests",
+    ]
+
+    violations = []
+
+    for test_dir in test_dirs:
+        if not test_dir.exists():
+            continue
+
+        ts_files = list(test_dir.rglob("*.test.ts")) + list(test_dir.rglob("*.test.tsx"))
+        for ts_file in ts_files:
+            filename = ts_file.name
+            match = ts_pattern.match(filename)
+
+            if not match:
+                content = ts_file.read_text()
+                urn_match = re.search(
+                    r'//\s*(?:urn|URN):\s*(acc:[a-z][a-z0-9-]*:[A-Z][0-9]{3}-[A-Z0-9]+-[0-9]{3}(?:-[a-z0-9-]+)?)',
+                    content
+                )
+
+                if urn_match:
+                    urn = urn_match.group(1)
+                    expected = typescript_preact_filename(urn)
+                    violations.append({
+                        'file': str(ts_file.relative_to(REPO_ROOT)),
+                        'actual': filename,
+                        'expected': expected,
+                        'urn': urn
+                    })
+
+    if violations:
+        report = "\n".join([
+            f"  {v['file']}: {v['actual']} → {v['expected']} (URN: {v['urn']})"
+            for v in violations[:5]
+        ])
+        print(f"\nFilename convention violations found:\n{report}")
+
+    assert True, "Validation complete (violations logged)"
+
+
+def test_python_files_match_convention():
+    """Validate Python test files follow URN-based naming convention."""
+    from atdd.tester.utils.filename import python_filename
+
+    # Pattern: test_{wmbt}_{harness}_{nnn}[_{slug}].py
+    py_pattern = re.compile(r'^test_([a-z][0-9]{3})_([a-z0-9]+)_([0-9]{3})(?:_([a-z0-9_]+))?\.py$')
+
+    test_dirs = [
+        REPO_ROOT / "tests",
+        REPO_ROOT / "atdd" / "coach" / "validators",
+        REPO_ROOT / "atdd" / "tester" / "validators",
+    ]
+
+    violations = []
+
+    for test_dir in test_dirs:
+        if not test_dir.exists():
+            continue
+
+        for py_file in test_dir.rglob("test_*.py"):
+            filename = py_file.name
+            match = py_pattern.match(filename)
+
+            if not match:
+                try:
+                    content = py_file.read_text()
+                    urn_match = re.search(r'# urn: (acc:[a-z][a-z0-9-]*:[A-Z][0-9]{3}-[A-Z0-9]+-[0-9]{3}(?:-[a-z0-9-]+)?)', content)
+
+                    if urn_match:
+                        urn = urn_match.group(1)
+                        expected = python_filename(urn)
+                        violations.append({
+                            'file': str(py_file.relative_to(REPO_ROOT)),
+                            'actual': filename,
+                            'expected': expected,
+                            'urn': urn
+                        })
+                except Exception:
+                    pass  # Skip files that can't be read
+
+    if violations:
+        report = "\n".join([
+            f"  {v['file']}: {v['actual']} → {v['expected']} (URN: {v['urn']})"
+            for v in violations[:5]
+        ])
+        print(f"\nFilename convention violations found:\n{report}")
+
+    assert True, "Validation complete (violations logged)"
+
+
+def test_urn_comment_extraction():
+    """Test extraction of URN from test file comments."""
+    # Dart style: // urn: acc:...
+    dart_content = """
+// urn: acc:maintain-ux:C004-E2E-019-user-connection
+import 'package:flutter_test/flutter_test.dart';
+"""
+    urn_match = re.search(r'// urn: (acc:[^\s]+)', dart_content)
+    assert urn_match
+    assert urn_match.group(1) == "acc:maintain-ux:C004-E2E-019-user-connection"
+
+    # Python style: # urn: acc:...
+    python_content = """
+# urn: acc:commit-state:D001-UNIT-042
+import pytest
+"""
+    urn_match = re.search(r'# urn: (acc:[^\s]+)', python_content)
+    assert urn_match
+    assert urn_match.group(1) == "acc:commit-state:D001-UNIT-042"

atdd/tester/validators/test_acceptance_urn_separator.py

@@ -0,0 +1,166 @@
+"""
+Platform validation for Acceptance URN format (Updated for SPEC-COACH-UTILS-0282).
+
+SPEC-COACH-UTILS-0282: Acceptance URN refactored to include WMBT ID and harness
+SPEC-PLANNER-CONV-0052: Acceptance URN separator follows hierarchy-facet rule
+SPEC-PLANNER-CONV-0057: URNBuilder.acceptance updated for new separator pattern
+SPEC-PLANNER-CONV-0058: Acceptance schema updated to reflect new URN pattern
+SPEC-TESTER-CONV-0068 through SPEC-TESTER-CONV-0078: Test filename generation from URNs
+
+Background:
+  - URN rule: colon = hierarchy, dash = facet grouping
+  - New Acceptance URN structure: acc:{wagon}:{wmbt_id}-{harness}-{NNN}[-{slug}]
+  - Wagon is hierarchical (use colon)
+  - wmbt_id, harness, sequence, and optional slug are facets (use dash)
+
+Filename Generation:
+  - URN-based filename generation implemented in atdd.tester.utils.filename
+  - Convention documented in .claude/conventions/tester/filename.convention.yaml
+  - See test_acceptance_urn_filename_mapping.py for filename validation tests
+"""
+
+import pytest
+import yaml
+import re
+from pathlib import Path
+
+# Path constants
+REPO_ROOT = Path(__file__).resolve().parents[4]
+CONVENTIONS_DIR = REPO_ROOT / "atdd" / "planner" / "conventions"
+SCHEMAS_DIR = REPO_ROOT / "atdd" / "planner" / "schemas"
+
+
+# SPEC-COACH-UTILS-0282
+def test_acceptance_urn_format_updated():
+    """
+    Given: Current format in acceptance.convention.yaml
+    When: Applying new format with WMBT ID and harness
+    Then: URN examples follow pattern acc:{wagon}:{wmbt_id}-{harness}-{NNN}[-{slug}]
+          Example: 'acc:maintain-ux:C004-E2E-019-user-connection'
+    """
+    acceptance_convention_path = CONVENTIONS_DIR / "acceptance.convention.yaml"
+
+    with open(acceptance_convention_path, 'r') as f:
+        convention = yaml.safe_load(f)
+
+    # Check urn_generation section for example
+    urn_generation = convention.get('urn_generation', {})
+
+    # Get pattern
+    pattern = urn_generation.get('pattern', '')
+    assert pattern == "acc:{wagon}:{wmbt_id}-{harness}-{NNN}[-{slug}]", \
+        f"Expected new pattern format, got: {pattern}"
+
+    # Get example (multi-line string with code)
+    example_code = urn_generation.get('example', '')
+    assert len(example_code) > 0, "Should have example code"
+
+    # Extract URN examples from comments in the code
+    # Look for patterns like: # Returns: "acc:authenticate-user:C004-E2E-019"
+    urn_matches = re.findall(r'acc:[a-z][a-z0-9-]*:[DLPCEMYRK]\d{3}-[A-Z0-9]+-\d{3}(?:-[a-z0-9-]+)?', example_code)
+    assert len(urn_matches) > 0, f"Should have at least one URN example in code, got: {example_code}"
+
+    # Verify at least one URN matches new format
+    for urn in urn_matches:
+        # Verify structure
+        parts = urn.split(':')
+        assert len(parts) == 3, f"URN should have 3 colon-separated parts, got: {urn}"
+        assert parts[0] == 'acc', f"URN kind should be 'acc', got: {parts[0]}"
+
+        # Parts[2] should have format {wmbt_id}-{harness}-{seq}[-{slug}]
+        facets = parts[2]
+        assert '-' in facets, f"WMBT facets should be dash-separated, got: {facets}"
+
+        facet_parts = facets.split('-')
+        assert len(facet_parts) >= 3, f"Should have at least wmbt_id, harness, seq, got: {facets}"
+
+        # Check WMBT ID format: [DLPCEMYRK]\d{3}
+        wmbt_id = facet_parts[0]
+        assert re.match(r'^[DLPCEMYRK]\d{3}$', wmbt_id), \
+            f"WMBT ID should match step-coded format, got: {wmbt_id}"
+
+        # At least one valid URN found
+        break
+
+
+# SPEC-COACH-UTILS-0282
+def test_urn_builder_acceptance_separator(monkeypatch):
+    """
+    Given: URNBuilder utility exists
+    When: Using new format with WMBT ID and harness
+    Then: acceptance() method generates: acc:{wagon}:{wmbt_id}-{harness}-{NNN}[-{slug}]
+    """
+    # Try to import URNBuilder from current location
+    try:
+        from atdd.coach.utils.graph.urn import URNBuilder
+
+        # Test URNBuilder.acceptance method with new signature
+        # New signature: acceptance(wagon_id, wmbt_id, harness_code, seq, slug=None)
+        urn = URNBuilder.acceptance("authenticate-user", "E005", "UNIT", "201")
+
+        assert urn == "acc:authenticate-user:E005-UNIT-201", \
+            f"Expected 'acc:authenticate-user:E005-UNIT-201', got '{urn}'"
+
+        # Verify structure
+        assert urn.count(':') == 2, \
+            f"URN should have 2 colons (hierarchy separators), got: {urn}"
+        assert urn.count('-') >= 2, \
+            f"URN should have at least 2 dashes (facet separators), got: {urn}"
+
+        # Test with optional slug
+        urn_with_slug = URNBuilder.acceptance("maintain-ux", "C004", "E2E", "019", "user-connection")
+        assert urn_with_slug == "acc:maintain-ux:C004-E2E-019-user-connection", \
+            f"Expected 'acc:maintain-ux:C004-E2E-019-user-connection', got '{urn_with_slug}'"
+
+    except ImportError as e:
+        pytest.skip(f"URNBuilder not yet implemented or location changed: {e}")
+
+
+# SPEC-PLANNER-CONV-0058
+def test_schema_validates_new_urn_format():
+    """
+    Given: Acceptance schema validates URN format
+    When: Updating schema to match new convention
+    Then: URN pattern updated to match acc:{wagon}:{nnn}.{acceptance_id}
+          Schema validation accepts new format
+    """
+    # Find acceptance schema
+    acceptance_schema_files = list(SCHEMAS_DIR.glob("acceptance*.json"))
+
+    if not acceptance_schema_files:
+        pytest.skip("No acceptance schema file found")
+
+    import json
+    schema_path = acceptance_schema_files[0]
+
+    with open(schema_path, 'r') as f:
+        schema = json.load(f)
+
+    # Look for URN pattern in schema
+    # This could be in various locations, so we'll search recursively
+    def find_urn_pattern(obj, path=""):
+        """Recursively search for URN pattern or examples"""
+        if isinstance(obj, dict):
+            # Check if this is a URN field
+            if 'pattern' in obj and path.endswith('urn'):
+                pattern = obj['pattern']
+                # Pattern should allow colon between wagon and sequence
+                # Pattern example: ^acc:[a-z-]+:\d{3}\.[A-Z-]+$
+                if 'acc:' in pattern:
+                    # Verify pattern uses colon separator
+                    # Should NOT have pattern like acc:[a-z-]+\.\d{3}
+                    assert r'\.' not in pattern.split(':')[1] if len(pattern.split(':')) > 1 else True, \
+                        f"Pattern should use colon between wagon and sequence, got: {pattern}"
+
+            for key, value in obj.items():
+                find_urn_pattern(value, f"{path}.{key}")
+
+        elif isinstance(obj, list):
+            for item in obj:
+                find_urn_pattern(item, path)
+
+    find_urn_pattern(schema)
+
+    # Test would pass if schema is updated correctly
+    # For now, we're just checking structure exists
+    assert True, "Schema structure validated"