Revision 90b5dff1
Added by Djebran Lezzoum about 6 years ago
| test/controllers/api/v2/smart_proxies_controller_test.rb | ||
|---|---|---|
|
assert_response :success
|
||
|
assert_not_nil assigns(:smart_proxies)
|
||
|
smart_proxies = ActiveSupport::JSON.decode(@response.body)
|
||
|
assert !smart_proxies.empty?
|
||
|
refute_empty smart_proxies
|
||
|
end
|
||
|
|
||
|
test "should get index filtered by feature" do
|
||
| ... | ... | |
|
get :show, params: { :id => smart_proxies(:one).to_param }
|
||
|
assert_response :success
|
||
|
show_response = ActiveSupport::JSON.decode(@response.body)
|
||
|
assert !show_response.empty?
|
||
|
refute_empty show_response
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '0ffe0dc5-675e-45f4-b7e1-a14d3dd81f6e'
|
||
|
test "should create smart_proxy" do
|
||
|
assert_difference('SmartProxy.unscoped.count') do
|
||
|
post :create, params: { :smart_proxy => valid_attrs }
|
||
|
end
|
||
|
assert_response :created
|
||
|
response = JSON.parse(@response.body)
|
||
|
assert response.key?('name')
|
||
|
assert_equal valid_attrs[:name], response['name']
|
||
|
assert response.key?('url')
|
||
|
assert_equal valid_attrs[:url], response['url']
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => 'f629cd9b-45ae-44ee-b724-9a768f958428'
|
||
|
test "should not create smart_proxy with invalid name" do
|
||
|
assert_difference('SmartProxy.unscoped.count', 0) do
|
||
|
post :create, params: { :smart_proxy => valid_attrs.merge(:name => '') }
|
||
|
end
|
||
|
assert_response :unprocessable_entity
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '2c5e70e0-75ea-4521-a9ef-1d1cdc5bcbf7'
|
||
|
test "should not create smart_proxy with invalid url" do
|
||
|
assert_difference('SmartProxy.unscoped.count', 0) do
|
||
|
post :create, params: { :smart_proxy => valid_attrs.merge(:url => '') }
|
||
|
end
|
||
|
assert_response :unprocessable_entity
|
||
|
assert_match 'URL must be valid and schema must be one of http and https', @response.body
|
||
|
end
|
||
|
|
||
|
test "should update smart_proxy" do
|
||
| ... | ... | |
|
assert_response :success
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => 'fbde9f87-33db-4b95-a5f7-71a618460c84'
|
||
|
test "should return any organization field when updating smart_proxy" do
|
||
|
any_needed_fields = %w[organization organization_ids organizations]
|
||
|
put :update, params: { :id => smart_proxies(:one).to_param, :smart_proxy => valid_attrs }
|
||
|
assert_response :success
|
||
|
response = JSON.parse(@response.body)
|
||
|
refute_empty response.keys.select { |key| any_needed_fields.include?(key) }
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '42d6b749-c047-4fd2-90ee-ffab7be558f9'
|
||
|
test "should return any location field when updating smart_proxy" do
|
||
|
any_needed_fields = %w[location location_ids locations]
|
||
|
put :update, params: { :id => smart_proxies(:one).to_param, :smart_proxy => valid_attrs }
|
||
|
assert_response :success
|
||
|
response = JSON.parse(@response.body)
|
||
|
refute_empty response.keys.select { |key| any_needed_fields.include?(key) }
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => 'f279640e-d7e9-48a3-aed8-7bf406e9d6f2'
|
||
|
test "should update smart_proxy name only" do
|
||
|
new_name = 'new_smart_proxy'
|
||
|
put :update, params: { :id => smart_proxies(:one).to_param, :smart_proxy => { :name => new_name } }
|
||
|
assert_response :success
|
||
|
response = JSON.parse(@response.body)
|
||
|
assert response.key?('name')
|
||
|
assert_equal new_name, response['name']
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '0305fd54-4e0c-4dd9-a537-d342c3dc867e'
|
||
|
test "should update smart_proxy url only" do
|
||
|
new_url = 'https://new-valid_server_url.com:8443'
|
||
|
put :update, params: { :id => smart_proxies(:one).to_param, :smart_proxy => { :url => new_url } }
|
||
|
assert_response :success
|
||
|
response = JSON.parse(@response.body)
|
||
|
assert response.key?('url')
|
||
|
assert_equal new_url, response['url']
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '62631275-7a92-4d34-a949-c56e0c4063f1'
|
||
|
test "should update smart_proxy organizations" do
|
||
|
organization_ids = [ taxonomies(:organization2).id, taxonomies(:empty_organization).id ].sort
|
||
|
smart_proxy = smart_proxies(:one)
|
||
|
put :update, params: { :id => smart_proxy.to_param, :smart_proxy => { :organization_ids => organization_ids } }
|
||
|
assert_response :success
|
||
|
response = JSON.parse(@response.body)
|
||
|
assert response.key?('organizations')
|
||
|
assert_equal organization_ids, response['organizations'].map { |org| org['id'] }.sort
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => 'e08eaaa9-7c11-4cda-bbe7-6d1f7c732569'
|
||
|
test "should update smart_proxy locations" do
|
||
|
location_ids = [ taxonomies(:location1).id, taxonomies(:location2).id ].sort
|
||
|
smart_proxy = smart_proxies(:one)
|
||
|
put :update, params: { :id => smart_proxy.to_param, :smart_proxy => { :location_ids => location_ids } }
|
||
|
assert_response :success
|
||
|
response = JSON.parse(@response.body)
|
||
|
assert response.key?('locations')
|
||
|
assert_equal location_ids, response['locations'].map { |loc| loc['id'] }.sort
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '4afbeac4-ac8c-4e03-9845-81923dbb5b3d'
|
||
|
test "should not update smart_proxy with invalid name" do
|
||
|
put :update, params: { :id => smart_proxies(:one).to_param, :smart_proxy => { :name => '' } }
|
||
|
assert_response :unprocessable_entity
|
||
|
assert_match 'Name can\'t be blank', @response.body
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '86df219d-f354-4a9c-97ee-72bb7acb7f5b'
|
||
|
test "should not update smart_proxy with invalid url" do
|
||
|
put :update, params: { :id => smart_proxies(:one).to_param, :smart_proxy => { :url => '' } }
|
||
|
assert_response :unprocessable_entity
|
||
|
assert_match 'URL must be valid and schema must be one of http and https', @response.body
|
||
|
end
|
||
|
|
||
|
test_attributes :pid => '872bf12e-736d-43d1-87cf-2923966b59d0'
|
||
|
test "should destroy smart_proxy" do
|
||
|
smart_proxy = smart_proxies(:four)
|
||
|
assert_difference('SmartProxy.unscoped.count', -1) do
|
||
|
delete :destroy, params: { :id => smart_proxies(:four).to_param }
|
||
|
delete :destroy, params: { :id => smart_proxy.to_param }
|
||
|
end
|
||
|
assert_response :success
|
||
|
refute SmartProxy.unscoped.exists?(smart_proxy.id)
|
||
|
end
|
||
|
|
||
|
# Pending - failure on .permission_failed?
|
||
| ... | ... | |
|
assert_equal 2, response['environments_with_new_puppetclasses']
|
||
|
end
|
||
|
|
||
|
# this is intentional: put test_attributes only for the first generated test eg: dryrun_param = {}
|
||
|
test_attributes :pid => '385efd1b-6146-47bf-babf-0127ce5955ed'
|
||
|
[{}, { :dryrun => false }, { :dryrun => 'false' }].each do |dryrun_param|
|
||
|
test "should import new puppetclasses" do
|
||
|
setup_import_classes
|
||
| ... | ... | |
|
end
|
||
|
end
|
||
|
assert_response :success
|
||
|
assert_match 'Successfully updated environment and puppetclasses from the on-disk puppet installation', @response.body
|
||
|
end
|
||
|
end
|
||
|
|
||
| test/models/smart_proxy_test.rb | ||
|---|---|---|
|
require 'test_helper'
|
||
|
|
||
|
class SmartProxyTest < ActiveSupport::TestCase
|
||
|
should allow_values(*valid_name_list).for(:name)
|
||
|
should_not allow_values('', ' ', '\t', nil, 'invalid url').for(:url)
|
||
|
|
||
|
test "should not create smart_proxy with invalid name" do
|
||
|
invalid_name_list.each do |invalid_name|
|
||
|
smart_proxy = FactoryBot.build(:smart_proxy, :name => invalid_name, :url => 'https://valid.url:4568')
|
||
|
refute_valid smart_proxy
|
||
|
assert_includes smart_proxy.errors.keys, :name
|
||
|
end
|
||
|
end
|
||
|
|
||
|
context 'url validations' do
|
||
|
setup do
|
||
|
@proxy = FactoryBot.
|
||
Also available in: Unified diff
Fixes #23605 - Port robottelo tests - smart_proxy