Libelektra: Homepage: Build schlägt fehl

Erstellt am 1. Mai 2019  ·  4Kommentare  ·  Quelle: ElektraInitiative/libelektra

Seit https://build.libelektra.org/jenkins/job/elektra-homepage/656/console schlägt der Homepage-Build mit folgendem Fehler fehl:

        Start  42: test_service_storageengine
 39/119 Test  #40: test_service_convertengine .......***Failed    7.44 sec
[==========] Running 12 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 12 tests from kdbrestServicesConvertengineTest
[ RUN      ] kdbrestServicesConvertengineTest.FindSuitablePluginCheck
[       OK ] kdbrestServicesConvertengineTest.FindSuitablePluginCheck (4223 ms)
[ RUN      ] kdbrestServicesConvertengineTest.ExportToFormatCheck
[       OK ] kdbrestServicesConvertengineTest.ExportToFormatCheck (80 ms)
[ RUN      ] kdbrestServicesConvertengineTest.ExportToFormatWithConfigCheck
[       OK ] kdbrestServicesConvertengineTest.ExportToFormatWithConfigCheck (54 ms)
[ RUN      ] kdbrestServicesConvertengineTest.ExportToFormatCheck2
/home/jenkins/workspace/elektra-homepage/src/tools/rest-backend/tests/test_service_convertengine.cpp:154: Failure
Value of: cf.isValidated ()
  Actual: true
Expected: false
[  FAILED  ] kdbrestServicesConvertengineTest.ExportToFormatCheck2 (106 ms)

@Namoshek weißt du, was dieser Fehler bedeutet? Wie können wir einen genaueren Validierungsfehler erhalten?

bug build good first issue help wanted

Hilfreichster Kommentar

@sanssecours hast du eine idee?

Soweit ich das anhand des Codes erkennen kann:

https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/model_configformat.hpp#L84 -L92

und

https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/model_configformat.hpp#L47 -L62

YAJL besteht eine Round-Trip-Validierung, nachdem PR #2580 in den master Zweig zusammengeführt wurde. Aus diesem begrenzten Wissen würde ich annehmen, dass ASSERT_FALSE in . ersetzt wird

https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/tests/test_service_convertengine.cpp#L154

mit ASSERT_TRUE sollte der richtige Weg sein, um das Problem zu beheben.

Alle 4 Kommentare

Ehrlich gesagt habe ich nicht die geringste Ahnung. Kann durch eine Änderung des zu testenden Plugins verursacht werden. Seitdem habe ich kein C++ mehr geschrieben, daher fällt es mir ziemlich schwer zu verstehen, was ich damals gemacht habe. 😄

Danke trotzdem für die schnelle Antwort.

Es scheint, als ob die Roundtrip-Validierung mit JSON jetzt fehlschlägt. @sanssecours hast du eine idee?

@sanssecours hast du eine idee?

Soweit ich das anhand des Codes erkennen kann:

https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/model_configformat.hpp#L84 -L92

und

https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/model_configformat.hpp#L47 -L62

YAJL besteht eine Round-Trip-Validierung, nachdem PR #2580 in den master Zweig zusammengeführt wurde. Aus diesem begrenzten Wissen würde ich annehmen, dass ASSERT_FALSE in . ersetzt wird

https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/tests/test_service_convertengine.cpp#L154

mit ASSERT_TRUE sollte der richtige Weg sein, um das Problem zu beheben.

Das klingt nach einer einfachen Lösung! Danke für die ausführliche Beschreibung!

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

e1528532 picture e1528532  ·  4Kommentare

markus2330 picture markus2330  ·  4Kommentare

markus2330 picture markus2330  ·  4Kommentare

mpranj picture mpranj  ·  3Kommentare

mpranj picture mpranj  ·  3Kommentare