You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
To improve testability of the visualization tool and adhere to generally good practices, we should add support for contributors to add unit tests that test the behavior of the application. We should also enforce this standard via enhancing the CI check to ensure all tests pass a pre-requisite to merging changes to the applicaiton
Proposed Solution
Introduce @testing-library/react framework to test behavior of the application rather than low level implementation details
Add unit tests on the existing components piece by piece
NOTE: I will add these piece by piece in the form of stacked PRs to reduce cognitive load on the reviewers
cc: @luciankt , can you review these PRs in ascending order as specified (eg: The [1/N] PR is reviewed before the [2/N] PR). This is due to me setting up what's called a stacked PR workflow via GitHub CLI to make each unit test addition self contained and make review easier.
🎉 Complete Test Coverage Stack [1/35] through [35/35]
All 35 stacked PRs for comprehensive unit and integration test coverage across the entire Georgia Tech CS 1332 Visualization Tool codebase have been submitted and are all passing CI:
Activity
karkir0003 commentedon Oct 3, 2026
@karkir0003 will take care of this so the rest of the team can focus on adding functionality
karkir0003 commentedon Oct 3, 2026
cc: @luciankt , can you review these PRs in ascending order as specified (eg: The
[1/N]PR is reviewed before the[2/N]PR). This is due to me setting up what's called a stacked PR workflow via GitHub CLI to make each unit test addition self contained and make review easier.karkir0003 commentedon Oct 3, 2026
🎉 Complete Test Coverage Stack [1/35] through [35/35]
All 35 stacked PRs for comprehensive unit and integration test coverage across the entire Georgia Tech CS 1332 Visualization Tool codebase have been submitted and are all passing CI:
Summary of Coverage
karkir0003 commentedon Oct 14, 2026
thanks @luciankt for merging these in. lmk if you see any issues and i can fix