RE: Fix for test1236 assertion

That seems reasonable to me at this time on a Friday night ;-)

so I changed:

<assert test="not(../following-sibling::log:message[@testcase=current()/../@testcase and @message='2']) or ../following-sibling::log:message[@testcase=current()/../@testcase and @message='2']/content[not(node())]"/>

to:

<assert 
test="not(../following-sibling::log:message[@testcase=current()/../@testcase and @message='2']) or 
count(../following-sibling::log:message[@testcase=current()/../@testcase and @message='2']/content/*) = '0'"/>

for 1236 and 1237 and turned the handle:

http://www.w3.org/2002/ws/addr/testsuite/report/

it's getting greener ..

Paul


-----Original Message-----
From: public-ws-addressing-tests-request@w3.org on behalf of David Illsley
Sent: Fri 1/27/2006 4:24 PM
To: Hugo Haas
Cc: public-ws-addressing-tests@w3.org; public-ws-addressing-tests-request@w3.org
Subject: Re: Fix for test1236 assertion
 
1236 still failed for the WS-I logs so I tried to understand the second 
XPath that Hugo added... and couldn't so I tried one of my own which seems 
to have the desired effect for both style logs:

<assert 
test="not(../following-sibling::log:message[@testcase=current()/../@testcase 
and @message='2']) or 
count(../following-sibling::log:message[@testcase=current()/../@testcase 
and @message='2']/content/*) = '0'"/>

Does anyone have any objections to this change being made for 1236 (and 
the optional test 1237 which is testing a similar thing)?
David

David Illsley
Web Services Development
IBM Hursley Park, SO21 2JN
+44 (0)1962 815049 (Int. 245049)
david.illsley@uk.ibm.com



Hugo Haas <hugo@w3.org> 
Sent by: public-ws-addressing-tests-request@w3.org
19/01/2006 18:54

To
public-ws-addressing-tests@w3.org
cc

Subject
Fix for test1236 assertion






test1236 is marked as failed for the WS-I log as it shows a reply with
an empty content, and the assertion was testing for the absence of an
entry for the reply in the log.

I changed:

  <assert 
test="not(../following-sibling::log:message[@testcase=current()/../@testcase 
and @message='2'])"/>

into:

  <assert 
test="not(../following-sibling::log:message[@testcase=current()/../@testcase 
and @message='2']) or 
../following-sibling::log:message[@testcase=current()/../@testcase and 
@message='2']/content[not(node())]"/>

Cheers,

Hugo

-- 
Hugo Haas - W3C
mailto:hugo@w3.org - http://www.w3.org/People/Hugo/

Received on Friday, 27 January 2006 19:45:23 UTC