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

Test App Updates #13

Merged
merged 11 commits into from
Feb 12, 2024
Merged

Conversation

simpsont-oci
Copy link
Contributor

Improves "random" sample generation in order to test more of dynamic data. Perhaps unsurprisingly, causes SEGV in the monitor for some generated samples when subscribing to the unmanaged test app (using dynamic data).

Given the test app IDL, the following samples have all been observed to cause SEGV:

 - {"origin":"unmanaged_testapp-lumoria-100374","count":1,"ct":{"bt":{"u8":1,"i8":1,"us":1,"s":1,"ul":1,"l":1,"ull":1,"ll":1,"f":1.0,"d":1.0,"b":true,"o":1,"c":"b","wc":"Y","str":""},"cuts":[{"$discriminator":"one","str":"A complex union string: 1216115135"},{"$discriminator":"four","tns":[]}]}}
 - {"origin":"unmanaged_testapp-lumoria-100426","count":2,"ct":{"bt":{"u8":2,"i8":2,"us":2,"s":2,"ul":2,"l":2,"ull":2,"ll":2,"f":2.0,"d":2.0,"b":false,"o":2,"c":"c","wc":"X","str":""},"cuts":[{"$discriminator":"one","str":"A complex union string: 1328248073"},{"$discriminator":"four","tns":[]}]}}
 - {"origin":"unmanaged_testapp-lumoria-100584","count":0,"ct":{"bt":{"u8":0,"i8":0,"us":0,"s":0,"ul":0,"l":0,"ull":0,"ll":0,"f":0.0,"d":0.0,"b":false,"o":0,"c":"a","wc":"Z","str":""},"cuts":[{"$discriminator":"two","ut":{"$discriminator":"one","str":"A union string: 1978156891"}},{"$discriminator":"three","tn":{"et":"three","ut":{"$discriminator":"four","btss":[[]]},"tns":[{"et":"four","ut":{"$discriminator":"two","bt":{"u8":64,"i8":87,"us":32458,"s":15036,"ul":1349045358,"l":406939873,"ull":1961668412,"ll":2068534139,"f":3123291904.0,"d":2162427736.0,"b":false,"o":42,"c":"e","wc":"M","str":"A basic string 2498535724"}},"tns":[]}]}}]}}
 - {"origin":"unmanaged_testapp-lumoria-100663","count":0,"ct":{"bt":{"u8":0,"i8":0,"us":0,"s":0,"ul":0,"l":0,"ull":0,"ll":0,"f":0.0,"d":0.0,"b":false,"o":0,"c":"a","wc":"Z","str":""},"cuts":[{"$discriminator":"four","tns":[]},{"$discriminator":"four","tns":[{"et":"four","ut":{"$discriminator":"two","bt":{"u8":156,"i8":118,"us":28891,"s":6011,"ul":4147023377,"l":1656741056,"ull":1456839787,"ll":2746201045,"f":414862368.0,"d":941259102.0,"b":true,"o":87,"c":"u","wc":"M","str":"A basic string 1861234030"}},"tns":[]}]}]}}
 - {"origin":"unmanaged_testapp-lumoria-100711","count":0,"ct":{"bt":{"u8":0,"i8":0,"us":0,"s":0,"ul":0,"l":0,"ull":0,"ll":0,"f":0.0,"d":0.0,"b":false,"o":0,"c":"a","wc":"Z","str":""},"cuts":[{"$discriminator":"three","tn":{"et":"three","ut":{"$discriminator":"three","bts":[{"u8":233,"i8":87,"us":38893,"s":19840,"ul":278126418,"l":1791332590,"ull":3253152003,"ll":1099833389,"f":3664764928.0,"d":3401278335.0,"b":false,"o":75,"c":"n","wc":"T","str":"A basic string 3643114909"}]},"tns":[]}},{"$discriminator":"one","str":"A complex union string: 1347159026"}]}}
 - {"origin":"unmanaged_testapp-lumoria-101031","count":2,"ct":{"bt":{"u8":2,"i8":2,"us":2,"s":2,"ul":2,"l":2,"ull":2,"ll":2,"f":2.0,"d":2.0,"b":false,"o":2,"c":"c","wc":"X","str":""},"cuts":[{"$discriminator":"one","str":"A complex union string: 1543224235"},{"$discriminator":"three","tn":{"et":"two","ut":{"$discriminator":"one","str":"A union string: 3453233805"},"tns":[]}}]}}

@simpsont-oci simpsont-oci marked this pull request as ready for review February 26, 2023 19:17
@simpsont-oci simpsont-oci marked this pull request as draft February 26, 2023 19:17
src/topic_table_model.cpp Outdated Show resolved Hide resolved
@iguessthislldo iguessthislldo marked this pull request as ready for review February 9, 2024 18:59
test/test.idl Outdated Show resolved Hide resolved
test/json_conversion.h Outdated Show resolved Hide resolved
test/json_conversion.h Outdated Show resolved Hide resolved
@iguessthislldo iguessthislldo merged commit b441282 into OpenDDS:master Feb 12, 2024
10 of 12 checks passed
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