-
Notifications
You must be signed in to change notification settings - Fork 15.8k
Open
0 / 30 of 3 issues completedOpen
0 / 30 of 3 issues completed
Copy link
Labels
Type: Maintainer OnlyThis issue is available only to maintainersThis issue is available only to maintainers
Description
Checks
- This is not a duplicate of an existing issue (please have a look through our open issues list to make sure)
- I have thoroughly read and understand The Odin Project Contributing Guide
- Would you like to work on this issue?
Describe your suggestion
Proposed changes
In both the Ruby and JavaScript courses, the testing section should be moved ahead of the computer science section. Then, for the projects in the computer science section, instructions should be updated to tell learners to write tests for their code. This will include:
- The recursion projects (merge sort and Fibonacci)
- The linked list project
- The binary search tree project
- The knight travails project
It may also end up including the hash map project depending on how #30247 is resolved.
Why?
This is an idea that's meant to solve a couple of different problems:
- Learners could use more practice writing their own tests, particularly in the JavaScript course, where Battleship is on the only project learners get to work on writing their own tests in a vanilla JS context.
- Because none of the computer science projects feature tests, verifying solutions is cumbersome. It often involves driver scripts and manually checking stdout, which is a decidedly inferior experience compared to automated testing.
Issue Tracking
This issue is going to be an epic I use to track the different pieces of this. The first two subissues can be done in any order, but the final item needs to come after the first two have been completed.
Path
Ruby / Rails, Node / JS
Lesson Url
Computer Science and Testing sections
(Optional) Discord Name
No response
(Optional) Additional Comments
No response
mao-sz
Sub-issues
Metadata
Metadata
Assignees
Labels
Type: Maintainer OnlyThis issue is available only to maintainersThis issue is available only to maintainers