search

keystonejs / keystone

The superpowered headless CMS for Node.js — built with GraphQL and React
https://keystonejs.com
MIT License
9.18k stars 1.15k forks source link

Update tests for atomicity, prefer /tmp, and add new access control tests #9032

Closed dcousens closed 7 months ago

dcousens commented 7 months ago

This pull request updates our tests to be atomic by design, by not sharing temporary directories or other resources. Additionally, this pull request adds new access control tests to verbosely test as many of the possible code paths as we can.

For many moons the tests have been using --runInBand to sidestep a number of problems in relation to concurrency and databases sharing resources. This pull request should fix that problem by using:

Thanks to @ChrisLaneAU for help with early work on this

codesandbox-ci[bot] commented 7 months ago

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 36d787c3bc04a988edc991831034ff0c80065996:

Sandbox Source
@keystone-6/sandbox Configuration