Browse Source

test uppy.io instead of localhost. Safar & edge are still having issues

Kevin van Zonneveld 9 years ago
parent
commit
a24a934e4d
2 changed files with 6 additions and 2 deletions
  1. 1 0
      CHANGELOG.md
  2. 5 2
      test/acceptance/index.js

+ 1 - 0
CHANGELOG.md

@@ -63,6 +63,7 @@ Scheduled to be released: June 03, 2016
 - [ ] progressdrawer: preview icons for all (@arturi)
 - [ ] server: Implement the `SERVER-PLAN.md` so that Google Drive files are actually uploaded to the endpoint
 - [ ] test: Get IE4 on windows 3.11 to run Uppy and see it fall back to regular form upload (@arturi)
+- [ ] test: Make Edge and Safari work via the tunnel so we can test localhost instead of uppy.io, and test the current build, vs the previous deploy that way
 - [ ] website: scrollbars on code samples (can’t reproduce!) (@arturi)
 
 ## 0.0.5

+ 5 - 2
test/acceptance/index.js

@@ -30,8 +30,11 @@ var isRemoteTest = !!accessKey
 
 var host = localHost
 if (isTravisTest) {
-  // We have a tunnel via the saucelabs addon on Travis
-  host = localHost
+  // @todo This should become localhost to utilize the Travis saucelabs addon tunnel
+  // But it seems Edge and Safar fail on that right now, so targeting uppy.io instead.
+  // That is unideal, as we are then testing a previous deploy, and not the current build
+  // host = localHost
+  host = remoteHost
 } else if (isRemoteTest) {
   // We're not too sure about a working tunnel otherwise, best just test uppy.io
   host = remoteHost