Merge pull request #2691 from ethereum/bumpRemixLib

Bump remix lib

package.json

@@ -55,13 +55,13 @@
     "npm-merge-driver": "^2.3.5",
     "npm-run-all": "^4.0.2",
     "onchange": "^3.2.1",
-    "remix-analyzer": "0.3.23",
+    "remix-analyzer": "0.5.0-beta.2",
-    "remix-debug": "0.3.27",
+    "remix-debug": "0.3.28",
-    "remix-lib": "0.4.23",
+    "remix-lib": "0.4.24",
-    "remix-solidity": "0.3.26",
+    "remix-simulator": "0.1.9-beta.2",
+    "remix-solidity": "0.3.27",
     "remix-tabs": "1.0.48",
-    "remix-tests": "0.1.29",
+    "remix-tests": "0.1.30",
-    "remix-simulator": "0.1.9-beta.1",
     "remixd": "0.1.8-alpha.10",
     "request": "^2.83.0",
     "rimraf": "^2.6.1",

src/app/tabs/staticanalysis/staticAnalysisView.js

@@ -182,11 +182,16 @@ staticAnalysisView.prototype.renderModules = function () {
 
 module.exports = staticAnalysisView
 
+/**
+ * @dev Process & categorize static analysis modules to show them on UI
+ * @param arr list of static analysis modules received from remix-analyzer module
+ */
 function preProcessModules (arr) {
-  return arr.map((item, i) => {
+  return arr.map((Item, i) => {
-    item['_index'] = i
+    const itemObj = new Item()
-    item.categoryDisplayName = item.category.displayName
+    itemObj['_index'] = i
-    item.categoryId = item.category.id
+    itemObj.categoryDisplayName = itemObj.category.displayName
-    return item
+    itemObj.categoryId = itemObj.category.id
+    return itemObj
   })
 }

src/app/tabs/test-tab.js

@@ -121,8 +121,12 @@ module.exports = class TestTab extends ViewPlugin {
 
   updateFinalResult (_errors, result, filename) {
     this.testsSummary.hidden = false
-    if (_errors) {
+    if (_errors && _errors.errors) {
-      _errors.forEach((err) => this.renderer.error(err.formattedMessage || err.message, this.testsSummary, {type: err.severity}))
+      _errors.errors.forEach((err) => this.renderer.error(err.formattedMessage || err.message, this.testsSummary, {type: err.severity}))
+      return
+    } else if (_errors && !_errors.errors && !Array.isArray(_errors)) {
+      // To track error like this: https://github.com/ethereum/remix/pull/1438
+      this.renderer.error(_errors.formattedMessage || _errors.message, this.testsSummary, {type: 'error'})
       return
     }
     this.testsSummary.appendChild(yo`<div class=${css.summaryTitle}> ${filename} </div>`)

test-browser/tests/recorder.js

@@ -70,7 +70,7 @@ var sources = [{'browser/testRecorder.sol': {content: `contract testRecorder {
 
 var records = `{
   "accounts": {
-    "account{3}": "0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c"
+    "account{10}": "0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c"
   },
   "linkReferences": {
     "testLib": "created{1512830014773}"
@@ -87,7 +87,7 @@ var records = `{
         "linkReferences": {},
         "inputs": "()",  
         "type": "constructor",
-        "from": "account{3}"
+        "from": "account{10}"
       }
     },
     {
@@ -113,7 +113,7 @@ var records = `{
         "name": "",
         "type": "constructor",
         "inputs": "(uint256)",
-        "from": "account{3}"
+        "from": "account{10}"
       }
     },
     {
@@ -129,7 +129,7 @@ var records = `{
         "name": "set",
         "inputs": "(uint256,address)",
         "type": "function",
-        "from": "account{3}"
+        "from": "account{10}"
       }
     }
   ],
@@ -228,7 +228,7 @@ var records = `{
 
 var scenario = {
   'accounts': {
-    'account{3}': '0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c'
+    'account{10}': '0xCA35b7d915458EF540aDe6068dFe2F44E8fa733c'
   },
   'linkReferences': {},
   'transactions': [
@@ -246,7 +246,7 @@ var scenario = {
         'name': '',
         'type': 'constructor',
         'inputs': '(uint256)',
-        'from': 'account{3}'
+        'from': 'account{10}'
       }
     },
     {
@@ -261,7 +261,7 @@ var scenario = {
         'name': 'set',
         'inputs': '(uint256)',
         'type': 'function',
-        'from': 'account{3}'
+        'from': 'account{10}'
       }
     }
   ],