Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not all TMAPI2 test are ran in the trunk, and fail when enabled #383

Closed
GoogleCodeExporter opened this issue Mar 16, 2015 · 3 comments
Closed

Comments

@GoogleCodeExporter
Copy link

GoogleCodeExporter commented Mar 16, 2015

The following TMAPI 2 test cases are not checked:

org.tmapi.core.TestTopicMapSystemFactory
org.tmapi.core.TestRFC3986

Enabling these results in the following:

Running tests
..E.F.F
Time: 0.282
There was 1 error:
1) testReadFromRuntimeProperty(org.tmapi.core.TestTopicMapSystemFactory) 
java.io.FileNotFoundException: 
/System/Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home/lib/tmapi.prope
rties (Permission denied)
...
There were 2 failures:
1) 
testReadFromResource(org.tmapi.core.TestTopicMapSystemFactory)junit.framework.As
sertionFailedError
2) 
test_RFC_3986__5_4_1_Normal_Examples(org.tmapi.core.TestRFC3986)junit.framework.
ComparisonFailure: Unexpected result for ?y expected:<http://a/b/c/[d;p]?y> but 
was:<http://a/b/c/[]?y>

Original issue reported on code.google.com by qsieb...@gmail.com on 6 Apr 2011 at 1:59

@GoogleCodeExporter
Copy link
Author

GoogleCodeExporter commented Mar 16, 2015

The RFC3986 test is a non-normative test case of TMAPI 2. Simply exclude it 
from the build. 
The TestTopicMapSystemFactory test is also not required. It is a test for TMAPI 
to check if TMAPI itself can gather the concrete implementation from different 
sources. Exclude that test as well.

Original comment by lars.he...@gmail.com on 6 Apr 2011 at 5:37

@GoogleCodeExporter
Copy link
Author

GoogleCodeExporter commented Mar 16, 2015

I've added the tests (disabled) to the test set, so that it is clear why they 
are not ran. See r2017

Original comment by qsieb...@gmail.com on 8 Apr 2011 at 7:11

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

GoogleCodeExporter commented Mar 16, 2015

Original comment by qsieb...@gmail.com on 27 Jan 2012 at 10:58

  • Added labels: Release5.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant