Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Website integration tests #416

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Conversation

FireBoyAJ24
Copy link
Contributor

@FireBoyAJ24 FireBoyAJ24 commented Jul 9, 2024

Description

I have added the website integration test functionality. Details on the structure of the integration tests and how to create tests is on the design specification.

Verification

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

  • All ROS topics work
  • All WEB tests work (take note above)

Resources

@SPDonaghy
Copy link
Contributor

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.
#406

@SPDonaghy
Copy link
Contributor

SPDonaghy commented Aug 9, 2024

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
Copy link
Contributor Author

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.

@FireBoyAJ24
Copy link
Contributor Author

We need to remove the Raye's data as I have already converted them into YAML.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding a HTTP POST Request for Integration Tests
2 participants