Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test] sort and merge nano tests #303

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

spalmer25
Copy link
Collaborator

depends on #301
This PR groups tests into folders and files to have a better global understanding of which tests exists.
Use pytest feature for better intelligibility.

@spalmer25 spalmer25 self-assigned this Nov 29, 2024
@spalmer25 spalmer25 changed the base branch from main to palmer@functori@test-explicit-messages November 29, 2024 09:12
@spalmer25 spalmer25 force-pushed the palmer@functori@test-explicit-messages branch from cd2d346 to 8d1c494 Compare November 29, 2024 12:03
@spalmer25 spalmer25 force-pushed the palmer@functori@sort-and-merge-nano-tests branch from 8b81fc8 to 474d918 Compare November 29, 2024 13:01
@spalmer25 spalmer25 force-pushed the palmer@functori@sort-and-merge-nano-tests branch from 474d918 to 17226a0 Compare December 9, 2024 08:19
@spalmer25 spalmer25 force-pushed the palmer@functori@test-explicit-messages branch from 8d1c494 to c00cf80 Compare December 12, 2024 13:25
@spalmer25 spalmer25 force-pushed the palmer@functori@sort-and-merge-nano-tests branch from 17226a0 to 666cf09 Compare December 12, 2024 13:26
@spalmer25 spalmer25 added the tests test - related isues label Dec 12, 2024
@spalmer25 spalmer25 force-pushed the palmer@functori@test-explicit-messages branch from c00cf80 to 0dc8f67 Compare December 12, 2024 15:37
@spalmer25 spalmer25 force-pushed the palmer@functori@sort-and-merge-nano-tests branch 2 times, most recently from b505dfa to 6a3d0d6 Compare December 13, 2024 11:35
@spalmer25 spalmer25 changed the base branch from palmer@functori@test-explicit-messages to main December 13, 2024 11:36
@spalmer25 spalmer25 marked this pull request as ready for review December 13, 2024 15:00
@spalmer25 spalmer25 requested review from ajinkyaraj-23 and removed request for ajinkyaraj-23 December 13, 2024 15:56
@spalmer25 spalmer25 marked this pull request as draft December 13, 2024 15:56
@spalmer25 spalmer25 force-pushed the palmer@functori@sort-and-merge-nano-tests branch from 5e8b369 to 6a3d0d6 Compare December 13, 2024 16:16
@spalmer25 spalmer25 marked this pull request as ready for review December 13, 2024 17:18
 - introduce `snapshot_dir` to locate the snapshots dir
 - use the `ids` of `pytest.mark.parametrize` to name tests according their parameters
@spalmer25 spalmer25 force-pushed the palmer@functori@sort-and-merge-nano-tests branch from 6a3d0d6 to be9315e Compare February 6, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests test - related isues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant