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

Fix failing CLI test #3567

Merged
merged 1 commit into from
Dec 18, 2019
Merged

Conversation

lexi-lambda
Copy link
Contributor

Description

Just patching this test that was broken by the v1.0.0 release process.

Affected components

  • Tests

@netlify
Copy link

netlify bot commented Dec 18, 2019

Deploy preview for hasura-docs ready!

Built with commit 45b7e9e

https://deploy-preview-3567--hasura-docs.netlify.com

@lexi-lambda lexi-lambda merged commit bdf79df into hasura:master Dec 18, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-3567.herokuapp.com is deleted

@lexi-lambda lexi-lambda deleted the fix-cli-migrate-test branch December 18, 2019 22:45
polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
hasura-bot pushed a commit that referenced this pull request Feb 14, 2022
## Description

This PR adds all the scaffolding for tests that require remote servers. It is mostly a refactor of `Feature`; where we listed for each test a list of individual backends, we now provide a list of `Context`s, that allows for tests to specify not only how it should be setup, but also what state needs to be carried around throughout the test. This will be useful when launching custom remote servers.

Additionally, this PR:
- cleans the way we generate logs in the engine as part of the tests
- cleans the cabal file
- introduce a few more helpers for sending commands to the engine (such as `postMetadata_`)
- allows for headers in queries sent to the engine (to support permissions tests)
- adds basic code to start / stop a "remote" server

This PR is a pre-requisite of #3567.

PR-URL: hasura/graphql-engine-mono#3573
Co-authored-by: jkachmar <8461423+jkachmar@users.noreply.github.com>
GitOrigin-RevId: 05f808c6b85729dbb3ea6648c3e10a3c16b641ef
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