From 2bb36d72a3874c3e0436edddf535e7e1dc6e95ca Mon Sep 17 00:00:00 2001 From: Blake Erickson Date: Wed, 27 Nov 2019 17:19:23 -0700 Subject: [PATCH] DEV: Fix failing test on CI due to whitespace differences I could not replicate the failure locally, but it was consistently failing on CI with: ``` Assertion Failed: it should escape watched words Expected: , Actual: ``` This commit removes an extra space that was added originally, but I don't think it is really needed after double checking how it displays in the UI. The `x` icon and the word have sufficient spacing between them. If we need to we can tweak it in css instead. --- .../admin/templates/components/admin-watched-word.hbs | 2 +- test/javascripts/acceptance/admin-watched-words-test.js.es6 | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/admin/templates/components/admin-watched-word.hbs b/app/assets/javascripts/admin/templates/components/admin-watched-word.hbs index 82e6b5431c1..c9e4fd9b041 100644 --- a/app/assets/javascripts/admin/templates/components/admin-watched-word.hbs +++ b/app/assets/javascripts/admin/templates/components/admin-watched-word.hbs @@ -1 +1 @@ -{{xIcon}} {{watchedWord}} +{{xIcon}}{{watchedWord}} diff --git a/test/javascripts/acceptance/admin-watched-words-test.js.es6 b/test/javascripts/acceptance/admin-watched-words-test.js.es6 index c602667fa6d..48bd7c146be 100644 --- a/test/javascripts/acceptance/admin-watched-words-test.js.es6 +++ b/test/javascripts/acceptance/admin-watched-words-test.js.es6 @@ -34,7 +34,7 @@ QUnit.test("list words in groups", async assert => { assert.equal( $(find(".watched-words-list .watched-word")[2]).text(), - ' ', + '', "it should escape watched words" );