Skip to content

testDynamicStructEmptyString always failed#3863

Merged
aleks-f merged 1 commit intopocoproject:develfrom
bas524:test-dynamic-struct
Nov 8, 2022
Merged

testDynamicStructEmptyString always failed#3863
aleks-f merged 1 commit intopocoproject:develfrom
bas524:test-dynamic-struct

Conversation

@bas524
Copy link
Copy Markdown
Contributor

@bas524 bas524 commented Nov 7, 2022

There was 1 failure:
1: N7CppUnit10TestCallerI7VarTestEE.testDynamicStructEmptyString
"expected: "{ "Empty": "", "Space": }" but was: "{ "Empty": "", "Space": " " }""
in "src/VarTest.cpp", line 2294

It's happen because in test we call aStruct.toString(false). It's mean that we don't expect wrapping, but expected result was written with wrapping

There was 1 failure:
 1: N7CppUnit10TestCallerI7VarTestEE.testDynamicStructEmptyString
    "expected: "{ "Empty": "", "Space":   }" but was: "{ "Empty": "", "Space": " " }""
    in "src/VarTest.cpp", line 2294

It's happen because in test we call aStruct.toString(false).
It's mean that we don't expect wrapping, but expected result was written with wrapping
@aleks-f aleks-f added this to the Release 1.12.5 milestone Nov 8, 2022
@aleks-f aleks-f merged commit aa48d39 into pocoproject:devel Nov 8, 2022
@aleks-f aleks-f modified the milestones: Release 1.12.5, Release 1.13.0 Nov 8, 2022
@bas524 bas524 deleted the test-dynamic-struct branch January 27, 2023 09:49
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.

2 participants