Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@
"axios-retry": "^4.4.2"
},
"devDependencies": {
"@seamapi/blueprint": "^1.7.0",
"@seamapi/blueprint": "^1.8.0",
"@seamapi/fake-seam-connect": "^2.0.4",
"@seamapi/smith": "^1.1.0",
"@seamapi/types": "1.1022.0",
Expand Down
4 changes: 2 additions & 2 deletions src/lib/routes/access-codes/access-codes.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 4 additions & 4 deletions src/lib/routes/access-grants/access-grants.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/access-methods/access-methods.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/acs/encoders/encoders.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 4 additions & 4 deletions src/lib/routes/acs/entrances/entrances.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/action-attempts/action-attempts.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/connect-webviews/connect-webviews.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/connected-accounts/connected-accounts.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/customers/customers.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/devices/devices.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/devices/unmanaged/unmanaged.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/events/events.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/locks/locks.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/noise-sensors/noise-sensors.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

12 changes: 6 additions & 6 deletions src/lib/routes/spaces/spaces.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/thermostats/thermostats.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/lib/routes/user-identities/user-identities.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 5 additions & 4 deletions test/seam/connect/retry.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,10 @@ test('SeamHttp: does not retry POST requests by default', async (t) => {
routes: ['/devices/list'],
})

const err = await t.throwsAsync(async () => await seam.devices.list(), {
instanceOf: AxiosError,
})
const err = await t.throwsAsync(
async () => await seam.client.post('/devices/list'),
{ instanceOf: AxiosError },
)

t.is(err?.response?.status, 503)
})
Expand Down Expand Up @@ -94,7 +95,7 @@ test('SeamHttp: does not replay a POST after a mid-flight connection reset', asy
endpoint: `http://127.0.0.1:${address.port}`,
})

await t.throwsAsync(async () => await seam.devices.list())
await t.throwsAsync(async () => await seam.client.post('/devices/list'))

t.is(attempts, 1)
})
36 changes: 16 additions & 20 deletions test/seam/connect/seam-http-request.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,27 +50,23 @@ test('SeamHttpRequest: url is a URL for requests without query string', async (t
)
})

// UPSTREAM: The Seam API does not yet consistently support GET requests, so only POST is used.
test.failing(
'SeamHttpRequest: url is a URL for requests with query string',
async (t) => {
const { seed, endpoint } = await getTestServer(t)
const seam = SeamHttp.fromApiKey(seed.seam_apikey1_token, { endpoint })
const { url } = seam.devices.list({
limit: 10,
device_ids: [seed.august_device_1, seed.ecobee_device_1],
})
t.true(url instanceof URL)
t.deepEqual(
toPlainUrlObject(url),
toPlainUrlObject(
new URL(
`${endpoint}/devices/get?device_ids=${seed.august_device_1}&device_ids=${seed.ecobee_device_1}&limit=10&_strict=true`,
),
test('SeamHttpRequest: url is a URL for requests with query string', async (t) => {
const { seed, endpoint } = await getTestServer(t)
const seam = SeamHttp.fromApiKey(seed.seam_apikey1_token, { endpoint })
const { url } = seam.devices.list({
limit: 10,
device_ids: [seed.august_device_1, seed.ecobee_device_1],
})
t.true(url instanceof URL)
t.deepEqual(
toPlainUrlObject(url),
toPlainUrlObject(
new URL(
`${endpoint}/devices/list?device_ids=${seed.august_device_1}&device_ids=${seed.ecobee_device_1}&limit=10&_strict=true`,
),
)
},
)
),
)
})

test('SeamHttpRequest: url is a URL when endpoint is a url without a path', async (t) => {
const { seed } = await getTestServer(t)
Expand Down
Loading