summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHombreLaser <sebastian-440@live.com>2023-03-10 21:40:37 -0600
committerHombreLaser <sebastian-440@live.com>2023-03-10 21:40:37 -0600
commitb174a733a5ba58f1d9802feaa4793b4a6c2df523 (patch)
tree71e1022301b0e860b1d65a85f493ea7fd2711200
parent5faf9807968ce9f7f548474afdc31029e57103dc (diff)
Añade specs del controlador de companies
-rw-r--r--spec/factories/company.rb9
-rwxr-xr-xspec/fixtures/files/tres castillos-2.pngbin0 -> 168179 bytes
-rw-r--r--spec/requests/companies_controller/create_companies_controller_spec.rb24
-rw-r--r--spec/requests/sessions_controller/create_sessions_controller_spec.rb (renamed from spec/requests/sessions_controller/post_sessions_controller_spec.rb)4
-rw-r--r--spec/requests/sessions_controller/destroy_sessions_controller_spec.rb (renamed from spec/requests/sessions_controller/delete_sessions_controller_spec.rb)0
-rw-r--r--spec/requests/user_accounts_controller/create_user_accounts_spec.rb (renamed from spec/requests/user_accounts_controller/post_user_accounts_spec.rb)5
-rw-r--r--spec/support/shared_examples/requests/post_request.rb6
7 files changed, 41 insertions, 7 deletions
diff --git a/spec/factories/company.rb b/spec/factories/company.rb
new file mode 100644
index 0000000..e4ae7d6
--- /dev/null
+++ b/spec/factories/company.rb
@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+FactoryBot.define do
+ factory :company, class: 'Company' do
+ name { Faker::Company.name }
+ short_name { Faker::Lorem.word }
+ country { ISO3166::Country.codes.sample }
+ end
+end
diff --git a/spec/fixtures/files/tres castillos-2.png b/spec/fixtures/files/tres castillos-2.png
new file mode 100755
index 0000000..95650b8
--- /dev/null
+++ b/spec/fixtures/files/tres castillos-2.png
Binary files differ
diff --git a/spec/requests/companies_controller/create_companies_controller_spec.rb b/spec/requests/companies_controller/create_companies_controller_spec.rb
new file mode 100644
index 0000000..53677f2
--- /dev/null
+++ b/spec/requests/companies_controller/create_companies_controller_spec.rb
@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'POST /api/companies', type: :request do
+ let(:user) { create(:user_account, role: 'master') }
+ let(:company) { build(:company) }
+ let(:logo) { fixture_file_upload('tres castillos-2.png', 'image/png') }
+ let(:token) { jwt(user) }
+
+ it_behaves_like 'a POST request' do
+ let(:headers) { { 'CONTENT_TYPE' => 'application/json', 'Authorization' => "Bearer #{token['token']}" } }
+ let(:route) { '/api/companies' }
+ let(:expected_error_messages) do
+ ["Name can't be blank", "Short name can't be blank", "Country can't be blank"]
+ end
+ let(:desired_error_status) { 422 }
+ let(:expected_text) { [company.name, company.short_name, company.country, 'logo', 'http'] }
+ let(:params) do
+ { name: company.name, short_name: company.short_name, country: company.country, logo: logo }
+ end
+ let(:wrong_params) { JSON.generate({ name: '', short_name: '', country: '' }) }
+ end
+end
diff --git a/spec/requests/sessions_controller/post_sessions_controller_spec.rb b/spec/requests/sessions_controller/create_sessions_controller_spec.rb
index c67b14d..69b7b42 100644
--- a/spec/requests/sessions_controller/post_sessions_controller_spec.rb
+++ b/spec/requests/sessions_controller/create_sessions_controller_spec.rb
@@ -10,7 +10,7 @@ RSpec.describe 'POST /api/login', type: :request do
let(:desired_error_status) { 401 }
let(:expected_text) { %w[token refresh] }
let(:user) { create(:user_account) }
- let(:params) { { email: user.email, password: user.password } }
- let(:wrong_params) { { email: user.email, password: 'wrong_password' } }
+ let(:params) { JSON.generate({ email: user.email, password: user.password }) }
+ let(:wrong_params) { JSON.generate({ email: user.email, password: 'wrong_password' }) }
end
end
diff --git a/spec/requests/sessions_controller/delete_sessions_controller_spec.rb b/spec/requests/sessions_controller/destroy_sessions_controller_spec.rb
index bc669fc..bc669fc 100644
--- a/spec/requests/sessions_controller/delete_sessions_controller_spec.rb
+++ b/spec/requests/sessions_controller/destroy_sessions_controller_spec.rb
diff --git a/spec/requests/user_accounts_controller/post_user_accounts_spec.rb b/spec/requests/user_accounts_controller/create_user_accounts_spec.rb
index ba77ae5..3fd5990 100644
--- a/spec/requests/user_accounts_controller/post_user_accounts_spec.rb
+++ b/spec/requests/user_accounts_controller/create_user_accounts_spec.rb
@@ -13,8 +13,9 @@ RSpec.describe 'POST /api/user_accounts', type: :request do
let(:expected_text) { %w[token refresh] }
let(:user) { create(:user_account) }
let(:params) do
- { email: user.email, password: user.password, first_name: user.first_name, last_name: user.last_name}
+ JSON.generate({ email: user.email, password: user.password,
+ first_name: user.first_name, last_name: user.last_name })
end
- let(:wrong_params) { { email: 'notanemail', password: '' } }
+ let(:wrong_params) { JSON.generate({ email: 'notanemail', password: '' }) }
end
end
diff --git a/spec/support/shared_examples/requests/post_request.rb b/spec/support/shared_examples/requests/post_request.rb
index b7226d0..c7b8d77 100644
--- a/spec/support/shared_examples/requests/post_request.rb
+++ b/spec/support/shared_examples/requests/post_request.rb
@@ -4,8 +4,8 @@ RSpec.shared_examples 'a POST request' do
let(:headers) { { 'CONTENT_TYPE' => 'application/json' } }
context 'with correct parameters' do
- it 'returns a successful http status and a response to the posted data' do
- post(route, params: params.to_json, headers:)
+ it 'returns a successful http status and a response with the posted data' do
+ post(route, params: params, headers:)
expect(response).to have_http_status(200)
expect(response.body).to include_strings(expected_text)
end
@@ -13,7 +13,7 @@ RSpec.shared_examples 'a POST request' do
context 'with incorrect parameters' do
it 'returns a client side error http status and a error message' do
- post(route, params: wrong_params.to_json, headers:)
+ post(route, params: wrong_params, headers:)
expect(response).to have_http_status(desired_error_status)
expect(response.body).to include_strings(expected_error_messages)
end