This commit fixes compiler bug where it fails when optional values are compiled into absent values in nested calls. - Throw exception with more context for easier future debugging. - Add better validation of argument values for nested calls. - Refactor `FunctionCallCompiler` for better clarity and modularize it to make it more maintainable and testable. - Refactor related interface to not have `I` prefix, and function/variable names for better clarity. Context: Discovered this issue while attempting to call `RunInlineCodeAsTrustedInstaller` which in turn invokes `RunPowerShell` for issue #246. This led to the realization that despite parameters flagged as optional, the nested argument compilation didn't support them.
111 lines
3.6 KiB
TypeScript
111 lines
3.6 KiB
TypeScript
import { describe, it, expect } from 'vitest';
|
|
import { CodeSubstituter } from '@/application/Parser/ScriptingDefinition/CodeSubstituter';
|
|
import { IExpressionsCompiler } from '@/application/Parser/Script/Compiler/Expressions/IExpressionsCompiler';
|
|
import { ProjectInformationStub } from '@tests/unit/shared/Stubs/ProjectInformationStub';
|
|
import { ExpressionsCompilerStub } from '@tests/unit/shared/Stubs/ExpressionsCompilerStub';
|
|
import { getAbsentObjectTestCases, getAbsentStringTestCases } from '@tests/unit/shared/TestCases/AbsentTests';
|
|
|
|
describe('CodeSubstituter', () => {
|
|
describe('throws with invalid parameters', () => {
|
|
// arrange
|
|
const testCases = [
|
|
...getAbsentStringTestCases().map((testCase) => ({
|
|
name: `given code: ${testCase.valueName}`,
|
|
expectedError: 'missing code',
|
|
parameters: {
|
|
code: testCase.absentValue,
|
|
info: new ProjectInformationStub(),
|
|
},
|
|
})),
|
|
...getAbsentObjectTestCases().map((testCase) => ({
|
|
name: `given info: ${testCase.valueName}`,
|
|
expectedError: 'missing info',
|
|
parameters: {
|
|
code: 'non empty code',
|
|
info: testCase.absentValue,
|
|
},
|
|
})),
|
|
];
|
|
for (const testCase of testCases) {
|
|
it(`${testCase.name} throws "${testCase.expectedError}"`, () => {
|
|
// arrange
|
|
const sut = new CodeSubstituterBuilder().build();
|
|
const { code, info } = testCase.parameters;
|
|
// act
|
|
const act = () => sut.substitute(code, info);
|
|
// assert
|
|
expect(act).to.throw(testCase.expectedError);
|
|
});
|
|
}
|
|
});
|
|
describe('substitutes parameters as expected values', () => {
|
|
// arrange
|
|
const info = new ProjectInformationStub();
|
|
const date = new Date();
|
|
const testCases: Array<{ parameter: string, argument: string }> = [
|
|
{
|
|
parameter: 'homepage',
|
|
argument: info.homepage,
|
|
},
|
|
{
|
|
parameter: 'version',
|
|
argument: info.version.toString(),
|
|
},
|
|
{
|
|
parameter: 'date',
|
|
argument: date.toUTCString(),
|
|
},
|
|
];
|
|
for (const testCase of testCases) {
|
|
it(`substitutes ${testCase.parameter} as expected`, () => {
|
|
const compilerStub = new ExpressionsCompilerStub();
|
|
const sut = new CodeSubstituterBuilder()
|
|
.withCompiler(compilerStub)
|
|
.withDate(date)
|
|
.build();
|
|
// act
|
|
sut.substitute('non empty code', info);
|
|
// assert
|
|
expect(compilerStub.callHistory).to.have.lengthOf(1);
|
|
const parameters = compilerStub.callHistory[0].args[1];
|
|
expect(parameters.hasArgument(testCase.parameter));
|
|
const { argumentValue } = parameters.getArgument(testCase.parameter);
|
|
expect(argumentValue).to.equal(testCase.argument);
|
|
});
|
|
}
|
|
});
|
|
it('returns code as it is', () => {
|
|
// arrange
|
|
const expected = 'expected-code';
|
|
const compilerStub = new ExpressionsCompilerStub();
|
|
const sut = new CodeSubstituterBuilder()
|
|
.withCompiler(compilerStub)
|
|
.build();
|
|
// act
|
|
sut.substitute(expected, new ProjectInformationStub());
|
|
// assert
|
|
expect(compilerStub.callHistory).to.have.lengthOf(1);
|
|
expect(compilerStub.callHistory[0].args[0]).to.equal(expected);
|
|
});
|
|
});
|
|
|
|
class CodeSubstituterBuilder {
|
|
private compiler: IExpressionsCompiler = new ExpressionsCompilerStub();
|
|
|
|
private date = new Date();
|
|
|
|
public withCompiler(compiler: IExpressionsCompiler) {
|
|
this.compiler = compiler;
|
|
return this;
|
|
}
|
|
|
|
public withDate(date: Date) {
|
|
this.date = date;
|
|
return this;
|
|
}
|
|
|
|
public build() {
|
|
return new CodeSubstituter(this.compiler, this.date);
|
|
}
|
|
}
|