feat: 🎸 Update Success Dialog when connecting to target (#1046)
Updated the Success Dialog in desktop app when connecting to a target to
dynamically show target type
✅ Closes: ICU-3647
This commit is contained in:
1 parent
8e884d53ae
commit
79596010e9
2 files changed
+44
-1
No files matched your search
@@ -20,7 +20,7 @@
|
||||
<:body>
|
||||
<h3>
|
||||
{{t 'resources.target.messages.proxy_url.title'}}
|
||||
({{t 'resources.target.types.tcp'}})
|
||||
({{t (concat 'resources.target.types.' target.type)}})
|
||||
</h3>
|
||||
<p>
|
||||
<Copyable
|
||||
|
||||
@@ -233,6 +233,49 @@ module('Acceptance | projects | targets', function (hooks) {
|
||||
);
|
||||
});
|
||||
|
||||
test('displays the correct target type (TCP) in the success dialog body', async function (assert) {
|
||||
assert.expect(1);
|
||||
stubs.ipcService.withArgs('cliExists').returns(true);
|
||||
stubs.ipcService.withArgs('connect').returns({
|
||||
session_id: instances.session.id,
|
||||
address: 'a_123',
|
||||
port: 'p_123',
|
||||
protocol: 'tcp',
|
||||
});
|
||||
const confirmService = this.owner.lookup('service:confirm');
|
||||
confirmService.enabled = true;
|
||||
await visit(urls.targets);
|
||||
await click(
|
||||
'tbody tr:first-child td:last-child button',
|
||||
'Activate connect mode'
|
||||
);
|
||||
|
||||
assert.ok(find('.rose-dialog-body h3').textContent.trim().includes('TCP'));
|
||||
});
|
||||
|
||||
test('displays the correct target type (SSH) in the success dialog body', async function (assert) {
|
||||
assert.expect(1);
|
||||
instances.target.update({
|
||||
type: 'ssh',
|
||||
});
|
||||
stubs.ipcService.withArgs('cliExists').returns(true);
|
||||
stubs.ipcService.withArgs('connect').returns({
|
||||
session_id: instances.session.id,
|
||||
address: 'a_123',
|
||||
port: 'p_123',
|
||||
protocol: 'ssh',
|
||||
});
|
||||
const confirmService = this.owner.lookup('service:confirm');
|
||||
confirmService.enabled = true;
|
||||
await visit(urls.targets);
|
||||
await click(
|
||||
'tbody tr:first-child td:last-child button',
|
||||
'Activate connect mode'
|
||||
);
|
||||
|
||||
assert.ok(find('.rose-dialog-body h3').textContent.trim().includes('SSH'));
|
||||
});
|
||||
|
||||
// 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
|
||||
|
||||
Reference in new issue
Block a user