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

Add hackernews functional integration test #413

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

runningcode
Copy link
Contributor

@runningcode runningcode commented Jan 16, 2025

This adds this repo as a composite build to the hacker news app and builds it.
You can see in this build scan that the composite build is being applied to the :app module.

https://scans.gradle.com/s/fq5pdezizjitk/build-dependencies?toggled=W1sxXSxbMSwwXSxbMSwwLFs0ODVdXV0

Copy link
Contributor Author

runningcode commented Jan 16, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@runningcode runningcode force-pushed the no/test_checkout_hackernews branch from 0aa8f81 to dd4c239 Compare January 16, 2025 21:29
@runningcode runningcode force-pushed the no/test_checkout_hackernews branch 7 times, most recently from 3d3b22a to b49586f Compare January 17, 2025 20:29
@@ -140,7 +140,7 @@ dependencies {
testRuntimeOnly(libs.junit.jupiter.engine)
testRuntimeOnly(libs.junit.platform.launcher)

"functionalTestImplementation"(projects.emergeGradlePlugin.plugin)
"functionalTestImplementation"(project(":plugin"))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This caused the same compilation issue we were seeing before when adding this as a composite build so I just removed the generated accessor.

@runningcode runningcode changed the title test checkout hackernews Add hackernews functional integration test Jan 17, 2025
@runningcode runningcode force-pushed the no/test_checkout_hackernews branch 2 times, most recently from d34cc93 to 93592c8 Compare January 17, 2025 21:23
@runningcode runningcode marked this pull request as ready for review January 17, 2025 21:24
@runningcode runningcode force-pushed the no/test_checkout_hackernews branch from 0e39253 to 1a4ade9 Compare January 17, 2025 22:24
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