Matthias, I did not look at those tests to see how important they are. But,
if they do test something important, rather than changing the expected result,
I think you should modify the tests to use an ordered collection, and add one
or two new tests for the error condition.
--
Review: Approve
--
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
The proposal to merge lp:~markos-za/zorba/markos-bugs into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
--
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
Your
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/markos-bugs-2011-10-03T08-56-11.421Z/log.html
--
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Markos Zaharioudakis has proposed merging lp:~markos-za/zorba/markos-bugs into
lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
--
Validation queue job markos-bugs-2011-10-03T08-56-11.421Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~markos-za/zorba/markos-bugs into lp:zorba has been
updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
--
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/77874
Your team
According to Launchpad, the change I suggested has not been made - why the new
Merge Proposal?
(Also confused by Launchpad's saying that the Approved Revision is no longer
in the source branch...)
Strange, I can see the following changes in the diff:
=== removed file
Review: Approve
Ah, I see the deleted files now. So, I'll approve the diff. However, I'm not
sure the remote queue will pick it up with the no longer in the source branch
message for Approved Revision. You may need to re-Approve the Merge Proposal.
--
The proposal to merge lp:~matthias-brantner/zorba/bug-863730 into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784
--
The proposal to merge lp:~matthias-brantner/zorba/bug-863730 into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-863730-2011-10-03T20-24-36.523Z/log.html
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job bug-863730-2011-10-03T20-24-36.523Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~matthias-brantner/zorba/bug-863730 into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-863730/+merge/77784
--
** Changed in: zorba
Status: New = Fix Committed
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/863730
Title:
static delete-node* functions don't raise ZDDY0012
Status in Zorba - The
Matthias Brantner has proposed merging lp:~matthias-brantner/zorba/bug-fixing
into lp:zorba.
Requested reviews:
Chris Hillery (ceejatec)
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
fix if Zorba is built with a different store (name of
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-fixing-2011-10-03T23-12-08.546Z/log.html
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Validation queue job bug-fixing-2011-10-03T23-12-08.546Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 0, Disapprove 1.
Got: 1 Pending.
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
--
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/78019
--
** Changed in: zorba
Status: New = In Progress
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/855715
Title:
Invalid escaped characters in regex not caught
Status in Zorba - The XQuery
23 matches
Mail list logo