Skip to content

tests: fix typos and misnamed platforms #41238

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.

Sign up for GitHub

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 1 commit into from
Dec 17, 2021
Merged

tests: fix typos and misnamed platforms #41238

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion samples/boards/arc_secure_services/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ tests:
sample.board.arc_secure_services:
# Requires multiple kernels in an AMP config. See README.rst
build_only: true
platform_allow: nsim_sem em_starterkit_em7d_secure
platform_allow: nsim_sem em_starterkit_em7d
tags: secure
harness: console
harness_config:
Expand Down
2 changes: 1 addition & 1 deletion samples/drivers/counter/maxim_ds3231/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,4 @@ sample:
tests:
sample.basic.maxim_ds3231:
build_only: true
platform_allow: efr32mg_sltb004a frdm_k64f nrf51_pca10028 nucleo_l476rg particle_xenon
platform_allow: efr32mg_sltb004a frdm_k64f nrf51dk_nrf51422 nucleo_l476rg particle_xenon
3 changes: 2 additions & 1 deletion samples/subsys/ipc/ipc_service/static_vrings_mi/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ tests:
sample.ipc.static_vrings_mi.nrf:
platform_allow: nrf5340dk_nrf5340_cpuapp bl5340_dvk_cpuapp
integration_platforms:
- nrf5340dk_nrf5340_cpuapp bl5340_dvk_cpuapp
- nrf5340dk_nrf5340_cpuapp
- bl5340_dvk_cpuapp
tags: ipc
build_only: true
2 changes: 1 addition & 1 deletion samples/subsys/ipc/openamp/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ sample:
name: OpenAMP example integration
tests:
sample.ipc.openamp:
platform_allow: lpcxpresso54114_m4 lpcxpresso55s69_cpu0 mps2_an521 v2m_musca v2m_musca_b1
platform_allow: lpcxpresso54114_m4 lpcxpresso55s69_cpu0 mps2_an521 v2m_musca_b1
tags: ipm
harness: console
harness_config:
Expand Down
3 changes: 2 additions & 1 deletion samples/subsys/ipc/rpmsg_service/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ tests:
sample.ipc.rpmsg_service.nrf:
platform_allow: nrf5340dk_nrf5340_cpuapp bl5340_dvk_cpuapp
integration_platforms:
- nrf5340dk_nrf5340_cpuapp bl5340_dvk_cpuapp
- nrf5340dk_nrf5340_cpuapp
- bl5340_dvk_cpuapp
tags: ipm
build_only: true
4 changes: 2 additions & 2 deletions samples/tfm_integration/tfm_secure_partition/sample.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
common:
tags: tfm
platform_allow: mps2_an521_nonsecure v2m_musca_s1_nonsecure
nrf5340dk_nrf5340_cpuappns nrf9160dk_nrf9160ns
platform_allow: mps2_an521_ns v2m_musca_s1_ns
nrf5340dk_nrf5340_cpuapp_ns nrf9160dk_nrf9160_ns
harness: console
harness_config:
type: multi_line
Expand Down
2 changes: 1 addition & 1 deletion tests/drivers/build_all/ieee802154/testcase.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ tests:
platform_allow: native_posix
extra_args: OVERLAY_CONFIG=external.conf
ieee802154.build.cc12xx_cc2520:
platform_allow: cc1352_sensortag
platform_allow: cc1352r_sensortag
extra_configs:
- CONFIG_IEEE802154_CC13XX_CC26XX=y
- CONFIG_IEEE802154_CC13XX_CC26XX_SUB_GHZ=y
Expand Down
2 changes: 1 addition & 1 deletion tests/drivers/flash/testcase.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ tests:
drivers.flash.stm32:
integration_platforms:
- nucleo_f091rc
- nucleo_f103rc
- nucleo_f103rb
- nucleo_f207zg
- stm32f3_disco
- nucleo_f429zi
Expand Down
4 changes: 2 additions & 2 deletions tests/kernel/workq/critical/testcase.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@ common:

tests:
kernel.workqueue:
platform_exclude: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard
platform_exclude: nsim_sem_mpu_stack_guard
filter: not CONFIG_WDT_SAM
kernel.workqueue.sam:
filter: CONFIG_WDT_SAM
extra_configs:
- CONFIG_WDT_DISABLE_AT_BOOT=y
kernel.workqueue.nsim:
platform_allow: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard
platform_allow: nsim_sem_mpu_stack_guard
extra_configs:
- CONFIG_TEST_HW_STACK_PROTECTION=n
kernel.workqueue.linker_generator:
Expand Down
12 changes: 6 additions & 6 deletions tests/posix/common/testcase.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,11 @@ common:

