Renamed visitor to anonymous

Evandro Junior · Leandro Santos
1 parent 99bf0aae
Showing 4 changed files with 37 additions and 43 deletions Show diff stats
test/api/categories_test.rb
test/api/communities_test.rb
test/api/profiles_test.rb
test/api/test_helper.rb
@@ -97,26 +97,24 @@ class CategoriesTest &lt; ActiveSupport::TestCase
     end
   end
-  ############## Visitors' tests #######################################################################33
-
-  should 'visitor list categories' do
-    visitor_setup
+  should 'anonymous list categories' do
+    anonymous_setup
     category = fast_create(Category, :environment_id => environment.id)
     get "/api/v1/categories/?#{params.to_query}"
     json = JSON.parse(last_response.body)
     assert_includes json["categories"].map { |c| c["name"] }, category.name
   end
-  should 'visitor get category by id' do
-    visitor_setup
+  should 'anonymous get category by id' do
+    anonymous_setup
     category = fast_create(Category, :environment_id => environment.id)
     get "/api/v1/categories/#{category.id}/?#{params.to_query}"
     json = JSON.parse(last_response.body)
     assert_equal category.name, json["category"]["name"]
   end
-  should 'visitor list parent and children when get category by id' do
-    visitor_setup
+  should 'anonymous list parent and children when get category by id' do
+    anonymous_setup
     parent = fast_create(Category, :environment_id => environment.id)
     child_1 = fast_create(Category, :environment_id => environment.id)
     child_2 = fast_create(Category, :environment_id => environment.id)
@@ -133,8 +131,8 @@ class CategoriesTest &lt; ActiveSupport::TestCase
     assert_equivalent [child_1.id, child_2.id], json['category']['children'].map { |c| c['id'] }
   end
-  should 'visitor include parent in categories list if params is true' do
-    visitor_setup
+  should 'anonymous include parent in categories list if params is true' do
+    anonymous_setup
     parent_1 = fast_create(Category, :environment_id => environment.id) # parent_1 has no parent category
     child_1 = fast_create(Category, :environment_id => environment.id)
     child_2 = fast_create(Category, :environment_id => environment.id)
@@ -156,8 +154,8 @@ class CategoriesTest &lt; ActiveSupport::TestCase
       json["categories"].map { |c| c['parent'] && c['parent']['id'] }
   end
-  should 'visitor include children in categories list if params is true' do
-    visitor_setup
+  should 'anonymous include children in categories list if params is true' do
+    anonymous_setup
     category = fast_create(Category, :environment_id => environment.id)
     child_1 = fast_create(Category, :environment_id => environment.id)
     child_2 = fast_create(Category, :environment_id => environment.id)
@@ -182,8 +180,8 @@ class CategoriesTest &lt; ActiveSupport::TestCase
   end
   expose_attributes.each do |attr|
-    should "visitor expose category #{attr} attribute by default" do
-      visitor_setup
+    should "anonymous expose category #{attr} attribute by default" do
+      anonymous_setup
       category = fast_create(Category, :environment_id => environment.id)
       get "/api/v1/categories/?#{params.to_query}"
       json = JSON.parse(last_response.body)
@@ -191,6 +189,6 @@ class CategoriesTest &lt; ActiveSupport::TestCase
     end
   end
-  ################################# End visitors' test ####################################################################################
+
 end
@@ -171,10 +171,8 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_not_includes json["communities"].map { |a| a["id"] }, community2.id
   end
-  ################### Visitor's tests ######################################3
-
-  should 'visitor list only communities' do
-    visitor_setup
+  should 'anonymous list only communities' do
+    anonymous_setup
     community = fast_create(Community, :environment_id => environment.id)
     enterprise = fast_create(Enterprise, :environment_id => environment.id) # should not list this enterprise
     get "/api/v1/communities?#{params.to_query}"
@@ -183,8 +181,8 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_includes json['communities'].map {|c| c['id']}, community.id
   end
-  should 'visitor list all communities' do
-    visitor_setup
+  should 'anonymous list all communities' do
+    anonymous_setup
     community1 = fast_create(Community, :environment_id => environment.id, :public_profile => true)
     community2 = fast_create(Community, :environment_id => environment.id)
     get "/api/v1/communities?#{params.to_query}"
@@ -192,8 +190,8 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_equivalent [community1.id, community2.id], json['communities'].map {|c| c['id']}
   end
-  should 'not visitor list invisible communities' do
-    visitor_setup
+  should 'not anonymous list invisible communities' do
+    anonymous_setup
     community1 = fast_create(Community, :environment_id => environment.id)
     fast_create(Community, :environment_id => environment.id, :visible => false)
@@ -202,8 +200,8 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_equal [community1.id], json['communities'].map {|c| c['id']}
   end
-  should 'visitor list private communities' do
-    visitor_setup
+  should 'anonymous list private communities' do
+    anonymous_setup
     community1 = fast_create(Community, :environment_id => environment.id)
     community2 = fast_create(Community, :environment_id => environment.id, :public_profile => false)
@@ -214,32 +212,32 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
-  should 'not visitor create a community' do
-    visitor_setup
+  should 'not anonymous create a community' do
+    anonymous_setup
     params[:community] = {:name => 'some'}
     post "/api/v1/communities?#{params.to_query}"
     json = JSON.parse(last_response.body)
     assert_equal 401, last_response.status
   end
