Add more tests #212
Merged
Add more tests #212
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
Sh3Rm4n
force-pushed
the
more-tests
branch
2 times, most recently
from
2ee8ed2
to
cc7f353
Compare
Sh3Rm4n
force-pushed
the
more-tests
branch
2 times, most recently
from
ff13865
to
027df7a
Compare
Sh3Rm4n
force-pushed
the
more-tests
branch
2 times, most recently
from
d45a0f6
to
d472554
Compare
Sh3Rm4n
force-pushed
the
more-tests
branch
2 times, most recently
from
e123b09
to
95b17fc
Compare
Sh3Rm4n
force-pushed
the
more-tests
branch
3 times, most recently
from
0733021
to
e0b0054
Compare
Which is the discovery board used for this testsuite
This is needed, so probe-run can get enough information to verify a test sucess even in release mode
The puller pin state was not updating fast enough. Insert a slight delay, so this does not happen
The extension trait is now only implemented for `u32`, so defining the type is not needed anymore and can be inferred.
In addition to the changes, test them
As this section gets bigger, I moved it into its own README.md in testsuite/.
Even after unplugging my STM32F3DISCOVERY MB1035D for several hours, I'm getting a noise error in this test case.
When using flip-link for stack overflow protection, it parses the linker scripts/fragments for RAM memory addresses and expects the keywords 'ORIGIN' and 'LENGTH' to be used in its verbose form. Allow friction-less integration of flip-link and this crate by using the verbose form.
Sh3Rm4n
referenced
this pull request
in sirhcel/stm32f3xx-hal
Jun 14, 2021
This struct could be converted from a Baud value already used in exisiting code and should therefor not break it.
|
SPI and I2C will have to wait for later PRs. This includes slight changes to watchdog and adc, which I did not expect. So to not blow this PR up anymore, I'll keep it as is |
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.
With focus on gpio tests for now. This is very much WIP.
I'd like to release a new version of this crate soon, but I'd also like to reduce the smoke-testing work (which I've done for 0.6.0 in #177) while automating some testing.