tests:
portability.posix.common:
platform_exclude: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard ehl_crb
platform_exclude: nsim_sem_mpu_stack_guard ehl_crb
extra_configs:
- CONFIG_NEWLIB_LIBC=n
portability.posix.common.newlib:
platform_exclude: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard ehl_crb
platform_exclude: nsim_sem_mpu_stack_guard ehl_crb
filter: TOOLCHAIN_HAS_NEWLIB == 1
extra_configs:
- CONFIG_NEWLIB_LIBC=y
Expand All @@ -19,26 +19,26 @@ tests:
extra_configs:
- CONFIG_ARCMWDT_LIBC=y
portability.posix.common.tls:
platform_exclude: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard ehl_crb
platform_exclude: nsim_sem_mpu_stack_guard ehl_crb
filter: CONFIG_ARCH_HAS_THREAD_LOCAL_STORAGE and CONFIG_TOOLCHAIN_SUPPORTS_THREAD_LOCAL_STORAGE
extra_configs:
- CONFIG_NEWLIB_LIBC=n
- CONFIG_THREAD_LOCAL_STORAGE=y
- CONFIG_MAIN_STACK_SIZE=1152
portability.posix.common.tls.newlib:
platform_exclude: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard ehl_crb
platform_exclude: nsim_sem_mpu_stack_guard ehl_crb
filter: TOOLCHAIN_HAS_NEWLIB == 1 and CONFIG_ARCH_HAS_THREAD_LOCAL_STORAGE and CONFIG_TOOLCHAIN_SUPPORTS_THREAD_LOCAL_STORAGE
extra_configs:
- CONFIG_NEWLIB_LIBC=y
- CONFIG_THREAD_LOCAL_STORAGE=y
- CONFIG_MAIN_STACK_SIZE=1152
portability.posix.common.nsim:
platform_allow: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard
platform_allow: nsim_sem_mpu_stack_guard
extra_configs:
- CONFIG_NEWLIB_LIBC=n
- CONFIG_TEST_HW_STACK_PROTECTION=n
portability.posix.common.newlib.nsim:
platform_allow: nsim_sem_mpu_stack_guard nsim_em_mpu_stack_guard
platform_allow: nsim_sem_mpu_stack_guard
filter: TOOLCHAIN_HAS_NEWLIB == 1
extra_configs:
- CONFIG_NEWLIB_LIBC=y
Expand Down
2 changes: 1 addition & 1 deletion tests/subsys/jwt/testcase.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ tests:
tags: jwt
filter: CONFIG_ENTROPY_HAS_DRIVER
integration_platforms:
- frdm_k64
- frdm_k64f
2 changes: 1 addition & 1 deletion tests/subsys/storage/stream/stream_flash/testcase.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,5 @@ tests:
tags: stream_flash
storage.stream_flash.mpu_allow_flash_write:
extra_args: OVERLAY_CONFIG=mpu_allow_flash_write.overlay
platform_allow: nrf52840_pca10056
platform_allow: nrf52840dk_nrf52840
Copy link
Contributor

@gopiotr gopiotr Dec 15, 2021

Choose a reason for hiding this comment

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

I built and tested this test scenario on real hardware and it failed.
@hakonfam could you look at this test, before merge this change into main?

Copy link
Contributor

@hakonfam hakonfam Dec 16, 2021

Choose a reason for hiding this comment

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

Will do

Copy link
Contributor

@hakonfam hakonfam Dec 16, 2021

Choose a reason for hiding this comment

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

This test passes locally on my HW (v2.0.1)

Copy link
Member

@PerMac PerMac Dec 16, 2021

Choose a reason for hiding this comment

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

It also works on my local setup.

Copy link
Contributor

@gopiotr gopiotr Dec 16, 2021

Choose a reason for hiding this comment

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

Hmmmm, I was also checking on board v2.0.1 with Zephyr SDK: zephyr-sdk-0.13.1 and it still failed. I will check my nrfjprog version and other things, but if you wrote, that it works on yours boards I will no longer block this PR.
@hakonfam @PerMac - thanks for checking :)

tags: stream_flash