search

OPUS4 / application

OPUS 4 application.
Other
15 stars 21 forks source link

Export and doi import formats update for new doctypes #1171

Closed alw-bsz closed 5 months ago

alw-bsz commented 10 months ago
alw-bsz commented 9 months ago

@j3nsch: Ich habe nun die von mir gefixten Tests – im Wesentlichen Änderungen bei der Feldbelegung durch die Standardisierung – gepusht.

Zu den Tests, die jetzt noch brechen:

1) CitationExport_Model_HelperTest::testGetPlainOutputRis 2) CitationExport_IndexControllerTest::testDownloadActionRis

Fixe ich noch (lokal schon erledigt, bei der Vorbereitung des Push „durchgerutscht“)

3) CitationExport_IndexControllerTest::testYearIsNotExportedWhenOnlyPublishedYearIsSet 4) CitationExport_IndexControllerTest::testBibtexYearExportWithOnlyPublishedDateSet 5) CitationExport_IndexControllerTest::testBibtexYearExportWithOnlyCompletedDateSet 6) CitationExport_IndexControllerTest::testBibtexYearExportWithOnlyCompletedYearSet 7) CitationExport_IndexControllerTest::testBibtexYearExportWithEveryDate 8) CitationExport_IndexControllerTest::testBibtexYearExportWithEmptyStrings

Die Dokumenttyp-spezifischen BibTeX-Einzeltrefferexporte entfallen ja. Ich weiß allerdings nicht, ob die Funktionen für das Testen der Jahresangaben erhalten bleiben sollten, oder das anderweitig abgedeckt ist. Ich hätte gerne noch geschaut, was in OPUSVIER-3289 steht, aber Euer Jira geht derzeit nicht,

9) CitationExport_IndexControllerTest::testBibtexTypeMasterthesis 10) CitationExport_IndexControllerTest::testBibtexTypeDoctoralthesis

Ich vermute, dass hier nicht der Dokumenttyp im Exportformat, sondern die URL für den Export geprüft wird. Diese Tests sind durch den Wegfall der Dokumenttyp-spezifischen BibTeX-Einzeltrefferexporte dann ebenfalls obsolet.

11) Export_BibtexExportTest::testExportSingleDocument 17) Oai_IndexControllerTest::testGetRecordMarc21OfDocId91 18) Oai_IndexControllerTest::testGenerationOfField856With2VisibleInOaiFiles

Schaue ich mir an. Vermutlich haben sich Belegungen geändert, die ich im Test korrigieren muss.

Die Failures wegen falschen Response Codes ( 12) – 16) ) müsste ich mir näher anschauen. Falls Du schon siehst, woran es liegt, bitte gerne beheben.

alw-bsz commented 8 months ago

Zu den Tests, die jetzt noch brechen:

1) CitationExport_Model_HelperTest::testGetPlainOutputRis

Dazu hattest Du bereits festgestellt, dass der Test ein XML mit negativen Zahlen als Year von CompletedDate und PublishedDate erzeugt, was eigentlich nicht passieren sollte:

<?xml version="1.0" encoding="UTF-8"?> 
<Opus version="1.0" xmlns:xlink="http://www.w3.org/1999/xlink">
    <Opus_Document Id="329" BelongsToBibliography="0" PublishedYear="2015" PublicationState="draft" ServerState="unpublished" Type="conferenceproceedings">
        <CompletedDate Year="-0001" Month="11" Day="30"/>
        <PublishedDate Year="-0001" Month="11" Day="30"/>
        <ServerDateCreated Year="2023" Month="12" Day="08" Hour="16" Minute="35" Second="24" Timezone="+01:00" UnixTimestamp="1702049724"/>
        <ServerDateModified Year="2023" Month="12" Day="08" Hour="16" Minute="35" Second="24" Timezone="+01:00" UnixTimestamp="1702049724"/>
    </Opus_Document>
</Opus>

Der Test schlägt an der fehlerhaften Jahresangabe in CompletedDate/@Year fehlt.

2) Export_Marc21ExportTest::testMarc21XmlExportWithUnpublishedDocAllowedForAdmin 3) Export_Marc21ExportTest::testMarc21XmlExportWithUnpublishedDocAllowedForNonAdmin 4) Export_Marc21ExportTest::testMarc21XmlExportWithPublishedDocAllowedForAdmin 5) Export_Marc21ExportTest::testMarc21XmlExportWithPublishedDocAllowedForGuest 6) Export_IndexControllerTest::testNonAdminAccessOnUnrestrictedMarc21ExportAllowed

Die Fehler 2-6 sind auf den Aufruf von <xsl:value-of select="php:functionString('Application_Xslt::dcType', @Type)"/> in /modules/oai/views/scripts/index/prefixes/marc21.xslt (Zeile 517) zurückzuführen. Ich verstehe nicht, warum der Fehler auftritt. Ähnliche Aufrufe finden sich auch in oai_dc.xslt sowie XMetaDissPlus.xslt. Dort funktionieren sie problemlos.