-  should 'visitor get community' do
-    visitor_setup
+  should 'anonymous get community' do
+    anonymous_setup
     community = fast_create(Community, :environment_id => environment.id)
     get "/api/v1/communities/#{community.id}"
     json = JSON.parse(last_response.body)
     assert_equal community.id, json['community']['id']
   end
-  should 'not visitor get invisible community' do
-    visitor_setup
+  should 'not anonymous get invisible community' do
+    anonymous_setup
     community = fast_create(Community, :environment_id => environment.id, :visible => false)
     get "/api/v1/communities/#{community.id}"
     json = JSON.parse(last_response.body)
     assert json['community'].blank?
   end
-  should 'visitor not get private communities' do
-    visitor_setup
+  should 'anonymous not get private communities' do
+    anonymous_setup
     community = fast_create(Community, :environment_id => environment.id)
     fast_create(Community, :environment_id => environment.id, :public_profile => false)
     get "/api/v1/communities/#{community.id}"
@@ -247,8 +245,8 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_equal community.id, json['community']['id']
   end
-  should 'visitor list communities with pagination' do
-    visitor_setup
+  should 'anonymous list communities with pagination' do
+    anonymous_setup
     community1 = fast_create(Community, :public_profile => true, :created_at => 1.day.ago)
     community2 = fast_create(Community, :created_at => 2.days.ago)
@@ -269,8 +267,8 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_not_includes json_page_two["communities"].map { |a| a["id"] }, community1.id
   end
-  should 'visitor list communities with timestamp' do
-    visitor_setup
+  should 'anonymous list communities with timestamp' do
+    anonymous_setup
     community1 = fast_create(Community, :public_profile => true)
     community2 = fast_create(Community)
@@ -285,6 +283,4 @@ class CommunitiesTest &lt; ActiveSupport::TestCase
     assert_not_includes json["communities"].map { |a| a["id"] }, community2.id
   end
-  ###################End Visitor's tests ######################################3
-
 end
@@ -80,7 +80,7 @@ class ProfilesTest &lt; ActiveSupport::TestCase
   end
-  should 'visitor list all profiles' do
+  should 'anonymous list all profiles' do
     person1 = fast_create(Person)
     person2 = fast_create(Person)
     community = fast_create(Community)
@@ -89,14 +89,14 @@ class ProfilesTest &lt; ActiveSupport::TestCase
     assert_equivalent [person1.id, person2.id, community.id], json.map {|p| p['id']}
   end
-  should 'visitor get person from profile id' do
+  should 'anonymous get person from profile id' do
     some_person = fast_create(Person)
     get "/api/v1/profiles/#{some_person.id}"
     json = JSON.parse(last_response.body)
     assert_equal some_person.id, json['id']
   end
-  should 'visitor get community from profile id' do
+  should 'anonymous get community from profile id' do
     community = fast_create(Community)
     get "/api/v1/profiles/#{community.id}"
     json = JSON.parse(last_response.body)
@@ -25,7 +25,7 @@ class ActiveSupport::TestCase
     @params = {:private_token => @private_token}
   end
-  def visitor_setup
+  def anonymous_setup
     @environment = Environment.default
     @params = {}
   end
	@@ -80,7 +80,7 @@ class ProfilesTest < ActiveSupport::TestCase		@@ -80,7 +80,7 @@ class ProfilesTest < ActiveSupport::TestCase
80		80
81	end	81	end
82		82
83	- should 'visitor list all profiles' do	83	+ should 'anonymous list all profiles' do
84	person1 = fast_create(Person)	84	person1 = fast_create(Person)
85	person2 = fast_create(Person)	85	person2 = fast_create(Person)
86	community = fast_create(Community)	86	community = fast_create(Community)
	@@ -89,14 +89,14 @@ class ProfilesTest < ActiveSupport::TestCase		@@ -89,14 +89,14 @@ class ProfilesTest < ActiveSupport::TestCase
89	assert_equivalent [person1.id, person2.id, community.id], json.map {\|p\| p['id']}	89	assert_equivalent [person1.id, person2.id, community.id], json.map {\|p\| p['id']}
90	end	90	end
91		91
92	- should 'visitor get person from profile id' do	92	+ should 'anonymous get person from profile id' do
93	some_person = fast_create(Person)	93	some_person = fast_create(Person)
94	get "/api/v1/profiles/#{some_person.id}"	94	get "/api/v1/profiles/#{some_person.id}"
95	json = JSON.parse(last_response.body)	95	json = JSON.parse(last_response.body)
96	assert_equal some_person.id, json['id']	96	assert_equal some_person.id, json['id']
97	end	97	end
98		98
99	- should 'visitor get community from profile id' do	99	+ should 'anonymous get community from profile id' do
100	community = fast_create(Community)	100	community = fast_create(Community)
101	get "/api/v1/profiles/#{community.id}"	101	get "/api/v1/profiles/#{community.id}"
102	json = JSON.parse(last_response.body)	102	json = JSON.parse(last_response.body)
	@@ -25,7 +25,7 @@ class ActiveSupport::TestCase		@@ -25,7 +25,7 @@ class ActiveSupport::TestCase
25	@params = {:private_token => @private_token}	25	@params = {:private_token => @private_token}
26	end	26	end
27		27
28	- def visitor_setup	28	+ def anonymous_setup
29	@environment = Environment.default	29	@environment = Environment.default
30	@params = {}	30	@params = {}
31	end	31	end