New issue

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