Skip to content

Commit d972aa3

Browse files
authored
cleanup unused code (#2409)
1 parent 386b08e commit d972aa3

File tree

2 files changed

+0
-31
lines changed

2 files changed

+0
-31
lines changed

app/institutions/dashboard/-components/chart-kpi-wrapper/chart-kpi/component-test.ts

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -81,31 +81,4 @@ module('Integration | institutions | dashboard | -components | chart-kpi', hooks
8181
assert.dom('[data-test-expanded-total="0"]')
8282
.hasText('100000');
8383
});
84-
85-
/**
86-
* I need to determine if this is going to be a feature or not
87-
test('it renders the without data correctly', async function(this: EnginesIntlTestContext, assert) {
88-
const data = Object({
89-
total: 0,
90-
title: 'This is the title',
91-
icon: 'building',
92-
});
93-
94-
this.set('data', data);
95-
96-
97-
await render(hbs`
98-
<Institutions::Dashboard::-Components::ChartKpiWrapper::ChartKpi
99-
@data={{this.data}}
100-
/>
101-
`);
102-
103-
assert.dom('[data-test-kpi-title]')
104-
.hasText('This is the title');
105-
assert.dom('[data-test-kpi-data]')
106-
.hasText('No data for institution found.');
107-
assert.dom('[data-test-kpi-icon]')
108-
.hasAttribute('data-icon', 'building');
109-
});
110-
*/
11184
});

app/institutions/dashboard/-components/chart-kpi-wrapper/component.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,6 @@ import InstitutionDepartmentModel from 'ember-osf-web/models/institution-departm
1111
import InstitutionSummaryMetricModel from 'ember-osf-web/models/institution-summary-metric';
1212
import SearchResultModel from 'ember-osf-web/models/search-result';
1313

14-
/*
15-
import InstitutionSummaryMetricModel from 'ember-osf-web/models/institution-summary-metric';
16-
*/
17-
1814
export interface ChartDataModel {
1915
label: string;
2016
total: number;

0 commit comments

Comments
 (0)