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

New testing post #584

Merged
merged 42 commits into from Apr 27, 2019

Conversation

1 participant
@phil-opp
Copy link
Owner

commented Apr 25, 2019

This PR deprecates the "Unit Tests" and "Integration Tests" posts in favor of a new "Testing" post. Instead of relying on bootimage test, we now use custom test frameworks and the bootimage runner. This has the following advantages:

  • All tests run on the target system in QEMU (instead of running unit tests on the host system)
  • Integration tests now follow the default cargo conventions and live in a tests directory (instead of creating additional executables in src/bin).
  • No more need for bootimage test: All tests are run with a simple cargo xtest.

This PR is still work in progress. There are a few things left to do:

  • Mention that bootimage runner uses a configurable timeout for tests (5 minutes by default)
  • Remove old #[cfg(not(test))] attributes from all posts
  • Update the post numbers
    • Folder names
    • Branch names
    • Post Weights
    • in Readme
  • Update the post-xx tags [prepared in #587]
    • Note that e.g. post-10 becomes post-09
  • Write Summary and What's Next
  • Add deprecation message to deprecated posts
  • Mention requirements in deprecated posts
  • Update publish date
  • [ ] Mention the --quiet flag for supressing output from bootimage runner?
  • [ ] Show how to create a attribute-like procedural macro for auto-generating the serial_println messages in tests? I have the macro ready, but it is too much effort just for avoiding a serial_println

Blocked on #582

@phil-opp phil-opp added the relnotes label Apr 25, 2019

@phil-opp phil-opp force-pushed the rewrite-test-post branch from 0a26150 to 54f582a Apr 26, 2019

@phil-opp phil-opp added this to In Progress in Roadmap Apr 26, 2019

@phil-opp phil-opp marked this pull request as ready for review Apr 27, 2019

@phil-opp phil-opp force-pushed the rewrite-test-post branch from 5f80852 to a8865e2 Apr 27, 2019

@phil-opp phil-opp merged commit 3bcb2ee into master Apr 27, 2019

1 check passed

phil-opp.blog_os Build #20190427.38 succeeded
Details

Roadmap automation moved this from In Progress to Done Apr 27, 2019

@bors bors bot deleted the rewrite-test-post branch Apr 27, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.