Skip to content

Hide self testing naming in tests#165

Merged
BabetteB merged 6 commits intomasterfrom
hide-self-testing-naming-in-tests
Mar 18, 2026
Merged

Hide self testing naming in tests#165
BabetteB merged 6 commits intomasterfrom
hide-self-testing-naming-in-tests

Conversation

@BabetteB
Copy link
Copy Markdown

Code Review Checklist

  • No out commented code
  • Good variable & method names
  • Code located in the right file/folder/package
  • No redundant or //TODO comments
  • Cannot be simplified
  • Right abstraction level
  • Modular enough
  • No code duplication
  • Cannot think of any inputs or events that could break the code

Question:

  • Are there any parts that are the slightest bit confusing and why?
    • Rather ask a question that is too small than to not ask any question

@BabetteB BabetteB marked this pull request as draft March 18, 2026 09:44
@BabetteB BabetteB force-pushed the hide-self-testing-naming-in-tests branch from 0c54028 to 0aa8b2a Compare March 18, 2026 09:50
@BabetteB BabetteB marked this pull request as ready for review March 18, 2026 09:54
@BabetteB BabetteB marked this pull request as draft March 18, 2026 09:55
@BabetteB BabetteB force-pushed the hide-self-testing-naming-in-tests branch from 2ecaea9 to fd97419 Compare March 18, 2026 09:57
@BabetteB BabetteB marked this pull request as ready for review March 18, 2026 09:59
@BabetteB BabetteB force-pushed the hide-self-testing-naming-in-tests branch from 5459f39 to 10c78d2 Compare March 18, 2026 12:07
lottejd
lottejd previously approved these changes Mar 18, 2026
@BabetteB BabetteB force-pushed the hide-self-testing-naming-in-tests branch from 860cc68 to 2ab8af4 Compare March 18, 2026 12:17
@BabetteB BabetteB force-pushed the hide-self-testing-naming-in-tests branch from 2ab8af4 to 121122a Compare March 18, 2026 12:19
@sonarqubecloud
Copy link
Copy Markdown

@BabetteB BabetteB merged commit 011540c into master Mar 18, 2026
4 checks passed
@BabetteB BabetteB deleted the hide-self-testing-naming-in-tests branch March 18, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants