fix(ci): apply ruff format and suppress vulture containment_path
CI / push-validation (pull_request) Successful in 22s
CI / helm (pull_request) Successful in 31s
CI / build (pull_request) Successful in 44s
CI / quality (pull_request) Successful in 50s
CI / lint (pull_request) Successful in 55s
CI / security (pull_request) Successful in 1m4s
CI / typecheck (pull_request) Successful in 1m11s
CI / unit_tests (pull_request) Successful in 6m6s
CI / coverage (pull_request) Failing after 1m5s
CI / docker (pull_request) Successful in 1m25s
CI / integration_tests (pull_request) Failing after 31m41s
CI / status-check (pull_request) Has been cancelled
CI / push-validation (pull_request) Successful in 22s
CI / helm (pull_request) Successful in 31s
CI / build (pull_request) Successful in 44s
CI / quality (pull_request) Successful in 50s
CI / lint (pull_request) Successful in 55s
CI / security (pull_request) Successful in 1m4s
CI / typecheck (pull_request) Successful in 1m11s
CI / unit_tests (pull_request) Successful in 6m6s
CI / coverage (pull_request) Failing after 1m5s
CI / docker (pull_request) Successful in 1m25s
CI / integration_tests (pull_request) Failing after 31m41s
CI / status-check (pull_request) Has been cancelled
Reformat features/steps/resource_handler_project_access_1444_steps.py and src/cleveragents/resource/handlers/_base.py to satisfy ruff format check (complex slice spacing, collapsing single-f-string assert parens). Add containment_path to vulture_whitelist.py — the parameter is required by the ResourceHandler.project_access interface (spec §21691-21698) but is intentionally unused in stub implementations that raise NotImplementedError.
This commit is contained in:
@@ -200,28 +200,21 @@ def step_tdd1444_instance_crosses_sandbox(context: Context, expected: str) -> No
|
||||
def step_tdd1444_instance_read_richness(context: Context, expected: int) -> None:
|
||||
assert context.tdd1444_result is not None, "No AccessProjection instance"
|
||||
assert context.tdd1444_result.read_richness == expected, (
|
||||
f"Expected read_richness={expected}, "
|
||||
f"got {context.tdd1444_result.read_richness}"
|
||||
f"Expected read_richness={expected}, got {context.tdd1444_result.read_richness}"
|
||||
)
|
||||
|
||||
|
||||
@then("tdd1444 the result is an AccessProjection or None")
|
||||
def step_tdd1444_result_is_projection_or_none(context: Context) -> None:
|
||||
assert context.tdd1444_error is None, (
|
||||
f"Unexpected error: {context.tdd1444_error}"
|
||||
)
|
||||
assert context.tdd1444_error is None, f"Unexpected error: {context.tdd1444_error}"
|
||||
assert context.tdd1444_result is None or isinstance(
|
||||
context.tdd1444_result, AccessProjection
|
||||
), (
|
||||
f"Expected AccessProjection or None, got {type(context.tdd1444_result)}"
|
||||
)
|
||||
), f"Expected AccessProjection or None, got {type(context.tdd1444_result)}"
|
||||
|
||||
|
||||
@then('tdd1444 the result is an AccessProjection with protocol "{expected}"')
|
||||
def step_tdd1444_result_protocol(context: Context, expected: str) -> None:
|
||||
assert context.tdd1444_error is None, (
|
||||
f"Unexpected error: {context.tdd1444_error}"
|
||||
)
|
||||
assert context.tdd1444_error is None, f"Unexpected error: {context.tdd1444_error}"
|
||||
assert context.tdd1444_result is not None, "Expected AccessProjection, got None"
|
||||
assert isinstance(context.tdd1444_result, AccessProjection), (
|
||||
f"Expected AccessProjection, got {type(context.tdd1444_result)}"
|
||||
@@ -235,6 +228,5 @@ def step_tdd1444_result_protocol(context: Context, expected: str) -> None:
|
||||
def step_tdd1444_result_read_richness(context: Context, expected: int) -> None:
|
||||
assert context.tdd1444_result is not None, "Expected AccessProjection, got None"
|
||||
assert context.tdd1444_result.read_richness == expected, (
|
||||
f"Expected read_richness={expected}, "
|
||||
f"got {context.tdd1444_result.read_richness}"
|
||||
f"Expected read_richness={expected}, got {context.tdd1444_result.read_richness}"
|
||||
)
|
||||
|
||||
@@ -388,7 +388,7 @@ class BaseResourceHandler:
|
||||
binding_loc = binding_resource.location or ""
|
||||
target_loc = target_resource.location
|
||||
if binding_loc and target_loc.startswith(binding_loc):
|
||||
access_path = target_loc[len(binding_loc):].lstrip("/")
|
||||
access_path = target_loc[len(binding_loc) :].lstrip("/")
|
||||
else:
|
||||
access_path = target_loc
|
||||
crosses = sandbox is not None
|
||||
|
||||
@@ -1244,3 +1244,7 @@ generate_session_id # noqa: B018, F821
|
||||
|
||||
# Pydantic @classmethod validator parameter required by decorator protocol
|
||||
cls # noqa: B018, F821
|
||||
|
||||
# ResourceHandler.project_access interface parameter — required by spec (§21691-21698)
|
||||
# but not referenced in stub/base implementations that raise NotImplementedError
|
||||
containment_path # noqa: B018, F821
|
||||
|
||||
Reference in New Issue
Block a user