Merge lp:~zorba-coders/zorba/fix_bug_1172276 into lp:zorba

Proposed by Sorin Marian Nasoi
Status: Merged
Merged at revision: 11422
Proposed branch: lp:~zorba-coders/zorba/fix_bug_1172276
Merge into: lp:zorba
Diff against target: 73 lines (+14/-18)
2 files modified
test/fots/CMakeLists.txt (+7/-11)
test/fots_driver/evaluate.xq (+7/-7)
To merge this branch: bzr merge lp:~zorba-coders/zorba/fix_bug_1172276
Reviewer Review Type Date Requested Status
Chris Hillery Approve
Sorin Marian Nasoi Approve
Review via email: mp+161771@code.launchpad.net

Commit message

- fix for eval:serialization-matches that made some of the "prod-OptionDecl.serialization" failing test case to pass because the serialization params defined in the prolog are the same as the default ones in Zorba
- marked the rest of the failures as DISPUTED: please see W3C bug #21868
- removed EXPECTED_FOTS_FAILURE for "K2-Serialization-11"

To post a comment you must log in.
Revision history for this message
Sorin Marian Nasoi (sorin.marian.nasoi) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Revision history for this message
Sorin Marian Nasoi (sorin.marian.nasoi) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

The attempt to merge lp:~zorba-coders/zorba/fix_bug_1172276 into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job fix_bug_1172276-2013-05-01T05-34-44.47Z is finished.
  The final status was:

  1 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

Revision history for this message
Chris Hillery (ceejatec) wrote :

Why are there two new moduleImport failures?

review: Needs Information
Revision history for this message
Sorin Marian Nasoi (sorin.marian.nasoi) wrote :

> Why are there two new moduleImport failures?
Fixed: it was caused by an error in the merge with Zorba trunk.

Revision history for this message
Chris Hillery (ceejatec) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job fix_bug_1172276-2013-05-01T13-59-44.925Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'test/fots/CMakeLists.txt'
--- test/fots/CMakeLists.txt 2013-04-30 23:54:17 +0000
+++ test/fots/CMakeLists.txt 2013-05-01 07:10:31 +0000
@@ -151,6 +151,13 @@
151EXPECTED_FOTS_FAILURE (DISPUTED prod-FunctionDecl function-decl-reserved-function-names-029 21568)151EXPECTED_FOTS_FAILURE (DISPUTED prod-FunctionDecl function-decl-reserved-function-names-029 21568)
152EXPECTED_FOTS_FAILURE (DISPUTED prod-FunctionDecl function-decl-reserved-function-names-031 21568)152EXPECTED_FOTS_FAILURE (DISPUTED prod-FunctionDecl function-decl-reserved-function-names-031 21568)
153EXPECTED_FOTS_FAILURE (DISPUTED prod-VarDefaultValue extvardef-006b 21832)153EXPECTED_FOTS_FAILURE (DISPUTED prod-VarDefaultValue extvardef-006b 21832)
154EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-003 21868)
155EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-004 21868)
156EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-005 21868)
157EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-026 21868)
158EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-028 21868)
159EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-031 21868)
160EXPECTED_FOTS_FAILURE (DISPUTED prod-OptionDecl.serialization Serialization-032 21868)
154161
155# Next three possibly unique to old RQ machine, but they need to be162# Next three possibly unique to old RQ machine, but they need to be
156# marked for the RQ to pass.163# marked for the RQ to pass.
@@ -252,7 +259,6 @@
252EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-6 0)259EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-6 0)
253EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-9 0)260EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-9 0)
254EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-10 0)261EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-10 0)
255EXPECTED_FOTS_FAILURE (misc-Serialization K2-Serialization-11 0)
256EXPECTED_FOTS_FAILURE (misc-XMLEdition XML10-4ed-Excluded-char-1 0)262EXPECTED_FOTS_FAILURE (misc-XMLEdition XML10-4ed-Excluded-char-1 0)
257EXPECTED_FOTS_FAILURE (misc-XMLEdition XML11-1ed-Included-char-1 0)263EXPECTED_FOTS_FAILURE (misc-XMLEdition XML11-1ed-Included-char-1 0)
258EXPECTED_FOTS_FAILURE (prod-Annotation annotation-3 0)264EXPECTED_FOTS_FAILURE (prod-Annotation annotation-3 0)
@@ -326,16 +332,6 @@
326EXPECTED_FOTS_FAILURE (prod-ModuleImport modules-collide-var-001 0)332EXPECTED_FOTS_FAILURE (prod-ModuleImport modules-collide-var-001 0)
327EXPECTED_FOTS_FAILURE (prod-ModuleImport modules-collide-fn-001 0)333EXPECTED_FOTS_FAILURE (prod-ModuleImport modules-collide-fn-001 0)
328EXPECTED_FOTS_FAILURE (prod-ModuleImport errata8-002a 0)334EXPECTED_FOTS_FAILURE (prod-ModuleImport errata8-002a 0)
329EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-003 0)
330EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-004 0)
331EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-005 0)
332EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-026 0)
333EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-027 0)
334EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-028 0)
335EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-029 0)
336EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-030 0)
337EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-031 0)
338EXPECTED_FOTS_FAILURE (prod-OptionDecl.serialization Serialization-032 0)
339EXPECTED_FOTS_FAILURE (prod-OrderByClause orderBy40 0)335EXPECTED_FOTS_FAILURE (prod-OrderByClause orderBy40 0)
340EXPECTED_FOTS_FAILURE (prod-OrderByClause orderBy41 0)336EXPECTED_FOTS_FAILURE (prod-OrderByClause orderBy41 0)
341EXPECTED_FOTS_FAILURE (prod-OrderByClause orderBy49 0)337EXPECTED_FOTS_FAILURE (prod-OrderByClause orderBy49 0)
342338
=== modified file 'test/fots_driver/evaluate.xq'
--- test/fots_driver/evaluate.xq 2013-04-17 19:10:37 +0000
+++ test/fots_driver/evaluate.xq 2013-05-01 07:10:31 +0000
@@ -548,19 +548,19 @@
548) as xs:string?548) as xs:string?
549{549{
550 try {550 try {
551 let $serResult := fn:serialize($result, $util:serParamXml)551 let $serResult := fn:serialize($result)
552 let $serExpResult := fn:serialize($expResult, $util:serParamXml)552 let $regex := fn:string($expResult)
553 let $matchesFlags := data($expResult/@flags)553 let $flags := data($expResult/@flags)
554 return554 return
555 if (exists($matchesFlags))555 if (exists($flags))
556 then556 then
557 if (matches($serResult, $serExpResult, $matchesFlags))557 if (matches($serResult, $regex, $flags))
558 then ()558 then ()
559 else concat("'serialization-matches' returned: result does not match expected result with flags '",559 else concat("'serialization-matches' returned: result does not match expected result with flags '",
560 $matchesFlags,560 $flags,
561 "'.")561 "'.")
562 else562 else
563 if (matches($serResult, $serExpResult))563 if (matches($serResult, $regex))
564 then ()564 then ()
565 else "'serialization-matches' returned: result does not match expected result."565 else "'serialization-matches' returned: result does not match expected result."
566 } catch * {566 } catch * {

Subscribers

People subscribed via source and target branches