Show “loading” when checking auth too

addresses this comment
https://github.com/transloadit/uppy/pull/265#issuecomment-317403651
@goto-bus-stop

src/generic-provider-views/AuthView.js

@@ -1,15 +1,21 @@
 const html = require('yo-yo')
 const onload = require('on-load')
+const LoaderView = require('./Loader')
 
 module.exports = (props) => {
   const demoLink = props.demo ? html`<button class="UppyProvider-authBtnDemo" onclick=${props.handleDemoAuth}>Proceed with Demo Account</button>` : null
-  return onload(html`
+  const AuthBlock = () => html`
     <div class="UppyProvider-auth">
-      <h1 class="UppyProvider-authTitle">
-        Please authenticate with <span class="UppyProvider-authTitleName">${props.pluginName}</span><br> to select files
-      </h1>
+      <h1 class="UppyProvider-authTitle">Please authenticate with <span class="UppyProvider-authTitleName">${props.pluginName}</span><br> to select files</h1>
       <button type="button" class="UppyProvider-authBtn" onclick=${props.handleAuth}>Authenticate</button>
       ${demoLink}
     </div>
-  `, props.checkAuth, null, `auth${props.pluginName}`)
+  `
+  return onload(html`
+    <div style="height: 100%;">
+      ${props.checkAuthInProgress
+        ? LoaderView()
+        : AuthBlock()
+      }
+    </div>`, props.checkAuth, null, `auth${props.pluginName}`)
 }

src/generic-provider-views/index.js

@@ -96,9 +96,16 @@ module.exports = class View {
   }
 
   checkAuth () {
+    this.updateState({ checkAuthInProgress: true })
     this.Provider.checkAuth()
-      .then(this.plugin.onAuth)
-      .catch(this.handleError)
+      .then((authenticated) => {
+        this.updateState({ checkAuthInProgress: false })
+        this.plugin.onAuth(authenticated)
+      })
+      .catch((err) => {
+        this.updateState({ checkAuthInProgress: false })
+        this.handleError(err)
+      })
   }
 
   /**
@@ -163,7 +170,7 @@ module.exports = class View {
         tagFile.preview = this.plugin.getItemThumbnailUrl(file)
       }
       this.plugin.core.log('Adding remote file')
-      this.plugin.core.emitter.emit('core:file-add', tagFile)
+      this.plugin.core.addFile(tagFile)
     })
   }
 
@@ -346,7 +353,7 @@ module.exports = class View {
   }
 
   render (state) {
-    const { authenticated, loading } = state[this.plugin.stateId]
+    const { authenticated, checkAuthInProgress, loading } = state[this.plugin.stateId]
 
     if (loading) {
       return LoaderView()
@@ -358,7 +365,8 @@ module.exports = class View {
         demo: this.plugin.opts.demo,
         checkAuth: this.checkAuth,
         handleAuth: this.handleAuth,
-        handleDemoAuth: this.handleDemoAuth
+        handleDemoAuth: this.handleDemoAuth,
+        checkAuthInProgress: checkAuthInProgress
       })
     }