Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [nebula-dev] Please fix failed tests!

Thanks Mickael,

I took a stab at fixing the CDateTime tests (again). Now all tests run inside Eclipse and from the command line in windows (mvn clean verify). However, they still fail on Hudson linux, one of the ui tests is hanging:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=420939




On Thu, Nov 7, 2013 at 6:51 PM, Mickael Istria <mistria@xxxxxxxxxx> wrote:

* CDateTime
Could not fix them, any help is welcome!
It may be a synchronization issue (tests deal with a lock of synchronous Event triggerings).

* CWT
This is a permission issue caused by OSGi: a class in another bundle can't access a "package" class from another bundle. It can only if it's in a fragment.
I worked around it by creating a "MorePublic" class for test purpose, which makes public some package methods used by tests. The fix also contain renaming of a VTestCase to AbstractVTestCase to avoid some confusion (for me and Tycho): https://git.eclipse.org/r/18182
* RadioGroup
It's a similar permission issue as described above, except that in this case, I found it better to make the methods public as it seems to me that they can be useful. https://git.eclipse.org/r/18184

Please add your comments and votes to this Gerrit patch set!

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

_______________________________________________
nebula-dev mailing list
nebula-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/nebula-dev



Back to the top