search

OPUS4 / opus4-import

OPUS 4 Import Library
Other
0 stars 1 forks source link

Migrate Import classes from Application #8

Closed j3nsch closed 2 years ago

j3nsch commented 2 years ago

…lication

j3nsch commented 2 years ago

Die Konfiguration fehlt noch. Insbesondere die sword. Einträge in application/configs/application.ini und evtl. in application/configs/config.ini. Beide werden auch bei den Tests geladen. Für das opus4-import Package bitte eine neue tests.ini anlegen, die die statische Konfiguration enthält. Die Datei config.ini.template sollte nur noch die Einträge enthalten, die lokal angepasst werden müssen, um die Tests laufen zu lassen. Das sind die Verbindungsdaten für die Datenbank.

j3nsch commented 2 years ago

Damit die Tests durchlaufen waren noch Setup und Cleanup notwendig. Die Tests erfordern die Existenz von bestimmten EnrichmentKeys. In der Application gibt es umfangreiche Testdaten. Außerdem müssen nicht die nur die Datenbank, sondern auch die Dateien im Workspace aufgeräumt werden, da es sonst zu Konflikten kommt.

Der Coding Style musste noch angepasst werden, vor allem die Verwendung von is_null und die Kommentare für Funktionen.