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

Port access control tests to new interfaces #4931

Merged
merged 1 commit into from
Mar 12, 2021

Conversation

timleslie
Copy link
Contributor

@timleslie timleslie commented Feb 23, 2021

We need to test the access control API of the new interfaces. This PR ports the existing tests to use the new interfaces.

This has exposed various internal fixes which need to happen before these tests will pass. I will ship separate PRs for each of these changes, and then put this PR up for review once they are complete.

@vercel
Copy link

vercel bot commented Feb 23, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/keystonejs/keystone-next-docs/CW17RVDGsxtiy1ZaSWD2rcGMupjZ
✅ Preview: https://keystone-next-docs-git-udpate-access-control-tests-keystonejs.vercel.app

@vercel vercel bot temporarily deployed to Preview February 23, 2021 22:34 Inactive
@vercel vercel bot temporarily deployed to Preview February 23, 2021 22:37 Inactive
@vercel vercel bot temporarily deployed to Preview February 23, 2021 22:57 Inactive
@vercel vercel bot temporarily deployed to Preview February 23, 2021 23:08 Inactive
@vercel vercel bot temporarily deployed to Preview February 23, 2021 23:23 Inactive
@vercel vercel bot temporarily deployed to Preview February 24, 2021 01:26 Inactive
@vercel vercel bot temporarily deployed to Preview February 24, 2021 01:31 Inactive
@vercel vercel bot temporarily deployed to Preview February 25, 2021 03:01 Inactive
@timleslie timleslie force-pushed the udpate-access-control-tests branch from 53a8eb8 to ea9aa9e Compare March 1, 2021 22:36
@changeset-bot
Copy link

changeset-bot bot commented Mar 1, 2021

🦋 Changeset detected

Latest commit: 499f089

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@keystone-next/keystone-legacy Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel vercel bot temporarily deployed to Preview March 1, 2021 22:36 Inactive
@timleslie timleslie force-pushed the udpate-access-control-tests branch from ea9aa9e to bb3ce3f Compare March 2, 2021 02:28
@vercel vercel bot temporarily deployed to Preview March 2, 2021 02:28 Inactive
@timleslie timleslie force-pushed the udpate-access-control-tests branch from bb3ce3f to 072a7fc Compare March 5, 2021 04:50
@vercel vercel bot temporarily deployed to Preview March 5, 2021 04:50 Inactive
@timleslie timleslie force-pushed the udpate-access-control-tests branch from 072a7fc to b00fdee Compare March 8, 2021 21:48
@vercel vercel bot temporarily deployed to Preview March 8, 2021 21:48 Inactive
@timleslie timleslie force-pushed the udpate-access-control-tests branch from b00fdee to 42379cd Compare March 9, 2021 21:49
@vercel vercel bot temporarily deployed to Preview March 9, 2021 21:49 Inactive
@timleslie timleslie force-pushed the udpate-access-control-tests branch from 42379cd to c81c0ef Compare March 9, 2021 23:34
@vercel vercel bot temporarily deployed to Preview March 9, 2021 23:34 Inactive
@vercel vercel bot temporarily deployed to Preview March 11, 2021 23:54 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 00:06 Inactive
@timleslie timleslie changed the title [WIP] Port access control tests to new interfaces Port access control tests to new interfaces Mar 12, 2021
@vercel vercel bot temporarily deployed to Preview March 12, 2021 00:38 Inactive
@timleslie timleslie requested a review from a team March 12, 2021 00:39
@vercel vercel bot temporarily deployed to Preview March 12, 2021 01:34 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 01:58 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 02:24 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 02:35 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 02:46 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 03:03 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 03:15 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 03:19 Inactive
@vercel vercel bot temporarily deployed to Preview March 12, 2021 03:35 Inactive
@timleslie timleslie requested a review from emmatown March 12, 2021 03:36
@timleslie timleslie enabled auto-merge (squash) March 12, 2021 03:36
@timleslie timleslie merged commit a4002b0 into master Mar 12, 2021
@timleslie timleslie deleted the udpate-access-control-tests branch March 12, 2021 03:40
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