Skip to content

Conversation

@nordic-krch
Copy link
Contributor

Due to missing Kconfig and wrong error check test was returning early before start of the test.

Additionally, remove power states which are now defined in the device dtsi.

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 2, 2024
@nordic-krch
Copy link
Contributor Author

Test works after applying nrfconnect/sdk-zephyr#2032

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 2, 2024

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: 3b5abcf764bf7cda390702882b33dbfb346d3540

more details

sdk-nrf:

PR head: 3b5abcf764bf7cda390702882b33dbfb346d3540
merge base: b10c1001bb44787e97dd52a44edaf79001900b51
target head (main): f28fe38a709e6e863b4ac173712249124f5201b2
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (3)
tests
│  ├── benchmarks
│  │  ├── multicore
│  │  │  ├── idle_uarte
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 3dd8985b56
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:3dd8985b56_81ed5a52d6

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 57
  • ✅ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@nordic-krch nordic-krch requested a review from a team as a code owner October 9, 2024 14:21
@nordic-bami
Copy link
Contributor

nordic-bami commented Oct 10, 2024

Please in addition do a small testcase.yml update as follows:

harness_config:
  fixture: ppk_power_measure
  pytest_root:
    - "${CUSTOM_ROOT_TEST_DIR}/test_measure_power_consumption.py::test_measure_and_data_dump_power_consumption_uarte"

@nordic-krch nordic-krch force-pushed the fix_idle_uart branch 3 times, most recently from 4a78269 to 638cf06 Compare October 10, 2024 12:23
@nordic-piks
Copy link
Contributor

fixture

Please also change fixture: ppk_power_measure -> gpio_loopback

Due to missing Kconfig and wrong error check test was returning
early before start of the test.

Additionally, remove power states which are now defined in the
device dtsi.

Fix pytest name.

Signed-off-by: Krzysztof Chruściński <[email protected]>
@rlubos rlubos merged commit 19fe584 into nrfconnect:main Oct 14, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants