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

build: gradle plugin download sdl / introspection integration tests #1656

Merged
merged 6 commits into from
Jan 25, 2023

Conversation

dariuszkuc
Copy link
Collaborator

📝 Description

Migrate download sdl task and introspection task integration tests to a composite build. Failure scenarios (i.e. timeout) is still run through GradleRunner in the plugin project.

🔗 Related Issues

Verified

This commit was signed with the committer’s verified signature. The key has expired.
chrischdi Christian Schlotter
import com.expediagroup.graphql.plugin.gradle.tasks.GraphQLDownloadSDLTask

plugins {
kotlin("jvm") version "1.7.21"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

we could potentially load this from version catalog (but would have to update the test logic to copy the toml file - will look into this later)

@dariuszkuc
Copy link
Collaborator Author

dariuszkuc commented Jan 24, 2023

NOTE: big size of the PR is due to introspection/sdl results (1946 lines changed) copied over for wiremock in the composite build.

@dariuszkuc dariuszkuc enabled auto-merge (squash) January 24, 2023 23:16
@samuelAndalon samuelAndalon self-requested a review January 25, 2023 16:23
@dariuszkuc dariuszkuc merged commit 0459612 into ExpediaGroup:master Jan 25, 2023
@dariuszkuc dariuszkuc deleted the plugin_it branch January 25, 2023 16:23
dariuszkuc added a commit that referenced this pull request Jan 26, 2023
### 📝 Description

Migrate last remaining Gradle integration tests to a composite build.

### 🔗 Related Issues

* #1654
* #1656
* #1657
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants