Bug 279679 - [XML Catalog] Test XML Catalog implementation against KDE test suite
Summary: [XML Catalog] Test XML Catalog implementation against KDE test suite
Status: ASSIGNED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.xml (show other bugs)
Version: 3.1   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: Future   Edit
Assignee: Jesper Moller CLA
QA Contact: Nick Sandonato CLA
URL: http://websvn.kde.org/trunk/kdenonbet...
Whiteboard: api
Keywords: needinfo
Depends on: 112284
Blocks:
  Show dependency tree
 
Reported: 2009-06-09 15:25 EDT by Jesper Moller CLA
Modified: 2013-06-19 11:14 EDT (History)
3 users (show)

See Also:


Attachments
ZIP archive of KDE XML Catalog tests (45.85 KB, application/x-zip-compressed)
2009-06-09 16:45 EDT, Jesper Moller CLA
no flags Details
Cleaned up distribution of KDE's OASIS Catalog test suite, under EPL v1.0. (96.91 KB, application/x-zip)
2009-08-14 08:43 EDT, Frans Englich CLA
no flags Details
Cleaned up distribution of KDE's OASIS Catalog test suite, under EPL v1.0.(second submit) (21.72 KB, application/zip)
2009-10-21 06:55 EDT, Frans Englich CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jesper Moller CLA 2009-06-09 15:25:32 EDT
As suggested in comment 28 in bug 112284 <https://bugs.eclipse.org/bugs/show_bug.cgi?id=112284#c28>, we should investigate using the KDE XML Catalog test suite for testing our implementation.
The original source was mentioned under http://markmail.org/message/g5dgrzqy3w7e4gr5

KDE has since been switched to Subversion, and i found the test suite at: 

svn co svn://anonsvn.kde.org/home/kde/trunk/kdenonbeta/kdom/catalog/TestSuite@718414

I'm assuming we need an IP QC if this suite is to be added to the WTP source editing build - the current license looks like this:


    Copyright 2005 Frans Englich <frans.englich@telia.com>

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is fur-
nished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in
all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FIT-
NESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER 
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING 
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS 
IN THE SOFTWARE.

(that's fairly liberal, isn't it?)

I'm going to try and run the test against the WTP XML Catalog implementation, and add any bugs as linked to this report.

You may assign the bug to me: jesper@selskabet.org
Comment 1 Jesper Moller CLA 2009-06-09 15:55:51 EDT
Added link to bug 112248
Comment 2 Jesper Moller CLA 2009-06-09 15:58:11 EDT
Bummer, I meant bug 112284.
Comment 3 David Carver CLA 2009-06-09 16:20:42 EDT
(In reply to comment #0)
> I'm assuming we need an IP QC if this suite is to be added to the WTP source
> editing build - the current license looks like this:

Yes, we'll need a CQ for this.  Can you attach a ZIP version of the test suite to this bug, as we'll need that for the CQ.

Nitin I don't mind working this through the CQ process.  I've already done this for the XQuery/Xpath 2.0 test suite from the W3C, and doing the same for the W3C DOM Level 1 test suite.

What would also help Jesper, is if there was already a set of Junit tests for this test suite created.  You can take a look at what I've done in XSL Tools for the W3C Test suite as a starting point.   I ended up writing an XSLT to generate the necessary Java JUnits for each of the tests that needed to be executed.

org.eclipse.wst.xml.xpath2.processor.tests

Comment 4 Jesper Moller CLA 2009-06-09 16:45:02 EDT
Created attachment 138720 [details]
ZIP archive of KDE XML Catalog tests

As exported from Subversion, for IPlog purposes.

svn export
svn://anonsvn.kde.org/home/kde/trunk/kdenonbeta/kdom/catalog/TestSuite@718414

I'll figure out how to run the tests from JUnit. If you want to keep the QC together, you'll have to wait for my test cases, I can't really commit to a date, but it'll be before M1. :-)
Comment 5 David Carver CLA 2009-06-09 16:58:09 EDT
No, I can get the XML Catalog Tests included separately with out the necessary JUnit tests.   I figure it may take it a bit to go through IP with the backlog that currently exists.

