Skip to content

🌿 introduce test fixtures for devices, action attempts, access codes#1

Open
dsinghvi wants to merge 9 commits intomainfrom
ds/setup-ete-tests
Open

🌿 introduce test fixtures for devices, action attempts, access codes#1
dsinghvi wants to merge 9 commits intomainfrom
ds/setup-ete-tests

Conversation

@dsinghvi
Copy link
Copy Markdown
Owner

@dsinghvi dsinghvi commented Oct 1, 2023

🌿 In this PR, we configure the generated Java SDK to test against fake-seam-connect. The tests here are largely inspired from the existing tests in the Seam Python SDK.

How does it work

  • Each resource gets its own test file (e.g. AccessCodesTest, DevicesTest)
  • Each resource invokes fake-seam-connect on startup
  • Each test uses the generated SDK and asserts conditions

Gaps

There are a couple gaps in the test because fake-seam-connect does not completely mock Seam's API. In particular,

  • Connecting acounts via email does not work
  • Deleting connected accounts endpoint is not served
  • Listing Locks endpoints is not served

However, these gaps. are relatively minor as the most important resources/tests work as expected.

@dsinghvi dsinghvi force-pushed the ds/setup-ete-tests branch 4 times, most recently from 2af6c5c to f999d24 Compare October 4, 2023 06:14
@dsinghvi dsinghvi changed the title [in progress] ETE Testing 🌿 introduce test fixtures for devices, action attempts, access codes, etc. Oct 4, 2023
@dsinghvi dsinghvi changed the title 🌿 introduce test fixtures for devices, action attempts, access codes, etc. 🌿 introduce test fixtures for devices, action attempts, access codes Oct 4, 2023
@dsinghvi dsinghvi force-pushed the ds/setup-ete-tests branch 2 times, most recently from e71b7ee to 6e3967d Compare October 9, 2023 19:54
@dsinghvi dsinghvi force-pushed the ds/setup-ete-tests branch from 6e3967d to 6109747 Compare October 9, 2023 20:09
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.

1 participant