ΠΠΎΡΠΊΠΎΠ»ΡΠΊΡ https://build.libelektra.org/jenkins/job/elektra-homepage/656/console, ΡΠ±ΠΎΡΠΊΠ° Π΄ΠΎΠΌΠ°ΡΠ½Π΅ΠΉ ΡΡΡΠ°Π½ΠΈΡΡ Π·Π°Π²Π΅ΡΡΠ°Π΅ΡΡΡ Π½Π΅ΡΠ΄Π°ΡΠ½ΠΎ ΡΠΎ ΡΠ»Π΅Π΄ΡΡΡΠ΅ΠΉ ΠΎΡΠΈΠ±ΠΊΠΎΠΉ:
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 ΡΡ Π·Π½Π°Π΅ΡΡ, ΡΡΠΎ ΠΎΠ·Π½Π°ΡΠ°Π΅Ρ ΡΡΠ° ΠΎΡΠΈΠ±ΠΊΠ°? ΠΠ°ΠΊ ΠΌΡ ΠΌΠΎΠΆΠ΅ΠΌ ΠΏΠΎΠ»ΡΡΠΈΡΡ Π±ΠΎΠ»Π΅Π΅ ΠΊΠΎΠ½ΠΊΡΠ΅ΡΠ½ΡΡ ΠΎΡΠΈΠ±ΠΊΡ ΠΏΡΠΎΠ²Π΅ΡΠΊΠΈ?
ΠΡΠ»ΠΈ ΡΠ΅ΡΡΠ½ΠΎ, ΠΏΠΎΠ½ΡΡΠΈΡ Π½Π΅ ΠΈΠΌΠ΅Ρ. ΠΠΎΠΆΠ΅Ρ Π±ΡΡΡ Π²ΡΠ·Π²Π°Π½ΠΎ ΠΈΠ·ΠΌΠ΅Π½Π΅Π½ΠΈΠ΅ΠΌ ΡΠ΅ΡΡΠΈΡΡΠ΅ΠΌΠΎΠ³ΠΎ ΠΏΠ»Π°Π³ΠΈΠ½Π°. Π‘ ΡΠ΅Ρ ΠΏΠΎΡ Ρ Π½Π΅ ΠΏΠΈΡΠ°Π» Π½ΠΈ ΠΎΠ΄Π½ΠΎΠ³ΠΎ C ++, ΠΏΠΎΡΡΠΎΠΌΡ ΠΌΠ½Π΅ Π΄ΠΎΠ²ΠΎΠ»ΡΠ½ΠΎ ΡΠ»ΠΎΠΆΠ½ΠΎ ΠΏΠΎΠ½ΡΡΡ, ΡΡΠΎ Ρ Π΄Π΅Π»Π°Π» ΡΠΎΠ³Π΄Π°. π
Π’Π΅ΠΌ Π½Π΅ ΠΌΠ΅Π½Π΅Π΅, ΡΠΏΠ°ΡΠΈΠ±ΠΎ Π·Π° Π±ΡΡΡΡΡΠΉ ΠΎΡΠ²Π΅Ρ.
ΠΠΎΡ ΠΎΠΆΠ΅, ΡΡΠΎ ΠΏΡΠΎΠ²Π΅ΡΠΊΠ° JSON ΡΡΠ΄Π° ΠΈ ΠΎΠ±ΡΠ°ΡΠ½ΠΎ ΡΠ΅ΠΏΠ΅ΡΡ Π½Π΅ ΡΠ°Π±ΠΎΡΠ°Π΅Ρ. @sanssecours Ρ Π²Π°Ρ Π΅ΡΡΡ ΠΈΠ΄Π΅Ρ?
@sanssecours Ρ Π²Π°Ρ Π΅ΡΡΡ ΠΈΠ΄Π΅Ρ?
ΠΠ°ΡΠΊΠΎΠ»ΡΠΊΠΎ Ρ ΠΌΠΎΠ³Ρ ΡΡΠ΄ΠΈΡΡ, ΠΈΠ· ΠΊΠΎΠ΄Π°:
ΠΈ
YAJL ΠΏΡΠΎΡ
ΠΎΠ΄ΠΈΡ ΠΏΡΠΎΠ²Π΅ΡΠΊΡ Π² ΠΎΠ±Π° ΠΊΠΎΠ½ΡΠ° ΠΏΠΎΡΠ»Π΅ ΡΠΎΠ³ΠΎ, ΠΊΠ°ΠΊ PR # 2580 Π±ΡΠ» ΠΎΠ±ΡΠ΅Π΄ΠΈΠ½Π΅Π½ Ρ Π²Π΅ΡΠΊΠΎΠΉ master
. ΠΡΡ
ΠΎΠ΄Ρ ΠΈΠ· ΡΡΠΈΡ
ΠΎΠ³ΡΠ°Π½ΠΈΡΠ΅Π½Π½ΡΡ
Π·Π½Π°Π½ΠΈΠΉ, Ρ Π±Ρ ΠΏΡΠ΅Π΄ΠΏΠΎΠ»ΠΎΠΆΠΈΠ», ΡΡΠΎ Π·Π°ΠΌΠ΅Π½ΠΈΠ» ASSERT_FALSE
Π²
Ρ ASSERT_TRUE
Π΄ΠΎΠ»ΠΆΠ½ΠΎ Π±ΡΡΡ ΠΏΡΠ°Π²ΠΈΠ»ΡΠ½ΡΠΌ ΡΠΏΠΎΡΠΎΠ±ΠΎΠΌ ΡΠ΅ΡΠ΅Π½ΠΈΡ ΠΏΡΠΎΠ±Π»Π΅ΠΌΡ.
ΠΠΎΡ ΠΎΠΆΠ΅, ΡΡΠΎ ΠΏΡΠΎΡΡΠΎΠ΅ ΡΠ΅ΡΠ΅Π½ΠΈΠ΅! Π‘ΠΏΠ°ΡΠΈΠ±ΠΎ Π·Π° ΠΏΠΎΠ΄ΡΠΎΠ±Π½ΠΎΠ΅ ΠΎΠΏΠΈΡΠ°Π½ΠΈΠ΅!
Π‘Π°ΠΌΡΠΉ ΠΏΠΎΠ»Π΅Π·Π½ΡΠΉ ΠΊΠΎΠΌΠΌΠ΅Π½ΡΠ°ΡΠΈΠΉ
ΠΠ°ΡΠΊΠΎΠ»ΡΠΊΠΎ Ρ ΠΌΠΎΠ³Ρ ΡΡΠ΄ΠΈΡΡ, ΠΈΠ· ΠΊΠΎΠ΄Π°:
https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/model_configformat.hpp#L84 -L92
ΠΈ
https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/model_configformat.hpp#L47 -L62
YAJL ΠΏΡΠΎΡ ΠΎΠ΄ΠΈΡ ΠΏΡΠΎΠ²Π΅ΡΠΊΡ Π² ΠΎΠ±Π° ΠΊΠΎΠ½ΡΠ° ΠΏΠΎΡΠ»Π΅ ΡΠΎΠ³ΠΎ, ΠΊΠ°ΠΊ PR # 2580 Π±ΡΠ» ΠΎΠ±ΡΠ΅Π΄ΠΈΠ½Π΅Π½ Ρ Π²Π΅ΡΠΊΠΎΠΉ
master
. ΠΡΡ ΠΎΠ΄Ρ ΠΈΠ· ΡΡΠΈΡ ΠΎΠ³ΡΠ°Π½ΠΈΡΠ΅Π½Π½ΡΡ Π·Π½Π°Π½ΠΈΠΉ, Ρ Π±Ρ ΠΏΡΠ΅Π΄ΠΏΠΎΠ»ΠΎΠΆΠΈΠ», ΡΡΠΎ Π·Π°ΠΌΠ΅Π½ΠΈΠ»ASSERT_FALSE
Π²https://github.com/ElektraInitiative/libelektra/blob/3d3c7eb52923cf4329197a9912c9439d9b33b920/src/tools/rest-backend/tests/test_service_convertengine.cpp#L154
Ρ
ASSERT_TRUE
Π΄ΠΎΠ»ΠΆΠ½ΠΎ Π±ΡΡΡ ΠΏΡΠ°Π²ΠΈΠ»ΡΠ½ΡΠΌ ΡΠΏΠΎΡΠΎΠ±ΠΎΠΌ ΡΠ΅ΡΠ΅Π½ΠΈΡ ΠΏΡΠΎΠ±Π»Π΅ΠΌΡ.