Merge branch 'master' into readd_gist_prefix

apps/remix-ide-e2e/src/tests/terminal.test.ts

@@ -119,7 +119,47 @@ module.exports = {
       .waitForElementContainsText('*[data-id="terminalJournal"]', 'newOwner', 60000)
       .waitForElementContainsText('*[data-id="terminalJournal"]', '0xd9145CCE52D386f254917e481eB44e9943F39138', 60000)
   },
-
+  'Run tests using Mocha script and check result logging in the terminal #group4': function (browser: NightwatchBrowser) {
+    browser
+      .addFile('scripts/storage.test.js', { content: storageMochaTests })
+      .pause(1000)
+      .openFile('contracts/1_Storage.sol')
+      .clickLaunchIcon('solidity')
+      .click('*[data-id="compilerContainerCompileBtn"]')
+      .pause(1000) // compile Storage
+      .executeScript('remix.execute(\'scripts/storage.test.js\')')
+      .pause(1000)
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Running tests....')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'storage contract Address:')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', '✓ test initial value')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', '✓ test updating and retrieving updated value')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', '✘ fail test updating and retrieving updated value')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Expected: 55')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Actual: 56')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Message: incorrect number: expected 56 to equal 55')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', '2 passing, 1 failing')
+  },
+  'Run tests using Mocha for a contract with library deployment and check result logging in the terminal #group4': function (browser: NightwatchBrowser) {
+    browser
+      .addFile('scripts/storageWithLib.test.js', { content: storageWithLibMochaTests })
+      .pause(1000)
+      .click('[data-id="treeViewDivtreeViewItemcontracts"]')
+      .addFile('contracts/StorageWithLib.sol', { content: storageWithLibContract })
+      .openFile('contracts/StorageWithLib.sol')
+      .clickLaunchIcon('solidity')
+      .click('*[data-id="compilerContainerCompileBtn"]')
+      .pause(1000) // compile StorageWithLib
+      .executeScript('remix.execute(\'scripts/storageWithLib.test.js\')')
+      .pause(1000)
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Running tests....')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Storage with lib')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'deploying lib:')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', '✘ test library integration by calling a lib method')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Expected: 34')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Actual: 14')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', 'Message: expected \'14\' to equal \'34\'')
+      .waitForElementContainsText('*[data-id="terminalJournal"]', '0 passing, 1 failing')
+  },
   'Should print hardhat logs #group4': function (browser: NightwatchBrowser) {
     browser
       .click('*[data-id="terminalClearConsole"]') // clear the terminal
@@ -261,6 +301,132 @@ const deployWithEthersJs = `
     }
 })()`
 
+const storageMochaTests = `
+const { expect } = require("chai");
+
+describe("Storage with lib", function () {
+  it("test initial value", async function () {
+    // Make sure contract is compiled and artifacts are generated
+    const metadata = JSON.parse(await remix.call('fileManager', 'getFile', 'contracts/artifacts/Storage.json'))
+    const signer = (new ethers.providers.Web3Provider(web3Provider)).getSigner()
+    let Storage = new ethers.ContractFactory(metadata.abi, metadata.data.bytecode.object, signer);
+    let storage = await Storage.deploy();
+    console.log('storage contract Address: ' + storage.address);
+    await storage.deployed()
+    expect((await storage.retrieve()).toNumber()).to.equal(0);
+  });
+
+  it("test updating and retrieving updated value", async function () {
+    const metadata = JSON.parse(await remix.call('fileManager', 'getFile', 'contracts/artifacts/Storage.json'))
+    const signer = (new ethers.providers.Web3Provider(web3Provider)).getSigner()
+    let Storage = new ethers.ContractFactory(metadata.abi, metadata.data.bytecode.object, signer);
+    let storage = await Storage.deploy();
+    await storage.deployed()
+    const setValue = await storage.store(56);
+    await setValue.wait();
+    expect((await storage.retrieve()).toNumber()).to.equal(56);
+  });
+
+  it("fail test updating and retrieving updated value", async function () {
+    const metadata = JSON.parse(await remix.call('fileManager', 'getFile', 'contracts/artifacts/Storage.json'))
+    const signer = (new ethers.providers.Web3Provider(web3Provider)).getSigner()
+    let Storage = new ethers.ContractFactory(metadata.abi, metadata.data.bytecode.object, signer);
+    let storage = await Storage.deploy();
+    await storage.deployed()
+    const setValue = await storage.store(56);
+    await setValue.wait();
+    expect((await storage.retrieve()).toNumber(), 'incorrect number').to.equal(55);
+  });
+});`
+
+const storageWithLibContract = `
+// SPDX-License-Identifier: GPL-3.0
+
+pragma solidity >=0.7.0 <0.9.0;
+
+library Lib {
+    function test () public view returns (uint) {
+        return 14;
+    }
+}
+/**
+ * @title Storage
+ * @dev Store & retrieve value inr a variable
+ */
+contract StorageWithLib {
+
+    uint256 number;
+
+    /**
+     * @dev Store valrue in variable
+     * @param num value to store
+     */
+    function store(uint256 num) public {        
+        number = num;        
+    }
+
+    /**
+     * @dev Return value 
+     * @return value of 'number'
+     */
+    function retrieve() public view returns (uint256){
+        return number;
+    }
+
+    function getFromLib() public view returns (uint) {
+        return Lib.test();
+    }
+}
+`
+
+const storageWithLibMochaTests = `
+const { expect } = require("chai");
+const { ethers } = require("hardhat");
+
+describe("Storage", function () {
+    it("test library integration by calling a lib method", async function () {
+        const metadataLib = JSON.parse(await remix.call('fileManager', 'readFile', 'contracts/artifacts/Lib.json'))
+        console.log('deploying lib:')
+        const artifactLib  = {
+            contractName: 'Lib',
+            sourceName: 'contracts/StorageWithLib.sol',
+            abi: metadataLib.abi,
+            bytecode: '0x' + metadataLib.data.bytecode.object,
+            deployedBytecode:  '0x' + metadataLib.data.deployedBytecode.object,
+            linkReferences: metadataLib.data.bytecode.linkReferences,
+            deployedLinkReferences: metadataLib.data.deployedBytecode.linkReferences,
+        }
+
+        const optionsLib = {}
+        const factoryLib = await ethers.getContractFactoryFromArtifact(artifactLib, optionsLib)
+        const lib = await factoryLib.deploy();
+        await lib.deployed()
+
+        const metadata = JSON.parse(await remix.call('fileManager', 'readFile', 'contracts/artifacts/StorageWithLib.json'))
+        const artifact  = {
+            contractName: 'StorageWithLib',
+            sourceName: 'contracts/StorageWithLib.sol',
+            abi: metadata.abi,
+            bytecode: '0x' + metadata.data.bytecode.object,
+            deployedBytecode:  '0x' + metadata.data.deployedBytecode.object,
+            linkReferences: metadata.data.bytecode.linkReferences,
+            deployedLinkReferences: metadata.data.deployedBytecode.linkReferences,
+        }
+        const options = {
+            libraries: { 
+                'Lib': lib.address
+            }
+        }
+        
+        const factory = await ethers.getContractFactoryFromArtifact(artifact, options)
+        const storage = await factory.deploy();
+        await storage.deployed()
+        const storeValue = await storage.store(333);
+        await storeValue.wait();
+        expect((await storage.getFromLib()).toString()).to.equal('34');
+    });
+});`
+
 const hardhatLog = `
 // SPDX-License-Identifier: GPL-3.0
 

apps/remix-ide/src/app/plugins/storage.ts

@@ -39,7 +39,7 @@ export class StoragePlugin extends Plugin {
       if (!localStorage.hasOwnProperty(_x)) {
         continue
       }
-      _xLen = ((localStorage[_x].length + _x.length) * 2)
+      _xLen = ((localStorage[_x].length + _x.length))
       _lsTotal += _xLen
     }
     return (_lsTotal / 1024).toFixed(2)

libs/remix-lib/src/execution/eventsDecoder.ts

@@ -56,21 +56,22 @@ export class EventsDecoder {
     return eventsABI
   }
 
-  _event (hash: string, eventsABI: Record<string, unknown>, contractName: string) {
-    const events = eventsABI[contractName]
-    if (!events) return null
-
-    if (events[hash]) {
-      const event = events[hash]
-      for (const input of event.inputs) {
-        if (input.type === 'function') {
-          input.type = 'bytes24'
-          input.baseType = 'bytes24'
+  _event (hash, eventsABI) {
+    // get all the events responding to that hash.
+    const contracts = []
+    for (const k in eventsABI) {
+      if (eventsABI[k][hash]) {
+        const event = eventsABI[k][hash]
+        for (const input of event.inputs) {
+          if (input.type === 'function') {
+            input.type = 'bytes24'
+            input.baseType = 'bytes24'
+          }
         }
+        contracts.push(event)
       }
-      return event
     }
-    return null
+    return contracts
   }
 
   _stringifyBigNumber (value): string {
@@ -95,16 +96,23 @@ export class EventsDecoder {
       // [address, topics, mem]
       const log = logs[i]
       const topicId = log.topics[0]
-      const eventAbi = this._event(topicId.replace('0x', ''), eventsABI, contractName)
-      if (eventAbi) {
-        const decodedlog = eventAbi.abi.parseLog(log)
-        const decoded = {}
-        for (const v in decodedlog.args) {
-          decoded[v] = this._stringifyEvent(decodedlog.args[v])
+      const eventAbis = this._event(topicId.replace('0x', ''), eventsABI)
+      for (const eventAbi of eventAbis) {
+        try {
+          if (eventAbi) {
+            const decodedlog = eventAbi.abi.parseLog(log)
+            const decoded = {}
+            for (const v in decodedlog.args) {
+              decoded[v] = this._stringifyEvent(decodedlog.args[v])
+            }
+            events.push({ from: log.address, topic: topicId, event: eventAbi.event, args: decoded })
+          } else {
+            events.push({ from: log.address, data: log.data, topics: log.topics })
+          }
+          break // if one of the iteration is successful
+        } catch (e) {
+          continue
         }
-        events.push({ from: log.address, topic: topicId, event: eventAbi.event, args: decoded })
-      } else {
-        events.push({ from: log.address, data: log.data, topics: log.topics })
       }
     }
     cb(null, { decoded: events, raw: logs })

libs/remix-ui/search/src/lib/context/context.tsx

@@ -202,18 +202,28 @@ export const SearchProvider = ({
     }
   }, [])
 
+  //*.sol, **/*.txt, contracts/*
+  const setGlobalExpression = (paths: string) => {
+    const results = []
+    paths.split(',').forEach(path => {
+      path = path.trim()
+      if(path.startsWith('*.')) path = path.replace(/(\*\.)/g, '**/*.')
+      if(path.endsWith('/*') && !path.endsWith('/**/*')) path = path.replace(/(\*)/g, '**/*.*')
+      results.push(path)
+    })
+    return results
+  }
+
   useEffect(() => {
     if (state.find) {
       (async () => {
         const files = await getDirectory('/', plugin)
         const pathFilter: any = {}
         if (state.include){
-          const includeWithGlobalExpression = state.include.replaceAll(/(?<!\/)(\*\.)/g, '**/*.')
-          pathFilter.include = includeWithGlobalExpression.split(',').map(i => i.trim())
+          pathFilter.include = setGlobalExpression(state.include)
         }
         if (state.exclude){
-          const excludeWithGlobalExpression = state.exclude.replaceAll(/(?<!\/)(\*\.)/g, '**/*.')
-          pathFilter.exclude = excludeWithGlobalExpression.split(',').map(i => i.trim())
+          pathFilter.exclude = setGlobalExpression(state.exclude)
         }
         const filteredFiles = files.filter(filePathFilter(pathFilter)).map(file => {
           const r: SearchResult = {