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]

m2e tests simply don't live @ eclipse.org

It was deemed too complex/tedious to try to CQ the sheer number of dependencies that are dragged while running tests (all these files are stored in the git repo to speed up test execution). 

So tests live in github https://github.com/tesla/m2e-core-tests and run on an external CI server https://ci.takari.io/view/m2e/job/m2eclipse-core-tests/

This is less than ideal, obviously, I'd prefer if we could have all code and run all tests on the same infra, but I just can't imagine having to open ~400 CQs (it'd be probably forbidden upon by the Geneva convention)

Fred




On Wed, Jan 13, 2016 at 3:45 PM, Chuck Bridgham <cbridgha@xxxxxxxxxx> wrote:
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, NC




From:        "Victor V. Rubezhny" <vrubezhny@xxxxxxxxxx>
To:        wtp-dev@xxxxxxxxxxx
Date:        01/13/2016 12:57 PM
Subject:        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-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._______________________________________________
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





--
"Have you tried turning it off and on again" - The IT Crowd
And if that fails, then http://goo.gl/tnBgH5

Back to the top