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

doc: Unit test documentation for nRF9160 Asset Tracker v2 #7494

Merged
merged 1 commit into from
May 13, 2022

Conversation

divipillai
Copy link
Contributor

Add unit test documentation for nRF9160 Asset Tracker v2

Signed-off-by: divya pillai divya.pillai@nordicsemi.no

@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label May 4, 2022
Copy link
Contributor

@coderbyheart coderbyheart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in commit message.

applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
@divipillai divipillai force-pushed the CIA-568_unit_test branch 2 times, most recently from 0db760b to dab3761 Compare May 4, 2022 15:07
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 4, 2022

Integration test specification

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-nrf-iot_thingy91 X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

Copy link

@balaji-nordic balaji-nordic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this. Requested a few changes.

@shantha-14 This is FYI. Here we are adding documentation on how to run the asset tracker v2 unit tests.

applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/doc/unit_test.rst Outdated Show resolved Hide resolved
doc/nrf/releases/release-notes-changelog.rst Outdated Show resolved Hide resolved
applications/asset_tracker_v2/README.rst Show resolved Hide resolved
@divipillai divipillai force-pushed the CIA-568_unit_test branch 3 times, most recently from ae8d510 to 8a1e7ac Compare May 10, 2022 11:05
Copy link

@balaji-nordic balaji-nordic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider joining line 68 and line 69 in unit_test.rst. I think line 69 does not need to be in a new line. Everything else looks good.

Added Unit test documentation for nRF9160 Asset Tracker v2

Signed-off-by: divya pillai <divya.pillai@nordicsemi.no>
@rlubos rlubos merged commit 2235a5b into nrfconnect:main May 13, 2022
@divipillai divipillai deleted the CIA-568_unit_test branch July 21, 2022 07:17

Enter the following west commands to execute the tests on different board targets:

* :ref:`zephyr:native_posix` board target:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* :ref:`zephyr:native_posix` board target:
* :ref:`zephyr:native_posix` board target: 🎯

@@ -103,6 +103,7 @@ nRF9160: Asset Tracker v2
* Support for QEMU x86 emulation.
* Support for the :ref:`lib_nrf_cloud_pgps` flash memory partition under certain conditions.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Run
applications/asset_tracker_v2/doc/unit_test.rstapplications/asset_tracker_v2/doc/unit_test.rst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants