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

Issue-68 Behaviour test info.json config.loader #165

Merged
merged 2 commits into from
Sep 8, 2021

Conversation

santhosh-thangavel
Copy link
Contributor

1.A cube is loaded using the info.json with a CSV.
2.Cube&qb validation succeeds.
3.Cube is serialised to CSV-W
4.csvlint validation succeeds
5.csv2rdf succeeds
6.RDF passes "SKOS, qb" SPARQL tests
7.Ouput ttl is compared with ttl as we do in gss-utils
8.Additional ttl is appended to the resulting RDF for multi-measure-data.csv as the Measure dimension and unit attribute defined in info.json is an existing dimension and attribute.

#68

2.Cube&qb validation succeeds.
3.Cube is serialised to CSV-W
4.csvlint validation succeeds
5.csv2rdf succeeds
6.RDF passes "SKOS, qb" SPARQL tests
7.Ouput ttl is compared with ttl as we do in gss-utils
8.Additional ttl is appended to the resulting RDF for multi-measure-data.csv as the Measure dimension and unit attribute defined in info.json is an existing dimension and attribute.
@santhosh-thangavel
Copy link
Contributor Author

@canwaf Please, review my pull request.

Copy link
Contributor

@canwaf canwaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests do what they say they do, changes don't impact other behaviour tests. Approved.

@santhosh-thangavel santhosh-thangavel merged commit 8629341 into main Sep 8, 2021
@santhosh-thangavel santhosh-thangavel deleted the santhosh/issue68 branch September 8, 2021 06:42
@@ -67,7 +67,7 @@ def test_structure_defined():
"Observed Value",
QbSingleMeasureObservationValue(
ExistingQbMeasure("http://example.org/units/some-existing-measure"),
ExistingQbUnit("http://example.org/units/some-existing-unit"),
ExistingQbUnit("http://example.org/units/some-exisiting-unit"),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've typoed my good spelling? :P

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robons Fixed it. Thank you.

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

Successfully merging this pull request may close these issues.

3 participants