Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .cursor/commands/improve-tests.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Is there any overlap in these tests, or are any tests engaging in testing theater?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

Clarify the purpose and audience of this Cursor command.

The phrasing "Is there any overlap in these tests, or are any tests engaging in testing theater?" is too vague and assumes context that isn't provided:

  • What constitutes "testing theater" in this codebase?
  • Are you asking an AI assistant (Cursor) to analyze tests, or is this a prompt for developers?
  • Without the tests present, this command is not actionable.

Revise to include explicit context, the problem statement, and success criteria. For example:

  • Explain what patterns constitute "testing theater"
  • Provide examples of what to look for
  • Define the expected output or action


Implement improvements if they are.
181 changes: 0 additions & 181 deletions tests/Fixtures/ContainerFixtures.php

This file was deleted.

137 changes: 0 additions & 137 deletions tests/Fixtures/MockFilesystem.php

This file was deleted.

Loading
Loading