vrr/Pulse
2
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-10-19 18:44:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 13

Extract API token manager state owner

Browse source
This commit is contained in:
rcourtman 2026-03-20 23:08:43 +00:00
parent 6e09daf9be
commit dcd88e2f64
10 changed files with 1022 additions and 802 deletions
Download patch file Download diff file Expand all files Collapse all files

36
scripts/release_control/canonical_completion_guard_test.py
View file

@ -2092,10 +2092,10 @@ class CanonicalCompletionGuardTest(unittest.TestCase):
],
)
def test_api_token_manager_change_requires_security_and_api_contracts(self):
required = infer_impacted_subsystems(
["frontend-modern/src/components/Settings/APITokenManager.tsx"]
)
def _assert_api_token_manager_change_requires_security_and_api_contracts(
self, touched_path: str
) -> None:
required = infer_impacted_subsystems([touched_path])
self.assertEqual(set(required), {"security-privacy", "api-contracts"})
security = required["security-privacy"]
@ -2105,7 +2105,7 @@ class CanonicalCompletionGuardTest(unittest.TestCase):
)
self.assertEqual(
security["touched_runtime_files"],
["frontend-modern/src/components/Settings/APITokenManager.tsx"],
[touched_path],
)
self.assertEqual(
security["verification_requirements"],
@ -2113,9 +2113,7 @@ class CanonicalCompletionGuardTest(unittest.TestCase):
{
"id": "security-settings-surfaces",
"label": "security settings surface proof",
"touched_runtime_files": [
"frontend-modern/src/components/Settings/APITokenManager.tsx"
],
"touched_runtime_files": [touched_path],
"allow_same_subsystem_tests": False,
"test_prefixes": [],
"exact_files": [
@ -2135,7 +2133,7 @@ class CanonicalCompletionGuardTest(unittest.TestCase):
)
self.assertEqual(
api_contracts["touched_runtime_files"],
["frontend-modern/src/components/Settings/APITokenManager.tsx"],
[touched_path],
)
self.assertEqual(
api_contracts["verification_requirements"],
@ -2143,18 +2141,32 @@ class CanonicalCompletionGuardTest(unittest.TestCase):
{
"id": "api-token-management-surface",
"label": "API token management surface proof",
"touched_runtime_files": [
"frontend-modern/src/components/Settings/APITokenManager.tsx"
],
"touched_runtime_files": [touched_path],
"allow_same_subsystem_tests": False,
"test_prefixes": [],
"exact_files": [
"frontend-modern/src/components/Settings/__tests__/APITokenManager.test.tsx",
"frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts",
],
}
],
)
def test_api_token_manager_change_requires_security_and_api_contracts(self):
self._assert_api_token_manager_change_requires_security_and_api_contracts(
"frontend-modern/src/components/Settings/APITokenManager.tsx"
)
def test_api_token_manager_model_change_requires_security_and_api_contracts(self):
self._assert_api_token_manager_change_requires_security_and_api_contracts(
"frontend-modern/src/components/Settings/apiTokenManagerModel.ts"
)
def test_api_token_manager_state_change_requires_security_and_api_contracts(self):
self._assert_api_token_manager_change_requires_security_and_api_contracts(
"frontend-modern/src/components/Settings/useAPITokenManagerState.ts"
)
def test_security_client_change_requires_security_and_api_contracts(self):
required = infer_impacted_subsystems(["frontend-modern/src/api/security.ts"])
self.assertEqual(set(required), {"security-privacy", "api-contracts"})

5
scripts/release_control/subsystem_lookup_test.py
View file

@ -773,7 +773,10 @@ class SubsystemLookupTest(unittest.TestCase):
)
self.assertEqual(
api_match["verification_requirement"]["exact_files"],
["frontend-modern/src/components/Settings/__tests__/APITokenManager.test.tsx"],
[
"frontend-modern/src/components/Settings/__tests__/APITokenManager.test.tsx",
"frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts",
],
)
security_match = next(
match for match in file_entry["matches"] if match["subsystem"] == "security-privacy"