Skip to content

Comments

Backend tests#322

Merged
ngc92 merged 6 commits intomainfrom
ngc92/backend-tests
Aug 25, 2025
Merged

Backend tests#322
ngc92 merged 6 commits intomainfrom
ngc92/backend-tests

Conversation

@ngc92
Copy link
Collaborator

@ngc92 ngc92 commented Aug 2, 2025

Built atop the database tests, and using the same principles. The tests aren't very fancy, they'll mostly detect if we cause unintended changes to our output I think.

@ngc92 ngc92 changed the base branch from main to ngc92/db-tests August 2, 2025 16:17
@github-actions
Copy link

github-actions bot commented Aug 2, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/libkernelbot
  backend.py
  leaderboard_db.py
  report.py 46
Project Total  

This report was generated by python-coverage-comment-action

@ngc92 ngc92 force-pushed the ngc92/db-tests branch 4 times, most recently from c48ba15 to 1dadc8e Compare August 9, 2025 09:28
Base automatically changed from ngc92/db-tests to main August 9, 2025 09:34
@ngc92 ngc92 force-pushed the ngc92/backend-tests branch 3 times, most recently from 91fd7d6 to a9c0f31 Compare August 21, 2025 09:23
@ngc92 ngc92 requested a review from msaroufim August 21, 2025 09:24
@ngc92 ngc92 force-pushed the ngc92/backend-tests branch from a37a00f to ad09cba Compare August 22, 2025 07:25
@ngc92 ngc92 force-pushed the ngc92/backend-tests branch from ad09cba to 75688e9 Compare August 22, 2025 07:30
@ngc92 ngc92 force-pushed the ngc92/backend-tests branch from 75688e9 to ccdf4f1 Compare August 22, 2025 09:19
@ngc92 ngc92 requested a review from S1ro1 August 25, 2025 23:03
Copy link
Member

@S1ro1 S1ro1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool!

@ngc92 ngc92 merged commit 8af2e18 into main Aug 25, 2025
4 checks passed
@ngc92 ngc92 deleted the ngc92/backend-tests branch August 25, 2025 23:33
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.

2 participants