Hey first of all thanks for the series and effort you put writing this out for us.
Noticed a couple things by just skimming and trying almost directly the code (not a good practice though) but hope this can be improved in this section of your articles.
1- It is mentioned that at some point jest configuration will be validated with the test itself.. I've noticed it's never configured... if someone is just copying/pasting (like me)... the sample wont work as the test script is not using Jest... npm's package.json by default states
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1"
I think as a minimum it should be edited to something like
"scripts": {
"test": "jest"
any other argument needed should be there.
2- I suggest to put commands like mkdir test && cd test and so on, as sometimes when this kind of details are just mixed with all the other text, it can be missed like I just did.
3- There are one or two typos, I recommend to check this out before posting the article to keep a higher quality.
Hey first of all thanks for the series and effort you put writing this out for us.
Noticed a couple things by just skimming and trying almost directly the code (not a good practice though) but hope this can be improved in this section of your articles.
1- It is mentioned that at some point jest configuration will be validated with the test itself.. I've noticed it's never configured... if someone is just copying/pasting (like me)... the sample wont work as the test script is not using Jest... npm's package.json by default states
I think as a minimum it should be edited to something like
any other argument needed should be there.
2- I suggest to put commands like
mkdir test && cd test
and so on, as sometimes when this kind of details are just mixed with all the other text, it can be missed like I just did.3- There are one or two typos, I recommend to check this out before posting the article to keep a higher quality.
Keep up the good work !
Thank you once again for your feedback, I forgot to update the package.json to;
The article has been updated accordingly.