mirror of
https://github.com/gristlabs/grist-core.git
synced 2024-10-27 20:44:07 +00:00
6908807236
This adds a config file that's loaded very early on during startup. It enables us to save/load settings from within Grist's admin panel, that affect the startup of the FlexServer. The config file loading: - Is type-safe, - Validates the config file on startup - Provides a path to upgrade to future versions. It should be extensible from other versions of Grist (such as desktop), by overriding `getGlobalConfig` in stubs. ---- Some minor refactors needed to occur to make this possible. This includes: - Extracting config loading into its own module (out of FlexServer). - Cleaning up the `loadConfig` function in FlexServer into `loadLoginSystem` (which is what its main purpose was before).
108 lines
3.8 KiB
TypeScript
108 lines
3.8 KiB
TypeScript
import { assert } from 'chai';
|
|
import * as sinon from 'sinon';
|
|
import { ConfigAccessors, createConfigValue, Deps, FileConfig } from "app/server/lib/config";
|
|
|
|
interface TestFileContents {
|
|
myNum?: number
|
|
myStr?: string
|
|
}
|
|
|
|
const testFileContentsExample: TestFileContents = {
|
|
myNum: 1,
|
|
myStr: "myStr",
|
|
};
|
|
|
|
const testFileContentsJSON = JSON.stringify(testFileContentsExample);
|
|
|
|
describe('FileConfig', () => {
|
|
const useFakeConfigFile = (contents: string) => {
|
|
const fakeFile = { contents };
|
|
sinon.replace(Deps, 'pathExists', sinon.fake.resolves(true));
|
|
sinon.replace(Deps, 'readFile', sinon.fake((path, encoding: string) => Promise.resolve(fakeFile.contents)) as any);
|
|
sinon.replace(Deps, 'writeFile', sinon.fake((path, newContents) => {
|
|
fakeFile.contents = newContents;
|
|
return Promise.resolve();
|
|
}));
|
|
|
|
return fakeFile;
|
|
};
|
|
|
|
afterEach(() => {
|
|
sinon.restore();
|
|
});
|
|
|
|
it('throws an error from create if the validator does not return a value', async () => {
|
|
useFakeConfigFile(testFileContentsJSON);
|
|
const validator = () => null;
|
|
await assert.isRejected(FileConfig.create<TestFileContents>("anypath.json", validator));
|
|
});
|
|
|
|
it('persists changes when values are assigned', async () => {
|
|
const fakeFile = useFakeConfigFile(testFileContentsJSON);
|
|
// Don't validate - this is guaranteed to be valid above.
|
|
const validator = (input: any) => input as TestFileContents;
|
|
const fileConfig = await FileConfig.create("anypath.json", validator);
|
|
await fileConfig.set("myNum", 999);
|
|
|
|
assert.equal(fileConfig.get("myNum"), 999);
|
|
assert.equal(JSON.parse(fakeFile.contents).myNum, 999);
|
|
});
|
|
|
|
// Avoid removing extra properties from the file, in case another edition of grist is doing something.
|
|
it('does not remove extra values from the file', async () => {
|
|
const configWithExtraProperties = {
|
|
...testFileContentsExample,
|
|
someProperty: "isPresent",
|
|
};
|
|
|
|
const fakeFile = useFakeConfigFile(JSON.stringify(configWithExtraProperties));
|
|
// It's entirely possible the validator can damage the extra properties, but that's not in scope for this test.
|
|
const validator = (input: any) => input as TestFileContents;
|
|
const fileConfig = await FileConfig.create("anypath.json", validator);
|
|
// Triggering a write to the file
|
|
await fileConfig.set("myNum", 999);
|
|
await fileConfig.set("myStr", "Something");
|
|
|
|
const newContents = JSON.parse(fakeFile.contents);
|
|
assert.equal(newContents.myNum, 999);
|
|
assert.equal(newContents.myStr, "Something");
|
|
assert.equal(newContents.someProperty, "isPresent");
|
|
});
|
|
});
|
|
|
|
describe('createConfigValue', () => {
|
|
const makeInMemoryAccessors = <T>(initialValue: T): ConfigAccessors<T> => {
|
|
let value: T = initialValue;
|
|
return {
|
|
get: () => value,
|
|
set: async (newValue: T) => { value = newValue; },
|
|
};
|
|
};
|
|
|
|
it('works without persistence', async () => {
|
|
const configValue = createConfigValue(1);
|
|
assert.equal(configValue.get(), 1);
|
|
await configValue.set(2);
|
|
assert.equal(configValue.get(), 2);
|
|
});
|
|
|
|
it('writes to persistence when saved', async () => {
|
|
const accessors = makeInMemoryAccessors(1);
|
|
const configValue = createConfigValue(1, accessors);
|
|
assert.equal(accessors.get(), 1);
|
|
await configValue.set(2);
|
|
assert.equal(accessors.get(), 2);
|
|
});
|
|
|
|
it('initialises with the persistent value if available', async () => {
|
|
const accessors = makeInMemoryAccessors(22);
|
|
const configValue = createConfigValue(1, accessors);
|
|
assert.equal(configValue.get(), 22);
|
|
|
|
const accessorsWithUndefinedValue = makeInMemoryAccessors<number | undefined>(undefined);
|
|
const configValueWithDefault = createConfigValue(333, accessorsWithUndefinedValue);
|
|
assert.equal(configValueWithDefault.get(), 333);
|
|
});
|
|
});
|
|
|