forked from BasedHardware/omi
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathtest_failure_class.py
More file actions
238 lines (202 loc) · 10.8 KB
/
Copy pathtest_failure_class.py
File metadata and controls
238 lines (202 loc) · 10.8 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
#!/usr/bin/env python3
"""Hermetic subprocess tests for scripts/failure-class."""
from __future__ import annotations
import json
import shutil
import subprocess
import sys
import tempfile
import unittest
from pathlib import Path
REPOSITORY_ROOT = Path(__file__).resolve().parents[1]
CLI = REPOSITORY_ROOT / "scripts" / "failure-class"
SEED_DIRECTORY = REPOSITORY_ROOT / ".github" / "failure-classes"
REPORT_FIXTURE = REPOSITORY_ROOT / "scripts" / "fixtures" / "failure_class" / "report-events.json"
SEED_IDS = {
"FC-malformed-doc-read",
"FC-unbounded-module-cache",
"FC-trapping-dict-merge",
"FC-per-hop-timeout",
"FC-split-mutation-authority",
}
def run(command: list[str], cwd: Path) -> subprocess.CompletedProcess[str]:
return subprocess.run(command, cwd=cwd, check=False, text=True, capture_output=True)
class FailureClassCliTests(unittest.TestCase):
def setUp(self) -> None:
self.temp_directory = tempfile.TemporaryDirectory()
self.root = Path(self.temp_directory.name)
definitions = self.root / ".github" / "failure-classes"
definitions.parent.mkdir(parents=True)
shutil.copytree(SEED_DIRECTORY, definitions)
run(["git", "init", "-q"], self.root)
run(["git", "config", "user.email", "failure-class@example.test"], self.root)
run(["git", "config", "user.name", "Failure Class Test"], self.root)
self.write("src/example.txt", "initial\n")
self.commit("chore: seed failure classes")
self.base = self.git("rev-parse", "HEAD")
def tearDown(self) -> None:
self.temp_directory.cleanup()
def git(self, *args: str) -> str:
result = run(["git", *args], self.root)
self.assertEqual(result.returncode, 0, result.stderr)
return result.stdout.strip()
def write(self, relative_path: str, content: str) -> Path:
path = self.root / relative_path
path.parent.mkdir(parents=True, exist_ok=True)
path.write_text(content, encoding="utf-8")
return path
def commit(self, subject: str) -> None:
self.git("add", ".")
self.git("commit", "-qm", subject)
def add_fix_commit(self) -> None:
self.write("src/example.txt", "fixed\n")
self.commit("fix(backend): protect read boundary")
def body(self, content: str) -> Path:
return self.write("pr-body.md", content)
def cli(self, *arguments: str) -> subprocess.CompletedProcess[str]:
return run([sys.executable, str(CLI), *arguments, "--root", str(self.root), "--format", "json"], self.root)
def payload(self, result: subprocess.CompletedProcess[str]) -> dict:
try:
return json.loads(result.stdout)
except json.JSONDecodeError as exc:
self.fail(f"CLI did not emit JSON: {exc}\nstdout={result.stdout}\nstderr={result.stderr}")
def validate(self, body: Path) -> subprocess.CompletedProcess[str]:
return self.cli("validate", "--base", self.base, "--head", "HEAD", "--pr-body-file", str(body))
def test_seed_definitions_are_valid_without_fix_commit(self) -> None:
result = self.validate(self.body("## Summary\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["ok"])
self.assertEqual(payload["validation"]["definition_count"], 5)
self.assertEqual({path.stem for path in SEED_DIRECTORY.glob("*.json")}, SEED_IDS)
def test_valid_existing_declaration(self) -> None:
self.add_fix_commit()
result = self.validate(self.body("Failure-Class: FC-malformed-doc-read\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["ok"])
self.assertEqual(payload["validation"]["declaration"], "FC-malformed-doc-read")
def test_missing_declaration_for_fix_commit_fails(self) -> None:
self.add_fix_commit()
result = self.validate(self.body("## Summary\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 1)
self.assertIn("missing_declaration", [item["code"] for item in payload["errors"]])
def test_unknown_declaration_fails(self) -> None:
self.add_fix_commit()
result = self.validate(self.body("Failure-Class: FC-not-in-registry\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 1)
self.assertIn("unknown_failure_class", [item["code"] for item in payload["errors"]])
def test_dormant_class_declaration_requires_explicit_reopen(self) -> None:
definition_path = self.root / ".github" / "failure-classes" / "FC-malformed-doc-read.json"
definition = json.loads(definition_path.read_text(encoding="utf-8"))
definition.update({"status": "dormant", "dormant_since": "2026-07-05T00:00:00Z"})
definition_path.write_text(json.dumps(definition, indent=2) + "\n", encoding="utf-8")
self.add_fix_commit()
result = self.validate(self.body("Failure-Class: FC-malformed-doc-read\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 1)
self.assertIn("dormant_failure_class_requires_reopen", [item["code"] for item in payload["errors"]])
def test_existing_class_fix_cannot_mutate_registry(self) -> None:
definition_path = self.root / ".github" / "failure-classes" / "FC-malformed-doc-read.json"
definition = json.loads(definition_path.read_text(encoding="utf-8"))
definition["canonical_prevention"] = "An incident fix must not update this registry record."
definition_path.write_text(json.dumps(definition, indent=2) + "\n", encoding="utf-8")
self.add_fix_commit()
result = self.validate(self.body("Failure-Class: FC-malformed-doc-read\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 1)
self.assertIn("instance_fix_mutates_registry", [item["code"] for item in payload["errors"]])
def test_registry_only_dormant_transition_is_valid(self) -> None:
definition_path = self.root / ".github" / "failure-classes" / "FC-malformed-doc-read.json"
definition = json.loads(definition_path.read_text(encoding="utf-8"))
definition.update({"status": "dormant", "dormant_since": "2026-07-05T00:00:00Z"})
definition_path.write_text(json.dumps(definition, indent=2) + "\n", encoding="utf-8")
self.commit("harden: record a dormant failure class")
result = self.validate(self.body("## Registry lifecycle transition\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["ok"])
def test_new_declaration_requires_and_accepts_new_definition(self) -> None:
definition = {
"schema_version": 1,
"id": "FC-new-test-boundary",
"violated_contract": "Test boundaries retain their contract.",
"canonical_prevention": "Keep the guard at the shared boundary.",
"evidence_prs": [1234],
"status": "open",
}
self.write(
".github/failure-classes/FC-new-test-boundary.json",
json.dumps(definition, indent=2) + "\n",
)
self.write("src/example.txt", "new class\n")
self.commit("fix: register a newly observed class")
result = self.validate(self.body("Failure-Class: new\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["ok"])
def test_new_declaration_without_added_definition_fails(self) -> None:
self.add_fix_commit()
result = self.validate(self.body("Failure-Class: new\n"))
payload = self.payload(result)
self.assertEqual(result.returncode, 1)
self.assertIn("new_definition_required", [item["code"] for item in payload["errors"]])
def test_prepare_emits_append_patch_and_registry_only_candidates(self) -> None:
self.add_fix_commit()
result = self.cli("prepare", "--base", self.base, "--head", "HEAD", "--pr-body-file", str(self.body("## Summary\n")))
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["requires_declaration"])
self.assertEqual(payload["pr_body_patch"]["operation"], "append")
self.assertEqual(payload["pr_body_patch"]["text"], "Failure-Class: none\n")
self.assertEqual(len(payload["advisory_candidates"]), 5)
self.assertIn("no class was inferred", payload["candidate_source"])
def test_explain_emits_versioned_definition(self) -> None:
result = self.cli("explain", "FC-trapping-dict-merge")
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["ok"])
self.assertEqual(payload["schema_version"], 1)
self.assertEqual(payload["failure_class"]["id"], "FC-trapping-dict-merge")
self.assertEqual(payload["failure_class"]["evidence_prs"], [6506, 9288])
def test_report_fixture_marks_old_open_instance_closure_eligible(self) -> None:
result = self.cli(
"report",
"--events-file",
str(REPORT_FIXTURE),
"--since",
"14d",
"--now",
"2026-07-16T00:00:00Z",
)
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
self.assertTrue(payload["advisory"])
self.assertFalse(payload["automatic_state_changes"])
by_id = {item["id"]: item for item in payload["classes"]}
self.assertTrue(by_id["FC-malformed-doc-read"]["closure_eligible"])
self.assertFalse(by_id["FC-trapping-dict-merge"]["closure_eligible"])
self.assertEqual(by_id["FC-malformed-doc-read"]["last_reported_instance"]["number"], 9494)
def test_report_flags_recurrence_after_a_dormant_transition(self) -> None:
definition_path = self.root / ".github" / "failure-classes" / "FC-trapping-dict-merge.json"
definition = json.loads(definition_path.read_text(encoding="utf-8"))
definition.update({"status": "dormant", "dormant_since": "2026-07-05T00:00:00Z"})
definition_path.write_text(json.dumps(definition, indent=2) + "\n", encoding="utf-8")
result = self.cli(
"report",
"--events-file",
str(REPORT_FIXTURE),
"--since",
"14d",
"--now",
"2026-07-16T00:00:00Z",
)
payload = self.payload(result)
self.assertEqual(result.returncode, 0, result.stderr)
by_id = {item["id"]: item for item in payload["classes"]}
self.assertTrue(by_id["FC-trapping-dict-merge"]["reopen_required"])
self.assertFalse(by_id["FC-trapping-dict-merge"]["closure_eligible"])
if __name__ == "__main__":
unittest.main()