compliance-tool: Fix compliance tool imports and unitests (#356)

Currently, there are several wrong imports and outdated unittests in the `compliance-tool` package. This fixes these issues. Fixes #354
eclipse-basyx · Jan 15, 2025 · 1ca495f · 1ca495f
1 parent 9cb5b89
commit 1ca495f
Show file tree

Hide file tree

Showing 20 changed files with 83 additions and 75 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -191,7 +191,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.8", "3.10", "3.12"]
+        python-version: ["3.9", "3.12"]
     defaults:
       run:
         working-directory: ./compliance_tool
@@ -224,10 +224,10 @@ jobs:
         working-directory: ./compliance_tool
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Python ${{ env.X_PYTHON_VERSION }}
+    - name: Set up Python ${{ env.X_PYTHON_MIN_VERSION }}
       uses: actions/setup-python@v2
       with:
-        python-version: ${{ env.X_PYTHON_VERSION }}
+        python-version: ${{ env.X_PYTHON_MIN_VERSION }}
     - name: Install Python dependencies
       run: |
         python -m pip install --upgrade pip
@@ -249,10 +249,10 @@ jobs:
         working-directory: ./compliance_tool
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Python ${{ env.X_PYTHON_VERSION }}
+    - name: Set up Python ${{ env.X_PYTHON_MIN_VERSION }}
       uses: actions/setup-python@v2
       with:
-        python-version: ${{ env.X_PYTHON_VERSION }}
+        python-version: ${{ env.X_PYTHON_MIN_VERSION }}
     - name: Install Python dependencies
       run: |
         python -m pip install --upgrade pip
@@ -277,10 +277,10 @@ jobs:
         working-directory: ./compliance_tool
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Python ${{ env.X_PYTHON_VERSION }}
+    - name: Set up Python ${{ env.X_PYTHON_MIN_VERSION }}
       uses: actions/setup-python@v2
       with:
-        python-version: ${{ env.X_PYTHON_VERSION }}
+        python-version: ${{ env.X_PYTHON_MIN_VERSION }}
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip

diff --git a/compliance_tool/__init__.py → 1.0.0 b/compliance_tool/__init__.py → 1.0.0
diff --git a/compliance_tool/aas_compliance_tool/cli.py b/compliance_tool/aas_compliance_tool/cli.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2020 the Eclipse BaSyx Authors
+# Copyright (c) 2024 the Eclipse BaSyx Authors
 #
 # This program and the accompanying materials are made available under the terms of the MIT License, available in
 # the LICENSE file of this project.
@@ -19,11 +19,12 @@
 
 from basyx.aas.adapter import aasx
 from basyx.aas.adapter.xml import write_aas_xml_file
-from basyx.aas.compliance_tool import compliance_check_xml as compliance_tool_xml, \
-    compliance_check_json as compliance_tool_json, compliance_check_aasx as compliance_tool_aasx
+from aas_compliance_tool import compliance_check_xml as compliance_tool_xml, \
+    compliance_check_json as compliance_tool_json, \
+    compliance_check_aasx as compliance_tool_aasx
 from basyx.aas.adapter.json import write_aas_json_file
 from basyx.aas.examples.data import create_example, create_example_aas_binding, TEST_PDF_FILE
-from basyx.aas.compliance_tool.state_manager import ComplianceToolStateManager, Status
+from aas_compliance_tool.state_manager import ComplianceToolStateManager, Status
 
 
 def parse_cli_arguments() -> argparse.ArgumentParser:

diff --git a/compliance_tool/aas_compliance_tool/compliance_check_aasx.py b/compliance_tool/aas_compliance_tool/compliance_check_aasx.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2020 the Eclipse BaSyx Authors
+# Copyright (c) 2024 the Eclipse BaSyx Authors
 #
 # This program and the accompanying materials are made available under the terms of the MIT License, available in
 # the LICENSE file of this project.
@@ -19,14 +19,14 @@
 
 import pyecma376_2
 
-from . import compliance_check_json, compliance_check_xml
+from aas_compliance_tool import compliance_check_json, compliance_check_xml
 from basyx.aas import model
 from basyx.aas.adapter import aasx
 from basyx.aas.adapter.xml import xml_deserialization
 from basyx.aas.adapter.json import json_deserialization
 from basyx.aas.examples.data import example_aas, create_example_aas_binding
 from basyx.aas.examples.data._helper import AASDataChecker, DataChecker
-from .state_manager import ComplianceToolStateManager, Status
+from aas_compliance_tool.state_manager import ComplianceToolStateManager, Status
 
 
 def check_deserialization(file_path: str, state_manager: ComplianceToolStateManager,

diff --git a/compliance_tool/aas_compliance_tool/compliance_check_json.py b/compliance_tool/aas_compliance_tool/compliance_check_json.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2020 the Eclipse BaSyx Authors
+# Copyright (c) 2024 the Eclipse BaSyx Authors
 #
 # This program and the accompanying materials are made available under the terms of the MIT License, available in
 # the LICENSE file of this project.
@@ -20,7 +20,7 @@
 from basyx.aas.adapter.json import json_deserialization
 from basyx.aas.examples.data import example_aas, create_example
 from basyx.aas.examples.data._helper import AASDataChecker
-from .state_manager import ComplianceToolStateManager, Status
+from aas_compliance_tool.state_manager import ComplianceToolStateManager, Status
 
 
 JSON_SCHEMA_FILE = os.path.join(os.path.dirname(__file__), 'schemas/aasJSONSchema.json')

diff --git a/compliance_tool/aas_compliance_tool/compliance_check_xml.py b/compliance_tool/aas_compliance_tool/compliance_check_xml.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2020 the Eclipse BaSyx Authors
+# Copyright (c) 2024 the Eclipse BaSyx Authors
 #
 # This program and the accompanying materials are made available under the terms of the MIT License, available in
 # the LICENSE file of this project.
@@ -20,7 +20,7 @@
 from basyx.aas.adapter.xml import xml_deserialization
 from basyx.aas.examples.data import example_aas, create_example
 from basyx.aas.examples.data._helper import AASDataChecker
-from .state_manager import ComplianceToolStateManager, Status
+from aas_compliance_tool.state_manager import ComplianceToolStateManager, Status
 
 
 XML_SCHEMA_FILE = os.path.join(os.path.dirname(__file__), 'schemas/aasXMLSchema.xsd')

diff --git a/compliance_tool/aas_compliance_tool/state_manager.py b/compliance_tool/aas_compliance_tool/state_manager.py
@@ -1,4 +1,4 @@
-# Copyright (c) 2020 the Eclipse BaSyx Authors
+# Copyright (c) 2024 the Eclipse BaSyx Authors
 #
 # This program and the accompanying materials are made available under the terms of the MIT License, available in
 # the LICENSE file of this project.

diff --git a/compliance_tool/setup.py b/compliance_tool/setup.py
@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-# Copyright (c) 2019-2021 the Eclipse BaSyx Authors
+# Copyright (c) 2024-2021 the Eclipse BaSyx Authors
 #
 # This program and the accompanying materials are made available under the terms of the MIT License, available in
 # the LICENSE file of this project.

diff --git a/compliance_tool/test/files/test_demo_full_example.json b/compliance_tool/test/files/test_demo_full_example.json
@@ -499,7 +499,7 @@
                 },
                 {
                     "language": "de",
-                    "text": "Ein Beispiel-BillofMaterial-Submodel f\u00fcr eine Test-Anwendung"
+                    "text": "Ein Beispiel-BillOfMaterial-Submodel f\u00fcr eine Test-Anwendung"
                 }
             ],
             "modelType": "Submodel",
