Skip to content

Commit da8169f

Browse files
committed
docs(testing): state the enforcement status of both new rules
The sibling mandatory conventions all end by naming the test that enforces them. These two do not have one, so say so explicitly rather than let the '(mandatory)' heading imply CI coverage: the CLI-customized-command rule has a lint tracked in #1402, and the list-ordering rule stays judgment-based by design (a lint cannot decide whether a given list is order-significant).
1 parent c083cd7 commit da8169f

1 file changed

Lines changed: 11 additions & 0 deletions

File tree

.claude/rules/testing.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,12 @@ existing fixtures do not set it and are fine, because the hang only bites the
206206
customized commands. `tests/integration/emr-instance-configs/verify.sh` is the
207207
reference.
208208

209+
NOT mechanically enforced yet — a lint over `tests/integration/*/verify.sh` is
210+
tracked in issue
211+
[#1402](https://github.com/go-to-k/cdkd/issues/1402), so until then this is a
212+
read-it-and-follow-it rule. User-facing writeup in
213+
[docs/testing.md](../../docs/testing.md).
214+
209215
### `verify.sh` list readbacks must be order-insensitive (mandatory)
210216

211217
AWS does not preserve the submitted order of list-valued members on readback.
@@ -232,6 +238,11 @@ order-significant (DNS resolver lists, preference orders — see
232238
`getDriftUnorderedPaths`) must stay unsorted, because sorting it would HIDE a
233239
real regression.
234240

241+
NOT mechanically enforced — whether a given list is order-significant is a
242+
judgment call a lint cannot make, so this one stays a read-it-and-follow-it
243+
rule by design. User-facing writeup in
244+
[docs/testing.md](../../docs/testing.md).
245+
235246
### Fixture stateful L2s need an explicit removalPolicy (mandatory)
236247

237248
Stateful CDK L2 constructs (`kinesis.Stream`, `dynamodb.Table`/`TableV2`,

0 commit comments

Comments
 (0)