Skip to content

Commit d517ab1

Browse files
committed
refactor: . Move mockApp to tests/__mocks__/obsidian.ts
1 parent 04652af commit d517ab1

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

tests/Renderer/QueryResultsRenderer.test.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
*/
44
import moment from 'moment';
55
import type { Task } from 'Task/Task';
6-
import type { App } from 'obsidian';
76
import { GlobalFilter } from '../../src/Config/GlobalFilter';
87
import { State } from '../../src/Obsidian/Cache';
98
import { QueryResultsRenderer } from '../../src/Renderer/QueryResultsRenderer';
@@ -18,6 +17,7 @@ import { prettifyHTML } from '../TestingTools/HTMLHelpers';
1817
import { TaskBuilder } from '../TestingTools/TaskBuilder';
1918
import { toMarkdown } from '../TestingTools/TestHelpers';
2019
import { resetSettings, updateSettings } from '../../src/Config/Settings';
20+
import { mockApp } from '../__mocks__/obsidian';
2121
import { mockHTMLRenderer } from './RenderingTestHelpers';
2222

2323
window.moment = moment;
@@ -33,13 +33,6 @@ afterEach(() => {
3333
resetSettings();
3434
});
3535

36-
/**
37-
* Since we don't use the app object's method or properties directly,
38-
* and just treat it as an "opaque object" for markdown rendering, there is
39-
* not a lot to mock in particular.
40-
*/
41-
const mockApp = {} as unknown as App;
42-
4336
function makeQueryResultsRenderer(source: string, tasksFile: TasksFile) {
4437
return new QueryResultsRenderer(
4538
'block-language-tasks',

tests/__mocks__/obsidian.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,14 @@
1-
import type { CachedMetadata } from 'obsidian';
1+
import type { App, CachedMetadata } from 'obsidian';
22

33
export {};
44

5+
/**
6+
* Since we don't use the app object's method or properties directly,
7+
* and just treat it as an "opaque object" for markdown rendering, there is
8+
* not a lot to mock in particular.
9+
*/
10+
export const mockApp = {} as unknown as App;
11+
512
export class MenuItem {
613
public title: string | DocumentFragment = '';
714
public callback: (evt: MouseEvent | KeyboardEvent) => any;

0 commit comments

Comments
 (0)