8000 test(solvers): add comprehensive test suite for ORToolsSolver by dominicusin · Pull Request #67 · dominicusin/package-maximizer · GitHub
[go: up one dir, main page]
Skip to content

test(solvers): add comprehensive test suite for ORToolsSolver - #67

Merged
dominicusin merged 1 commit into
mainfrom
test/ortools-solver-coverage
Sep 1, 2026
Merged

test(solvers): add comprehensive test suite for ORToolsSolver#67
dominicusin merged 1 commit into
mainfrom
test/ortools-solver-coverage

Conversation

@dominicusin
@dominicusin dominicusin commented Sep 1, 2026
Copy link
Copy Markdown
Owner

Summary

  • Added 21 tests for OR-Tools CP-SAT solver
  • Coverage: ortools_solver 30% → 84%
  • Tests: 601 → 622 passed

Test plan

  • 622 passed, 2 skipped
  • Coverage 88%
  • CI green (pending)

@github-actions
test(solvers): add comprehensive test suite for ORToolsSolver
61ca03e 
- Added 21 tests for OR-Tools CP-SAT solver
- Coverage: ortools_solver 30% → 84%
- Tests: 601 → 622 passed
@qodo-code-review
qodo-code-review Bot commented Sep 1, 2026
Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@coderabbitai
coderabbitai Bot commented Sep 1, 2026
Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 50 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: df6db395-ac8a-4cda-ab25-2833d789249b

📥 Commits

Reviewing files that changed from the base of the PR and between a9f5ad9 and 61ca03e.

📒 Files selected for processing (1)
  • tests/test_ortools_solver.py

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@dominicusin
dominicusin merged commit 1b116fa into main Sep 1, 2026
28 of 30 checks passed
@dominicusin
dominicusin deleted the test/ortools-solver-coverage branch September 1, 2026 06:24
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

@dominicusin
0