Skip to content

add section on test names - #13276

Closed
ShahakShama wants to merge 1 commit into
shahak/style-guide-short-unit-test-recommendationfrom
shahak/style-guide-test-names
Closed

ShahakShama wants to merge 1 commit into
shahak/style-guide-short-unit-test-recommendationfrom
shahak/style-guide-test-names

Conversation

@ShahakShama

@ShahakShama ShahakShama commented Mar 15, 2026 •

Copy link
Copy Markdown
Contributor

Note

Low Risk
Documentation-only change with no impact on runtime behavior or build/test execution.

Overview
Adds a new Test Names subsection to style.md under Testing that standardizes unit test naming: tests should use a test_ prefix and can be long/descriptive to clearly distinguish tests from helper functions.

Written by Cursor Bugbot for commit 0448af8. This will update automatically on new commits. Configure here.

@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch from 0448af8 to 33c2542 Compare April 9, 2026 10:28
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch 2 times, most recently from 123b641 to ebd0ed1 Compare April 9, 2026 11:50
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch from 33c2542 to 8780138 Compare April 9, 2026 11:50
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from ebd0ed1 to c83e5f5 Compare April 9, 2026 12:12
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch 2 times, most recently from 32661c4 to 6c01bd8 Compare April 9, 2026 12:15
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch 2 times, most recently from 4ab0e1c to cac881d Compare April 9, 2026 12:37
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch from 6c01bd8 to 36c715b Compare April 9, 2026 12:37
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from cac881d to b770f94 Compare April 9, 2026 15:05
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch from 36c715b to 3afc1cd Compare April 9, 2026 15:05
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from b770f94 to e719fcb Compare April 13, 2026 10:51
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch 2 times, most recently from 0b8b63c to 8f19659 Compare April 15, 2026 05:53
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from e719fcb to 52158a3 Compare April 15, 2026 05:53
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from 52158a3 to a6cda9d Compare April 26, 2026 10:57
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch 2 times, most recently from 6a22bec to f1db1df Compare April 26, 2026 11:04
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from a6cda9d to 286c4a7 Compare April 26, 2026 11:04
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-short-unit-test-recommendation branch from 286c4a7 to 0475dd7 Compare April 28, 2026 13:43
@ShahakShama
ShahakShama force-pushed the shahak/style-guide-test-names branch from f1db1df to 090cfed Compare April 28, 2026 13:43
@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions github-actions Bot added the stale label May 29, 2026
@github-actions github-actions Bot closed this Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants