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

Add mocking-based unit test for the status command #67

Merged
merged 34 commits into from
Dec 23, 2024

Conversation

SimonL22
Copy link
Contributor

@SimonL22 SimonL22 commented Sep 9, 2024

Also update pyproject.toml s.t. it contains settings for the ruff formatter and linter that are consistent with the GitHub action.

Copy link
Member

@joka921 joka921 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also make sure to run an autoformatter (autopep8) or fix the format issues yourself.

test/qlever/commands/test_status_execute.py Outdated Show resolved Hide resolved
test/qlever/commands/test_status_execute.py Outdated Show resolved Hide resolved
test/qlever/commands/test_status_execute.py Outdated Show resolved Hide resolved
test/qlever/commands/test_status_execute.py Outdated Show resolved Hide resolved
test/qlever/commands/test_status_execute.py Outdated Show resolved Hide resolved
test/qlever/commands/test_status_other_methods.py Outdated Show resolved Hide resolved
test/qlever/commands/test_status_other_methods.py Outdated Show resolved Hide resolved
@joka921 joka921 changed the title Test status with mocking Add mocking-based unit test for the status command Dec 23, 2024
@joka921 joka921 merged commit 6272c06 into ad-freiburg:main Dec 23, 2024
9 checks passed
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.

2 participants