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.

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

Jump to bottom

Improve test infrastructure #154

Merged
merged 31 commits into from
Nov 10, 2023
Merged

Improve test infrastructure #154

merged 31 commits into from
Nov 10, 2023

Conversation

rexut
Copy link
Member

@rexut rexut commented Nov 10, 2023 •
edited by tobiaskaestner
Loading

This fulfills issues #151 and #152 with following improvements:

  • All tests for Grove Interconnect, Button, and LED Shields converted to be more generic and guided by a list of well-known integration platforms currently supported by Bridle. This leads to a super-matrix of test suites and cases, thus all invalid board/shield conditions, not yet well supported boards or not finished implementations are now excluded by the new global quarantine list in tests/quarantine.yaml.
  • All SAMD21 targets doesn't yet support the LED shell command against the internal LED driver API properly in case of mixed GPIO and PWM LEDs on same I/O pin (shared pin function), thus related CONFIG_LED and CONFIG_LED_SHELL were disable for this boards as default.
  • The Grove Interconnect Shield demonstration has been expanded to also include the Raspberry Pi Pico W and Waveshare RP2040-LCD-0.96 boards on the Grove Shield for Pi Pico V1.
  • In conjunction with the new Twister upstream support for filtering tests by board manufacturer or vendor, Bridle now also provides the vendor entry for each supported board.

rexut added 30 commits November 10, 2023 06:12
@rexut
tests: grove: base: set Seeeduino equal to Arduino Zero
b487ec0 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: grove: btn: set Seeeduino equal to Arduino Zero
f67478f 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: grove: led: set Seeeduino equal to Arduino Zero
87e5725 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
boards: seeeduino_lotus: expand Grove PWM mapping
5ca5d7f 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
boards: seeeduino_cm0: expand Grove PWM mapping
e2dd3e5 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shields: grove: set Seeeduino equal to Arduino Zero
1cd59a9 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
boards: arm: seeed_xiao_samd21: add XIAO DAC feature
7a8916e 
For better testcase filter by 'depends_on' the board must provide
the supported DAC feature on XIAO connector.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
shields: grove: set all RPi Pico based boards equal
5661ca2 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
boards: waveshare_rp2040: add RPi Pico features
b715531 
For better testcase filter by 'depends_on' the board must provide
all supported RPi Pico connector features.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: add global quarantine list
10fb149 
This global quarantine list defines all valid exclusion
constraints of the upcoming supertest matrix for all
integration platforms supported by Bridle in one place.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: grove: base: use integration platform tests
3a7990e 
Together with the new global quarantine list all specific
test cases are now more generic and independent of special
white or black lists. When ever an exclusion will be needed
this rule have to add to the new global quarantine list.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: grove: btn: use integration platform tests
41f69df 
Together with the new global quarantine list all specific
test cases are now more generic and independent of special
white or black lists. When ever an exclusion will be needed
this rule have to add to the new global quarantine list.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: grove: led: use integration platform tests
f5375a4 
Together with the new global quarantine list all specific
test cases are now more generic and independent of special
white or black lists. When ever an exclusion will be needed
this rule have to add to the new global quarantine list.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: x_grove_testbed: use integration platform tests
ec3fe6a 
Together with the new global quarantine list all specific
test cases are now more generic and independent of special
white or black lists. When ever an exclusion will be needed
this rule have to add to the new global quarantine list.

In addition, the "Grove Base V2" shield constellation
is also being tested on the Seeed Studio Lotus board.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
shield: x_grove_testbed: avoid duplicate code
4cf1630 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
doc: shields: grove: nucleo_f413zh: add PWM mapping
be7f3fb 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
boards: seeeduino_lotus: add label grove_pwm_d8 in DTS
0aa9616 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
samples: helloshell: add RPi Pico (W) for integration
353c9d3 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
samples: helloshell: add WS RP4020 Plus for integration
a3e47c8 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: x_grove_testbed: support RPi Pico W
6c81feb 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shield: x_grove_testbed: add RPi Pico W
7fc600f 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
tests: x_grove_testbed: support WS RP2040-LCD-0.96
5529976 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shield: x_grove_testbed: add WS RP2040-LCD-0.96
b8912b7 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shields: grove: restruct RPi Pico shield interface
e0a5841 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shields: grove: set Nucleo F413ZH equal to F767ZI
916054d 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shields: grove: add note on additional resistores
d82512e 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
shields: grove: fix typos in comments
6e23c2f 
Signed-off-by: Stephan Linz <[email protected]>
@rexut
dts: vendor-prefixes: update entries
d0388b4 
- add new prefix 'nxp'
- add new prefix 'tiac'
- add new prefix 'waveshare'
- fix full name of prefix 'st'

Signed-off-by: Stephan Linz <[email protected]>
@rexut
boards: add vendor to board yaml
be888fa 
In conjunction with the new Twister upstream support for
filtering tests by board manufacturer or vendor, Bridle
now also provides the vendor entry for each supported board.

Signed-off-by: Stephan Linz <[email protected]>
@rexut
release: update draft notes for upcoming 3.5.1
db3cd4d 
Signed-off-by: Stephan Linz <[email protected]>
@rexut rexut self-assigned this Nov 10, 2023
@rexut rexut added this to the v3.5.99 milestone Nov 10, 2023
@github-actions GitHub Actions
Copy link

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.

@rexut rexut requested a review from tobiaskaestner November 10, 2023 13:03
tobiaskaestner
tobiaskaestner approved these changes Nov 10, 2023
View reviewed changes
Copy link
Contributor

@tobiaskaestner tobiaskaestner left a comment

Choose a reason for hiding this comment

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

LGTM, many thanks

@rexut rexut merged commit 95ffbd4 into main Nov 10, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobiaskaestner tobiaskaestner tobiaskaestner approved these changes

Assignees

@rexut rexut

Labels
area: Boards area: Devicetree Binding area: Devicetree area: Documentation area: Release Notes area: Samples area: Shields area: Tests platform: RPi
Projects
None yet
Milestone
v3.5.99
Development

Successfully merging this pull request may close these issues.
2 participants
@rexut @tobiaskaestner