[ARVADOS] updated: 7edde4785f7cc325a8c2b109d6fcda176af2650c

[git at public.curoverse.com]

Summary of changes:
 .../test/controllers/application_controller_test.rb        | 13 ++++---------
 services/api/test/fixtures/links.yml                       | 14 ++++++++++++++
 2 files changed, 18 insertions(+), 9 deletions(-)

       via  7edde4785f7cc325a8c2b109d6fcda176af2650c (commit)
      from  ec27ba2a576189755d443d54213318741f73d125 (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 7edde4785f7cc325a8c2b109d6fcda176af2650c
Author: Radhika Chippada <radhika at curoverse.com>
Date:   Tue Feb 3 10:20:45 2015 -0500

    2659: Finally understood the mystery behind the "anonymous tests are failing when run with all other tests".
    There is a permission link fixture for anonymous group. I needed to create read permission link for anonymous user.
    (Tom thinks) the groups permission cache may be getting out of sync somehow.

diff --git a/apps/workbench/test/controllers/application_controller_test.rb b/apps/workbench/test/controllers/application_controller_test.rb
index 463a131..f48bdb6 100644
--- a/apps/workbench/test/controllers/application_controller_test.rb
+++ b/apps/workbench/test/controllers/application_controller_test.rb
@@ -338,16 +338,11 @@ class ApplicationControllerTest < ActionController::TestCase
 
       get(:show, {id: fixture['uuid']})
 
-      if Rails.configuration.anonymous_user_token
-        assert_response 200
-        if controller.class == JobsController
-          assert_includes @response.inspect, fixture['script']
-        else
-          assert_includes @response.inspect, fixture['name']
-        end
+      assert_response 200
+      if controller.class == JobsController
+        assert_includes @response.inspect, fixture['script']
       else
-        assert_response :redirect
-        assert_match /\/users\/welcome/, @response.redirect_url
+        assert_includes @response.inspect, fixture['name']
       end
     end
   end
diff --git a/services/api/test/fixtures/links.yml b/services/api/test/fixtures/links.yml
index a6fd4de..1b34868 100644
--- a/services/api/test/fixtures/links.yml
+++ b/services/api/test/fixtures/links.yml
@@ -512,6 +512,20 @@ anonymous_group_can_read_anonymously_accessible_project:
   head_uuid: zzzzz-j7d0g-zhxawtyetzwc5f0
   properties: {}
 
+anonymous_user_can_read_anonymously_accessible_project:
+  uuid: zzzzz-o0j2j-82nbli3jptwksj1
+  owner_uuid: zzzzz-tpzed-000000000000000
+  created_at: 2014-05-30 14:30:00.184389725 Z
+  modified_by_client_uuid: zzzzz-ozdt8-brczlopd8u8d0jr
+  modified_by_user_uuid: zzzzz-tpzed-000000000000000
+  modified_at: 2014-05-30 14:30:00.184019565 Z
+  updated_at: 2014-05-30 14:30:00.183829316 Z
+  link_class: permission
+  name: can_read
+  tail_uuid: zzzzz-tpzed-anonymouspublic
+  head_uuid: zzzzz-j7d0g-zhxawtyetzwc5f0
+  properties: {}
+
 user_agreement_readable_by_anonymously_accessible_project:
   uuid: zzzzz-o0j2j-o5ds5gvhkztdc8h
   owner_uuid: zzzzz-j7d0g-zhxawtyetzwc5f0

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


hooks/post-receive
--