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

Conformance tests are running against the buildpack version, not source tree #231

Closed
di opened this issue Oct 21, 2020 · 1 comment · Fixed by #236
Closed

Conformance tests are running against the buildpack version, not source tree #231

di opened this issue Oct 21, 2020 · 1 comment · Fixed by #236

Comments

@di
Copy link
Member

di commented Oct 21, 2020

The conformance tests for this repo are configured to use buildpacks, so they're using the version of this framework that is already released and supported in the buildpack rather than what's in the current source tree.

Instead, the conformance tests should be run locally against the current source tree. For example, this is what the .NET framework does.

This will ensure that we are aware if a given PR breaks the conformance tests, as the status will match the current state of the repo.

@di
Copy link
Member Author

di commented Oct 22, 2020

See GoogleCloudPlatform/functions-framework-python#95 for another example of how to do this.

hdp617 added a commit that referenced this issue Nov 11, 2020
* run conformance tests against the current FF version specified by  $GITHUB_SHA instead of the default version in buildpacks
* update conformance action version
* remove no longer used script
* remove nodejs14 since this runtime is not yet live and does not have tag :latest.

fixes #231
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant