Skip to content

Added a Testing Guide and updated the specification in some JS files #907

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
cjyuan wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Added a Testing Guide and updated the specification in some JS files #907

cjyuan wants to merge 8 commits into from

Conversation

@cjyuan
Copy link
Contributor

@cjyuan cjyuan commented Jan 23, 2026

Changelist

  • Added a Testing Functions Guide -- Sprint-3/testing-guide.md
  • Updated the specification and description in the JS files in Sprint-3/1-implement-and-rewrite-tests and in Sprint-3/2-practice-tdd
    • Made the problem description more specific
    • Removed some suggested test cases to encourage trainees to think about what values should be used to test their functions

@github-actions

This comment was marked as off-topic.

Sign in to view
@cjyuan cjyuan requested a review from illicitonion January 23, 2026 04:14
@cjyuan cjyuan added 📅 Sprint 3 Assigned during Sprint 3 of this module NotCoursework labels Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@illicitonion illicitonion Awaiting requested review from illicitonion

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

NotCoursework 📅 Sprint 3 Assigned during Sprint 3 of this module

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cjyuan