[ARVADOS] updated: 1.3.0-1405-g8d152808d

Git user git at public.curoverse.com
Thu Jul 25 20:41:36 UTC 2019


Summary of changes:
 apps/workbench/config/application.default.yml               | 12 ------------
 apps/workbench/test/integration/repositories_browse_test.rb |  5 +----
 apps/workbench/test/integration/user_settings_menu_test.rb  |  2 +-
 services/api/app/models/repository.rb                       |  3 +++
 4 files changed, 5 insertions(+), 17 deletions(-)

       via  8d152808d49ad6b1767b26ac10dffb53fb7940fe (commit)
      from  4512b2324d203673a7b76927631a9bac4361d0de (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 8d152808d49ad6b1767b26ac10dffb53fb7940fe
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Thu Jul 25 16:41:03 2019 -0400

    14717: Make sure tests use git http and keep-web URLs from config.yml
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/apps/workbench/config/application.default.yml b/apps/workbench/config/application.default.yml
index 8d1dff995..9456e6145 100644
--- a/apps/workbench/config/application.default.yml
+++ b/apps/workbench/config/application.default.yml
@@ -79,18 +79,6 @@ test:
   profiling_enabled: true
   secret_token: <%= rand(2**256).to_s(36) %>
   secret_key_base: <%= rand(2**256).to_s(36) %>
-  # This setting is to allow workbench start when running tests, it should be
-  # set to a correct value when testing relevant features.
-  keep_web_url: http://example.com/c=%{uuid_or_pdh}
-
-  # When you run the Workbench's integration tests, it starts the API
-  # server as a dependency.  These settings should match the API
-  # server's Rails defaults.  If you adjust those, change these
-  # settings in application.yml to match.
-  arvados_login_base: https://localhost:3000/login
-  arvados_v1_base: https://localhost:3000/arvados/v1
-  arvados_insecure_https: true
-
   site_name: Workbench:test
 
   # Enable user profile with one required field
diff --git a/apps/workbench/test/integration/repositories_browse_test.rb b/apps/workbench/test/integration/repositories_browse_test.rb
index e668b8ca1..056598ef1 100644
--- a/apps/workbench/test/integration/repositories_browse_test.rb
+++ b/apps/workbench/test/integration/repositories_browse_test.rb
@@ -41,12 +41,9 @@ class RepositoriesTest < ActionDispatch::IntegrationTest
 
   test "browse using arv-git-http" do
     repo = api_fixture('repositories')['foo']
-    portfile =
-      File.expand_path('../../../../../tmp/arv-git-httpd-ssl.port', __FILE__)
-    gitsslport = File.read(portfile)
     Repository.any_instance.
       stubs(:http_fetch_url).
-      returns "https://localhost:#{gitsslport}/#{repo['name']}.git"
+      returns "#{Rails.configuration.Services.GitHTTP.ExternalURL.to_s}/#{repo['name']}.git"
     commit_sha1 = '1de84a854e2b440dc53bf42f8548afa4c17da332'
     visit page_with_token('active', "/repositories/#{repo['uuid']}/commit/#{commit_sha1}")
     assert_text "Date:   Tue Mar 18 15:55:28 2014 -0400"
diff --git a/apps/workbench/test/integration/user_settings_menu_test.rb b/apps/workbench/test/integration/user_settings_menu_test.rb
index 562dc7d1f..5f2886c7a 100644
--- a/apps/workbench/test/integration/user_settings_menu_test.rb
+++ b/apps/workbench/test/integration/user_settings_menu_test.rb
@@ -188,7 +188,7 @@ class UserSettingsMenuTest < ActionDispatch::IntegrationTest
     end
     assert_text ":active/workbenchtest.git"
     assert_match /git at git.*:active\/workbenchtest.git/, page.text
-    assert_match /https:\/\/git.*\/active\/workbenchtest.git/, page.text
+    assert_match /#{Rails.configuration.Services.GitHTTP.ExternalURL.to_s}\/active\/workbenchtest.git/, page.text
   end
 
   [
diff --git a/services/api/app/models/repository.rb b/services/api/app/models/repository.rb
index 5e0e39f9b..e6a079540 100644
--- a/services/api/app/models/repository.rb
+++ b/services/api/app/models/repository.rb
@@ -115,6 +115,9 @@ class Repository < ArvadosModel
     else
       base = URI(default_base_fmt % prefix)
     end
+    if base.path == ""
+      base.path = "/"
+    end
     if base.scheme == "ssh"
       '%s@%s:%s.git' % [base.user, base.host, name]
     else

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list