Libelektra: page d'accueil : la compilation échoue

Créé le 1 mai 2019  ·  4Commentaires  ·  Source: ElektraInitiative/libelektra

Depuis https://build.libelektra.org/jenkins/job/elektra-homepage/656/console, la construction de la page d'accueil échoue avec l'erreur suivante :

        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 savez-vous ce que signifie cette erreur ? Comment pouvons-nous obtenir une erreur de validation plus spécifique ?

bug build good first issue help wanted

Commentaire le plus utile

@sanssecours vous avez une idée ?

Pour autant que je sache, d'après le code:

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

et

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

YAJL ne passe une validation aller-retour après PR # 2580 a été fusionné dans le master branche. À partir de ces connaissances limitées, je supposerais de remplacer ASSERT_FALSE dans

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

avec ASSERT_TRUE devrait être la bonne façon de résoudre le problème.

Tous les 4 commentaires

Pour être honnête, je n'en ai pas la moindre idée. Peut être causé par une modification du plugin en cours de test. Je n'ai plus écrit de C++ depuis, donc j'ai du mal à comprendre ce que je faisais à l'époque. 😄

Merci quand même pour la réponse rapide.

Il semble que la validation aller-retour avec JSON échoue maintenant. @sanssecours vous avez une idée ?

@sanssecours vous avez une idée ?

Pour autant que je sache, d'après le code:

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

et

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

YAJL ne passe une validation aller-retour après PR # 2580 a été fusionné dans le master branche. À partir de ces connaissances limitées, je supposerais de remplacer ASSERT_FALSE dans

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

avec ASSERT_TRUE devrait être la bonne façon de résoudre le problème.

Cela ressemble à une solution facile ! Merci pour la description détaillée !

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

mpranj picture mpranj  ·  3Commentaires

sanssecours picture sanssecours  ·  3Commentaires

dominicjaeger picture dominicjaeger  ·  3Commentaires

mpranj picture mpranj  ·  3Commentaires

markus2330 picture markus2330  ·  3Commentaires