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

CRS Tests done via string comparison #34

Closed
bermud opened this issue Jul 20, 2018 · 4 comments
Closed

CRS Tests done via string comparison #34

bermud opened this issue Jul 20, 2018 · 4 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@bermud
Copy link

bermud commented Jul 20, 2018

The NSG profile tests in the Geopackage CITE tests for CRS are not being done in a proper fashion. The test is doing a simple string compare of the entire CRS WKT. This is not a proper approach to validating a CRS. The CRS must be evaluated on a component by component basis to establish if the CRS is correct.

A simple example of why the current CITE test string comparison won’t work – a CRS WKT may contain comments – the NSG string compared against contains comments. I can get a test failure simply by putting in a description such as “Break the CITE Test”…

Horizontal component of 3D system. Used by the GPS satellite navigation system and for NATO military geodetic surveying.

It’s also possible to break the test by using alternate approved/documented keywords for the same object or object value. Or changing the order in a non-ambiguous fashion.

@lgoltz lgoltz added the duplicate This issue or pull request already exists label Jul 27, 2018
@lgoltz
Copy link
Contributor

lgoltz commented Jul 27, 2018

I close this as duplicate with reference to #27.

@lgoltz lgoltz closed this as completed Jul 27, 2018
@lgoltz lgoltz reopened this Jul 31, 2018
@lgoltz lgoltz added status:in-progress and removed duplicate This issue or pull request already exists labels Aug 1, 2018
@lgoltz
Copy link
Contributor

lgoltz commented Aug 1, 2018

#27 handles a test failure when the description of the CRS in the GeoPackage does not match the one from the specification.

In the test

the WKT is compared, but of they are not equal this is not a test failure.

@bermud Which test fails exactly with which version of the ETS and geopackage?

@lgoltz lgoltz added the help wanted Extra attention is needed label Aug 1, 2018
@ghost
Copy link

ghost commented Sep 12, 2018

Is this issue in reference to the CRS Description of Requirement 19 of the profile? https://nsgreg.nga.mil/doc/view?i=4379

If so, then the description should be the ones from Table 7 through Table 19. I don't see an issue with the way it is being tested in the test tool.

@lgoltz
Copy link
Contributor

lgoltz commented Sep 13, 2018

@melvchan Thank you for commenting!
I close this ticket as it is not clear which test is failing.

@bermud If there are remaining issues, please create a tickets. The exact test name, test resource and version of the ets and teamengine would be helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants