Skip to content

Add integration tests using vitest #53

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

Merged
merged 12 commits into from
Jul 9, 2025
Merged

Add integration tests using vitest #53

merged 12 commits into from
Jul 9, 2025

Conversation

joachimvh
Copy link
Contributor

All the other commits are random issues that popped up while implementing the tests. Some cleanup, some related to things still being hardcoded in the code.

@joachimvh joachimvh force-pushed the test/integration branch 17 times, most recently from b879fda to 69ab89b Compare July 2, 2025 11:13
@joachimvh joachimvh requested a review from termontwouter July 4, 2025 09:35
termontwouter
termontwouter previously approved these changes Jul 7, 2025
joachimvh added 9 commits July 9, 2025 07:41
This prevents Components.js build errors
When looking into contracts in the future,
this should be handled in a more robust way.
See the comment in AuxiliaryModesExtractor.ts for more details.
This allows policies on resources to also apply
to their auxiliary resources, such as .meta resources.
This includes derived resources,
which is referenced in the demo setup,
but not used in the example script.
This can be re-added in the future should it be required.
joachimvh added 3 commits July 9, 2025 07:41
For some reason, tsx could not exit gracefully when running these.
Either it got stuck, or exited but left the process running.
These issues did not occur with ts-node,
but the solution in this commit seemed more acceptable
than having a new dependency.
Future work could include investigating
if this is an issue with the server.
@joachimvh joachimvh merged commit de2fab0 into main Jul 9, 2025
3 checks passed
@joachimvh joachimvh deleted the test/integration branch July 9, 2025 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants