Repository navigation
Description
Student: Aniruddha Bongale
Task: R50-T048 · 8 of 10
Repository: https://github.com/S-AI-Projects/TheAlgorithms-JavaScript
Assigned working branch: student/e26b07f0827
Goal
Work on Graphs/FloydWarshall.js. Exercise a documented failure or absent-result branch of FloydWarshall. Where the contract is total, use its valid empty/missing-result case instead of inventing an exception. Fixture: Use the smallest valid input and one ordinary input for FloydWarshall from Graphs/FloydWarshall.js. Write the literal inputs and independently calculated expected outputs in the tests before editing the implementation. The check must name the source line establishing each precondition.
Steps
- Read Graphs/FloydWarshall.js, especially
FloydWarshallnear line 9. Record its public inputs, result, mutation behavior and any external dependencies before changing it. - Locate the actual return, raise, rejection or fallback branch in the source and construct the smallest input reaching it.
- Assert the intended exception/sentinel and verify that caller-owned state remains valid afterwards; include a successful call after the failure.
- Implement work/R50-T048/check.py as a focused runner that executes the new checks and propagates failures. Native check to incorporate: run the repository test script in non-watch mode after installing from its lockfile; for browser-only code use a browser test instead of plain Node. Record the actual command and nonzero test count.
- Create the checks specified below and run
python3 work/R50-T048/check.py. Save inputs, expected versus actual results and tool versions in work/R50-T048/; a check must exit non-zero when an assertion fails. - Commit the repository change on branch R50-T048; prepare a PR to the organisation fork explaining the result and linking the checks. Keep each task independently reviewable. Remove work/R50-T048/ from a later upstream branch.
Done when
☐ The checked behavior is tied to FloydWarshall in Graphs/FloydWarshall.js; every assertion has an explicit expected value.
☐ Use the smallest valid input and one ordinary input for FloydWarshall from Graphs/FloydWarshall.js. Write the literal inputs and independently calculated expected outputs in the tests before editing the implementation. The check must name the source line establishing each precondition.
☐ Existing supported behavior and the public interface are preserved unless a failing regression demonstrates and documents a defect.
☐ The tested branch is named by source line and the input actually reaches it.
☐ An ordinary valid call still succeeds after the failure case.
Additional task guidance
Additional curated task guidance
Item:
Graphs/FloydWarshall.js
Task type:
Failure-path coverage
Code change (goes into the PR):
Graphs/FloydWarshall.js: add focused tests and a short explanation of the chosen failure/empty-result path. Repair error handling only if the regression proves it violates the contract.
Automated check:
python3 work/R50-T048/check.py
Prerequisites:
Node matching the project engines/CI, package manager selected by its lockfile, and Python 3.
Upstream PR rules:
Preserve the repository JavaScript style, use its test framework, and include JSDoc for changed public behavior. Submit coursework to the organisation fork first; keep work/ evidence out of any later upstream PR. Upstream acceptance is not a grading requirement. Policy: https://github.com/TheAlgorithms/JavaScript/blob/5c39e87a9a31f279c60f830ad74a845e4788a517/CONTRIBUTING.md
Source (pinned):
https://github.com/TheAlgorithms/JavaScript/blob/5c39e87a9a31f279c60f830ad74a845e4788a517/Graphs/FloydWarshall.js#L9
Change:
Replaces the shared task template with an item-specific contribution brief.
PR destination:
Upstream candidate after review
Submission checklist
- Complete this task and its checks.
- Add the required evidence and link your pull request here.
- Select this S-AI-Projects fork as the PR base repository.
- Request instructor review.
Keep this PR focused on this task. Do not submit assignment PRs to the upstream repository.
Activity