diff --git a/package-lock.json b/package-lock.json index 1a1083c1..40600517 100644 --- a/package-lock.json +++ b/package-lock.json @@ -3108,7 +3108,8 @@ }, "node_modules/fast-glob": { "version": "3.3.2", - "license": "MIT", + "resolved": "https://registry.npmjs.org/fast-glob/-/fast-glob-3.3.2.tgz", + "integrity": "sha512-oX2ruAFQwf/Orj8m737Y5adxDQO0LAB7/S5MnxCdTNDd4p6BsyIVsv9JQsATbTSq8KHRpLwIHbVlUNatxd+1Ow==", "dependencies": { "@nodelib/fs.stat": "^2.0.2", "@nodelib/fs.walk": "^1.2.3", @@ -6538,7 +6539,7 @@ }, "packages/code-analyzer-flowtest-engine": { "name": "@salesforce/code-analyzer-flowtest-engine", - "version": "0.14.0", + "version": "0.14.1", "license": "BSD-3-Clause", "dependencies": { "@salesforce/code-analyzer-engine-api": "0.14.0", diff --git a/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/__main__.py b/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/__main__.py index 312a38e4..58212365 100644 --- a/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/__main__.py +++ b/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/__main__.py @@ -176,6 +176,8 @@ def parse_args(my_args: list[str], default: str = None) -> argparse.Namespace: type=check_not_exist) parser.add_argument("--debug", action='store_true', help="whether to set logging level to debug") + parser.add_argument("--no_log", action='store_true', help="disables logging") + """ Options for crawl-spec generation @@ -248,12 +250,16 @@ def main(argv: list[str] = None) -> str | None: return # logging - if args.debug is True: - log_level = logging.DEBUG + if args.no_log is True: + logging.getLogger().setLevel(logging.CRITICAL + 1) else: - log_level = logging.WARNING + if args.debug is True: + log_level = logging.DEBUG + + else: + log_level = logging.WARNING - setup_logger(level=log_level, log_file=args.log_file) + setup_logger(level=log_level, log_file=args.log_file) if args.query_path is not None and args.query_class is None: raise argparse.ArgumentTypeError("A query_class must be provided if a query_path is set") diff --git a/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/version.py b/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/version.py index 09b2f79e..d36ada1e 100644 --- a/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/version.py +++ b/packages/code-analyzer-flowtest-engine/FlowTest/flowtest/version.py @@ -1,5 +1,5 @@ -# Store the version here so: +1# Store the version here so: # 1) we don't load dependencies by storing it in __init__.py # 2) we can import it in setup.py for the same reason # 3) we can import it into your module -__version__ = '0.7.0-ALPHA' +__version__ = '0.7.1' diff --git a/packages/code-analyzer-flowtest-engine/src/python/FlowTestCommandWrapper.ts b/packages/code-analyzer-flowtest-engine/src/python/FlowTestCommandWrapper.ts index 55bc0910..99d45351 100644 --- a/packages/code-analyzer-flowtest-engine/src/python/FlowTestCommandWrapper.ts +++ b/packages/code-analyzer-flowtest-engine/src/python/FlowTestCommandWrapper.ts @@ -48,6 +48,7 @@ export class RunTimeFlowTestCommandWrapper implements FlowTestCommandWrapper { const pythonArgs: string[] = [ '-m', 'flowtest', + '--no_log', '-j', tmpFile, '-d', diff --git a/packages/code-analyzer-flowtest-engine/test/python/FlowTestCommandWrapper.test.ts b/packages/code-analyzer-flowtest-engine/test/python/FlowTestCommandWrapper.test.ts index d20d803d..66d84567 100644 --- a/packages/code-analyzer-flowtest-engine/test/python/FlowTestCommandWrapper.test.ts +++ b/packages/code-analyzer-flowtest-engine/test/python/FlowTestCommandWrapper.test.ts @@ -55,6 +55,12 @@ describe('FlowTestCommandWrapper implementations', () => { it('Correctly parses status updates from stdout', () => { expect(completionPercentages).toEqual([0, 25, 50, 75]); }); + + it('Generates no log file', async () => { + const logFileMatcher = /\.flowtest_log_.+\.log/; + const logFiles = (await fs.readdir('.')).filter(f => f.match(logFileMatcher)); + expect(logFiles).toHaveLength(0); + }); }); describe('Failure Modes', () => {