summaryrefslogtreecommitdiff
path: root/spec/requests/addresses_controller/update_spec.rb
diff options
context:
space:
mode:
authorHombreLaser <sebastian-440@live.com>2023-03-30 17:29:40 -0600
committerHombreLaser <sebastian-440@live.com>2023-03-30 17:29:40 -0600
commit2ae0cd8a8dc6d630ee1a7a84ddf6111609dbdeb7 (patch)
treebbcaadbea3da32ee974d28cfb37c1af4be018215 /spec/requests/addresses_controller/update_spec.rb
parentefc5eb10894fc95487c55628b94024e97cd60139 (diff)
Mejora la estructura de los specs
Diffstat (limited to 'spec/requests/addresses_controller/update_spec.rb')
-rw-r--r--spec/requests/addresses_controller/update_spec.rb54
1 files changed, 54 insertions, 0 deletions
diff --git a/spec/requests/addresses_controller/update_spec.rb b/spec/requests/addresses_controller/update_spec.rb
new file mode 100644
index 0000000..0c3662e
--- /dev/null
+++ b/spec/requests/addresses_controller/update_spec.rb
@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'PUT /api/account/addresses', type: :request do
+ let(:user) { create(:user_account, role: 'regular') }
+ let(:address) { create(:address) }
+ let(:new_address) { build(:address) }
+ let(:token) { jwt(user) }
+ let(:headers) { { 'Authorization' => "Bearer #{token['token']}" } }
+ let(:params) do
+ { number: new_address.number, street: new_address.street, country: new_address.country, zip_code: new_address.zip_code,
+ city: new_address.city }
+ end
+
+ before(:each) do
+ relate_user_and_address(user, address)
+ end
+
+ it_behaves_like 'a PUT request' do
+ let(:route) { "/api/account/addresses/#{address.id}" }
+ let(:wrong_route) { "/api/account/addresses/#{SecureRandom.hex(8)}" }
+ let(:expected_error_messages) do
+ ["Number can't be blank", "Street can't be blank", "Zip code can't be blank",
+ 'Country is invalid', "City can't be blank"]
+ end
+ let(:expected_text) do
+ [new_address.number.to_s, new_address.zip_code.to_s, new_address.country, new_address.street,
+ new_address.city]
+ end
+ let(:wrong_params) { { number: '', street: '', zip_code: '', city: '', country: 'Not a country' } }
+ end
+
+ context 'when the update params coincide with an already existing address for the user' do
+ it 'returns a 422 http status and an error message' do
+ new_address.save
+ relate_user_and_address(user, new_address)
+ put("/api/account/addresses/#{address.id}", params:, headers:)
+ expect(response).to have_http_status(422)
+ expect(response.body).to include('Ya cuenta con esta dirección')
+ end
+ end
+
+ context 'when the update params coincide with an already existing address' do
+ it 'builds a relationship between the user and the address, without creating another address' do
+ new_address.save
+ another_user = create(:user_account, role: 'regular')
+ relate_user_and_address(another_user, new_address)
+ put("/api/account/addresses/#{address.id}", params:, headers:)
+ expect(Address.count).to eq(1)
+ expect(another_user.addresses.first).to eq(user.addresses.find(new_address.id))
+ end
+ end
+end