Un-skip inventory integration tests and adapt to cross-location adjustment fields#196
Closed
menglinblock wants to merge 5 commits into
Closed
Un-skip inventory integration tests and adapt to cross-location adjustment fields#196menglinblock wants to merge 5 commits into
menglinblock wants to merge 5 commits into
Conversation
Generated by Fern CLI Version: unknown Generators: - fernapi/fern-python-sdk: 4.55.4
…tment fields Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…flakiness Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
Author
|
🤖 CI-verification vehicle only — the 3 commits were cherry-picked onto fern-bot/2026-07-13_21-32-27_131 (release PR #195), which is now green. Closing. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 Opened by menglin's AI agent.
Do not merge — CI verification only
This draft PR exists solely to run the integration test suite (which has access to
TEST_SQUARE_TOKEN) against the changes in release PR #195 plus a test fix. The final commit (Un-skip inventory integration tests and adapt to cross-location adjustment fields) is intended to be cherry-picked ontofern-bot/2026-07-13_21-32-27_131.What changed
In the 2026-07-15 Square API version,
InventoryAdjustment.location_idwas replaced byfrom_location_idandto_location_id(the API now returns 400adjustment.from_location_id cannot be emptyiffrom_location_idis missing on anADJUSTMENTchange). This PR:@pytest.mark.skip(...)decorators fromtests/integration/test_inventory.py(added in 🌿 Fern Regeneration -- November 21, 2025 #175, Nov 2025, as "Temporarily skip tests with false failures" / server-side errors).ADJUSTMENTpayloads (helpercreate_initial_adjustment,test_batch_change_inventory,test_retrieve_inventory_adjustments), replaces the singlelocation_idfield withfrom_location_idandto_location_idboth set to the same location ID (same-location adjustment) — matching the fix already applied in the Java/PHP/.NET/Node SDKs.location_ids=[...]query parameters are unchanged (InventoryPhysicalCount.location_idis unchanged in this API version).The base branch here is
masteronly because the new field names exist only in the regenerated SDK on the fern-bot release branch; the first two commits are the fern-bot regeneration itself.Made with Cursor