@@ -1309,7 +1309,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispielswert f\u00fcr ein MulitLanguageProperty-Element"
+                                    "text": "Beispielwert f\u00fcr ein MultiLanguageProperty-Element"
                                 }
                             ],
                             "valueId": {
@@ -2281,7 +2281,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispiel MulitLanguageProperty Element"
+                                    "text": "Beispiel MultiLanguageProperty Element"
                                 }
                             ],
                             "modelType": "MultiLanguageProperty",
@@ -2301,7 +2301,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispielswert f\u00fcr ein MulitLanguageProperty-Element"
+                                    "text": "Beispielwert f\u00fcr ein MultiLanguageProperty-Element"
                                 }
                             ]
                         },
@@ -2825,7 +2825,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispiel MulitLanguageProperty Element"
+                                    "text": "Beispiel MultiLanguageProperty Element"
                                 }
                             ],
                             "modelType": "MultiLanguageProperty",

diff --git a/compliance_tool/test/files/test_demo_full_example.xml b/compliance_tool/test/files/test_demo_full_example.xml
@@ -486,7 +486,7 @@
         </aas:langStringTextType>
         <aas:langStringTextType>
           <aas:language>de</aas:language>
-          <aas:text>Ein Beispiel-BillofMaterial-Submodel für eine Test-Anwendung</aas:text>
+          <aas:text>Ein Beispiel-BillOfMaterial-Submodel für eine Test-Anwendung</aas:text>
         </aas:langStringTextType>
       </aas:description>
       <aas:administration>
