Skip to content

Codex/unit tests refactoring - #65

Merged
koreyba merged 11 commits into
stagefrom
codex/unit-tests-refactoring
Mar 7, 2026
Merged

koreyba merged 11 commits into
stagefrom
codex/unit-tests-refactoring

test: honor style rules in sanitizer mock

d85e622
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 7, 2026 in 40s

Quality Gate failed

Annotations

Check warning on line 8 in ui/web/tests/unit/hooks/useSearchMode.test.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLszOQh-IuXSnTUso&open=AZzJLszOQh-IuXSnTUso&pullRequest=65

Check warning on line 34 in core/tests/unit/core-services-smartPaste.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Member 'doc: FakeDocument' is never reassigned; mark it as `readonly`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs2rQh-IuXSnTUsy&open=AZzJLs2rQh-IuXSnTUsy&pullRequest=65

Check warning on line 28 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE6&open=AZzJxPK2L5GvtGFBxBE6&pullRequest=65

Check warning on line 41 in jest.config.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs3YQh-IuXSnTUtF&open=AZzJLs3YQh-IuXSnTUtF&pullRequest=65

Check warning on line 104 in core/tests/unit/core-services-smartPaste.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use 'Object.hasOwn()' instead of 'Object.prototype.hasOwnProperty.call()'.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs2rQh-IuXSnTUs1&open=AZzJLs2rQh-IuXSnTUs1&pullRequest=65

Check warning on line 31 in jest.config.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs3YQh-IuXSnTUtE&open=AZzJLs3YQh-IuXSnTUtE&pullRequest=65

Check warning on line 49 in ui/web/tests/unit/hooks/useSearchMode.test.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLszOQh-IuXSnTUsr&open=AZzJLszOQh-IuXSnTUsr&pullRequest=65

Check warning on line 2 in core/tests/integration/smartPaste.integration.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `node:path` over `path`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs22Qh-IuXSnTUs6&open=AZzJLs22Qh-IuXSnTUs6&pullRequest=65

Check failure on line 35 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJyfubbUJJ94dzl9yC&open=AZzJyfubbUJJ94dzl9yC&pullRequest=65

Check warning on line 4 in ui/mobile/jest.config.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs1XQh-IuXSnTUss&open=AZzJLs1XQh-IuXSnTUss&pullRequest=65

Check warning on line 36 in ui/web/tests/unit/hooks/useSearchMode.test.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLszOQh-IuXSnTUsq&open=AZzJLszOQh-IuXSnTUsq&pullRequest=65

Check warning on line 4 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE3&open=AZzJxPK2L5GvtGFBxBE3&pullRequest=65

Check warning on line 21 in jest.config.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs3YQh-IuXSnTUtD&open=AZzJLs3YQh-IuXSnTUtD&pullRequest=65

Check warning on line 1 in core/tests/integration/smartPaste.integration.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `node:fs` over `fs`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs22Qh-IuXSnTUs5&open=AZzJLs22Qh-IuXSnTUs5&pullRequest=65

Check warning on line 28 in ui/web/tests/unit/hooks/useSearchMode.test.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLszOQh-IuXSnTUsp&open=AZzJLszOQh-IuXSnTUsp&pullRequest=65

Check warning on line 95 in core/tests/unit/core-services-smartPaste.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use 'Object.hasOwn()' instead of 'Object.prototype.hasOwnProperty.call()'.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs2rQh-IuXSnTUs0&open=AZzJLs2rQh-IuXSnTUs0&pullRequest=65

Check warning on line 29 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE9&open=AZzJxPK2L5GvtGFBxBE9&pullRequest=65

Check warning on line 28 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE7&open=AZzJxPK2L5GvtGFBxBE7&pullRequest=65

Check warning on line 518 in core/tests/unit/core-services-smartPaste.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Don't use a zero fraction in the number.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs2rQh-IuXSnTUs4&open=AZzJLs2rQh-IuXSnTUs4&pullRequest=65

Check warning on line 32 in core/tests/unit/core-services-smartPaste.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Member 'tag: string' is never reassigned; mark it as `readonly`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs2rQh-IuXSnTUsx&open=AZzJLs2rQh-IuXSnTUsx&pullRequest=65

Check warning on line 91 in core/tests/unit/core-services-smartPaste.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJLs2rQh-IuXSnTUsz&open=AZzJLs2rQh-IuXSnTUsz&pullRequest=65

Check warning on line 4 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE4&open=AZzJxPK2L5GvtGFBxBE4&pullRequest=65

Check warning on line 126 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE-&open=AZzJxPK2L5GvtGFBxBE-&pullRequest=65

Check warning on line 29 in tests/jest/core.setup.cjs

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`.

See more on https://sonarcloud.io/project/issues?id=koreyba_EverFreeNote&issues=AZzJxPK2L5GvtGFBxBE8&open=AZzJxPK2L5GvtGFBxBE8&pullRequest=65