Skip to content

Commit 80e9f44

Browse files
committed
Update tests following re-org
1 parent e30dc24 commit 80e9f44

File tree

2 files changed

+11
-14
lines changed

2 files changed

+11
-14
lines changed

src/__tests__/inbox.test.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { TFile, TFolder, Vault } from 'obsidian';
1+
import { FileManager, TFile, TFolder, Vault } from 'obsidian';
22
import { INBOX_FOLDER } from '..';
33
import { Inbox } from '../inbox';
44

@@ -7,14 +7,16 @@ describe('Inbox', () => {
77
let sut: Inbox;
88

99
let vault: Vault;
10+
let fileManager: FileManager;
1011

1112
beforeEach(() => {
1213
vault = jest.fn() as unknown as Vault;
1314
vault.getAllFolders = jest.fn();
1415
vault.getFolderByPath = jest.fn();
15-
vault.rename = jest.fn();
16+
fileManager = jest.fn() as unknown as FileManager;
17+
fileManager.renameFile = jest.fn();
1618

17-
sut = new Inbox(vault);
19+
sut = new Inbox(vault, fileManager);
1820
});
1921

2022
it('returns empty when the inbox folder is not found', () => {
@@ -70,13 +72,13 @@ describe('Inbox', () => {
7072
});
7173

7274
it('moves a file', async () => {
73-
const vaultRename = jest.spyOn(vault, 'rename');
75+
const fileManagerRename = jest.spyOn(fileManager, 'renameFile');
7476

7577
const file = new TFile();
7678
file.name = 'example.md';
7779

7880
await sut.move(file, 'some/new/path');
7981

80-
expect(vaultRename).toHaveBeenCalledWith(file, 'some/new/path/example.md');
82+
expect(fileManagerRename).toHaveBeenCalledWith(file, 'some/new/path/example.md');
8183
});
8284
});
Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { App, TFile, TFolder } from 'obsidian';
2-
import { Inbox } from '../inbox';
3-
import { CLS_PREFIX, OrganiserModal } from '../modal';
2+
import { Inbox } from '../../inbox';
3+
import { CLS_PREFIX, OrganiserModal } from '../../modal';
44

55
const WAIT_TIME: number = 20;
66

@@ -55,8 +55,7 @@ describe('OrganiserModal', () => {
5555
await sut.onOpen();
5656

5757
const select = sut.contentEl.find(`.${CLS_PREFIX}top .${CLS_PREFIX}dropdown`);
58-
expect(select).toBeInstanceOf(HTMLSelectElement);
59-
expect(select.children.length).toEqual(1);
58+
expect(select).toBeInstanceOf(HTMLInputElement);
6059

6160
const table = sut.contentEl.find(`.${CLS_PREFIX}files table`);
6261
expect(table).toBeInstanceOf(HTMLTableElement);
@@ -70,11 +69,7 @@ describe('OrganiserModal', () => {
7069
await sut.onOpen();
7170

7271
const selectEl = sut.contentEl.find(`.${CLS_PREFIX}top .${CLS_PREFIX}dropdown`);
73-
expect(selectEl).toBeInstanceOf(HTMLSelectElement);
74-
expect(selectEl.children.length).toEqual(4);
75-
expect(selectEl.children[1].innerHTML).toEqual('Level One');
76-
expect(selectEl.children[2].innerHTML).toEqual('Level Two (Level One)');
77-
expect(selectEl.children[3].innerHTML).toEqual('Level Three (Level One > Level Two)');
72+
expect(selectEl).toBeInstanceOf(HTMLInputElement);
7873

7974
const tableEl = sut.contentEl.find(`.${CLS_PREFIX}files table`);
8075
expect(tableEl).toBeInstanceOf(HTMLTableElement);

0 commit comments

Comments
 (0)