@@ -1424,7 +1424,7 @@
                 </aas:langStringTextType>
                 <aas:langStringTextType>
                   <aas:language>de</aas:language>
-                  <aas:text>Beispielswert für ein MulitLanguageProperty-Element</aas:text>
+                  <aas:text>Beispielwert für ein MultiLanguageProperty-Element</aas:text>
                 </aas:langStringTextType>
               </aas:value>
               <aas:valueId>
@@ -2109,7 +2109,7 @@
                 </aas:langStringTextType>
                 <aas:langStringTextType>
                   <aas:language>de</aas:language>
-                  <aas:text>Beispiel MulitLanguageProperty Element</aas:text>
+                  <aas:text>Beispiel MultiLanguageProperty Element</aas:text>
                 </aas:langStringTextType>
               </aas:description>
               <aas:semanticId>
@@ -2128,7 +2128,7 @@
                 </aas:langStringTextType>
                 <aas:langStringTextType>
                   <aas:language>de</aas:language>
-                  <aas:text>Beispielswert für ein MulitLanguageProperty-Element</aas:text>
+                  <aas:text>Beispielwert für ein MultiLanguageProperty-Element</aas:text>
                 </aas:langStringTextType>
               </aas:value>
             </aas:multiLanguageProperty>
@@ -2632,7 +2632,7 @@
                     </aas:langStringTextType>
                     <aas:langStringTextType>
                       <aas:language>de</aas:language>
-                      <aas:text>Beispiel MulitLanguageProperty Element</aas:text>
+                      <aas:text>Beispiel MultiLanguageProperty Element</aas:text>
                     </aas:langStringTextType>
                   </aas:description>
                   <aas:semanticId>

diff --git a/compliance_tool/test/files/test_demo_full_example_json_aasx/aasx/data.json b/compliance_tool/test/files/test_demo_full_example_json_aasx/aasx/data.json
@@ -507,7 +507,7 @@
                 },
                 {
                     "language": "de",
-                    "text": "Ein Beispiel-BillofMaterial-Submodel f\u00fcr eine Test-Anwendung"
+                    "text": "Ein Beispiel-BillOfMaterial-Submodel f\u00fcr eine Test-Anwendung"
                 }
             ],
             "modelType": "Submodel",
@@ -1317,7 +1317,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispielswert f\u00fcr ein MulitLanguageProperty-Element"
+                                    "text": "Beispielwert f\u00fcr ein MultiLanguageProperty-Element"
                                 }
                             ],
                             "valueId": {
@@ -2289,7 +2289,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispiel MulitLanguageProperty Element"
+                                    "text": "Beispiel MultiLanguageProperty Element"
                                 }
                             ],
                             "modelType": "MultiLanguageProperty",
@@ -2309,7 +2309,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispielswert f\u00fcr ein MulitLanguageProperty-Element"
+                                    "text": "Beispielwert f\u00fcr ein MultiLanguageProperty-Element"
                                 }
                             ]
                         },
