Skip to content

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

Merged
merged 8 commits into from
Sep 4, 2018
Merged

Add documentation for unit testing #677

merged 8 commits into from
Sep 4, 2018

Conversation

lorjala
Copy link
Contributor

@lorjala lorjala commented Aug 22, 2018

Introductory documentation for unit testing framework.

depends on ARMmbed/mbed-os#7819 and ARMmbed/mbed-cli#734

@lorjala
Copy link
Contributor Author

lorjala commented Aug 22, 2018

@OPpuolitaival

@@ -0,0 +1,58 @@
## Unit testing
Copy link
Contributor

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.

Amanda Butler added 3 commits August 27, 2018 15:32
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",
Copy link
Contributor

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.

@AnotherButler
Copy link
Contributor

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.

@AnotherButler
Copy link
Contributor

Copy link
Contributor

@ashok-rao ashok-rao left a 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.

@theotherjimmy
Copy link
Contributor

I changed the Mbed CLI invocation. I'll create another PR with this content + that change.

@kegilbert
Copy link
Contributor

kegilbert commented Sep 4, 2018

@theotherjimmy Should this PR be closed if #703 has the same changes?

@AnotherButler AnotherButler merged commit 4b1cde8 into ARMmbed:development Sep 4, 2018
@cmonr
Copy link
Contributor

cmonr commented Sep 5, 2018

@kegilbert The description in #703 has this line:

Superseeds #677

@theotherjimmy
Copy link
Contributor

That should be okay. Let's check on #703

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants