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

Jump to bottom

Test description typo in PriorityQueue.test.js #315

Closed

iamlockon opened this issue Feb 21, 2019 · 1 comment

iamlockon commented Feb 21, 2019

As title, it appears to me that the test description in line 37 :

  it('should be possible to change priority of internal nodes', () => {

and in line 54 :

  it('should be possible to change priority of head node', () => {

should be exchanged, since the node with item of value '100' should be the head node in my opinion. Thank you.

trekhleb added a commit that referenced this issue


          Fix issue #315.

Owner

trekhleb commented Apr 12, 2019

@iamlockon I guess you're right. The naming should be fixed now.

trekhleb closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment