Skip to content

Commit

Permalink
update: fix issue with insufficient buffer size for cli test
Browse files Browse the repository at this point in the history
  • Loading branch information
Brianspha committed Jan 16, 2025
1 parent d7f56f5 commit ac918b8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
3 changes: 2 additions & 1 deletion crates/solidity/src/tests/cli-tests/src/helper.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ export const executeCommand = (command: string, stdin?: string): CommandResult =
const process = spawnSync(command, [], {
input: stdin,
shell: true,
encoding: 'utf-8'
encoding: 'utf8',
maxBuffer: 30 * 1024 * 1024
});

return {
Expand Down
5 changes: 1 addition & 4 deletions crates/solidity/src/tests/cli-tests/tests/common.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import * as shell from 'shelljs';
import * as path from 'path';



//id1762
describe("Run resolc without any options", () => {
const command = 'resolc';
Expand Down Expand Up @@ -185,9 +184,7 @@ describe("Standard JSON compilation with path options", () => {
});

it("Compiler run successful without emiting warnings", () => {
const cleanOutput = result.output.trim().replace(/[\r\n]+/g, '');
const parsedResults = JSON.parse(cleanOutput);
console.log("result.output: ", parsedResults)
const parsedResults = JSON.parse(result.output)
expect(parsedResults.errors.filter((error: { type: string; }) => error.type != 'Warning')).toEqual([]);
});
});
Expand Down

0 comments on commit ac918b8

Please sign in to comment.