Merge lp:~zorba-coders/zorba/fix_bug_dc into lp:zorba/data-cleaning-module

Proposed by Sorin Marian Nasoi
Status: Merged
Approved by: Chris Hillery
Approved revision: 35
Merged at revision: 35
Proposed branch: lp:~zorba-coders/zorba/fix_bug_dc
Merge into: lp:zorba/data-cleaning-module
Diff against target: 45 lines (+3/-7)
5 files modified
CMakeLists.txt (+0/-4)
test/ExpQueryResults/data-cleaning/conversion/phone-from-address.xml.res (+1/-1)
test/ExpQueryResults/data-cleaning/conversion/phone-from-user.xml.res (+0/-1)
test/ExpQueryResults/data-cleaning/conversion/unit-convert.xml.res (+1/-0)
test/ExpQueryResults/data-cleaning/conversion/user-from-address.xml.res (+1/-1)
To merge this branch: bzr merge lp:~zorba-coders/zorba/fix_bug_dc
Reviewer Review Type Date Requested Status
Bruno Martins Approve
Diogo Simões Approve
Gabriel Petrovay (community) Approve
Sorin Marian Nasoi Approve
Matthias Brantner Pending
Review via email: mp+86706@code.launchpad.net

Commit message

Fix for bug 871051.

Description of the change

Fix for bug 871051.

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
Gabriel Petrovay (gabipetrovay) wrote :

This doesn't fix the issue:
The following tests FAILED:
        2516 - zorba_data-cleaning_module/data-cleaning/conversion/address-from-user.xq (Failed)
        2520 - zorba_data-cleaning_module/data-cleaning/conversion/phone-from-user.xq (Failed)
        2523 - zorba_data-cleaning_module/data-cleaning/conversion/user-from-phone.xq (Failed)
Errors while running CTest

Moreover, I don't see any changes in the diff in any files related to the "address-from-user.xq" test.

review: Needs Fixing
Revision history for this message
Gabriel Petrovay (gabipetrovay) wrote :

I was wrong. It FIXES the issue.

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_dc into lp:zorba/data-cleaning-module failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:272 (message):
  Validation queue job fix_bug_dc-2011-12-23T13-04-38.956Z is finished. The
  final status was:

  Undetermined, probably an error - please email <email address hidden> with the
  number of this job!

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

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_dc into lp:zorba/data-cleaning-module failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:272 (message):
  Validation queue job fix_bug_dc-2011-12-23T13-08-35.284Z is finished. The
  final status was:

  Undetermined, probably an error - please email <email address hidden> with the
  number of this job!

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

Revision history for this message
Diogo Simões (diogo-simoes89) :
review: Approve
Revision history for this message
Chris Hillery (ceejatec) wrote :

Not sure why the "conflicting tags" problem only seems to have affected this merge proposal, but at any rate, I've fixed it on the remote queue machine and put code in to prevent it happening in future.

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_dc-2011-12-23T20-23-35.246Z is finished. The final status was:

All tests succeeded!

Revision history for this message
Bruno Martins (bgmartins) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2011-10-09 00:22:42 +0000
3+++ CMakeLists.txt 2011-12-22 14:00:41 +0000
4@@ -25,10 +25,6 @@
5
6 ADD_TEST_DIRECTORY("${PROJECT_SOURCE_DIR}/test" "${EXCEPTION_LIST}")
7
8-EXPECTED_FAILURE(zorba_data-cleaning_module/data-cleaning/conversion/address-from-user.xq 871051)
9-EXPECTED_FAILURE(zorba_data-cleaning_module/data-cleaning/conversion/phone-from-user.xq 871051)
10-EXPECTED_FAILURE(zorba_data-cleaning_module/data-cleaning/conversion/user-from-phone.xq 871051)
11-
12 ADD_SUBDIRECTORY("src")
13
14 DONE_DECLARING_ZORBA_URIS()
15
16=== modified file 'test/ExpQueryResults/data-cleaning/conversion/phone-from-address.xml.res'
17--- test/ExpQueryResults/data-cleaning/conversion/phone-from-address.xml.res 2011-07-19 19:12:03 +0000
18+++ test/ExpQueryResults/data-cleaning/conversion/phone-from-address.xml.res 2011-12-22 14:00:41 +0000
19@@ -1,1 +1,1 @@
20-(520) 824-3160
21\ No newline at end of file
22+(520) 824-3160 (520) 824-3160
23\ No newline at end of file
24
25=== modified file 'test/ExpQueryResults/data-cleaning/conversion/phone-from-user.xml.res'
26--- test/ExpQueryResults/data-cleaning/conversion/phone-from-user.xml.res 2011-10-17 10:05:58 +0000
27+++ test/ExpQueryResults/data-cleaning/conversion/phone-from-user.xml.res 2011-12-22 14:00:41 +0000
28@@ -1,1 +0,0 @@
29-(661) 397-4236 (310) 513-0752 (510) 259-0456 (831) 385-3605 (213) 627-0188 (323) 846-1235 (661) 224-1072 (909) 820-3137 (916) 627-1090 (707) 938-9861 (805) 648-6417
30
31=== added file 'test/ExpQueryResults/data-cleaning/conversion/unit-convert.xml.res'
32--- test/ExpQueryResults/data-cleaning/conversion/unit-convert.xml.res 1970-01-01 00:00:00 +0000
33+++ test/ExpQueryResults/data-cleaning/conversion/unit-convert.xml.res 2011-12-22 14:00:41 +0000
34@@ -0,0 +1,1 @@
35+1.609344
36\ No newline at end of file
37
38=== modified file 'test/ExpQueryResults/data-cleaning/conversion/user-from-address.xml.res'
39--- test/ExpQueryResults/data-cleaning/conversion/user-from-address.xml.res 2011-07-19 19:12:03 +0000
40+++ test/ExpQueryResults/data-cleaning/conversion/user-from-address.xml.res 2011-12-22 14:00:41 +0000
41@@ -1,1 +1,1 @@
42-Stan Smith
43\ No newline at end of file
44+Sunizona Greenhouses Inc Stan Smith
45\ No newline at end of file

Subscribers

People subscribed via source and target branches

to all changes: