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.

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

tests/sema/: added simple sema test #15540

Merged
merged 1 commit into from Apr 2, 2021

Conversation

JulianHolzwarth
Copy link
Contributor

Contribution description

This is a simple test for sema.

Testing procedure

BOARD=native make -C tests/sema flash test

Issues/PRs references

tests/sema/main.c Outdated Show resolved Hide resolved
tests/sema/main.c Outdated Show resolved Hide resolved
tests/sema/main.c Outdated Show resolved Hide resolved
tests/sema/main.c Outdated Show resolved Hide resolved
tests/sema/main.c Outdated Show resolved Hide resolved
tests/sema/main.c Outdated Show resolved Hide resolved
@JulianHolzwarth
Copy link
Contributor Author

@benpicco thanks. Changed it.

#define TEST_TIME_US 1000
#define TEST_ITERATIONS 4

static char stack[THREAD_STACKSIZE_DEFAULT];
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess THREAD_STACKSIZE_SMALL would do as well

tests/sema/main.c Outdated Show resolved Hide resolved
@fjmolinas
Copy link
Contributor

Please squash @JulianHolzwarth

@jeandudey jeandudey added Area: tests Area: tests and testing framework Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Mar 20, 2021
@jeandudey
Copy link
Contributor

@JulianHolzwarth this might need a rebase so the check-pr / check-commit check succeeds

@fjmolinas fjmolinas added the CI: run tests If set, CI server will run tests on hardware for the labeled PR label Apr 2, 2021
@fjmolinas fjmolinas added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 2, 2021
@fjmolinas
Copy link
Contributor

Rebased

@fjmolinas
Copy link
Contributor

Added BOARD to insufficient memory list and added missing header

Copy link
Contributor

@fjmolinas fjmolinas left a comment

Choose a reason for hiding this comment

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

ACK, thanks for the test @JulianHolzwarth!!

@fjmolinas fjmolinas merged commit 73ccd1e into RIOT-OS:master Apr 2, 2021
@kaspar030 kaspar030 added this to the Release 2021.04 milestone Apr 23, 2021
@JulianHolzwarth JulianHolzwarth deleted the pr/software/test/sema branch May 3, 2021 20:17
@JulianHolzwarth
Copy link
Contributor Author

@fjmolinas sorry I did not reply. And thank you for merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants