From f6c9ffcf608e78bb9d9bb55fc6e8b1c085ffbe40 Mon Sep 17 00:00:00 2001 From: Kevin Ansfield Date: Tue, 19 Apr 2022 20:42:47 +0100 Subject: [PATCH] Fixed conflicting test selector in refs https://github.com/TryGhost/Admin/commit/8e120a74d69de11993f88773271ec6d77acc3363 - we use `data-test-state` inside of some modals which also use `` which caused unrelated tests to start failing due to a change in matching element counts --- ghost/admin/app/components/gh-task-button.hbs | 8 ++++---- .../tests/integration/components/gh-task-button-test.js | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/ghost/admin/app/components/gh-task-button.hbs b/ghost/admin/app/components/gh-task-button.hbs index a02b5a2731..f4adbb40e8 100644 --- a/ghost/admin/app/components/gh-task-button.hbs +++ b/ghost/admin/app/components/gh-task-button.hbs @@ -6,8 +6,8 @@ isFailure=this.isFailure )}} {{else}} - {{#if this.isRunning}}{{svg-jar "spinner" class="gh-icon-spinner"}}{{this.runningText}}{{/if}} - {{#if this.isIdle}}{{this.buttonText}}{{/if}} - {{#if this.isSuccess}}{{svg-jar "check-circle"}} {{this.successText}}{{/if}} - {{#if this.isFailure}}{{svg-jar "retry"}} {{this.failureText}}{{/if}} + {{#if this.isRunning}}{{svg-jar "spinner" class="gh-icon-spinner"}}{{this.runningText}}{{/if}} + {{#if this.isIdle}}{{this.buttonText}}{{/if}} + {{#if this.isSuccess}}{{svg-jar "check-circle"}} {{this.successText}}{{/if}} + {{#if this.isFailure}}{{svg-jar "retry"}} {{this.failureText}}{{/if}} {{/if}} diff --git a/ghost/admin/tests/integration/components/gh-task-button-test.js b/ghost/admin/tests/integration/components/gh-task-button-test.js index cbea83bff2..b967e9006e 100644 --- a/ghost/admin/tests/integration/components/gh-task-button-test.js +++ b/ghost/admin/tests/integration/components/gh-task-button-test.js @@ -155,7 +155,7 @@ describe('Integration: Component: gh-task-button', function () { await render(hbs``); this.myTask.perform(); - await waitFor('[data-test-state="idle"]', {timeout: 50}); + await waitFor('[data-test-task-button-state="idle"]', {timeout: 50}); await settled(); });