I'll see about getting the CQ submitted though tonight so we aren't waiting on it.  I'm going to tentatively target this for the 3.2 time frame, but no guarantee that it'll make it.
Comment 6 David Carver CLA 2009-06-10 22:38:07 EDT
I've submitted the CQ for this, so it's at least in the queue to be reviewed.
Comment 7 David Carver CLA 2009-08-05 09:56:52 EDT
We are getting closer, the plan is to maintain and migrate part of the above set of files and tests to being maintained officially at eclipse.   We have aggreement from one of the parties to relicense his contributions as EPL, still waiting on the other party.

Comment 8 Frans Englich CLA 2009-08-14 08:43:07 EDT
Created attachment 144524 [details]
Cleaned up distribution of KDE's OASIS Catalog test suite, under EPL v1.0.

This ZIP contains a git repository with some minor changes, but most of all the change to the Eclipse license. This was cleared by the two copyright holders(me and Rob Lugt), and discussed with David Carver, Sharon Corbett, and Barb Cochrane.

If you don't want to work with git, simply remove the .git/ folder in the root directory and go from there.

So this test suite is now moving over to the Eclipse Foundation, and I'll stop maintaining it, and you Eclipse people take over :)
Comment 9 David Carver CLA 2009-08-14 10:23:05 EDT
(In reply to comment #8)
> Created an attachment (id=144524) [details]
> Cleaned up distribution of KDE's OASIS Catalog test suite, under EPL v1.0.
> 
> This ZIP contains a git repository with some minor changes, but most of all the
> change to the Eclipse license. This was cleared by the two copyright holders(me
> and Rob Lugt), and discussed with David Carver, Sharon Corbett, and Barb
> Cochrane.
> 
> If you don't want to work with git, simply remove the .git/ folder in the root
> directory and go from there.
> 
> So this test suite is now moving over to the Eclipse Foundation, and I'll stop
> maintaining it, and you Eclipse people take over :)
> 

Thanks Frans.   We'll keep good care of it.  Keep an eye on this bug for a bit as IP team may have a few more questions, but I think we should be good.
Comment 10 Frans Englich CLA 2009-10-21 06:55:21 EDT
Created attachment 150091 [details]
 Cleaned up distribution of KDE's OASIS Catalog test suite, under EPL v1.0.(second submit)

This second attachment incorporates feedback by Sharon Corbett, via email "RE: XML Catalogue Test Suite 1.0 Follow On Questions"(Tue, 13 Oct). Notably:

* runTests.py is excluded
* .git/ is excluded
* Changelog has also been excluded
Comment 11 David Carver CLA 2009-10-21 09:17:40 EDT
(In reply to comment #10)
> Created an attachment (id=150091) [details]
>  Cleaned up distribution of KDE's OASIS Catalog test suite, under EPL
> v1.0.(second submit)
> 
> This second attachment incorporates feedback by Sharon Corbett, via email "RE:
> XML Catalogue Test Suite 1.0 Follow On Questions"(Tue, 13 Oct). Notably:
> 
> * runTests.py is excluded
> * .git/ is excluded
> * Changelog has also been excluded

Thanks, Frans.  I've updated the eclipse CQ with the attachment and turned it back over to IP.  I think we are close.
Comment 12 Jesper Moller CLA 2009-12-29 19:16:35 EST
We should be good to go with this since the corresponding IPzilla is resolved long ago.

Should this go in a bundle for itself like the XQTS stuff did?
Comment 13 David Carver CLA 2009-12-29 21:52:49 EST
(In reply to comment #12)
> We should be good to go with this since the corresponding IPzilla is resolved
> long ago.
> 
> Should this go in a bundle for itself like the XQTS stuff did?

Yes, I would keep it in it's own bundle.  This way it can be used by other's outside of eclipse as well.
Comment 14 David Carver CLA 2010-02-21 08:58:40 EST
Jesper I'm re-assigning this to you to shepard the rest of the way through the process.
Comment 15 David Carver CLA 2011-01-28 19:04:30 EST
(In reply to comment #14)
> Jesper I'm re-assigning this to you to shepard the rest of the way through the
> process.

Jesper just a reminder that this is still out there, we have everything in order we just need to push it over the finish line.