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

Introducing Unit Test Framework #432

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

IanTrudel
Copy link

  • Provides a TestCase, TestSuite, TestReporter and a Mock class.
  • Testing is bootstrapped using the new unittest package to test the said package.

- Provides a TestCase, TestSuite, TestReporter and a Mock class.
- Testing is bootstrapped using the new unittest package to test the said package.
@IanTrudel IanTrudel mentioned this pull request Apr 20, 2024
18 tasks
uni/lib/unittest.icn Outdated Show resolved Hide resolved
uni/lib/unittest.icn Outdated Show resolved Hide resolved
- Simplification suggested by @Jafaral.
- Reordered assertions in Mock.testExpect().
@Jafaral Jafaral changed the title Initial commit for Unicon Unit Test Framework Introduce Unit Test Framework Apr 25, 2024
- `assertEqual()` now expects `expected` and `actual` to be of the same type based on the feedback from @Jafaral.
- Implemented `assertFail` and `assertNotEqual()`, and corresponding tests.
- Reworked the results tally in TestReporter and returned in the `summary()` method.
- Added tests for the TestReporter.
- Completing TestTestSuite run tests and assertion failure tests.
- Adding a few more assertions.
- Tidying up the output.
- Inspired from `tests/unicon/tester.icn`
- Example output: ran 20 tests in 2.004114s
@IanTrudel IanTrudel changed the title Introduce Unit Test Framework Introducing Unit Test Framework Apr 29, 2024
@IanTrudel IanTrudel marked this pull request as ready for review May 4, 2024 22:14
A Unicon Techincal Report for the Unicon Unit Testing Framework including:
- Introduction
- Components of the framework
- Unit test examples
- References
@Jafaral
Copy link
Member

Jafaral commented May 25, 2024

It is better to rebase your branch on top of the master branch instead of merging.

@IanTrudel
Copy link
Author

It is better to rebase your branch on top of the master branch instead of merging.

Will do next time. Thank you!

@IanTrudel
Copy link
Author

@Jafaral sorry about the merge. I did set git to rebase but something clearly went wrong.

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.

3 participants