Bug #107240 for RDF-Trine-Store-File: failling tests in t/store-file.t

Tue Sep 22 09:05:14 2015 mestre.smash [...] gmail.com - Ticket created

Subject:

failling tests in t/store-file.t

Hello, First of all thank you for this module. The current version (including the development version 0.11_2) is failing a test in t/store_file.t. The problem is that the all_triple_store_tests function call uses the get_contexts function which is not supported with this module. At least that's is what I figured out in a quick glimpse at the source. Thank you

Tue Sep 22 15:53:45 2015 kjetilk [...] cpan.org - Correspondence added

ty. 22. sep. 2015 09.05.14 skreiv SMASH: Show quoted text

> Hello, > > First of all thank you for this module.

Great, glad you like it! Show quoted text

> The current version (including > the development version 0.11_2) is failing a test in t/store_file.t. > The problem is that the all_triple_store_tests function call uses the > get_contexts function which is not supported with this module. At > least that's is what I figured out in a quick glimpse at the source.

Ah, yeah, I think this is due to that we changed the semantics og get_contexts in RDF::Trine. I think this is fixed on the https://github.com/kjetilk/RDF-Trine-Store-File master branch, actually. Would it be possible for you to test it? The reason why I haven't made a release in a while is that I had the ambition of making it support quads, but it turned out to be harder than expected because the Trine tests were somewhat confusing. However, if this fixes a problem, I can make a release from master. Cheers, Kjetil

Tue Sep 22 15:53:45 2015 The RT System itself - Status changed from 'new' to 'open'

Tue Sep 22 15:54:01 2015 kjetilk [...] cpan.org - Taken

Wed Sep 23 05:07:55 2015 mestre.smash [...] gmail.com - Correspondence added

From:

mestre.smash [...] gmail.com

Hello, Thank your for your quick reply. On Tue Sep 22 15:53:45 2015, KJETILK wrote: Show quoted text

> Ah, yeah, I think this is due to that we changed the semantics og > get_contexts in RDF::Trine. I think this is fixed on the > https://github.com/kjetilk/RDF-Trine-Store-File master branch, > actually. Would it be possible for you to test it?

I'm getting the same error in t/store-file.t using the master branch version. Your dist.ini file seems empty, I created a minimal version to test the module. And also created a PR on github if you want to use it: https://github.com/kjetilk/RDF-Trine-Store-File/pull/1 Show quoted text

> Cheers, > > Kjetil

Chheers, Thank you

Wed Sep 23 17:28:40 2015 kjetilk [...] cpan.org - Correspondence added

on. 23. sep. 2015 05.07.55 skreiv SMASH: Show quoted text

> Hello, > > Thank your for your quick reply. > > On Tue Sep 22 15:53:45 2015, KJETILK wrote:

> > Ah, yeah, I think this is due to that we changed the semantics og > > get_contexts in RDF::Trine. I think this is fixed on the > > https://github.com/kjetilk/RDF-Trine-Store-File master branch, > > actually. Would it be possible for you to test it?

> > I'm getting the same error in t/store-file.t using the master branch > version.

Strange! I'm pushing to github now, so lets see what travis says. Kjetil

Wed Sep 23 17:36:13 2015 kjetilk [...] cpan.org - Correspondence added

on. 23. sep. 2015 17.28.40 skreiv KJETILK: Show quoted text

> on. 23. sep. 2015 05.07.55 skreiv SMASH:

> > I'm getting the same error in t/store-file.t using the master branch > > version.

> > Strange! I'm pushing to github now, so lets see what travis says.

Ah, there it fails, and now I see it failing for me too! Let see, this is harder than I expected, I don't know if I can find the time, but I'll try. Kjetil