{
  "context": "Tests choosing fakes/mocks and behaviour-focused tests.",
  "type": "weighted_checklist",
  "checklist": [
    {
      "name": "Behaviour focus",
      "description": "Tests observable returned values, persisted effects, or outgoing side effects rather than private choreography.",
      "max_score": 12
    },
    {
      "name": "Avoid internals",
      "description": "Does not assert private methods or exact internal call order except where the outgoing interaction is the behaviour.",
      "max_score": 10
    },
    {
      "name": "Fakes preferred",
      "description": "Uses fakes or spies for owned repo/payment/email boundaries rather than heavy mocks for everything.",
      "max_score": 12
    },
    {
      "name": "Mock rationale",
      "description": "Uses mocks only for side effects that are the behaviour, such as sending receipt email, or explains why.",
      "max_score": 10
    },
    {
      "name": "Own types",
      "description": "Substitutes owned interfaces/adapters rather than mocking third-party library objects directly.",
      "max_score": 8
    },
    {
      "name": "Public seam",
      "description": "Tests through the service public API or stable seam production uses.",
      "max_score": 10
    },
    {
      "name": "AAA",
      "description": "Tests are arranged in Arrange-Act-Assert form with short domain names.",
      "max_score": 10
    },
    {
      "name": "Precise assertions",
      "description": "Assertions check coherent behaviours and meaningful values, not broad snapshots or weak existence checks.",
      "max_score": 10
    },
    {
      "name": "Refactor support",
      "description": "Explains why rewritten tests should survive harmless internal refactoring.",
      "max_score": 10
    },
    {
      "name": "Smell review",
      "description": "Identifies over-mocking/overspecification in the old suite.",
      "max_score": 8
    }
  ]
}