From a7e597bad0e60aa23b7be0a89e7ada549ad30759 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 9 Jan 2020 10:19:16 +0100
Subject: [PATCH 1/3] Export to Gist

if "id" is not defined, it is not a gist update but a creation. we have to take the file from the browser explorer
---
 src/app/files/file-explorer.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 899da13702..8eb9703692 100644
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -470,7 +470,8 @@ fileExplorer.prototype.toGist = function (id) {
     return data.files || []
   }
 
-  this.packageFiles(this.files, 'browser/gists/' + id, (error, packaged) => {
+  const folder = id ? 'browser/gists/' + id : 'browser/'
+  this.packageFiles(this.files, folder, (error, packaged) => {
     if (error) {
       console.log(error)
       modalDialogCustom.alert('Failed to create gist: ' + error)

From a80bb0420888aa0ce4eeb5151cc56e0e465a6a38 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Fri, 10 Jan 2020 12:12:17 +0100
Subject: [PATCH 2/3] add browser tests

---
 package.json                                  |  1 +
 src/app/files/file-explorer.js                |  4 +-
 test-browser/commands/executeScript.js        |  1 +
 test-browser/commands/getModalBody.js         | 16 +++++++
 .../commands/modalFooterCancelClick.js        | 17 +++++++
 test-browser/commands/modalFooterOKClick.js   |  2 +-
 test-browser/commands/switchFile.js           |  1 +
 test-browser/tests/gist.js                    | 45 +++++++++++++++++++
 8 files changed, 84 insertions(+), 3 deletions(-)
 create mode 100644 test-browser/commands/getModalBody.js
 create mode 100644 test-browser/commands/modalFooterCancelClick.js
 create mode 100644 test-browser/tests/gist.js

diff --git a/package.json b/package.json
index ec518ecf41..ea5e12e8d9 100644
--- a/package.json
+++ b/package.json
@@ -170,6 +170,7 @@
     "nightwatch_local_solidityUnittests": "nightwatch ./test-browser/tests/solidityUnittests.js --config nightwatch.js --env chrome ",
     "nightwatch_local_remixd": "nightwatch ./test-browser/tests/remix.js --config nightwatch.js --env chrome ",
     "nightwatch_local_console": "nightwatch ./test-browser/tests/console.js --config nightwatch.js --env chrome ",
+    "nightwatch_local_gist": "nightwatch ./test-browser/tests/gist.js --config nightwatch.js --env chrome ",
     "onchange": "onchange build/app.js -- npm-run-all lint",
     "prepublish": "mkdirp build; npm-run-all -ls downloadsolc_root build",
     "remixd": "remixd -s ./contracts --remix-ide http://127.0.0.1:8080",
diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 8eb9703692..340cd90e01 100644
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -633,7 +633,7 @@ fileExplorer.prototype.renderMenuItems = function () {
     items = this.menuItems.map(({action, title, icon}) => {
       if (action === 'uploadFile') {
         return yo`
-          <label class="${icon} ${css.newFile}"  title="${title}">
+          <label id=${action} class="${icon} ${css.newFile}"  title="${title}">
             <input type="file" onchange=${(event) => {
               event.stopPropagation()
               this.uploadFile(event)
@@ -642,7 +642,7 @@ fileExplorer.prototype.renderMenuItems = function () {
         `
       } else {
         return yo`
-        <span onclick=${(event) => { event.stopPropagation(); this[ action ]() }} class="newFile ${icon} ${css.newFile}" title=${title}></span>
+        <span id=${action} onclick=${(event) => { event.stopPropagation(); this[ action ]() }} class="newFile ${icon} ${css.newFile}" title=${title}></span>
         `
       }
     })
diff --git a/test-browser/commands/executeScript.js b/test-browser/commands/executeScript.js
index c668411726..eb673dad69 100644
--- a/test-browser/commands/executeScript.js
+++ b/test-browser/commands/executeScript.js
@@ -6,6 +6,7 @@ class ExecuteScript extends EventEmitter {
       .click('#terminalCli')
       .keys(script)
       .keys(this.api.Keys.ENTER)
+      .keys(this.api.Keys.ENTER) // that's a bug... sometimes we need to press 2 times to execute a command
       .perform(() => {
         this.emit('complete')
       })
diff --git a/test-browser/commands/getModalBody.js b/test-browser/commands/getModalBody.js
new file mode 100644
index 0000000000..5467275c12
--- /dev/null
+++ b/test-browser/commands/getModalBody.js
@@ -0,0 +1,16 @@
+const EventEmitter = require('events')
+
+class GetModalBody extends EventEmitter {
+  command (callback) {
+    this.api.waitForElementVisible('.modal-body')
+    .getText('.modal-body', (result) => {
+      console.log(result)
+      callback(result.value, () => {
+        this.emit('complete')
+      })
+    })
+    return this
+  }
+}
+
+module.exports = GetModalBody
diff --git a/test-browser/commands/modalFooterCancelClick.js b/test-browser/commands/modalFooterCancelClick.js
new file mode 100644
index 0000000000..86adbc63b3
--- /dev/null
+++ b/test-browser/commands/modalFooterCancelClick.js
@@ -0,0 +1,17 @@
+const EventEmitter = require('events')
+
+class ModalFooterOKClick extends EventEmitter {
+  command () {
+    this.api.waitForElementVisible('#modal-footer-cancel').perform((client, done) => {
+      this.api.execute(function () {
+        document.querySelector('#modal-footer-cancel').click()
+      }, [], (result) => {
+        done()
+        this.emit('complete')
+      })
+    })
+    return this
+  }
+}
+
+module.exports = ModalFooterOKClick
diff --git a/test-browser/commands/modalFooterOKClick.js b/test-browser/commands/modalFooterOKClick.js
index 5a6dcc61d1..b6c0db5b7c 100644
--- a/test-browser/commands/modalFooterOKClick.js
+++ b/test-browser/commands/modalFooterOKClick.js
@@ -2,7 +2,7 @@ const EventEmitter = require('events')
 
 class ModalFooterOKClick extends EventEmitter {
   command () {
-    this.api.perform((client, done) => {
+    this.api.waitForElementVisible('#modal-footer-ok').perform((client, done) => {
       this.api.execute(function () {
         document.querySelector('#modal-footer-ok').click()
       }, [], (result) => {
diff --git a/test-browser/commands/switchFile.js b/test-browser/commands/switchFile.js
index 8d48a85495..004697b8fe 100644
--- a/test-browser/commands/switchFile.js
+++ b/test-browser/commands/switchFile.js
@@ -15,6 +15,7 @@ class SwitchFile extends EventEmitter {
 // click on fileExplorer can toggle it. We go through settings to be sure FE is open
 function switchFile (browser, name, done) {
   browser.clickLaunchIcon('settings').clickLaunchIcon('fileExplorers')
+      .waitForElementVisible('li[key="' + name + '"]')
       .click('li[key="' + name + '"]')
       .pause(2000)
       .perform(() => {
diff --git a/test-browser/tests/gist.js b/test-browser/tests/gist.js
new file mode 100644
index 0000000000..b3153f5aa3
--- /dev/null
+++ b/test-browser/tests/gist.js
@@ -0,0 +1,45 @@
+'use strict'
+const init = require('../helpers/init')
+const sauce = require('./sauce')
+// 99266d6da54cc12f37f11586e8171546c7700d67
+
+module.exports = {
+  before: function (browser, done) {
+    init(browser, done)
+  },
+  'UploadToGists': function (browser) {
+      /*
+       - set the access token
+       - publish to gist
+       - retrieve the gist
+       - switch to a file in the new gist
+      */
+    console.log('token', process.env.gist_token)
+    browser
+    .waitForElementVisible('#icon-panel', 10000)
+    .clickLaunchIcon('settings')
+    .setValue('#gistaccesstoken', process.env.gist_token)
+    .click('#savegisttoken')
+    .clickLaunchIcon('fileExplorers')
+    .click('#publishToGist')
+    .modalFooterOKClick()
+    .getModalBody((value, done) => {
+      const reg = /gist.github.com\/([^.]+)/
+      const id = value.match(reg)
+      console.log('gist regex', id)
+      if (!id) {
+        browser.assert.fail('cannot get the gist id', '', '')
+      } else {
+        let gistid = id[1]
+        browser
+          .modalFooterCancelClick()
+          .executeScript(`remix.loadgist('${gistid}')`)
+          .switchFile('browser/gists')
+          .switchFile(`browser/gists/${gistid}`)
+          .switchFile(`browser/gists/${gistid}/1_Storage.sol`)
+          .end()
+      }
+    })
+  },
+  tearDown: sauce
+}

From 8b7cdfeef31418407c426c17c4c25d82e1dcd3bf Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Mon, 13 Jan 2020 10:44:31 +0100
Subject: [PATCH 3/3] add comments

---
 src/app/files/file-explorer.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 340cd90e01..9a74ffee0d 100644
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -470,6 +470,7 @@ fileExplorer.prototype.toGist = function (id) {
     return data.files || []
   }
 
+  // If 'id' is not defined, it is not a gist update but a creation so we have to take the files from the browser explorer.
   const folder = id ? 'browser/gists/' + id : 'browser/'
   this.packageFiles(this.files, folder, (error, packaged) => {
     if (error) {