punkfairie/ed-safari
1
0
Fork 0
Code Issues Pull requests 1 Projects Releases 1 Packages Wiki Activity Actions

Improve Settings test coverage.

Browse source
This commit is contained in:
marley 2023-05-18 21:20:15 -07:00
parent 60d2298a16
commit 56cec7a835
2 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/models/Settings.ts
View file

@ -19,24 +19,20 @@ interface settingsFile {
} }
export class Settings extends EventEmitter { export class Settings extends EventEmitter {
static #instance: Settings|undefined; static #instance: Settings | undefined;
readonly #file: string; readonly #file: string;
minValue: number; minValue: number;
maxDistance: number; maxDistance: number;
#matrixFile: null|string; #matrixFile: null | string;
matrix?: EliteMatrix; matrix?: EliteMatrix;
private constructor(isPackaged: boolean) { private constructor(isPackaged: boolean) {
super(); super();
if (!isPackaged && os.platform() === 'linux') {
this.#file = '/mnt/c/Users/marle/ed-safari-settings.json';
} else {
this.#file = path.join(os.homedir(), 'ed-safari-settings.json'); this.#file = path.join(os.homedir(), 'ed-safari-settings.json');
}
// Check if settings file exists, and create it if not. Using sync since it's such a small // Check if settings file exists, and create it if not. Using sync since it's such a small
// file, and this information is necessary to build the UI. // file, and this information is necessary to build the UI.

1
test/Settings.test.ts
View file

@ -4,6 +4,7 @@ import { Settings } from '../src/models/Settings';
import { mockSettings } from './mockData'; import { mockSettings } from './mockData';
jest.mock('fs'); jest.mock('fs');
const fs = require('fs');
describe('Settings', () => { describe('Settings', () => {
const settingsFile = { const settingsFile = {