From ff0356e6984af5f957279d766346f3487c93fb82 Mon Sep 17 00:00:00 2001 From: Mathieu Magnin Date: Tue, 12 Mar 2019 17:36:16 +0100 Subject: [PATCH 1/2] Fix Sentry 165 : api crash when siret is unknown --- app/lib/api_entreprise/entreprise_adapter.rb | 4 +- .../api_entreprise/entreprises_not_found.json | 6 + .../api_entreprise/entreprise_adapter_spec.rb | 112 ++++++++++-------- 3 files changed, 71 insertions(+), 51 deletions(-) create mode 100644 spec/fixtures/files/api_entreprise/entreprises_not_found.json diff --git a/app/lib/api_entreprise/entreprise_adapter.rb b/app/lib/api_entreprise/entreprise_adapter.rb index 989136360..c1b4b8f4c 100644 --- a/app/lib/api_entreprise/entreprise_adapter.rb +++ b/app/lib/api_entreprise/entreprise_adapter.rb @@ -7,9 +7,9 @@ class ApiEntreprise::EntrepriseAdapter < ApiEntreprise::Adapter end def process_params - params = data_source[:entreprise].slice(*attr_to_fetch) + params = data_source[:entreprise]&.slice(*attr_to_fetch) - if valid_params?(params) + if params.present? && valid_params?(params) params[:date_creation] = Time.zone.at(params[:date_creation]).to_datetime params.transform_keys { |k| :"entreprise_#{k}" } else diff --git a/spec/fixtures/files/api_entreprise/entreprises_not_found.json b/spec/fixtures/files/api_entreprise/entreprises_not_found.json new file mode 100644 index 000000000..bf73067df --- /dev/null +++ b/spec/fixtures/files/api_entreprise/entreprises_not_found.json @@ -0,0 +1,6 @@ +{ + "errors": [ + "Le siret ou siren indiqué n'existe pas, n'est pas connu ou ne comporte aucune information pour cet appel" + ], + "gateway_error": true +} diff --git a/spec/lib/api_entreprise/entreprise_adapter_spec.rb b/spec/lib/api_entreprise/entreprise_adapter_spec.rb index a8559c024..a8fb2e50e 100644 --- a/spec/lib/api_entreprise/entreprise_adapter_spec.rb +++ b/spec/lib/api_entreprise/entreprise_adapter_spec.rb @@ -8,60 +8,74 @@ describe ApiEntreprise::EntrepriseAdapter do before do stub_request(:get, /https:\/\/entreprise.api.gouv.fr\/v2\/entreprises\/#{siren}?.*token=/) - .to_return(body: File.read('spec/fixtures/files/api_entreprise/entreprises.json', status: 200)) + .to_return(body: body, status: status) end - it '#to_params class est une Hash ?' do - expect(subject).to be_an_instance_of(Hash) + context "when SIRET is OK" do + let(:body) { File.read('spec/fixtures/files/api_entreprise/entreprises.json') } + let(:status) { 200 } + + it '#to_params class est une Hash ?' do + expect(subject).to be_an_instance_of(Hash) + end + + context 'Attributs Entreprises' do + it 'L\'entreprise contient bien un siren' do + expect(subject[:entreprise_siren]).to eq(siren) + end + + it 'L\'entreprise contient bien un capital_social' do + expect(subject[:entreprise_capital_social]).to eq(462308) + end + + it 'L\'entreprise contient bien un numero_tva_intracommunautaire' do + expect(subject[:entreprise_numero_tva_intracommunautaire]).to eq('FR16418166096') + end + + it 'L\'entreprise contient bien une forme_juridique' do + expect(subject[:entreprise_forme_juridique]).to eq('SA à directoire (s.a.i.)') + end + + it 'L\'entreprise contient bien un forme_juridique_code' do + expect(subject[:entreprise_forme_juridique_code]).to eq('5699') + end + + it 'L\'entreprise contient bien un nom_commercial' do + expect(subject[:entreprise_nom_commercial]).to eq('OCTO-TECHNOLOGY') + end + + it 'L\'entreprise contient bien une raison_sociale' do + expect(subject[:entreprise_raison_sociale]).to eq('OCTO-TECHNOLOGY') + end + + it 'L\'entreprise contient bien un siret_siege_social' do + expect(subject[:entreprise_siret_siege_social]).to eq('41816609600051') + end + + it 'L\'entreprise contient bien un code_effectif_entreprise' do + expect(subject[:entreprise_code_effectif_entreprise]).to eq('31') + end + + it 'L\'entreprise contient bien une date_creation' do + expect(subject[:entreprise_date_creation]).to eq('Wed, 01 Apr 1998 00:00:00.000000000 +0200') + end + + it 'L\'entreprise contient bien un nom' do + expect(subject[:entreprise_nom]).to eq('test_nom') + end + + it 'L\'entreprise contient bien un prenom' do + expect(subject[:entreprise_prenom]).to eq('test_prenom') + end + end end - context 'Attributs Entreprises' do - it 'L\'entreprise contient bien un siren' do - expect(subject[:entreprise_siren]).to eq(siren) - end + context "when SIRET is KO" do + let(:body) { File.read('spec/fixtures/files/api_entreprise/entreprises_not_found.json') } + let(:status) { 206 } - it 'L\'entreprise contient bien un capital_social' do - expect(subject[:entreprise_capital_social]).to eq(462308) - end - - it 'L\'entreprise contient bien un numero_tva_intracommunautaire' do - expect(subject[:entreprise_numero_tva_intracommunautaire]).to eq('FR16418166096') - end - - it 'L\'entreprise contient bien une forme_juridique' do - expect(subject[:entreprise_forme_juridique]).to eq('SA à directoire (s.a.i.)') - end - - it 'L\'entreprise contient bien un forme_juridique_code' do - expect(subject[:entreprise_forme_juridique_code]).to eq('5699') - end - - it 'L\'entreprise contient bien un nom_commercial' do - expect(subject[:entreprise_nom_commercial]).to eq('OCTO-TECHNOLOGY') - end - - it 'L\'entreprise contient bien une raison_sociale' do - expect(subject[:entreprise_raison_sociale]).to eq('OCTO-TECHNOLOGY') - end - - it 'L\'entreprise contient bien un siret_siege_social' do - expect(subject[:entreprise_siret_siege_social]).to eq('41816609600051') - end - - it 'L\'entreprise contient bien un code_effectif_entreprise' do - expect(subject[:entreprise_code_effectif_entreprise]).to eq('31') - end - - it 'L\'entreprise contient bien une date_creation' do - expect(subject[:entreprise_date_creation]).to eq('Wed, 01 Apr 1998 00:00:00.000000000 +0200') - end - - it 'L\'entreprise contient bien un nom' do - expect(subject[:entreprise_nom]).to eq('test_nom') - end - - it 'L\'entreprise contient bien un prenom' do - expect(subject[:entreprise_prenom]).to eq('test_prenom') + it '#to_params class est une Hash ?' do + expect(subject).to eq({}) end end end From 7252c26e934b90027c409be152aebe25a3f73b41 Mon Sep 17 00:00:00 2001 From: Mathieu Magnin Date: Tue, 12 Mar 2019 17:36:16 +0100 Subject: [PATCH 2/2] Gem web-console is back ! --- Gemfile | 1 + Gemfile.lock | 7 +++++++ 2 files changed, 8 insertions(+) diff --git a/Gemfile b/Gemfile index ab377aff5..0d559692a 100644 --- a/Gemfile +++ b/Gemfile @@ -90,6 +90,7 @@ group :development do gem 'rubocop', require: false gem 'rubocop-rspec-focused', require: false gem 'scss_lint', require: false + gem 'web-console' gem 'xray-rails' end diff --git a/Gemfile.lock b/Gemfile.lock index 83db7583d..f2250a9da 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -101,6 +101,7 @@ GEM axlsx (>= 2.0, < 4) bcrypt (3.1.12) bindata (2.4.4) + bindex (0.5.0) bootstrap-sass (3.4.1) autoprefixer-rails (>= 5.2.1) sassc (>= 2.0.0) @@ -620,6 +621,11 @@ GEM vcr (4.0.0) warden (1.2.8) rack (>= 2.0.6) + web-console (3.7.0) + actionview (>= 5.0) + activemodel (>= 5.0) + bindex (>= 0.4.0) + railties (>= 5.0) webfinger (1.1.0) activesupport httpclient (>= 2.4) @@ -734,6 +740,7 @@ DEPENDENCIES typhoeus vcr warden + web-console webmock webpacker (>= 4.0.x) xray-rails