-
Notifications
You must be signed in to change notification settings - Fork 178
Add documentation for unit testing #677
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
Conversation
docs/tools/offline/cli-unittest.md
Outdated
@@ -0,0 +1,58 @@ | |||
## Unit testing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure this should be its own page. I think it would make more sense in the CLI test and debug section.
Copy edit for consistent tense across docs and minor grammar nits.
Copy edit file for active voice.
Copy edit file, mostly for active voice.
docs.json
Outdated
@@ -722,6 +722,10 @@ | |||
"type": "markdown", | |||
"url": "https://github.com/ARMmbed/mbed-os-5-docs/blob/development/docs/tools/offline/cli-test-debug.md" | |||
}, | |||
{ | |||
"type": "markdown", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we've removed this page.
I've copy edited this file, mostly for active voice. Please review my changes to make sure I didn't accidentally change the meaning of anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Docs look good @lorjala . Let's have a detailed run & execution of these instructions and tests during full OoB.
I changed the Mbed CLI invocation. I'll create another PR with this content + that change. |
@theotherjimmy Should this PR be closed if #703 has the same changes? |
@kegilbert The description in #703 has this line:
|
That should be okay. Let's check on #703 |
Introductory documentation for unit testing framework.
depends on ARMmbed/mbed-os#7819 and ARMmbed/mbed-cli#734