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.

Sign up for GitHub

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

Jump to bottom

refactor: organize tests by endpoint and provide endpoint-specific make targets #566

Merged
merged 5 commits into from
Aug 14, 2024
Merged

refactor: organize tests by endpoint and provide endpoint-specific make targets #566

merged 5 commits into from
Aug 14, 2024

Commits on Aug 14, 2024

  1. refactor: organize tests by service and then by control vs data 

    In preparation for introducing per-endpoint make targets, we organize
the integration tests by service first and then control vs data
after. We add fully qualified namespaces for this organization so we
can filter on the command line.

We also add namespaces for `Integration` vs `Unit`.
    @malandis
    malandis committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    e2a7b96 View commit details
    Browse the repository at this point in the history

  2. feat: group integration tests targets by endpoint 

    Groups the integration test targets by endpoint and provides
endpoint-specific test targets:

- test-cache
- test-control
- test-token
- test-storage (a no-op since the storage client isn't implemented)

Each of these targets is conditional on the OS as in the previous PR.
    @malandis
    malandis committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    0738010 View commit details
    Browse the repository at this point in the history

  3. chore: format

    @malandis
    malandis committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    b04a488 View commit details
    Browse the repository at this point in the history

  4. refactor: test target runs dotnet test unconditionally 

    We decided to let the `test` target run the `test` command
unconditionally so as to not let any tests slip through the cracks. We
leave the endpoint patterns as before. We also remove the unit test
target as it is not necessary anymore.
    @malandis
    malandis committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    7c73222 View commit details
    Browse the repository at this point in the history

  5. refactor: rename targets totest-{cache,control,token}-endpoint 

    For clarity in other SDKs we will use the format
`test-{endpointname}-endpoint`.
    @malandis
    malandis committed Aug 14, 2024
    Configuration menu
    Copy the full SHA
    19ee8ce View commit details
    Browse the repository at this point in the history