[YANGIDE-11] (Re)enable non-UI tests for build and proposed Gerrit changes Created: 09/May/16 Updated: 26/Oct/17 |
|
| Status: | Open |
| Project: | yangide |
| Component/s: | General |
| Affects Version/s: | unspecified |
| Fix Version/s: | None |
| Type: | Improvement | ||
| Reporter: | Michael Vorburger | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||||||||||
| Description |
|
I just noticed that the yangide source tree actually includes x4 test projects, but as of today none of them are currently not run during its build and for proposed Gerrit changes. Having tests but not running them seems a shame! (Perhaps something like YANGIDE-10 would even have been detected by this?) This bug is to discuss what it would take in particular the existing tests in org.opendaylight.yangide.core.tests and org.opendaylight.yangide.m2e.yang.tests for build and proposed Gerrit changes. (I'll create a separate new bug for two other test bundles which I found.) I can see that these are commented out in the <modules> of the root pom.xml, but I don't really understand why .. do they not run (anymore?) - need help to get them green? Or were they perhaps commented out fearing that they will not run on a build server due to UI dependencies? As far as I saw, these two actually do NOT contain UI tests, so should be fine. |
| Comments |
| Comment by Michael Vorburger [ 09/May/16 ] |
|
> I'll create a separate new bug for two other test bundles which I found
|
| Comment by Michael Vorburger [ 09/May/16 ] |
|
> these are commented out in the <modules> of the root pom.xml BTW: This has the curious side effect that in a https://github.com/vorburger/opendaylight-eclipse-setup provision workspace, or even when one does a manual Import Existing Maven Projects from the yangide root directory, then org.opendaylight.yangide.core.tests and org.opendaylight.yangide.m2e.yang.tests do not get imported. It appears that M2E appears to be (IMHO too..) smart and "filters" out projects which are not in <modules>. |
| Comment by Michael Vorburger [ 09/May/16 ] |
|
https://git.opendaylight.org/gerrit/#/c/38586/ raised for this, and: Locally it passed just fine (without opening any UI related pop-ups), see: Running org.opendaylight.yangide.m2e.yang.tests.YangGenerationTest [INFO] org.opendaylight.yangide.m2e.yang.tests ............ SUCCESS [03:22 min] That YangGenerationTest does seem slow though at 3 minutes. On https://jenkins.opendaylight.org/releng/job/yangide-verify-boron/jdk=openjdk8,nodes=dynamic_verify/31/console this failed with: org.eclipse.swt.SWTError: No more handles [gtk_init_check() failed] So this "non-UI" test behind still requires a UI (because of a YangUIPreferenceInitializer from org.opendaylight.yangide.org.opendaylight.yangide.ui. Because you would need to have at least a "virtual" UI anyway for |
| Comment by David M. Karr [ 09/May/16 ] |
|
Getting the tests fully working (reliably) will be a project in itself. I commented out those modules in the POM because I often saw them failing in the command line build, even when they would succeed running interactively in Eclipse. I've already informally broached the subject of running automated UI tests with Thanh and Andy (Grimberg). I had more important things to get working. |
| Comment by Michael Vorburger [ 10/May/16 ] |
|
Next step: Waiting for reply to helpdesk@opendaylight.org helpdesk ticket #22870, and/or https://lists.opendaylight.org/pipermail/dev/2016-May/001999.html .. |
| Comment by Michael Vorburger [ 02/Jun/16 ] |
|
Helpdesk meanwhile gave me access to Jenkins sandbox. Email below clarifies what needs to be done next. Assigning to myself to indicate that I'm hoping to eventually get to this, but likely only during July and not June. If anyone gets to this earlier, please do and update status here. ---------- Forwarded message ---------- On Mon May 09 15:15:23 2016, vorburger@redhat.com wrote: I installed both plugins in the sandbox and will entrust you to test and recommend to us which one we should deploy to production. They both appear to be supported by JJB which is a good thing: http://docs.openstack.org/infra/jenkins-job-builder/wrappers.html If you find there are features that you cannot configure with JJB and need support for it. Feel free to post on the dev mailing list with the [releng] topic. We have an intern this summer who is working on JJB who can implement any missing functionality if necessary. > > This would be useful for That's great, please keep us in the loop on your findings. Our systems build using dynamically allocated build slaves so you may find that additional dependencies might be needed to be installed on the slave systems. Our CI Jenkins page which I linked to you today should have details on the different slave types and you can find the vagrant definitions for those slaves in the releng/builder repo under the "vagrant" directory. Unfortunately updates to these files currently require helpdesk to respin the images and we're in the middle of transitioning to the private cloud so there might be some blocking issues here if you need vagrants updated until the private cloud migration is complete. Thanks, |
| Comment by Michael Vorburger [ 26/Oct/17 ] |
|
Un-assigning issues I'll never work on... (I've no interest in yangide anymore.) |