Skip to content

Conversation

@Flavore669
Copy link
Contributor

Helloooo, This is the foundation for uploader testing, it's only a matter of time before I finish. In the meantime, I feel it would be beneficial if you could look over my changes in advance because why not.

Changes

  • Added dependency injection pattern with connectDBFunc for testability in uploader.go
  • Added test data files for courses, professors, and sections in a folder under uploader directory

Testing Approach

  • Uses minimal test data to verify control flow
  • Mocks database dependencies to enable unit testing

I will hopefully finish testing the Uploader function, and start testing buildStaticAggregation function soon

Please let me know if you have any suggestions or concern for testing uploader.go

@Flavore669 Flavore669 marked this pull request as draft October 27, 2025 04:51
@Flavore669 Flavore669 changed the title Issue #40 - Add unit tests for api-tools/utils/methods [In-Progress] Issue #40 - Add unit tests for api-tools/uploader [In-Progress] Nov 2, 2025
@Flavore669 Flavore669 marked this pull request as ready for review November 6, 2025 07:19
@Flavore669 Flavore669 changed the title Issue #40 - Add unit tests for api-tools/uploader [In-Progress] Issue #40 - Add unit tests for api-tools/uploader [Finished] Nov 7, 2025
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.

1 participant