From 2149d34ce47f1de89826634d11f2c7b8952be6b6 Mon Sep 17 00:00:00 2001 From: yann300 Date: Tue, 24 Aug 2021 14:13:33 +0200 Subject: [PATCH] e2e test: running static analysis with remixd and hardhat --- apps/remix-ide-e2e/src/tests/remixd.test.ts | 22 +++++++++++++++++++ .../src/lib/remix-ui-static-analyser.tsx | 2 +- 2 files changed, 23 insertions(+), 1 deletion(-) diff --git a/apps/remix-ide-e2e/src/tests/remixd.test.ts b/apps/remix-ide-e2e/src/tests/remixd.test.ts index 24a6ce9b96..e76561b20f 100644 --- a/apps/remix-ide-e2e/src/tests/remixd.test.ts +++ b/apps/remix-ide-e2e/src/tests/remixd.test.ts @@ -34,6 +34,13 @@ const sources = [ }, { 'test_import_node_modules_with_github_import.sol': { content: 'import "openzeppelin-solidity/contracts/sample.sol";' } + }, + { + 'test_static_analysis_with_remixd_and_hardhat.sol': { + content: ` + import "hardhat/console.sol"; + contract test5 { function get () public returns (uint) { return 8; }}` + } } ] @@ -71,6 +78,21 @@ module.exports = { .setSolidityCompilerVersion('soljson-v0.8.0+commit.c7dfd78e.js') // open-zeppelin moved to pragma ^0.8.0 .testContracts('test_import_node_modules_with_github_import.sol', sources[4]['test_import_node_modules_with_github_import.sol'], ['ERC20', 'test11']) }, + 'Static Analysis run with remixd': function (browser) { + browser.testContracts('Untitled.sol', sources[5]['test_static_analysis_with_remixd_and_hardhat'], ['test5']) + .clickLaunchIcon('solidityStaticAnalysis') + .click('#staticanalysisButton button') + .waitForElementPresent('#staticanalysisresult .warning', 2000, true, function () { + browser + .click('[data-id="staticAnalysisModuleMiscellaneous1"') + .click('[data-id="staticAnalysisModuleMiscellaneous1"') + .getEditorValue((content) => { + browser.assert.ok(content.indexOf( + 'function _sendLogPayload(bytes memory payload) private view {') !== -1, + 'code has not been loaded') + }) + }) + }, 'Run git status': '' + function (browser) { browser diff --git a/libs/remix-ui/static-analyser/src/lib/remix-ui-static-analyser.tsx b/libs/remix-ui/static-analyser/src/lib/remix-ui-static-analyser.tsx index c16de17803..ace945e95e 100644 --- a/libs/remix-ui/static-analyser/src/lib/remix-ui-static-analyser.tsx +++ b/libs/remix-ui/static-analyser/src/lib/remix-ui-static-analyser.tsx @@ -471,7 +471,7 @@ export const RemixUiStaticAnalyser = (props: RemixUiStaticAnalyserProps) => { {element[0]} {element[1]['map']((x, i) => ( // eslint-disable-line dot-notation x.hasWarning ? ( // eslint-disable-next-line dot-notation -
+