Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create SOC Modul for VWID #595

Merged
merged 16 commits into from
Nov 4, 2022
Merged

Create SOC Modul for VWID #595

merged 16 commits into from
Nov 4, 2022

Conversation

rleidner
Copy link
Collaborator

Ein erster Versuch, zur owb2.0 etwas beizutragen.
Bitte prüfen und melden wenn es nicht passt.

@rleidner
Copy link
Collaborator Author

Es sollten jetzt keine flake8-Fehler mehr auftreten.

@LKuemmel
Copy link
Contributor

Beim Standard-Image fehlen noch Python-Packages. Deshalb kann das SoC-Modul bei der Erstellung der Liste der verfügbaren SoC-Module nicht importiert werden, das verursacht den Fehler im Workflow.

@rleidner
Copy link
Collaborator Author

OK.
Die zusätzlichen python packages (aiohttp, lxml) sind im geänderten requirements.txt enthalten und werden beim Reboot automatisch installiert.

@LKuemmel
Copy link
Contributor

Wenn der Test auf GitHub ausgeführt wird, fehlen die packages. Diese müssen gemockt werden:

sys.modules['jq'] = type(sys)('jq')

Wenn Du diese Zeile analog für aiohttp und lxml ergänzt, sollte der Test durchlaufen.

@rleidner
Copy link
Collaborator Author

Hallo Lena,
ich habe aiohttp und lxml in die conftest.py eingetragen.
Leider bin ich mit dem test workflo win github noch nicht sehr vertraut.
Es bleibt dieser Fehler - ich wüsste aber nicht was ich da machen könnte:

packages/modules/common/simcount/_simcounter_store.py:75
/home/runner/work/core/core/packages/modules/common/simcount/_simcounter_store.py:75: DeprecationWarning: private variables, such as '_SimCountPrefix__read_file', will be normal attributes in 3.10
def __read_file(file: str) -> float:

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=========================== short test summary info ============================
FAILED packages/modules/configuration_test.py::test_pub_configurable - AssertionError: assert 0 == 1

  • where 1 = .call_count
    =================== 1 failed, 246 passed, 1 warning in 2.01s ===================
    Error: Process completed with exit code 1.

@vuffiraa72
Copy link
Contributor

Es fehlt noch ein Mock für lxml.html
Damit sollten die Tests laufen.

sys.modules['lxml.html'] = type(sys)('lxml.html')

@rleidner
Copy link
Collaborator Author

rleidner commented Nov 1, 2022

@vuffiraa72,
"Es fehlt noch ein Mock für lxml.html
Damit sollten die Tests laufen."
Danke - das war es!
Die Fehlermeldungen sind nicht wirklich hilfreich...

@LKuemmel LKuemmel merged commit 764b631 into openWB:master Nov 4, 2022
@rleidner rleidner deleted the soc_vwid branch April 7, 2023 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants