From ba1045129337be692a3b6d490bf170c4be42795f Mon Sep 17 00:00:00 2001 From: Randall Morey Date: Thu, 16 Dec 2021 17:38:37 -0500 Subject: [PATCH] fix: skip flawed test --- ui/desktop/tests/acceptance/projects/targets-test.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ui/desktop/tests/acceptance/projects/targets-test.js b/ui/desktop/tests/acceptance/projects/targets-test.js index fc40f6611..4971f209f 100644 --- a/ui/desktop/tests/acceptance/projects/targets-test.js +++ b/ui/desktop/tests/acceptance/projects/targets-test.js @@ -255,7 +255,11 @@ module('Acceptance | projects | targets', function (hooks) { await visit(urls.targets); }); - test('cannot connect to a target without proper authorization', async function (assert) { + // Skipping because this test doesn't make sense. Users will never even see + // targets for which they do not have authorize-session. The only reason this + // test ever passed was because filtering in target mocks wasn't previously + // implemented. + test.skip('cannot connect to a target without proper authorization', async function (assert) { assert.expect(3); instances.target.update({ authorized_actions: instances.target.authorized_actions.filter(