@@ -2833,7 +2833,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispiel MulitLanguageProperty Element"
+                                    "text": "Beispiel MultiLanguageProperty Element"
                                 }
                             ],
                             "modelType": "MultiLanguageProperty",

diff --git a/compliance_tool/test/files/test_demo_full_example_wrong_attribute.json b/compliance_tool/test/files/test_demo_full_example_wrong_attribute.json
@@ -499,7 +499,7 @@
                 },
                 {
                     "language": "de",
-                    "text": "Ein Beispiel-BillofMaterial-Submodel f\u00fcr eine Test-Anwendung"
+                    "text": "Ein Beispiel-BillOfMaterial-Submodel f\u00fcr eine Test-Anwendung"
                 }
             ],
             "modelType": "Submodel",
@@ -1309,7 +1309,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispielswert f\u00fcr ein MulitLanguageProperty-Element"
+                                    "text": "Beispielwert f\u00fcr ein MultiLanguageProperty-Element"
                                 }
                             ],
                             "valueId": {
@@ -2281,7 +2281,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispiel MulitLanguageProperty Element"
+                                    "text": "Beispiel MultiLanguageProperty Element"
                                 }
                             ],
                             "modelType": "MultiLanguageProperty",
@@ -2301,7 +2301,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispielswert f\u00fcr ein MulitLanguageProperty-Element"
+                                    "text": "Beispielwert f\u00fcr ein MultiLanguageProperty-Element"
                                 }
                             ]
                         },
@@ -2825,7 +2825,7 @@
                                 },
                                 {
                                     "language": "de",
-                                    "text": "Beispiel MulitLanguageProperty Element"
+                                    "text": "Beispiel MultiLanguageProperty Element"
                                 }
                             ],
                             "modelType": "MultiLanguageProperty",

diff --git a/compliance_tool/test/files/test_demo_full_example_wrong_attribute.xml b/compliance_tool/test/files/test_demo_full_example_wrong_attribute.xml
@@ -486,7 +486,7 @@
         </aas:langStringTextType>
         <aas:langStringTextType>
           <aas:language>de</aas:language>
-          <aas:text>Ein Beispiel-BillofMaterial-Submodel für eine Test-Anwendung</aas:text>
+          <aas:text>Ein Beispiel-BillOfMaterial-Submodel für eine Test-Anwendung</aas:text>
         </aas:langStringTextType>
       </aas:description>
       <aas:administration>
@@ -1424,7 +1424,7 @@
                 </aas:langStringTextType>
                 <aas:langStringTextType>
                   <aas:language>de</aas:language>
-                  <aas:text>Beispielswert für ein MulitLanguageProperty-Element</aas:text>
+                  <aas:text>Beispielwert für ein MultiLanguageProperty-Element</aas:text>
                 </aas:langStringTextType>
               </aas:value>
               <aas:valueId>
@@ -2109,7 +2109,7 @@
                 </aas:langStringTextType>
                 <aas:langStringTextType>
                   <aas:language>de</aas:language>
-                  <aas:text>Beispiel MulitLanguageProperty Element</aas:text>
+                  <aas:text>Beispiel MultiLanguageProperty Element</aas:text>
                 </aas:langStringTextType>
               </aas:description>
               <aas:semanticId>
@@ -2128,7 +2128,7 @@
                 </aas:langStringTextType>
                 <aas:langStringTextType>
                   <aas:language>de</aas:language>
-                  <aas:text>Beispielswert für ein MulitLanguageProperty-Element</aas:text>
+                  <aas:text>Beispielwert für ein MultiLanguageProperty-Element</aas:text>
                 </aas:langStringTextType>
               </aas:value>
             </aas:multiLanguageProperty>
@@ -2632,7 +2632,7 @@
                     </aas:langStringTextType>
                     <aas:langStringTextType>
                       <aas:language>de</aas:language>
-                      <aas:text>Beispiel MulitLanguageProperty Element</aas:text>
+                      <aas:text>Beispiel MultiLanguageProperty Element</aas:text>
                     </aas:langStringTextType>
                   </aas:description>
                   <aas:semanticId>