Skip to content

Update pysofar tests#37

Closed
tcj wants to merge 13 commits intomasterfrom
feature/update-tests
Closed

Update pysofar tests#37
tcj wants to merge 13 commits intomasterfrom
feature/update-tests

Conversation

@tcj
Copy link
Copy Markdown
Collaborator

@tcj tcj commented Jun 2, 2023

These commits update the pysofar tests with a variety of goals:

  • change the Spotter being tested most often to the one that is granted to API trial users
  • open up testing of SST sensor data
  • open up testing of Smart Mooring sensor data
  • further attempts to re-use existing API instance between workers (this expands beyond tests)
  • support limiting the number of concurrent processes by request
  • update copyright info
  • mark one test as an expected failure

Currently, all tests except two pass, with one expected failure. The two tests that don't pass are due to a documented difference in API responses between Spotters 'owned by' the token user versus Spotters granted read-only access via other means.

@tcj tcj requested review from evanShap, g-mo and pieterbartsmit June 2, 2023 01:31
@tcj tcj self-assigned this Jun 2, 2023
@tcj tcj marked this pull request as draft June 6, 2023 21:01
@tcj tcj closed this Sep 27, 2023
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