-
Notifications
You must be signed in to change notification settings - Fork 3
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
Tests rework #428
Tests rework #428
Conversation
Python 3.11 compatibility
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bitte dann noch die github action run-unit-tests umbenennen zu run tests oder so
# doctag1_update_read_response["parent_tag_id"] == doctag1_update["parent_tag_id"] | ||
# ) | ||
# assert doctag1_update_read_response["id"] == doctag1["id"] | ||
# assert doctag1_update_read_response["project_id"] == doctag1_update["project_id"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
die kommentare alle raus, wenn nicht gebraucht wird
backend/src/nano.3535870.save
Outdated
@@ -0,0 +1 @@ | |||
user.name=Alienmaster [email protected] sequence.editor=code --wait core.editor=code --wait |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
die datei muss weg
backend/src/test/conftest.py
Outdated
|
||
request.addfinalizer(lambda: crud_code.remove(db=db, id=code_id)) | ||
# request.addfinalizer(lambda: crud_code.remove(db=db, id=code_id)) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
die finalizer dann acuh alle komplett weg, nicht nur rauskommentieren
backend/src/test/conftest.py
Outdated
|
||
request.addfinalizer(lambda: crud_project.remove(db, id=project_id)) | ||
# request.addfinalizer(lambda: crud_project.remove(db, id=project_id)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hier auch
backend/src/test/conftest.py
Outdated
|
||
request.addfinalizer(lambda: crud_user.remove(db=db, id=user_id)) | ||
# request.addfinalizer(lambda: crud_user.remove(db=db, id=user_id)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hier auch
backend/src/test/conftest.py
Outdated
|
||
# def __del__(self): | ||
# for user in self.userList.values(): | ||
# client.delete(f"/user/{user['id']}", headers=user["AuthHeader"]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
auskommentieten code bitte löschen
backend/src/test/conftest.py
Outdated
# "Authorization": f"{login['token_type']} {login['access_token']}" | ||
# } | ||
# for project in self.projectList.values(): | ||
# client.delete(f"/project/{project['id']}", headers=superuser_authheader) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hier
backend/src/pytest.ini
Outdated
@@ -0,0 +1,3 @@ | |||
[pytest] | |||
; testpaths = | |||
; test/app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
braucht man die datei? sieht iwie unnötig aus^^
No description provided.