TheOdinProject / curriculum

The open curriculum for learning web development
https://www.theodinproject.com/
Other
9.99k stars 13.35k forks source link

JavaScript/More_Testing: Emphasize that setting up jest for webpack is not required #26548

Closed fcasibu closed 10 months ago

fcasibu commented 12 months ago

Describe your suggestion

I've seen a commonly asked question where in the lesson More Testing assignement 6 describes:

And finally, if you wish, you can add Jest to your webpack setup. Read about that process here.

Problem: Even though we already have completed projects where we only use jest/babel to test it still gives an impression that we still need to setup jest for webpack in the following project, but this is not the case because like the documentation describes, it's useful for handling static assets, css modules, etc in tests. These things are not necessary in fully testing the modules in the following project Battleship. It is also encouraged to create the project and create the modules without handling any kind of DOM manipulation as a good practice, so we won't really see any issues coming up even if they ignored this assignment.

Solution: Emphasize that setting up jest for webpack is not necessary. This I think could also be more of an additional resource than an assignment (IMO)

Path

Node / JS

Lesson Url

https://www.theodinproject.com/lessons/node-path-javascript-more-testing

Checks

(Optional) Discord Name

nevz

(Optional) Additional Comments

No response

KevinMulhern commented 12 months ago

Can someone from @TheOdinProject/javascript take a look at this please?

github-actions[bot] commented 11 months ago

This issue is stale because it has had no activity for the last 30 days.

wise-king-sullyman commented 10 months ago

I agree that can probably be moved to the additional resources section.

aconstas commented 10 months ago

I would love to work on this! Can I be assigned?

wise-king-sullyman commented 10 months ago

All yours @aconstas, thanks!