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

Misc integration test improvements #1634

Merged
merged 1 commit into from
Sep 23, 2024
Merged

Misc integration test improvements #1634

merged 1 commit into from
Sep 23, 2024

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Sep 20, 2024

As part of adding Poetry + .python-version file support, a fair amount of buildpack refactoring will be required.

This backports some of the additional test scenarios we now have in the new Python CNB, along with some test general improvements.

It's been split out of later PRs for easier review.

Prep for #796 / #932.

GUS-W-16807487.

@edmorley edmorley self-assigned this Sep 20, 2024
@edmorley edmorley force-pushed the test-improvements branch 2 times, most recently from a5f0fa8 to 6230534 Compare September 20, 2024 17:32
@edmorley edmorley marked this pull request as ready for review September 20, 2024 17:37
@edmorley edmorley requested a review from a team as a code owner September 20, 2024 17:37
@edmorley edmorley force-pushed the test-improvements branch 2 times, most recently from 0ace7bb to b735d3b Compare September 22, 2024 14:10
As part of adding Poetry + `.python-version` file support, a fair
amount of buildpack refactoring will be required.

This backports some of the additional test scenarios we now have
in the new Python CNB, along with some test general improvements.

It's been split out of later PRs for easier review.
@edmorley edmorley enabled auto-merge (squash) September 23, 2024 07:51
@edmorley edmorley merged commit 80e7781 into main Sep 23, 2024
7 checks passed
@edmorley edmorley deleted the test-improvements branch September 23, 2024 08:01
@heroku-linguist heroku-linguist bot mentioned this pull request Sep 24, 2024
edmorley added a commit that referenced this pull request Oct 24, 2024
Since Heroku-22 has now been migrated to the new build infrastructure,
so no longer has the small system env var differences that were causing
these tests to fail on that stack (which is why they had been skipped as
part of #1634).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants