Corrige le crash de l'app lors d'un SIRET en erreur | Merge pull request #3604 from betagouv/fix-sentry-165

Corrige le crash de l'app lors d'un SIRET en erreur

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
 

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

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

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
+}

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