fix rest of firefox tests

ether · Mar 22, 2020 · 6a7bcfd · 6a7bcfd
1 parent 5662287
commit 6a7bcfd
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/tests/frontend/specs/helper.js b/tests/frontend/specs/helper.js
@@ -136,6 +136,8 @@ describe("the test helper", function(){
  helper.selectLines($startLine, $endLine, startOffset, endOffset);
 
  var selection = inner$.document.getSelection();
+ // .replace is required here because Firefox keeps the line breaks. I'm not sure this is ideal behavior of getSelection where the text
+ // is not consistant between browsers but that's the situation so that's how I'm covering it in this test.
  expect(cleanText(selection.toString().replace(/(\r\n|\n|\r)/gm,""))).to.be("ort lines to t");
 
  done();
@@ -154,7 +156,7 @@ describe("the test helper", function(){
  helper.selectLines($startLine, $endLine, startOffset, endOffset);
 
  var selection = inner$.document.getSelection();
- expect(cleanText(selection.toString())).to.be("ort lines to test");
+ expect(cleanText(selection.toString().replace(/(\r\n|\n|\r)/gm,""))).to.be("ort lines to test");
 
  done();
  });
@@ -172,7 +174,7 @@ describe("the test helper", function(){
  helper.selectLines($startLine, $endLine, startOffset, endOffset);
 
  var selection = inner$.document.getSelection();
- expect(cleanText(selection.toString())).to.be("ort lines ");
+ expect(cleanText(selection.toString().replace(/(\r\n|\n|\r)/gm,""))).to.be("ort lines ");
 
  done();
  });
@@ -190,7 +192,7 @@ describe("the test helper", function(){
  helper.selectLines($startLine, $endLine, startOffset, endOffset);
 
  var selection = inner$.document.getSelection();
- expect(cleanText(selection.toString())).to.be("ort lines to test");
+ expect(cleanText(selection.toString().replace(/(\r\n|\n|\r)/gm,""))).to.be("ort lines to test");
 
  done();
  });
@@ -205,7 +207,7 @@ describe("the test helper", function(){
  helper.selectLines($startLine, $endLine);
 
  var selection = inner$.document.getSelection();
- expect(cleanText(selection.toString())).to.be("short lines to test");
+ expect(cleanText(selection.toString().replace(/(\r\n|\n|\r)/gm,""))).to.be("short lines to test");
 
  done();
  });