Skip to content

Modify test commands in README.md#313

Open
security-companion wants to merge 1 commit intoninoseki:masterfrom
security-companion:patch-1
Open

Modify test commands in README.md#313
security-companion wants to merge 1 commit intoninoseki:masterfrom
security-companion:patch-1

Conversation

@security-companion
Copy link
Contributor

Hi @ninoseki
thanks for adapting the readme, however these commands do not work for me in VS Code.
If I adapt them they run.
However when trying to run the pytest I get some errors.
Also if I create a virtualenv with uv venv and start it with .venv\Scripts\activate (in Windows) and run uv run pytest afterwards I get the same errors.
Any ideas?

Also with which command do you start the local python webserver?
Greetings
security-companion

(eml_analyzer) C:\Users\Joachim\Documents\Programmierprojekte\eml_analyzer>uv run pytest
Installed 132 packages in 4.49s
ImportError while loading conftest 'C:\Users\Joachim\Documents\Programmierprojekte\eml_analyzer\tests\conftest.py'.
tests\conftest.py:10: in
from backend import clients, factories, schemas
backend_init_.py:1: in
from .clients.emailrep import EmailRep # noqa: F401
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
backend\clients_init_.py:3: in
from .emailrep import EmailRep # noqa: F401
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
backend\clients\emailrep.py:4: in
from backend import schemas
backend\schemas_init_.py:3: in
from .payload import FilePayload, Payload # noqa: F401
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
backend\schemas\payload.py:3: in
from backend.validator import is_eml_or_msg_file
backend\validator.py:1: in
import magic
.venv\Lib\site-packages\magic_init_.py:209: in
libmagic = loader.load_lib()
^^^^^^^^^^^^^^^^^
.venv\Lib\site-packages\magic\loader.py:49: in load_lib
raise ImportError('failed to find libmagic. Check your installation')
E ImportError: failed to find libmagic. Check your installation

Updated test commands for backend and frontend sections.
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.

1 participant