feat(django): add urls and ignore_template_errors markers (#35)

- Add positional argument extraction to Rust marker scanner
- Implement @pytest.mark.urls('module') for ROOT_URLCONF override
- Implement @pytest.mark.ignore_template_errors for template suppression
- Integrate URL/template markers into run_test() with proper cleanup
- Update roadmap: merge Django markers into 0.2.3, shift Landlock to 0.2.4
- Defer transaction=True, reset_sequences, multi-db to 0.3.0 (#49)

Author: NikkeTryHard

docs/research/roadmap.md

@@ -57,13 +57,14 @@ flowchart TB
         P2_0["0.2.0 Hook Framework ✅"]
         P2_1["0.2.1 pytest-django ✅"]
         P2_2["0.2.2 pytest-asyncio ✅"]
-        P2_3["0.2.3 pytest-mock/env/timeout ✅"]
+        P2_3["0.2.3 pytest-mock/env/timeout + Django Markers ✅"]
         P2_4["0.2.4 Landlock V4-V6"]
         P2_5["0.2.5 Plugin Stabilization"]
 
         P2_0 --> P2_1
         P2_0 --> P2_2
         P2_0 --> P2_3
+        P2_1 --> P2_3
         P2_0 --> P2_4
         P2_1 --> P2_5
         P2_2 --> P2_5
@@ -274,7 +275,7 @@ flowchart TB
 **Current Status:**
 
 - Phase 1 (0.1.x): Complete
-- Phase 2 (0.2.x): In Progress - 0.2.0 done, 0.2.1 done, 0.2.2 done, 0.2.3 done, 0.2.4 can start
+- Phase 2 (0.2.x): In Progress - 0.2.0-0.2.4 done, 0.2.5 (Landlock) can start
 - Phases 3-4: Blocked by Phase 2 plugin work
 - Phases 5-9: **Many items can start NOW** - see blue "Can Start" nodes in flowchart
 - Phase 10: Not started
@@ -629,13 +630,13 @@ See #39 for tracking:
 - [x] Support gather/wait patterns
 - [x] Handle TaskGroup cleanup
 
-### 0.2.3 - Additional Plugin Support
+### 0.2.3 - Additional Plugin Support + Django Markers
 
-**Target**: Support for commonly used pytest plugins.
+**Target**: Support for commonly used pytest plugins and additional Django markers.
 
 **Status**: ✅ COMPLETE
 
-> **Parallelization**: Can be developed in parallel with 0.2.1, 0.2.2, and 0.2.3.1. Only requires 0.2.0 (hook framework) to be complete. No dependencies on other 0.2.x versions.
+> **Parallelization**: Can be developed in parallel with 0.2.1 and 0.2.2. Only requires 0.2.0 (hook framework) to be complete.
 
 #### pytest-mock
 
@@ -672,6 +673,22 @@ See #39 for tracking:
 - [x] Per-phase timeouts (setup, call, teardown)
   > Note: Current implementation is aggregate timeout. Per-phase is future enhancement.
 
+#### Django URL and Template Markers (Issue #35)
+
+- [x] `@pytest.mark.urls('myapp.test_urls')` - Override ROOT_URLCONF per test
+- [x] `@pytest.mark.ignore_template_errors` - Suppress template errors
+- [x] Positional argument extraction in Rust scanner
+- [x] URL cache clearing on override/restore
+- [x] Template debug mode toggle
+
+#### Deferred to 0.3.x (Database Integration)
+
+The following django_db marker options require deeper database transaction support:
+
+- `transaction=True` - Use real transactions (not savepoints)
+- `reset_sequences=True` - Reset auto-increment sequences
+- `databases=['default', 'secondary']` - Multi-database support
+
 #### pytest-cov (Deferred)
 
 - [ ] Detect pytest-cov and warn about Tach's native coverage
@@ -691,7 +708,7 @@ See #39 for tracking:
 
 **Target**: Use Landlock for network isolation when available, reducing reliance on CLONE_NEWNET.
 
-> **Parallelization**: Fully independent. Can be developed at any time after 0.2.0. This is a kernel feature enhancement with no dependencies on plugin shims (0.2.1-0.2.3).
+> **Parallelization**: Fully independent. Can be developed at any time after 0.2.0. This is a kernel feature enhancement with no dependencies on plugin shims (0.2.1-0.2.4).
 
 #### Network Restriction Rules
 
@@ -713,7 +730,7 @@ allow_bind_ports = [8000, 8080]  # Empty = no binding allowed
 
 > **External Ref:** [Landlock Kernel Docs - Network](https://docs.kernel.org/userspace-api/landlock.html)
 
-### 0.2.5 - Plugin Testing and Stabilization
+### 0.2.6 - Plugin Testing and Stabilization
 
 **Target**: Ensure plugin shims work correctly with real-world projects.
 

src/discovery/scanner.rs

@@ -646,9 +646,19 @@ fn expr_to_json_value(expr: &ast::Expr) -> Option<serde_json::Value> {
 /// Extract keyword arguments from a Call expression
 ///
 /// Returns a HashMap of argument name -> JSON value
+/// Positional args are stored with numeric keys ("0", "1", etc.)
 fn extract_marker_arguments(call: &ast::ExprCall) -> HashMap<String, serde_json::Value> {
     let mut args = HashMap::new();
 
+    // Capture positional arguments with numeric keys
+    // e.g., @pytest.mark.urls('myapp.test_urls') -> {"0": "myapp.test_urls"}
+    for (i, arg) in call.args.iter().enumerate() {
+        if let Some(value) = expr_to_json_value(arg) {
+            args.insert(i.to_string(), value);
+        }
+    }
+
+    // Capture keyword arguments
     for keyword in &call.keywords {
         if let Some(ref arg_name) = keyword.arg
             && let Some(value) = expr_to_json_value(&keyword.value)

src/tach_harness.py

@@ -2298,6 +2298,170 @@ def _cleanup_django_db_isolation(savepoints: list[tuple[str, str]]) -> None:
             print(f"[tach:harness] WARN: Failed to rollback savepoint for '{alias}': {e}", file=sys.stderr)
 
 
+# =============================================================================
+# Django URL and Template Markers (v0.2.4 - Issue #35)
+# =============================================================================
+
+
+def _parse_urls_marker(marker_info: list[dict[str, Any]] | None) -> str | None:
+    """Extract URL module path from @pytest.mark.urls marker.
+
+    The urls marker accepts a single positional argument specifying the
+    ROOT_URLCONF module to use for the test.
+
+    Example:
+        @pytest.mark.urls('myapp.test_urls')
+
+    Args:
+        marker_info: List of marker dictionaries from discovery.
+
+    Returns:
+        The URL module path string, or None if marker not present.
+    """
+    if not marker_info:
+        return None
+
+    for marker in marker_info:
+        if isinstance(marker, dict) and marker.get("name") == "urls":
+            args = marker.get("args", {})
+            # Positional arg is stored with key "0"
+            return args.get("0")
+
+    return None
+
+
+def _apply_urls_override(urlconf: str | None) -> str | None:
+    """Override ROOT_URLCONF for a test and clear URL resolver cache.
+
+    Args:
+        urlconf: The URL module path to use, or None to skip override.
+
+    Returns:
+        The original ROOT_URLCONF value for restoration, or None if not applied.
+    """
+    if urlconf is None or not _is_django_available():
+        return None
+
+    try:
+        from django.conf import settings
+        from django.urls import clear_url_caches
+
+        original = getattr(settings, "ROOT_URLCONF", None)
+        settings.ROOT_URLCONF = urlconf
+        clear_url_caches()
+        return original
+    except Exception as e:
+        print(f"[tach:harness] WARN: Failed to apply urls override: {e}", file=sys.stderr)
+        return None
+
+
+def _cleanup_urls_override(original_urlconf: str | None) -> None:
+    """Restore original ROOT_URLCONF after test.
+
+    Args:
+        original_urlconf: The original ROOT_URLCONF value to restore,
+            or None if no override was applied.
+    """
+    if original_urlconf is None:
+        return
+
+    try:
+        from django.conf import settings
+        from django.urls import clear_url_caches
+
+        settings.ROOT_URLCONF = original_urlconf
+        clear_url_caches()
+    except Exception as e:
+        print(f"[tach:harness] WARN: Failed to restore urls: {e}", file=sys.stderr)
+
+
+def _parse_ignore_template_errors_marker(marker_info: list[dict[str, Any]] | None) -> bool:
+    """Check if @pytest.mark.ignore_template_errors marker is present.
+
+    Args:
+        marker_info: List of marker dictionaries from discovery.
+
+    Returns:
+        True if the marker is present, False otherwise.
+    """
+    if not marker_info:
+        return False
+
+    for marker in marker_info:
+        if isinstance(marker, dict) and marker.get("name") == "ignore_template_errors":
+            return True
+
+    return False
+
+
+def _apply_ignore_template_errors(ignore: bool) -> dict[str, Any] | None:
+    """Disable template debug mode to suppress template errors.
+
+    When ignore_template_errors is set, we disable DEBUG and template
+    debugging to prevent TemplateSyntaxError from being raised.
+
+    Args:
+        ignore: Whether to ignore template errors.
+
+    Returns:
+        Dictionary of original settings for restoration, or None if not applied.
+    """
+    if not ignore or not _is_django_available():
+        return None
+
+    try:
+        from django.conf import settings
+
+        originals: dict[str, Any] = {}
+
+        # Save and modify DEBUG
+        originals["DEBUG"] = getattr(settings, "DEBUG", False)
+
+        # Save and modify TEMPLATES debug settings
+        if hasattr(settings, "TEMPLATES"):
+            originals["TEMPLATES"] = []
+            for i, template_config in enumerate(settings.TEMPLATES):
+                if isinstance(template_config, dict):
+                    orig_debug = template_config.get("OPTIONS", {}).get("debug")
+                    originals["TEMPLATES"].append(orig_debug)
+                    # Set debug to False to suppress template errors
+                    if "OPTIONS" not in template_config:
+                        template_config["OPTIONS"] = {}
+                    template_config["OPTIONS"]["debug"] = False
+
+        return originals
+    except Exception as e:
+        print(f"[tach:harness] WARN: Failed to apply ignore_template_errors: {e}", file=sys.stderr)
+        return None
+
+
+def _cleanup_ignore_template_errors(originals: dict[str, Any] | None) -> None:
+    """Restore original template settings after test.
+
+    Args:
+        originals: Dictionary of original settings from _apply_ignore_template_errors,
+            or None if no override was applied.
+    """
+    if originals is None:
+        return
+
+    try:
+        from django.conf import settings
+
+        # Restore TEMPLATES debug settings
+        if "TEMPLATES" in originals and hasattr(settings, "TEMPLATES"):
+            for i, orig_debug in enumerate(originals["TEMPLATES"]):
+                if i < len(settings.TEMPLATES):
+                    template_config = settings.TEMPLATES[i]
+                    if isinstance(template_config, dict) and "OPTIONS" in template_config:
+                        if orig_debug is None:
+                            template_config["OPTIONS"].pop("debug", None)
+                        else:
+                            template_config["OPTIONS"]["debug"] = orig_debug
+    except Exception as e:
+        print(f"[tach:harness] WARN: Failed to restore template settings: {e}", file=sys.stderr)
+
+
 def run_test(
     file_path: str,
     node_id: str,
@@ -2422,9 +2586,19 @@ def sync_wrapper(*args, **kwargs):
         django_db_args = _parse_django_db_marker(marker_info)
         django_savepoints = _apply_django_db_isolation(django_db_args)
 
+        # Django URL and Template Markers (v0.2.4 - Issue #35)
+        urlconf = _parse_urls_marker(marker_info)
+        original_urlconf = _apply_urls_override(urlconf)
+
+        ignore_templates = _parse_ignore_template_errors_marker(marker_info)
+        template_originals = _apply_ignore_template_errors(ignore_templates)
+
         try:
             reports = _pytest.runner.runtestprotocol(target_item, nextitem=None, log=False)
         finally:
+            # Cleanup in reverse order of application
+            _cleanup_ignore_template_errors(template_originals)
+            _cleanup_urls_override(original_urlconf)
             # Rollback savepoints to restore database state
             _cleanup_django_db_isolation(django_savepoints)