From 3cf761e9c0228df3b9340e0ce8225a1d16c3bfc4 Mon Sep 17 00:00:00 2001 From: Scott Anderson <662325+scottanderson@users.noreply.github.com> Date: Fri, 1 Aug 2025 00:43:07 -0400 Subject: [PATCH] Loosen coverage requirements slightly (#1669) ## Description: Tight code coverage requirements and low coverage are making it difficult to merge code, because a small increase in lines of code can decrease coverage enough to trigger a test failure. ## Please complete the following: - [x] I have added screenshots for all UI updates - [x] I process any text displayed to the user through translateText() and I've added it to the en.json file - [x] I have added relevant tests to the test directory - [x] I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced - [ ] I have read and accepted the CLA agreement (only required once). --- jest.config.ts | 8 ++++---- src/client/graphics/layers/EmojiTable.ts | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/jest.config.ts b/jest.config.ts index 014158c29..4722a979e 100644 --- a/jest.config.ts +++ b/jest.config.ts @@ -29,10 +29,10 @@ export default { collectCoverageFrom: ["src/**/*.ts", "!src/**/*.d.ts"], coverageThreshold: { global: { - statements: 22.2, - branches: 17.5, - lines: 22.7, - functions: 21.2, + statements: 22.0, + branches: 17.0, + lines: 22.5, + functions: 21.0, }, }, coverageReporters: ["text", "lcov", "html"], diff --git a/src/client/graphics/layers/EmojiTable.ts b/src/client/graphics/layers/EmojiTable.ts index b61c7d6a0..aefec7ade 100644 --- a/src/client/graphics/layers/EmojiTable.ts +++ b/src/client/graphics/layers/EmojiTable.ts @@ -48,7 +48,7 @@ export class EmojiTable extends LitElement { this.hideTable(); }); }); - this.eventBus.on(CloseViewEvent, (e) => { + eventBus.on(CloseViewEvent, (e) => { if (!this.hidden) { this.hideTable(); }