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

Group posix tests #9912

Closed
nashif opened this issue Sep 11, 2018 · 2 comments
Closed

Group posix tests #9912

nashif opened this issue Sep 11, 2018 · 2 comments
Assignees
@nniranjhana @spoorthik
Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@nashif
Copy link
Member

nashif commented Sep 11, 2018

Right now we have lots of duplication in the posix tests and we only test minimal functionality in each test, like we did with CMSIS RTOS v1, lets group the tests as much as possible into 1 or 2 test projects and reduce duplication and speed up test execution.
@nashif nashif added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug labels Sep 11, 2018
@nniranjhana nniranjhana mentioned this issue Sep 14, 2018
Closed
@spoorthik
Copy link
Contributor

spoorthik commented Sep 18, 2018 •
edited
Loading

@nashif @nniranjhana: Started working on it - #10017.
Also found few inconsistencies on return codes in API implementation, raised an issue - #9993

@nniranjhana nniranjhana mentioned this issue Oct 8, 2018
Merged
@nniranjhana
Copy link

Corresponding PRs have been merged for grouping tests, closing now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nniranjhana nniranjhana

@spoorthik spoorthik

Labels
bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nashif @nniranjhana @spoorthik