medplum / medplum / 23459635441

92%

push

github

Various server cleanups (#8758)

* Fix variable shadowing in login test

Rename inner project to newProject in the 'require Google auth'
test to avoid shadowing the outer project variable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Long <ml@medplum.com>

* Fix typo in token.ts comment

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Long <ml@medplum.com>

* Remove unnecessary export from createPatient

The function is only used within newpatient.ts, so the export is
unnecessary public API surface.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Long <ml@medplum.com>

* Narrow return types to WithId<> where IDs are guaranteed

- getProjectsByGoogleClientId: Project[] → WithId<Project>[]
- getStandardClientById: ClientApplication → WithId<ClientApplication>

Resources read from the database or initialized with known IDs always
have an id field, so the narrower types are more accurate.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Long <ml@medplum.com>

* Minor test cleanups

- Fix describe block name in db-configure-indexes.test.ts
- Remove empty imports in transaction.test.ts and oauth/utils.test.ts
- Use node:crypto instead of crypto in reindex.test.ts

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Long <ml@medplum.com>

* Clarify variable names in revoke test

Rename accessToken to aliceAccessToken and avoid destructuring
shadowing to make the cross-user revocation test easier to follow.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Long <ml@medplum.com>

---------

Signed-off-by: Matt Long <ml@medplum.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

17924 of 20350 branches covered (88.08%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

32578 of 34410 relevant lines covered (94.68%)

14572.91 hits per line