Merge pull request #18 from arXiv/ARXIVCE-3162-fix-xml-errors

Arxivce 3162 fix xml errors

Author: kyokukou

oaipmh/templates/record_formats.xml

@@ -131,12 +131,12 @@
         {% if record.current_meta.report_num %}
             <report-no>{{ record.current_meta.report_num }}</report-no>
         {% endif %}
-        {% if record.current_meta.msc_class %}
-            <msc-class>{{ record.current_meta.msc_class }}</msc-class>
-        {% endif %}
         {% if record.current_meta.acm_class %}
             <acm-class>{{ record.current_meta.msc_class }}</acm-class>
         {% endif %}
+        {% if record.current_meta.msc_class %}
+            <msc-class>{{ record.current_meta.msc_class }}</msc-class>
+        {% endif %}
         {% if record.current_meta.journal_ref %}
             <journal-ref>{{ record.current_meta.journal_ref }}</journal-ref>
         {% endif %}
@@ -179,11 +179,11 @@
         {% if record.current_meta.doi %}
             <doi>{{record.current_meta.doi}}</doi>
         {% endif %}
-        {% if record.current_meta.license %}
-            <license>{{record.current_meta.license}}</license>
-        {% endif %}
         {% if record.current_meta.abstract %}
             <abstract>{{record.current_meta.abstract}}</abstract>
         {% endif %}
+        {% if record.current_meta.license %}
+            <license>{{record.current_meta.license}}</license>
+        {% endif %}
     </arXivOld>
 {% endmacro %}

oaipmh/templates/token.xml

@@ -3,6 +3,6 @@
     {% if token.empty %}
         <resumptionToken/>
     {% else %}
-        <resumptionToken expires='{{token.expires.strftime("%Y-%m-%dT%H:%M:%SZ")}}'>{{token.token_str}}</resumptionToken>
+        <resumptionToken expirationDate='{{token.expires.strftime("%Y-%m-%dT%H:%M:%SZ")}}'>{{token.token_str}}</resumptionToken>
     {% endif %}
 {% endmacro %}

tests/test_list_data.py

@@ -654,7 +654,7 @@ def test_final_resToken(test_client):
         response = test_client.get("/oai", query_string=params)
         assert response.status_code == 200 
         text=response.get_data(as_text=True)
-        assert "<resumptionToken expires=" in text
+        assert "<resumptionToken expirationDate=" in text
         token=__get_res_token(text)
         assert len(token) >1