Skip to content

Create Unit Tests #13

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

Open
chrisdedman opened this issue Mar 9, 2025 · 0 comments · Fixed by #14
Open

Create Unit Tests #13

chrisdedman opened this issue Mar 9, 2025 · 0 comments · Fixed by #14
Labels
testing Need testing

Comments

@chrisdedman
Copy link
Member

Description

We need to add unit tests to ensure the software's reliability and correctness. This will help us catch regressions early and improve overall code quality.

Tasks

  • Identify key functions and methods that require testing.
  • Write unit tests for the identified functions using a suitable testing framework (Qt Test).
  • Ensure tests cover a variety of scenarios, including edge cases.
  • Document the testing process and update the Wiki with instructions on running tests locally.

Requirements

  • Tests should be written in a separate test file, e.g., test_[module_name].cpp.
  • Use mock objects or stubs where necessary to isolate the unit under test.
  • Target at least 80% code coverage.
  • Ensure that tests are independent and can be run in any order.
@chrisdedman chrisdedman added the testing Need testing label Mar 9, 2025
@chrisdedman chrisdedman moved this to Todo in CodeAstra MVP Mar 9, 2025
@chrisdedman chrisdedman self-assigned this Mar 9, 2025
@chrisdedman chrisdedman moved this from Todo to In Progress in CodeAstra MVP Mar 9, 2025
@chrisdedman chrisdedman linked a pull request Mar 27, 2025 that will close this issue
@chrisdedman chrisdedman removed their assignment Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Need testing
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

1 participant