From 28b8818c4de4b8fb7947edfeec5377345525c07e Mon Sep 17 00:00:00 2001 From: Laurent Cozic Date: Fri, 5 Apr 2024 12:19:57 +0100 Subject: [PATCH] Chore: Disabled not very useful test that can fail easily as we add more sort fields --- .../services/sortOrder/notesSortOrderUtils.test.ts | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.ts b/packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.ts index 2e04da295..08f31e50d 100644 --- a/packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.ts +++ b/packages/app-desktop/services/sortOrder/notesSortOrderUtils.test.ts @@ -1,4 +1,4 @@ -import { notesSortOrderFieldArray, notesSortOrderNextField, setNotesSortOrder } from './notesSortOrderUtils'; +import { notesSortOrderNextField, setNotesSortOrder } from './notesSortOrderUtils'; import Setting from '@joplin/lib/models/Setting'; const { shimInit } = require('@joplin/lib/shim-init-node.js'); @@ -9,11 +9,13 @@ describe('notesSortOrderUtils', () => { Setting.autoSaveEnabled = false; }); - it('should always provide the same ordered fields', async () => { - const expected = ['user_updated_time', 'user_created_time', 'title', 'order']; - expect(notesSortOrderFieldArray()).toStrictEqual(expected); - expect(notesSortOrderFieldArray()).toStrictEqual(expected); - }); + // Disabling as the test doesn't seem very useful. + + // it('should always provide the same ordered fields', async () => { + // const expected = ['user_updated_time', 'user_created_time', 'title', 'order']; + // expect(notesSortOrderFieldArray()).toStrictEqual(expected); + // expect(notesSortOrderFieldArray()).toStrictEqual(expected); + // }); it('should provide the next field cyclically', async () => { expect(notesSortOrderNextField('user_updated_time')).toBe('user_created_time');