Skip to content

Tests#6

Merged
AlexsanderHamir merged 6 commits intomainfrom
tests
Jul 8, 2025
Merged

Tests#6
AlexsanderHamir merged 6 commits intomainfrom
tests

Conversation

@AlexsanderHamir
Copy link
Copy Markdown
Owner

@AlexsanderHamir AlexsanderHamir commented Jul 8, 2025

What was done ?

  1. Organized the test package to make sure everything is clear.
  2. Added unit tests to ensure functionality.
  3. Improved integration tests to check for accounting inconsistencies.

What's missing ?

  1. Higher coverage of unit tests.
  2. Code re-design to help with unit testing.

@AlexsanderHamir
Tests ##1
6e3f159 
1. Changed start function to remove verbosity during tests.
2. Made a long test faster.
3. Added comments to a tricky function.
@AlexsanderHamir
Test ## 2
8fa68dc 
1. Some errors need multiple interations for a error to even happen.
2. Added priority for test errors, not every error is something to work on immediately
@AlexsanderHamir
Test ##3
48863ee 
1. Renamed files for better understanding.
2. Made issue #4 reproducible.
3. Marked some potential areas where issues could come from.
@AlexsanderHamir
Test ##4
c68927a 
1. I was incorrect about where issue #4 was coming from.
2. Check for more stats on current integration test.
@AlexsanderHamir
Test ##5
e869f8a 
1. Standardized some errors for testing purposes.
2. Implemented unit tests covering basic edge cases.
3. Removed extra function when starting the simulation.
4. Improved integration test.
5. Got a testing package for assertion.
@AlexsanderHamir
done with testing
c1eacf9
@AlexsanderHamir AlexsanderHamir merged commit 645e9f1 into main Jul 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@AlexsanderHamir