Skip to content

fix(test): allow to run test locally on mac#5672

Merged
kisvegabor merged 1 commit intolvgl:masterfrom
XuNeo:fix-test-on-mac
Feb 20, 2024
Merged

fix(test): allow to run test locally on mac#5672
kisvegabor merged 1 commit intolvgl:masterfrom
XuNeo:fix-test-on-mac

Conversation

@XuNeo
Copy link
Collaborator

@XuNeo XuNeo commented Feb 17, 2024

Description of the feature or fix

Make libinput and xbkcommon optional in test because they are not available on mac.
Add custom target run to make it easier to run all tests.

Now the tests can be run locally in below steps

  • cd lvgl/tests
  • config. cmake -Bbuild -GNinja -DOPTIONS_TEST_DEFHEAP=1
  • ninja -Cbuild run will firstly build all testcases and then call ctest

Notes

@C47D
Copy link
Contributor

C47D commented Feb 18, 2024

Now the tests can be run locally in below steps
cd lvgl/tests
config. cmake -Bbuild -GNinja -DOPTIONS_TEST_DEFHEAP=1
ninja -Cbuild run will firstly build all testcases and then call ctest

Can we document this somewhere? Does the test directory has a README?

@XuNeo
Copy link
Collaborator Author

XuNeo commented Feb 18, 2024

Can we document this somewhere? Does the test directory has a README?

There's doc in lvgl/tests/README.md, and it lists recommended method to run test. The method mentioned above is manually calling cmake instead of via tests/main.py.

@kisvegabor
Copy link
Member

Cool!

@kisvegabor kisvegabor enabled auto-merge (squash) February 19, 2024 21:06
Make libinput and xbkcommon optional in test because they are not available on mac.
Add custom target 'run' to make it easier to run all tests.

Signed-off-by: Neo Xu <neo.xu1990@gmail.com>
@kisvegabor kisvegabor merged commit 8a85a49 into lvgl:master Feb 20, 2024
HongChao6 pushed a commit to HongChao6/lvgl that referenced this pull request Oct 18, 2024
Signed-off-by: Neo Xu <neo.xu1990@gmail.com>
HongChao6 pushed a commit to HongChao6/lvgl that referenced this pull request Oct 18, 2024
Signed-off-by: Neo Xu <neo.xu1990@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants