-
Notifications
You must be signed in to change notification settings - Fork 42
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
2.11.6: test suite is failing #196
Comments
Sorry for the (very long) delay in looking at this. I'm getting ready to make a new release and I can't replicate this now on either the previous release 2.11.6 or the current stable branch. I can't really see exactly what's failing from eyeballing the output either. The output looks the same for me when the tests pass. It also look like all Github Actions were passing at the time teh release was made. If you can tell me more, I'll take a look at it. |
Just retested 2.11.11. Here i test suite output: + /usr/bin/make -O -j48 V=1 VERBOSE=1 test
cd CoinUtils; /usr/bin/make test
Making all in src
/usr/bin/make all-am
make[2]: Nothing to be done for 'all-am'.
cd test; /usr/bin/make test
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinMessageHandlerTest.o CoinMessageHandlerTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinErrorTest.o CoinErrorTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinDenseVectorTest.o CoinDenseVectorTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinMpsIOTest.o CoinMpsIOTest.cpp
CoinMpsIOTest.cpp: In function ‘void CoinMpsIOUnitTest(const std::string&)’:
CoinMpsIOTest.cpp:381:25: warning: unused variable ‘numberEntries’ [-Wunused-variable]
381 | int numberEntries = SOS[0]->numberEntries();
| ^~~~~~~~~~~~~
CoinMpsIOTest.cpp:391:25: warning: unused variable ‘numberEntries’ [-Wunused-variable]
391 | int numberEntries = SOS[1]->numberEntries();
| ^~~~~~~~~~~~~
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinLpIOTest.o CoinLpIOTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinIndexedVectorTest.o CoinIndexedVectorTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinModelTest.o CoinModelTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinPackedMatrixTest.o CoinPackedMatrixTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinShallowPackedVectorTest.o CoinShallowPackedVectorTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o CoinPackedVectorTest.o CoinPackedVectorTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/usr/bin/g++ -DHAVE_CONFIG_H -I. -I`echo .` -I../src -I`echo ./../src` -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -c -o unitTest.o unitTest.cpp
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
/bin/sh ../../libtool --tag=CXX --mode=link /usr/bin/g++ -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--gc-sections -Wl,--as-needed -Wl,--build-id=sha1 -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -Wl,-z,pack-relative-relocs -flto=auto -fuse-linker-plugin -o unitTest CoinLpIOTest.o CoinDenseVectorTest.o CoinErrorTest.o CoinIndexedVectorTest.o CoinMessageHandlerTest.o CoinModelTest.o CoinMpsIOTest.o CoinPackedMatrixTest.o CoinPackedVectorTest.o CoinShallowPackedVectorTest.o unitTest.o ../src/libCoinUtils.la -lz -lglpk -lflexiblas -lflexiblas -lm
libtool: link: /usr/bin/g++ -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -pipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -DCOINUTILS_BUILD -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--gc-sections -Wl,--as-needed -Wl,--build-id=sha1 -Wl,-z -Wl,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -Wl,-z -Wl,pack-relative-relocs -flto=auto -fuse-linker-plugin -o .libs/unitTest CoinLpIOTest.o CoinDenseVectorTest.o CoinErrorTest.o CoinIndexedVectorTest.o CoinMessageHandlerTest.o CoinModelTest.o CoinMpsIOTest.o CoinPackedMatrixTest.o CoinPackedVectorTest.o CoinShallowPackedVectorTest.o unitTest.o ../src/.libs/libCoinUtils.so -lz -lglpk -lflexiblas -lm
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[2]: Entering directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
./unitTest
Testing CoinFinite ... finite value: ok; infinite value: ok.
ERROR: No functional CoinIsnan.
Testing CoinError
Testing CoinShallowPackedVector
Testing CoinPackedVector
Testing CoinIndexedVector
Testing CoinPackedMatrix
Testing printMatrixElement:
Expecting 1.1, 0.0, -1.2.
a(1,2) = 1.1; a(3,5) = 0; a(3,6) = -1.2
Expecting bounds error messages.
a(-1,-1) = Major index -1 not in range 0..4
a(0,-1) = Minor index -1 not in range 0..7
a(4,8) = Minor index 8 not in range 0..7
a(5,8) = Major index 5 not in range 0..4
Testing CoinDenseVector
Testing CoinMessageHandler
This should print if the constructor sets format_ to null.
By default, the log level has no effect for on-the-fly messages.
Log level is now 0 .
But we can specify a log level and have it take effect.
If you saw a message that said 'should not print', there's a problem.
A core dump here indicates a cloning failure.
This should print if cloning sets format_ to null.
Test0001I This message has no parts and no fields.
Test0003I This message has an integer field: (42)
Test0004I This message has a double field: (42.562857)
Changing to 10 significant digits.
Test0004I This message has a double field: (42.56285714)
And back to 8 significant digits.
Test0004I This message has a double field: (42.562857)
Test0004I This message has an explicit precision .3: (42.6)
Test0005I This message has a char field: (w)
Test0006I This message has a string field: (forty-two)
Test0007I Prefix Part 1 Part 2 with integer in hex 0x002a Part 3 suffix.
Test0007I Prefix suffix.
Test0007I Prefix Part 1 suffix.
Test0007I Prefix Part 2 with integer in hex 0x002a suffix.
Test0007I Prefix Part 3 suffix.
Test0008I Standardised prefix, free form remainder: An int 42 A double 4.2 a new line
and done.
No standardised prefix, free form reminder: integer ( 42 ).
Tran0027I Transition message.
Trying a clone in mid-message.
Test0007I Prefix Part 1 Part 2 with integer in hex 0x002a Part 3 suffix.
Test0007I Prefix Part 1 Part 2 with integer in hex 0x0042
The second copy should be missing Part 3 and suffix.
CoinMessageHandlerTest completed without error.
ERROR: Errors occurred during testing; please check the output.
make[2]: *** [Makefile:776: test] Error 1
make[2]: Leaving directory '/home/tkloczko/rpmbuild/BUILD/CoinUtils-releases-2.11.11/CoinUtils/test'
make[1]: *** [Makefile:1004: test] Error 2
make: *** [Makefile:855: test] Error 2 If you have any propositions of what I can try to do to diagnose this issue please let me know. |
All tests are still passing on all platforms on Github Actions. If you can isolate what is different, that would give us something to look at. It looks like you are adding a lot of extra flags. If you would like, you can post your exact recipe for building, what OS, what compiler version, etc. But since I still can't replicate, I guess there is not much I can do. |
Any propositions of what I can try to do to diagnose this issue? 🤔 |
I would start by tracking down the line the test is failing on and see why that is happening. Just grepping for CoinUtils/CoinUtils/test/unitTest.cpp Line 174 in a9f7bd3
That line is reached when COIN_C_ISNAN is not defined, which happens when the function isnan cannot be found during configuration. This won't happen on any platforms we regularly test on with default flags and I haven't gotten any other complaints about it, so it's probably something pretty specific to your setup.
The reason can probably be gleaned by looking at |
Looks like something is wrong with test suite
The text was updated successfully, but these errors were encountered: