Skip to content

remove test using soon-to-be-gone endpoint #1468

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

Merged
merged 3 commits into from
Mar 13, 2025

Conversation

xavdid-stripe
Copy link
Member

@xavdid-stripe xavdid-stripe commented Mar 13, 2025

Why?

The /v1/invoices/upcoming/lines endpoint is being removed soon. As a result, this test is causing CI failures upstream. I'm removing it for now in the hopes that we can replace it with a endpoint that accomplishes the same thing, but still exists.

Another test was removed entirely.

We don't need to change anything auto-generated; that'll get picked up when the spec is updated. These changes are purely for manual files.

What?

  • comment out test
  • remove endpoint-specific test

See Also

@xavdid-stripe xavdid-stripe requested a review from a team as a code owner March 13, 2025 00:19
@xavdid-stripe xavdid-stripe requested review from prathmesh-stripe and removed request for a team March 13, 2025 00:19
@xavdid-stripe xavdid-stripe enabled auto-merge (squash) March 13, 2025 00:19
@xavdid-stripe xavdid-stripe merged commit bfb694b into master Mar 13, 2025
15 checks passed
@xavdid-stripe xavdid-stripe deleted the remove-upcoming-lines-test branch March 13, 2025 12:47
ramya-stripe pushed a commit that referenced this pull request Mar 19, 2025
* upgrade ruff version (#1456)

* upgrade ruff version

* remove old comment

* update comment

* add codeowners file (#1457)

* Remove incorrect changelog entry about parse_snapshot_event (#1461)

* Update generated code (#1450)

* Update generated code for v1463

* Update generated code for v1494

* Update generated code for v1495

* Update generated code for v1501

* Update generated code for v1505

---------

Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@users.noreply.github.com>
Co-authored-by: prathmesh-stripe <165320323+prathmesh-stripe@users.noreply.github.com>

* Bump version to 11.6.0

* changed the public test api key to match what is currently published on docs.stripe.com (#1466)

* remove test using soon-to-be-gone endpoint (#1468)

---------

Co-authored-by: David Brownman <109395161+xavdid-stripe@users.noreply.github.com>
Co-authored-by: helenye-stripe <111009531+helenye-stripe@users.noreply.github.com>
Co-authored-by: stripe-openapi[bot] <105521251+stripe-openapi[bot]@users.noreply.github.com>
Co-authored-by: jar-stripe <jar@stripe.com>
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