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

chore(cli): Add additional tests for sdl generator #6200

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

dac09
Copy link
Collaborator

@dac09 dac09 commented Aug 10, 2022

Adds additional tests to prevent regression fixed in #6195

Why?

When we run sdl... internally we also generate service files. We had a regression where the scenario file wasn't being generated - these tests just make sure we don't regress on it again!

@nx-cloud
Copy link

nx-cloud bot commented Aug 10, 2022

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 5dd30be. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 14 targets

Sent with 💌 from NxCloud.

@netlify
Copy link

netlify bot commented Aug 10, 2022

Deploy Preview for redwoodjs-docs canceled.

Name Link
🔨 Latest commit 5dd30be
🔍 Latest deploy log https://app.netlify.com/sites/redwoodjs-docs/deploys/62f3babe1309aa0008178e31

@dac09 dac09 added release:chore This PR is a chore (means nothing for users) fixture-ok Override the test project fixture check labels Aug 10, 2022
@dac09 dac09 requested a review from dthyresson August 10, 2022 15:50
Copy link
Contributor

@dthyresson dthyresson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice to have this in addition to snapshots. Thanks

@dthyresson dthyresson merged commit 0e9754b into redwoodjs:main Aug 10, 2022
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Aug 10, 2022
dac09 added a commit to dac09/redwood that referenced this pull request Aug 15, 2022
…9/redwood into fix/cell-prerender-graphql-fallback

* 'fix/cell-prerender-graphql-fallback' of github.com:dac09/redwood:
  Stylistic suggestions
  Leverage PROJECT_CWD to set cwd in yarn 3 bin "proxies" (redwoodjs#6199)
  chore(cli): Add additional tests for sdl generator (redwoodjs#6200)
@jtoar jtoar modified the milestones: next-release, v3.0.0 Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixture-ok Override the test project fixture check release:chore This PR is a chore (means nothing for users)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants