Skip to content

Make tests more clear #8999

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

Closed
jennabelle wants to merge 8 commits into from
Closed

Make tests more clear #8999

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
af9e4d0
replace ==, !=, some instances of var, and assert.equal()
Sep 25, 2016
f04c78c
change var to const
Oct 10, 2016
3d27c89
change 'var' to 'const' for clarity
Oct 10, 2016
055e479
trying to complete git pull upstream master
jennabelle Oct 10, 2016
95b1fcd
completing merge
jennabelle Oct 11, 2016
be11d1a
Change last `var` to `const`, add `assert.strictEqual()`
Oct 11, 2016
393a059
add semicolon after `assert.strictEqual()`
Oct 11, 2016
e02413c
Refactor for readability and change `var` to `const`
Oct 11, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
completing merge
  • Loading branch information
@jennabelle
jennabelle committed Oct 11, 2016
commit 95b1fcdf19cf67f2d418dc8ceb48a03755f9b870

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.