Description
This issue is about updating some of the tests in the curriculum to use specific assert methods. The documentation on the methods can be found here: https://www.chaijs.com/api/assert/
Each group of asserts to replace has indicated which is the suggested assert
method to use, make sure to not change what is being tested. For any questions, you can reach out in our chat room or our forum.
If some of the lines changed do not end with a semicolon, let's take this occasion to add it.
These should be assert.exists
, the argument stays the same
The following assert()
should be replaced with assert.equal()
removing the ===
assert.equal
takes two arguments, which will be the two operands of the comparison operator
ex. from assert(a === b)
to assert.equal(a, b)
This looks like something that can be fixed by "first-time" code contributors to this repository.
Please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or our forum if you need help contributing; our moderators will guide you through this.
Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing.