Use explicit unit test marker

Rename the synthesized marker to match its semantics and keep the local unit target from collecting the integration test directory.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: kevinjqliu

.github/workflows/python-ci.yml

@@ -89,7 +89,7 @@ jobs:
     - name: Install
       run: uv sync --all-extras --no-extra hive-kerberos
     - name: Run unit tests
-      run: uv run python -m pytest tests/ -m unmarked --ignore=tests/integration -v -x
+      run: uv run python -m pytest tests/ -m unit --ignore=tests/integration -v -x
 
   cibw-dev-env-smoke-test:
     runs-on: ubuntu-latest

Makefile

@@ -104,7 +104,7 @@ lint: ## Run code linters via prek (pre-commit hooks)
 ##@ Testing
 
 test: ## Run all unit tests (excluding integration)
-	$(TEST_RUNNER) pytest tests/ -m unmarked $(PYTEST_ARGS)
+	$(TEST_RUNNER) pytest tests/ -m unit --ignore=tests/integration $(PYTEST_ARGS)
 
 test-integration: test-integration-setup test-integration-exec test-integration-cleanup ## Run integration tests
 

pyproject.toml

@@ -229,7 +229,7 @@ quote-style = "double"
 testpaths = ["tests"]
 
 markers = [
-  "unmarked: marks a test as a unittest",
+  "unit: marks a test as a unit test",
   "s3: marks a test as requiring access to s3 compliant storage (use with --aws-access-key-id, --aws-secret-access-key, and --endpoint args)",
   "adls: marks a test as requiring access to adls compliant storage (use with --adls.account-name, --adls.account-key, and --adls.endpoint args)",
   "integration: marks integration tests against Apache Spark",

tests/conftest.py

@@ -110,7 +110,7 @@
 def pytest_collection_modifyitems(items: list[pytest.Item]) -> None:
     for item in items:
         if not any(marker.name in NON_UNIT_TEST_MARKERS for marker in item.iter_markers()):
-            item.add_marker("unmarked")
+            item.add_marker("unit")
 
 
 @pytest.fixture(autouse=True, scope="session")