cessda / cessda.cdc.aggregator.shared-library

Python library containing shared code for the CESSDA CDC Aggregator
European Union Public License 1.2
0 stars 0 forks source link

Fix interpretation of Nesstar DDI1.2.2. in tests #15

Closed cessda-bitbucket-importer closed 3 years ago

cessda-bitbucket-importer commented 3 years ago

Original report on BitBucket by Toni Sissala (GitHub: toni-sissala).


In testcode the Nesstar DDI records have XML namespaces wrong. In Nesstar repos the xmlns is cleared in child elements and only declared in root (see http://harvester.sodanet.gr:8080/oai/request?verb=GetRecord&identifier=http://nesstar-server.sodanet.gr:80/obj/fStudy/ekke-1-0001-GR&metadataPrefix=oai_ddi and https://nesstar2.adp.fdv.uni-lj.si/oai-pmh/?verb=GetRecord&metadataPrefix=oai_ddi&identifier=http://nesstar2.adp.fdv.uni-lj.si:80/obj/fStudy/aa&)

cessda-bitbucket-importer commented 3 years ago

Original comment by Toni Sissala (GitHub: toni-sissala).


Merged in bugfix/15-fix-interpretation-of-nesstar-ddi-in-tests (pull request #11)

Fix interpretation of nesstar DDI in tests

The test XML did not clear the xmlns in child elements of the root. The nesstar repos seem to do that. Now the test xml also clears the xmlns after the root element.

Fixes #15 @ BB