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

Enhancing Tests in Roudi #454

Open
2 of 6 tasks
chiranjeevimaddi opened this issue Dec 21, 2020 · 2 comments · Fixed by #463, #485, #487, #527 or #587
Open
2 of 6 tasks

Enhancing Tests in Roudi #454

chiranjeevimaddi opened this issue Dec 21, 2020 · 2 comments · Fixed by #463, #485, #487, #527 or #587
Assignees
@chiranjeevimaddi
Labels
test A module/integration/stress/etc test for a component

Comments

@chiranjeevimaddi
Copy link
Contributor

chiranjeevimaddi commented Dec 21, 2020 •
edited by elBoberido
Loading

Brief feature description

Introduce Module tests for RouDi component.

Detailed information

Increase the code coverage.

Classes in RouDi components which needs test cases to be added.
In file roudi.cpp, class

  • RouDi.

In file roudi_process.cpp class

In file roudi_lock.cpp class

  • RouDiLock
@dkroenke
Copy link
Member

@chiranjeevimaddi please use the templates (feature template) for this issue and make it a bit more descriptive. Also you can use checkboxes for having a list and track which file you are currently working on so that no one else begin to work on this.
For future we can extend this issue with other RouDi classes.

  • checkbox

@dkroenke dkroenke added the test A module/integration/stress/etc test for a component label Dec 22, 2020
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Dec 22, 2020
@chiranjeevimaddi
iox-eclipse-iceoryx#454 Added testcases for RoudiProcess class
459ac89 
Signed-off-by: Chiranjeevi Maddi (RBEI/EBB1) <[email protected]>
@dkroenke dkroenke linked a pull request Dec 22, 2020 that will close this issue
iox-#454 Added testcases for RoudiProcess class #463
Merged
19 tasks
@chiranjeevimaddi chiranjeevimaddi mentioned this issue Dec 23, 2020
Merged
19 tasks
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Dec 24, 2020
@chiranjeevimaddi
iox-eclipse-iceoryx#454 Updated the test code as per comments
0852bb6 
Signed-off-by: Chiranjeevi Maddi (RBEI/EBB1) <[email protected]>
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Jan 4, 2021
@chiranjeevimaddi
Merge remote-tracking branch 'upstream/master' into iox-eclipse-iceor…
49a6cf3 
…yx#454-Enhancing-Test-in-RouDi
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 5, 2021
@JimmyBch
iox-eclipse-iceoryx#454 merge eclipse github fork master
cafa7a1 
Signed-off-by: Jimmy Belloche <[email protected]>
@budrus budrus mentioned this issue Jan 5, 2021
Closed
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 5, 2021
@JimmyBch
iox-eclipse-iceoryx#454 solve issue protected static method in unit t…
d6e7431 
…est of memory provider

Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 7, 2021
@JimmyBch
iox-eclipse-iceoryx#454 add new element in memory provider error arra…
01d9ba8 
…y test

Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 7, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findings from PR
970c235 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 7, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findings from PR
bf9ecd5 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 7, 2021
@JimmyBch
iox-eclipse-iceoryx#454 add removed code and remove added method getE…
fa1db31 
…rrorString

Signed-off-by: Jimmy Belloche <[email protected]>
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Jan 8, 2021
@chiranjeevimaddi
iox-eclipse-iceoryx#454 updated the rouDiProcess::sendMQ with error h…
ec5b895 
…andler and added test case

