Skip to content

Remove test line without effect because no should.

Ramya Authappan requested to merge github/fork/cirosantilli/rm-empty-eq into master

Created by: cirosantilli

I think the should == was forgotten, as the return value is never used, and the function is only used in intermediate steps, not on the conclusions.

If I add the should, tests start to fail. Not sure if Poltergeist can handle window.location stuff.

Even if it could, tests should still fail, because when you edit the issue search all possible parameters get added to the URL, not just the issue_search.

So let's just remove this.

Merge request reports

Loading