Skip to content

test: cross-check rodCut contract for Issue #23 - #2025

Open
9808Tarun wants to merge 1 commit into
TheAlgorithms:masterfrom
9808Tarun:R50-T023
Open

9808Tarun wants to merge 1 commit into
TheAlgorithms:masterfrom
9808Tarun:R50-T023

Conversation

@9808Tarun

Copy link
Copy Markdown

Description
Cross-checked rodCut in Dynamic-Programming/RodCutting.js against a 20-input explicit reference domain table and created the required check runner script.

Related Issue
Fixes: #23

Type of Change

  • Bug Fix
  • New Feature
  • UI/UX Improvement
  • Performance Improvement
  • Tests
  • Documentation
  • Refactoring / Maintenance
  • CI/CD

Changes Made

  • Tested smallest valid input ([1], n=1) and ordinary input ([1, 5, 8, 9, 10, 17, 17, 20], n=8), referencing precondition source line 6 of Dynamic-Programming/RodCutting.js.
  • Added 20-domain reference table cross-checks and array non-mutation assertions in Dynamic-Programming/test/RodCutting.test.js.
  • Created work/R50-T023/check.py to execute the test suite in non-watch mode and capture test output, fixture results, and tool versions.

Screenshots / Recordings
N/A

Testing

  • Executed python3 work/R50-T023/check.py locally.
  • Verified test suite passes for Dynamic-Programming/test/RodCutting.test.js.

Checklist

  • I have tested my changes locally.
  • I have added or updated tests, if needed.
  • I have updated documentation, if needed.
  • I have followed the project's coding guidelines.
  • I have run the relevant tests and checks.
  • I have addressed previous review feedback, if applicable.

Additional Notes
N/A

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