Re: proposed set of SHACL tests

Hi!

> Does the commit below look correct?

yes, that should do the trick.

thx,
simon

---
DDipl.-Ing. Simon Steyskal
Institute for Information Business, WU Vienna

www: http://www.steyskal.info/  twitter: @simonsteys

Am 2016-01-15 06:20, schrieb Holger Knublauch:
> On 8/01/2016 5:27 PM, Simon Steyskal wrote:
>> 
>> @ Holger
>> If one states that an arbitrary number of results should be returned 
>> by using "ms:result false", your engine throws a 
>> ResourceRequiredException.
>> I fixed that by adding
>> 
>>     else if(JenaDatatypes.FALSE.equals(resultS.getObject())){
>>         if(results.isEmpty())
>>             fail("Validation was expected to produce validation 
>> results for " + testResource);
>>     }
>> 
>> to AbstractSHACLTestClass.java
> 
> Thanks for reporting this - that isn't a case that I have encountered
> yet, and in general my test framework isn't complete. Does the commit
> below look correct?
> 
> https://github.com/TopQuadrant/shacl/commit/47da5a8c6f25ad5c36fb2258c6d162bfd3e832fb
> 
> Thanks
> Holger