[ARVADOS] updated: 1.3.0-1404-g4512b2324

Git user git at public.curoverse.com
Thu Jul 25 18:28:27 UTC 2019


Summary of changes:
 apps/workbench/test/integration/anonymous_access_test.rb |  2 --
 apps/workbench/test/integration/collections_test.rb      |  2 --
 apps/workbench/test/integration/download_test.rb         |  2 --
 apps/workbench/test/integration/jobs_test.rb             |  3 ---
 sdk/python/tests/run_test_server.py                      | 13 ++++++++++---
 services/api/config/application.default.yml              |  1 -
 6 files changed, 10 insertions(+), 13 deletions(-)

       via  4512b2324d203673a7b76927631a9bac4361d0de (commit)
      from  35ba053c83b7ad18e1a336d50d3a8d5a53adce9f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 4512b2324d203673a7b76927631a9bac4361d0de
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Thu Jul 25 14:28:08 2019 -0400

    14717: More test fixes.  Configure correct websockets address.
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/apps/workbench/test/integration/anonymous_access_test.rb b/apps/workbench/test/integration/anonymous_access_test.rb
index d3f08c62b..0842635f6 100644
--- a/apps/workbench/test/integration/anonymous_access_test.rb
+++ b/apps/workbench/test/integration/anonymous_access_test.rb
@@ -117,8 +117,6 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
   end
 
   test 'view file' do
-    use_keep_web_config
-
     magic = rand(2**512).to_s 36
     owner = api_fixture('groups')['anonymously_accessible_project']['uuid']
     col = upload_data_and_get_collection(magic, 'admin', "Hello\\040world.txt", owner)
diff --git a/apps/workbench/test/integration/collections_test.rb b/apps/workbench/test/integration/collections_test.rb
index 024a14605..87d3d678d 100644
--- a/apps/workbench/test/integration/collections_test.rb
+++ b/apps/workbench/test/integration/collections_test.rb
@@ -53,8 +53,6 @@ class CollectionsTest < ActionDispatch::IntegrationTest
   end
 
   test "can download an entire collection with a reader token" do
-    use_keep_web_config
-
     token = api_token('active')
     data = "foo\nfile\n"
     datablock = `echo -n #{data.shellescape} | ARVADOS_API_TOKEN=#{token.shellescape} arv-put --no-progress --raw -`.strip
diff --git a/apps/workbench/test/integration/download_test.rb b/apps/workbench/test/integration/download_test.rb
index e2d3dc8ef..6ae9f2927 100644
--- a/apps/workbench/test/integration/download_test.rb
+++ b/apps/workbench/test/integration/download_test.rb
@@ -9,8 +9,6 @@ class DownloadTest < ActionDispatch::IntegrationTest
   @@wrote_test_data = false
 
   setup do
-    use_keep_web_config
-
     # Make sure Capybara can download files.
     need_selenium 'for downloading', :selenium_with_download
     DownloadHelper.clear
diff --git a/apps/workbench/test/integration/jobs_test.rb b/apps/workbench/test/integration/jobs_test.rb
index 5dcdab748..a66dfd803 100644
--- a/apps/workbench/test/integration/jobs_test.rb
+++ b/apps/workbench/test/integration/jobs_test.rb
@@ -39,7 +39,6 @@ class JobsTest < ActionDispatch::IntegrationTest
 
   test 'view partial job log' do
     need_selenium 'to be able to see the CORS response headers (PhantomJS 1.9.8 does not)'
-    use_keep_web_config
 
     # This config will be restored during teardown by ../test_helper.rb:
     Rails.configuration.Workbench.LogViewerMaxBytes = 100
@@ -59,8 +58,6 @@ class JobsTest < ActionDispatch::IntegrationTest
   end
 
   test 'view log via keep-web redirect' do
-    use_keep_web_config
-
     token = api_token('active')
     logdata = fakepipe_with_log_data.read
     logblock = `echo -n #{logdata.shellescape} | ARVADOS_API_TOKEN=#{token.shellescape} arv-put --no-progress --raw -`.strip
diff --git a/sdk/python/tests/run_test_server.py b/sdk/python/tests/run_test_server.py
index b073eff40..6c3585dd8 100644
--- a/sdk/python/tests/run_test_server.py
+++ b/sdk/python/tests/run_test_server.py
@@ -26,6 +26,11 @@ import time
 import unittest
 import yaml
 
+try:
+    from urllib.parse import urlparse
+except ImportError:
+    from urlparse import urlparse
+
 MY_DIRNAME = os.path.dirname(os.path.realpath(__file__))
 if __name__ == '__main__' and os.path.exists(
       os.path.join(MY_DIRNAME, '..', 'arvados', '__init__.py')):
@@ -395,10 +400,12 @@ def get_config():
         return yaml.safe_load(f)
 
 def internal_port_from_config(service):
-    return int(list(get_config()["Clusters"]["zzzzz"]["Services"][service]["InternalURLs"].keys())[0].split(":")[2])
+    return int(urlparse(
+        list(get_config()["Clusters"]["zzzzz"]["Services"][service]["InternalURLs"].keys())[0]).
+               netloc.split(":")[1])
 
 def external_port_from_config(service):
-    return int(get_config()["Clusters"]["zzzzz"]["Services"][service]["ExternalURL"].split(":")[2])
+    return int(urlparse(get_config()["Clusters"]["zzzzz"]["Services"][service]["ExternalURL"]).netloc.split(":")[1])
 
 def run_controller():
     if 'ARVADOS_TEST_PROXY_SERVICES' in os.environ:
@@ -691,7 +698,7 @@ def setup_config():
             "InternalURLs": { }
         },
         "Websocket": {
-            "ExternalURL": "https://localhost:%s" % websocket_external_port,
+            "ExternalURL": "wss://localhost:%s/websocket" % websocket_external_port,
             "InternalURLs": { }
         },
         "GitHTTP": {
diff --git a/services/api/config/application.default.yml b/services/api/config/application.default.yml
index 3ad3cac2b..4e1936b77 100644
--- a/services/api/config/application.default.yml
+++ b/services/api/config/application.default.yml
@@ -86,6 +86,5 @@ test:
   workbench_address: https://localhost:3001/
   git_repositories_dir: <%= Rails.root.join 'tmp', 'git', 'test' %>
   git_internal_dir: <%= Rails.root.join 'tmp', 'internal.git' %>
-  websocket_address: "wss://0.0.0.0:<%= ENV['ARVADOS_TEST_WSS_PORT'] %>/websocket"
   trash_sweep_interval: -1
   docker_image_formats: ["v1"]

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list