[WIP] code coverage / mixer test failure #6273

Merged
merged 3 commits into from Jan 9, 2017

Projects

None yet

1 participant

@dagar
Member
dagar commented Jan 7, 2017 edited

Mixer test failure when built with code coverage options (optimization disabled).

INFO  [tests] RUNNING TEST: loadIOPass
INFO  [tests] TEST PASSED: loadIOPass
INFO  [tests] RUNNING TEST: loadQuadTest
ERROR [tests] Mixer text length overflow for file: ROMFS/px4fmu_test/mixers/quad_test.mix. Is PX4IO_MAX_MIXER_LENGHT too small? (curr len: 230)
ERROR [tests] Mixer load failed with chunk size 59
ERROR [tests] TEST FAILED: loadQuadTest
INFO  [tests] RUNNING TEST: loadVTOL1Test
ERROR [modules__unit_test] Compare failed: check number of mixers loaded - (mixer_group.count():2) (expected_count:4) (../src/systemcmds/tests/test_mixer.cpp:353)
ERROR [tests] Mixer load failed with chunk size 13
ERROR [tests] TEST FAILED: loadVTOL1Test
INFO  [tests] RUNNING TEST: loadVTOL2Test
ERROR [tests] Mixer text length overflow for file: ROMFS/px4fmu_test/mixers/vtol2_test.mix. Is PX4IO_MAX_MIXER_LENGHT too small? (curr len: 230)
ERROR [tests] Mixer load failed with chunk size 13
ERROR [tests] TEST FAILED: loadVTOL2Test
INFO  [tests] RUNNING TEST: loadComplexTest
INFO  [tests] TEST PASSED: loadComplexTest
INFO  [tests] RUNNING TEST: loadAllTest
INFO  [tests] Testing all mixers in ROMFS/px4fmu_common/mixers
ERROR [tests] Mixer text length overflow for file: ROMFS/px4fmu_common/mixers/Viper.main.mix. Is PX4IO_MAX_MIXER_LENGHT too small? (curr len: 230)
ERROR [tests] Mixer load failed with chunk size 20
ERROR [tests] Error testing mixer ROMFS/px4fmu_common/mixers/Viper.main.mix
ERROR [tests] TEST FAILED: loadAllTest
INFO  [tests] RUNNING TEST: mixerTest
INFO  [tests] TEST PASSED: mixerTest
ERROR [modules__unit_test] SOME TESTS FAILED
INFO  [modules__unit_test]   Tests passed :      3
INFO  [modules__unit_test]   Tests failed :      4
INFO  [modules__unit_test]   Tested assertions : 1520
mixer FAILED
@dagar dagar referenced this pull request Jan 7, 2017
Merged

docker run helper script #6270

@dagar dagar changed the title from [WIP] restore tests code coverage to [WIP] code coverage / mixer test failure Jan 7, 2017
@dagar dagar added the bug label Jan 7, 2017
dagar added some commits Jan 6, 2017
@dagar dagar circleci restore code coverage build a313834
@dagar dagar code coverage try at regular optimization 0b7cf1f
@dagar dagar docker update to 2017-01-08
bebfa8c
@dagar
Member
dagar commented Jan 9, 2017

@LorenzMeier @AndreasAntener the mixer test passes consistently with regular optimization and code coverage results are nearly the same, so I'll stop disabling optimization and re-enable the coverage build.

For future reference the mixer does seem to have intermittent issues when built with -O0.

@dagar dagar merged commit dc78783 into master Jan 9, 2017

6 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/sitl/pr Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 18.198%
Details
semaphoreci The build passed on Semaphore.
Details
@dagar dagar deleted the codecoverage branch Jan 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment