Fix Sentry 165 : api crash when siret is unknown

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