Skip to content
This repository was archived by the owner on Jul 21, 2021. It is now read-only.

Conversation

@ChiefGokhlayehBosch
Copy link
Contributor

@ChiefGokhlayehBosch ChiefGokhlayehBosch commented Sep 7, 2020

First import of integration test capabilities from Kiso's predecessor, based on preceding work by @khalifima (changed and refactored his work a little). Not much effort has been put into upgrading integration test code quality. What you see pretty much how it was sourced. The things we changed are manly cosmetic, one notable exception being the license headers, which were updated to EPL-2.0.

This draft currently only contains a fraction of the original integration test suite (namely only a single Essentials MCU UART test), more tests are to be added later on (in future PRs). The test spec has been updated accordingly.

To build an integration test, you first have to configure your CMake workspace with the ENABLE_INTEGRATION_TESTING setting set to ON and KISO_INTEGRATION_TEST_NAME set to a package containing integration tests.

    "cmake.configureSettings": {
        "KISO_BOARD_NAME": "NucleoF767",
        "KISO_INTEGRATION_TEST_NAME": "core/essentials",
        "ENABLE_INTEGRATION_TESTING": true
    }

The added test is only available for NUCLEO-F767ZI boards. To build the binary, run cmake with target testentry (or just build all). As per usual, you can use target flash to flash the binary.

Note KISO_APPLICATION_NAME is not needed when configuring with ENABLE_INTEGRATION_TESTING = ON and is instead replaced by KISO_INTEGRATION_TEST_NAME. This path points to the package which defines the testentry build target. It expects to find the necessary CMakeLists.txt file in ${KISO_INTEGRATION_TEST_NAME}/test/integration.

Relates to #17.

@ChiefGokhlayehBosch
Copy link
Contributor Author

ChiefGokhlayehBosch commented Sep 8, 2020

  • Document integration test framework in hugo
  • Manual testing of testapp
  • Integration of test coordinator aka PyKiso

This PR combines #159 and #105.

@ChiefGokhlayehBosch ChiefGokhlayehBosch force-pushed the feature/#17-add-integration-test-framework branch 2 times, most recently from c94c9e6 to ef4bcd8 Compare September 8, 2020 12:35
khalifima and others added 6 commits September 8, 2020 14:38
Co-authored-by: ChiefGokhlayehBosch <[email protected]>
Signed-off-by: ChiefGokhlayehBosch <[email protected]>
Co-authored-by: ChiefGokhlayehBosch <[email protected]>
Signed-off-by: ChiefGokhlayehBosch <[email protected]>
Signed-off-by: ChiefGokhlayehBosch <[email protected]>
@ChiefGokhlayehBosch ChiefGokhlayehBosch force-pushed the feature/#17-add-integration-test-framework branch from ef4bcd8 to 7b56b18 Compare September 8, 2020 12:38
@ChiefGokhlayehBosch ChiefGokhlayehBosch deleted the feature/#17-add-integration-test-framework branch September 8, 2020 14:53
@ChiefGokhlayehBosch ChiefGokhlayehBosch restored the feature/#17-add-integration-test-framework branch September 9, 2020 09:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants