UBCSailbot / sailbot_workspace

UBC Sailbot's monorepo
https://ubcsailbot.github.io/sailbot_workspace/main/
Apache License 2.0
4 stars 1 forks source link

Adding Website integration tests #416

Open FireBoyAJ24 opened 2 months ago

FireBoyAJ24 commented 2 months ago

Description

Verification

Note: Error not from the integration tests side is considered acceptable but preferably avoided

Resources

SPDonaghy commented 1 month ago

Just putting this here as its relevant to the web integration tests. Thinking of mocking the HTTP endpoints for the unit tests and leaving it to these integration tests to actually test the functionality of the endpoints. https://github.com/UBCSailbot/sailbot_workspace/issues/406

SPDonaghy commented 1 month ago

Note: Error not from the integration tests side is considered acceptable but preferably avoided

Also are we going to wait until web endpoints are ready (and the WEB integration tests work) before merging this? Just to verify there's no bugs with the test itself

FireBoyAJ24 commented 1 month ago

Note: Error not from the integration tests side is considered acceptable but preferably avoided

Also are we going to wait until web endpoints are ready (and the WEB integration tests work) before merging this? Just to verify there's no bugs with the test itself

I would prefer to wait until the remote transciver is done to remove any bugs that pops out of the integration tests.