Add card.binary HIL tests. #124
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
small fixes for reliability
fix: NoteBinaryMaxEncodedLength wasn't adding the transport framing overhead.
chore: add cron schedule so the runner is in weekly use. Also adds more tags that relate to the required configuration of the runner.
chore: astyle fixes
remove md5srv tests for now
chore: move workflows out of subdirectory
fix: unique name for proxy event aliases
fix: workflow linter works locally in VSCode now that they are in the main workflow folder. (I read docs that said workflows can appear in subsirectories.)
fix: apparently cannot reference the `env` context. Remove the wait-for since the local tunnel check also tests connectivity to md5srv
fix: wait-on is required.
chore: setup python. This is a pain keeping 2-3 environments working (act locally, act with containers, github self-hosted runners)
fix: jiggle use of env vars
fix: the NOTE_C_DEP location contains spaces
crazy: runner environment variables are not available in the `env` context.
fix: alternate way to set the build flags
fix: pio build flags not propagating
fix: propagate env vars so they are available to later steps.
fix working directory
fix: try again with the working directory
chore: increase number of retries waiting for note.add to complete and be routed.
fix: I'm an idiot. still half asleep.
fix: my previous comment still stands
fix: run test project dir
chore: replace NoteBInaryReceiveAll
chore: back out previous change.
fix: typo in test name
fix: id-10-t error
fixes bats tests for md5srv after MD5SRV_XXX environment variables were reworked to serve as defaults.
chore: add longer and configurable route timeouts
chore: retry transfer handler when it fails.
chore: retry web.get validation on more http errors.
```
{"req":"web.get","route":"card.binary.6126291556","name":"Random_1234?chunk=3","content":"application/octet-stream","binary":true,"crc":"0025:AD91438A"}
{"result":504,"body":{"err":"wire: no response from service {service}: socket read {network}: network timeout {network-timeout} {network}"}}
```
feat: add defaults so that platformio.ini doesn't have to be continually switched between a local dev build and CI build.
fix: remove debug echo
fix: ensure python venv is setup before starting the md5 server.
feat: support for retrying handlers and a way for them to indicate they should not be retried.
chore: astyle
fix: vscode screwed up the indentation
feat: adds support for using tunnelmole proxy
feat: include md5srv BATS tests
fix: workflow path
chore: rework code to use the latest API
chore: use self-hosted runner for md5srv tests
fix: workflow didn't work on self-hosted runner. trying a containerized build instead.
fix: set TERM
fix: wait for server.
fix: wait for a non-server error. Ensure md5 server is cleaned up after each test.
fix: add logging for json decode errors
feat: add output logging
fix: rerun tests on failure.
fix: token
fix: add missing file
fix: remove pause so jobs complete
- Fix various typos. - Comment out various tests that aren't passing, yet. Most of these are exercising binary transfers over the Notecard's LPUART. There's a bug here that we're actively debugging in the Notecard firmware. - Tweak some of the tests so that they send a carriage return when doing a binary transmit of arbitrary data. We had a bug in the past where \r was not handled properly by the Notecard firmware.
- mikepenz/action-junit-report#23 (comment) - Update platformio.ini to remove extra_scripts.
zfields
approved these changes
Dec 5, 2023
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Moving the work from #91 over to this PR.