[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [wtp-dev] JSDT Sync call minutes [20160112]
|
Regarding the testing restrictions downloading
dependencies via npm or bower.. I wonder what m2e(Maven) does
for tests? because this is a similar repository issue?ccing Fred on this.
Thanks - Chuck
IBM Tools development, Eclipse WTP PMC
IBM Software Lab - Research Triangle Park, NCFrom:
"Victor V. Rubezhny"
<vrubezhny@xxxxxxxxxx>To:
wtp-dev@xxxxxxxxxxxDate:
01/13/2016 12:57 PMSubject:
Re: [wtp-dev]
JSDT Sync call minutes [20160112]Sent by:
wtp-dev-bounces@xxxxxxxxxxx
+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.teststhat 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-devCONFIDENTIALITY 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._______________________________________________
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