Turn on basic testing #23
Closed
Commits on Mar 10, 2024
-
I _wanted_ to turn on the automated tests that we have, but it fails because pytest is not installed. @couryrr-afs can you please fix the automated tests so we can include them as well. <details> <summary>Existing automated e2e tests fail</summary> ``` $ bash demo-automated-testing.sh [+] Building 0.0s (0/0) docker:desktop-linux [+] Running 2/0 ✔ Container everest-ac-demo-mqtt-server-1 Created 0.0s ✔ Container everest-ac-demo-manager-1 Created 0.0s Attaching to everest-ac-demo-manager-1, everest-ac-demo-mqtt-server-1 everest-ac-demo-mqtt-server-1 | 1710048535: mosquitto version 2.0.10 starting everest-ac-demo-mqtt-server-1 | 1710048535: Config loaded from /mosquitto/config/mosquitto.conf. everest-ac-demo-mqtt-server-1 | 1710048535: Opening ipv4 listen socket on port 1883. everest-ac-demo-mqtt-server-1 | 1710048535: Opening websockets listen socket on port 9001. everest-ac-demo-mqtt-server-1 | 1710048535: mosquitto version 2.0.10 running everest-ac-demo-manager-1 | ./run-test.sh: line 3: pytest: not found everest-ac-demo-manager-1 exited with code 127 ``` </details> Signed-off-by: Shankari <[email protected]>
-
Turn it on for pull requests so we can test that it works
Signed-off-by: Shankari <[email protected]>
-
-
Set a timeout so that the action doesn't run forever
When we have true automated tests, we will not need this, but the scripts all currently run without -d and expect UI interaction, so we will timeout. Note that this means that the action will always be canceled, but not much we can do to avoid that now. Signed-off-by: Shankari <[email protected]>
-
Set a timeout so that the action doesn't run forever
When we have true automated tests, we will not need this, but the scripts all currently run without -d and expect UI interaction, so we will timeout. Note that this means that the action will always be canceled, but not much we can do to avoid that now. Signed-off-by: Shankari <[email protected]>
-
Merge branch 'turn_on_basic_testing' of https://github.com/US-JOET/ev…
…erest-demo into turn_on_basic_testing
-
Remove timeout from command since we have it as an action parameter
Signed-off-by: Shankari <[email protected]>
-
Switch to mac and install docker
To be as close to the laptop environment as possible
-
Since 10 minutes is too short for OSX ``` c9a245e33d92 Extracting [=================> ] 357.1MB/1.048GB c9a245e33d92 Extracting [=================> ] 357.6MB/1.048GB c9a245e33d92 Extracting [=================> ] 358.2MB/1.048GB c9a245e33d92 Extracting [=================> ] 358.7MB/1.048GB c9a245e33d92 Extracting [=================> ] 359.3MB/1.048GB Error: The action 'Download and launch' has timed out after 10 minutes. ```
-
Bump up the timeout to 30 mins
Download takes 20 mins Build should take another 10 mins
-
Verify docker compose version to address issue with build mounts
Signed-off-by: Shankari <[email protected]>
-
Let's check the colima version
Signed-off-by: Shankari <[email protected]>
-
And create the host folder if we need to
Although the error is likely because we are trying to mount a directory outside $HOME and /tmp/colima
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.