Tools/Notes
1
0
Fork 0
mirror of https://github.com/TriliumNext/Notes.git synced 2025-10-31 21:11:30 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1194 from TriliumNext/test-unskip-tests

Browse Source 
test: fix and enable ex-flaky tests in `services/search.spec.ts`
This commit is contained in:
Elian Doran 2025-02-14 23:43:01 +02:00 committed by GitHub
commit 552f87e2b3
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
package.json
View File

@ -41,8 +41,8 @@
"build-docs": "npm run build-backend-docs && npm run build-frontend-docs", "build-docs": "npm run build-backend-docs && npm run build-frontend-docs",
"webpack": "tsx node_modules/webpack/bin/webpack.js -c webpack.config.ts", "webpack": "tsx node_modules/webpack/bin/webpack.js -c webpack.config.ts",
"test-playwright": "playwright test", "test-playwright": "playwright test",
"test": "cross-env TRILIUM_DATA_DIR=./data-test vitest", "test": "cross-env TRILIUM_DATA_DIR=./integration-tests/db vitest",
"test-coverage": "cross-env TRILIUM_DATA_DIR=./data-test vitest --coverage", "test-coverage": "cross-env TRILIUM_DATA_DIR=./integration-tests/db vitest --coverage",
"start-electron-forge": "npm run prepare-dist && electron-forge start", "start-electron-forge": "npm run prepare-dist && electron-forge start",
"make-electron": "npm run webpack && npm run prepare-dist && electron-forge make", "make-electron": "npm run webpack && npm run prepare-dist && electron-forge make",
"package-electron": "electron-forge package", "package-electron": "electron-forge package",

16
src/services/search/services/search.spec.ts
View File

@ -22,7 +22,7 @@ describe("Search", () => {
}); });
}); });
it.skip("simple path match", () => { it("simple path match", () => {
rootNote.child(note("Europe").child(note("Austria"))); rootNote.child(note("Europe").child(note("Austria")));
const searchContext = new SearchContext(); const searchContext = new SearchContext();
@ -32,7 +32,7 @@ describe("Search", () => {
expect(findNoteByTitle(searchResults, "Austria")).toBeTruthy(); expect(findNoteByTitle(searchResults, "Austria")).toBeTruthy();
}); });
it.skip("normal search looks also at attributes", () => { it("normal search looks also at attributes", () => {
const austria = note("Austria"); const austria = note("Austria");
const vienna = note("Vienna"); const vienna = note("Vienna");
@ -50,7 +50,7 @@ describe("Search", () => {
expect(findNoteByTitle(searchResults, "Vienna")).toBeTruthy(); expect(findNoteByTitle(searchResults, "Vienna")).toBeTruthy();
}); });
it.skip("normal search looks also at type and mime", () => { it("normal search looks also at type and mime", () => {
rootNote.child(note("Effective Java", { type: "book", mime: "" })).child(note("Hello World.java", { type: "code", mime: "text/x-java" })); rootNote.child(note("Effective Java", { type: "book", mime: "" })).child(note("Hello World.java", { type: "code", mime: "text/x-java" }));
const searchContext = new SearchContext(); const searchContext = new SearchContext();
@ -69,7 +69,7 @@ describe("Search", () => {
expect(searchResults.length).toEqual(2); expect(searchResults.length).toEqual(2);
}); });
it.skip("only end leafs are results", () => { it("only end leafs are results", () => {
rootNote.child(note("Europe").child(note("Austria"))); rootNote.child(note("Europe").child(note("Austria")));
const searchContext = new SearchContext(); const searchContext = new SearchContext();
@ -79,7 +79,7 @@ describe("Search", () => {
expect(findNoteByTitle(searchResults, "Europe")).toBeTruthy(); expect(findNoteByTitle(searchResults, "Europe")).toBeTruthy();
}); });
it.skip("only end leafs are results", () => { it("only end leafs are results", () => {
rootNote.child(note("Europe").child(note("Austria").label("capital", "Vienna"))); rootNote.child(note("Europe").child(note("Austria").label("capital", "Vienna")));
const searchContext = new SearchContext(); const searchContext = new SearchContext();
@ -132,7 +132,7 @@ describe("Search", () => {
expect(findNoteByTitle(searchResults, "Czech Republic")).toBeTruthy(); expect(findNoteByTitle(searchResults, "Czech Republic")).toBeTruthy();
}); });
it.skip("inherited label comparison", () => { it("inherited label comparison", () => {
rootNote.child(note("Europe").label("country", "", true).child(note("Austria")).child(note("Czech Republic"))); rootNote.child(note("Europe").label("country", "", true).child(note("Austria")).child(note("Czech Republic")));
const searchContext = new SearchContext(); const searchContext = new SearchContext();
@ -527,7 +527,7 @@ describe("Search", () => {
expect(becca.notes[searchResults[0].noteId].title).toEqual("Europe"); expect(becca.notes[searchResults[0].noteId].title).toEqual("Europe");
}); });
it.skip("test note.text *=* something", () => { it("test note.text *=* something", () => {
const italy = note("Italy").label("capital", "Rome"); const italy = note("Italy").label("capital", "Rome");
const slovakia = note("Slovakia").label("capital", "Bratislava"); const slovakia = note("Slovakia").label("capital", "Bratislava");
@ -540,7 +540,7 @@ describe("Search", () => {
expect(becca.notes[searchResults[0].noteId].title).toEqual("Slovakia"); expect(becca.notes[searchResults[0].noteId].title).toEqual("Slovakia");
}); });
it.skip("test that fulltext does not match archived notes", () => { it("test that fulltext does not match archived notes", () => {
const italy = note("Italy").label("capital", "Rome"); const italy = note("Italy").label("capital", "Rome");
const slovakia = note("Slovakia").label("capital", "Bratislava"); const slovakia = note("Slovakia").label("capital", "Bratislava");