Signed-off-by: Chiranjeevi Maddi (RBEI/EBB1) <[email protected]>
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Jan 8, 2021
@chiranjeevimaddi
Merge remote-tracking branch 'upstream/master' into iox-eclipse-iceor…
31ee8de 
…yx#454-Enhancing-Test-in-RouDi
@dkroenke dkroenke linked a pull request Jan 8, 2021 that will close this issue
Iox #285 added unit test for roudi memory block manager provider #478
Closed
19 tasks
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Jan 8, 2021
@chiranjeevimaddi
iox-eclipse-iceoryx#454 updated the sendMQ fail test function
4437cf0 
Signed-off-by: Chiranjeevi Maddi (RBEI/EBB1) <[email protected]>
chiranjeevimaddi added a commit to chiranjeevimaddi/iceoryx that referenced this issue Jan 8, 2021
@chiranjeevimaddi
Merge remote-tracking branch 'upstream/master' into iox-eclipse-iceor…
c05d6e0 
…yx#454-Enhancing-Test-in-RouDi
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findigs on operator logstream
cec476f 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findings for class IceoryxRoudiMem…
cda6607 
…oryManager

Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 add new test memory manager error operator
035e6e4 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findings from PR
f82976d 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 remove commented code
8cb7cab 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 add removed code and remove added method getE…
f82ab5f 
…rrorString

Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix missing review findings
2f57434 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findigs on operator logstream
cb2c166 
Signed-off-by: Jimmy Belloche <[email protected]>
JimmyBch added a commit to nihalchari/iceoryx that referenced this issue Jan 8, 2021
@JimmyBch
iox-eclipse-iceoryx#454 fix review findings for class IceoryxRoudiMem…
453549b 
…oryManager

Signed-off-by: Jimmy Belloche <[email protected]>
@dkroenke dkroenke mentioned this issue Jan 8, 2021
Closed
19 tasks
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 split into two tests, the overflow check and …
15f8a6f 
…the remove with acquire a new one

Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 split destroy ports test into two tests, dest…
84af95c 
…roy and cleanup

Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 fix review findings
0172e47 
Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 add method exhaustInterfacePorts with optiona…
173ae44 
…l lambda.

Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 add method exhaustCondVarPorts with optional …
498d9f5 
…lambda

Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 add method exhaustNodePorts with three option…
44306da 
…al lambda

Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 fix review findings to improve code style
777819e 
Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 update test cases after fixing some review re…
417526f 
…marks

Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@manel94 @marthtz
iox-eclipse-iceoryx#454 update naming and delete extra variables
0c6d991 
Signed-off-by: Manel Melaouhi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@elBoberido @marthtz
Merge pull request eclipse-iceoryx#527 from chiranjeevimaddi/iox-#454…
727629a 
…-Enhancing-coverage-for-class-roudi-and-port-manager-in-roudi-folder

Iox eclipse-iceoryx#454 enhancing coverage for class roudi and port manager in roudi folder
marthtz added a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@marthtz
Merge pull request eclipse-iceoryx#564 from marthtz/iox-#454-enhance-…
72f9b2c 
…cmd-line-parser-tests

Iox eclipse-iceoryx#454 enhance cmd line parser tests
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@chiranjeevimaddi @marthtz
Merge remote-tracking branch 'upstream/master' into iox-eclipse-iceor…
a33b543 
…yx#454-create-test-file-for-iceoryx-roudi-and-roudi-app
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 improve test for roudi app
717688d 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 improve existing test
d7e6483 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 change test name
62d1f5d 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 improve test assertion
d7db668 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 revert change and add copyrighet header update
4256ca7 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 fix review findings
76738cf 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 fix review findings again
bd482de 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 change header file again
5ba5114 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 update header file
ec2bd58 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@chiranjeevimaddi @marthtz
iox-eclipse-iceoryx#454 revert roudi_app file
f264c38 
Signed-off-by: chiranjeevi maddi <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 add new line at end of file
fa5abc4 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 add include getopt.hpp for windows build
d4f4ba2 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 remove useless function from test class
9953d5d 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 add new tests with empty config
8d8b042 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 remove useless function
432b49e 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz pushed a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@JimmyBch @marthtz
iox-eclipse-iceoryx#454 add override to teardown method
9be514c 
Signed-off-by: Jimmy Belloche <[email protected]>
marthtz added a commit to boschglobal/iceoryx that referenced this issue May 12, 2021
@marthtz
Merge pull request eclipse-iceoryx#587 from chiranjeevimaddi/iox-#454…
73c7838 
…-create-test-file-for-iceoryx-roudi-and-roudi-app

Iox eclipse-iceoryx#454 create test file for iceoryx roudi and roudi app
@mossmaurice
Copy link
Contributor

@chiranjeevimaddi Will you continue to work on this issue? Can we plan this for iceoryx 2.0 (~December 2021)

@CarolinaGGB

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

@chiranjeevimaddi chiranjeevimaddi

Labels
test A module/integration/stress/etc test for a component
Projects
None yet
Milestone
No milestone
4 participants
@mossmaurice @dkroenke @elBoberido @chiranjeevimaddi