Skip to content

Commit

Permalink
fix: fix some lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
romgere committed May 31, 2024
1 parent b6804c3 commit 60bd7a2
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
3 changes: 3 additions & 0 deletions tests/unit/controllers/app/generator-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ function mockFont(name: string, variant: Variant | undefined) {
module('Unit | Controller | app/generator', function (hooks) {
setupTest(hooks);

// eslint-disable-next-line qunit/require-expect
test('it handles font change & font loading (google font)', async function (assert) {
assert.expect(4);

Expand Down Expand Up @@ -50,6 +51,7 @@ module('Unit | Controller | app/generator', function (hooks) {
);
});

// eslint-disable-next-line qunit/require-expect
test('it handles font change & font loading (custom font)', async function (assert) {
assert.expect(3);
this.owner.lookup('service:font-manager').loadCustomFont = async function (file: Blob) {
Expand Down Expand Up @@ -80,6 +82,7 @@ module('Unit | Controller | app/generator', function (hooks) {
);
});

// eslint-disable-next-line qunit/require-expect
test('it generate a STL with mesh', async function (assert) {
assert.expect(7);
const controller = this.owner.lookup('controller:app/generator') as GeneratorController;
Expand Down
2 changes: 2 additions & 0 deletions tests/unit/services/file-exporter-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import type { Mesh } from 'three';
module('Unit | Service | file-exporter', function (hooks) {
setupTest(hooks);

// eslint-disable-next-line qunit/require-expect
test(`downloadMeshFile works [STL]`, function (assert) {
assert.expect(5);
const service = this.owner.lookup('service:file-exporter') as FileExporterService;
Expand All @@ -28,6 +29,7 @@ module('Unit | Service | file-exporter', function (hooks) {
service.downloadMeshFile('a_mesh' as unknown as Mesh, 'stl');
});

// eslint-disable-next-line qunit/require-expect
test(`downloadMeshFile works [OBJ]`, function (assert) {
assert.expect(4);
const service = this.owner.lookup('service:file-exporter') as FileExporterService;
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/services/font-manager-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ module('Unit | Service | font-manager', function (hooks) {
title: 'Font name with variant',
},
]) {
// eslint-disable-next-line qunit/require-expect
test(`it fetch font [${title}]`, async function (assert) {
assert.expect(3);
const service = this.owner.lookup('service:font-manager') as FontManagerService;
Expand Down Expand Up @@ -88,7 +89,6 @@ module('Unit | Service | font-manager', function (hooks) {
},
]) {
test(`it throw error when font can't be load [${title}]`, async function (assert) {
assert.expect(1);
const service = this.owner.lookup('service:font-manager') as FontManagerService;
service.fontList = mockedFontList;

Expand Down Expand Up @@ -129,6 +129,7 @@ module('Unit | Service | font-manager', function (hooks) {
assert.strictEqual(`${font}`, 'a-parsed-font', 'Font is returned');
});

// eslint-disable-next-line qunit/require-expect
test('it can load custom font file', async function (assert) {
assert.expect(2);

Expand Down

0 comments on commit 60bd7a2

Please sign in to comment.