fix: 🐛 Removing status badging from desktop client (#1749)
* fix: 🐛 Removing status badging from desktop client Removed status badges for `targets` and `sessions` in desktop client app. ✅ Closes: ICU-9790 * refactor: 💡 Bring back status badging for session Session status badging can stay due to session can only be canceled by the user in desktop client and does give an immediate status to the session. ✅ Closes: ICU-9790 * refactor: 💡 merge changes
This commit is contained in:
1 parent
f1e612385d
commit
508d66e375
4 files changed
-62
No files matched your search
@@ -124,13 +124,6 @@
|
||||
{{#if model.description}}
|
||||
<p>{{model.description}}</p>
|
||||
{{/if}}
|
||||
{{#if model.isActive}}
|
||||
<Hds::Badge
|
||||
@text={{t 'states.active'}}
|
||||
@color='success'
|
||||
@icon='check'
|
||||
/>
|
||||
{{/if}}
|
||||
</row.headerCell>
|
||||
<row.cell>
|
||||
<Copyable
|
||||
|
||||
@@ -10,10 +10,6 @@
|
||||
<page.header>
|
||||
<h2>
|
||||
{{@model.displayName}}
|
||||
{{#if @model.isActive}}
|
||||
|
||||
<Hds::Badge @color='success' @text={{t 'states.active'}} />
|
||||
{{/if}}
|
||||
</h2>
|
||||
<p>{{@model.description}}</p>
|
||||
<p>
|
||||
|
||||
@@ -5,10 +5,6 @@
|
||||
|
||||
<h2>
|
||||
{{@model.displayName}}
|
||||
{{#if @model.isActive}}
|
||||
|
||||
<Hds::Badge @color='success' @text={{t 'states.active'}} />
|
||||
{{/if}}
|
||||
</h2>
|
||||
<p>{{@model.description}}</p>
|
||||
<p>
|
||||
|
||||
@@ -193,53 +193,6 @@ module('Acceptance | projects | targets | sessions', function (hooks) {
|
||||
);
|
||||
});
|
||||
|
||||
test('can identify target with active sessions', 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.sessions);
|
||||
await click('.rose-layout-page-actions button', 'Activate connect mode');
|
||||
await click('.rose-dialog-footer .rose-button-secondary');
|
||||
assert.ok(find('.rose-layout-page-header .hds-badge--color-success'));
|
||||
});
|
||||
|
||||
test('can identify target with pending sessions', async function (assert) {
|
||||
assert.expect(1);
|
||||
|
||||
instances.session.update({ status: 'pending' });
|
||||
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.sessions);
|
||||
await click('.rose-layout-page-actions button', 'Activate connect mode');
|
||||
await click('.rose-dialog-footer .rose-button-secondary');
|
||||
assert.ok(find('.rose-layout-page-header .hds-badge--color-success'));
|
||||
});
|
||||
|
||||
test('cannot identify target with terminated sessions', async function (assert) {
|
||||
assert.expect(1);
|
||||
instances.session.update({ status: 'terminated' });
|
||||
|
||||
await visit(urls.sessions);
|
||||
assert.notOk(find('.rose-layout-page-header .hds-badge--color-success'));
|
||||
});
|
||||
|
||||
test('cancelling a session', async function (assert) {
|
||||
assert.expect(2);
|
||||
|
||||
|
||||
Reference in new issue
Block a user