Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [wtp-dev] JSDT Sync call minutes [20160112]
+1 from me too, but...

Yes, I've asked not to include any JUnit tests into the contribution. Otherwise I have had to open a number of additional CQs for those tests, while we are limited in time.

Any JUnit Tests which aren't bringing a licensing problems are welcomed and can be added at any time after the contribution is done.

There is a couple of little licensing problems regarding these tests:
1) any json file (like bower.json, for example) is to be written by a committer or a contributor (we must respect authors) - I think it's the smallest problem.
2) bower.json and package.json files specify dependencies for a project, but we have no our own projects in bower/npm repositories published under Eclipse Public License, so it's very problematic for us to reference any products (in these files) in our tests. At least I don't know a single existing _javascript_ library published under EPL in npm. The same problem is for Bower/NPM tools contributed by Ilya.

So, we CAN write some common JUnit tests for testing some functionality, like creating some json file, finding content assist proposals according to its JSON Schema, validating it against the schema, but we CANNOT use a json file to download any dependency into a test project or, for example, to show proposals of possible versions of some _javascript_ library (because it requires us to download it from outside of Eclipse's repositories) and/or request any info from bower or npm repositories.

As far as I understood, we have to open a CQ every time we need to use something "from outside" in case we're not sure that it's license allows us to do it from Eclipse.

Ilya has some JUnit tests for bower/npm tools - but those tests don't download anything afaik. So, if we have some tests which aren't using a content not authored by a contributor (Angelo or the other ones) or a committer - we CAN push such tests into Source Editing repo. Otherwise we have to open a CQ for every file or a library to be used and/or downloaded by JUnit test during its work.

In most cases there are no problems to open a CQ... It just takes to long to resolve sometimes.

Regards,
Victor



On 01/13/2016 11:29 AM, Mickael Istria wrote:
On 01/13/2016 09:23 AM, Angelo zerr wrote:
I think it should be very good to integrate JUnit tests https://github.com/angelozerr/eclipse-wtp-json/tree/master/core/org.eclipse.wst.json.core.tests that I have not included in the gerrit patch (Victor told me to remove it).
+1.
Unless there are complex legal issues with those, can you please turn them as a patch to webtools.sourceediting and push it onto Gerrit?

--
Mickael Istria
Eclipse developer at JBoss, by Red Hat
My blog - My Tweets


_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/wtp-dev


CONFIDENTIALITY NOTICE: This email and files attached to it are confidential. If you are not the intended recipient you are hereby notified that using, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. If you have received this email in error please notify the sender and delete this email.

Back to the top