DGNum/demarches-normaliennes
13
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

add fix for nil values on numeroVoie and typeVoie

Browse source
This commit is contained in:
clemkeirua 2020-09-22 09:26:46 +02:00
parent 1173f1e459
commit 412a87e532
2 changed files with 14 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

9
app/graphql/types/personne_morale_type.rb
View file

@ -7,7 +7,7 @@ module Types
end end
field :siren, String, null: false field :siren, String, null: false
field :capital_social, GraphQL::Types::BigInt, null: false field :capital_social, GraphQL::Types::BigInt, null: false, description: "capital social de l’entreprise. -1 si inconnu."
field :numero_tva_intracommunautaire, String, null: false field :numero_tva_intracommunautaire, String, null: false
field :forme_juridique, String, null: false field :forme_juridique, String, null: false
field :forme_juridique_code, String, null: false field :forme_juridique_code, String, null: false
@ -48,7 +48,8 @@ module Types
end end
def code_effectif_entreprise def code_effectif_entreprise
object.code_effectif_entreprise || '' # we need this in order to bypass Hashie::Dash deserialization issue on nil values
object.code_effectif_entreprise
end end
def effectif_annuel def effectif_annuel
@ -86,8 +87,8 @@ module Types
field :naf, String, null: false field :naf, String, null: false
field :libelle_naf, String, null: false field :libelle_naf, String, null: false
field :adresse, String, null: false field :adresse, String, null: false
field :numero_voie, String, null: false field :numero_voie, String, null: true
field :type_voie, String, null: false field :type_voie, String, null: true
field :nom_voie, String, null: false field :nom_voie, String, null: false
field :complement_adresse, String, null: false field :complement_adresse, String, null: false
field :code_postal, String, null: false field :code_postal, String, null: false

11
spec/controllers/api/v2/graphql_controller_spec.rb
View file

@ -396,6 +396,8 @@ describe API::V2::GraphqlController do
... on PersonneMorale { ... on PersonneMorale {
siret siret
siegeSocial siegeSocial
numeroVoie
typeVoie
entreprise { entreprise {
siren siren
dateCreation dateCreation
@ -421,11 +423,13 @@ describe API::V2::GraphqlController do
id: dossier.etablissement.to_typed_id, id: dossier.etablissement.to_typed_id,
siret: dossier.etablissement.siret, siret: dossier.etablissement.siret,
siegeSocial: dossier.etablissement.siege_social, siegeSocial: dossier.etablissement.siege_social,
numeroVoie: dossier.etablissement.numero_voie.to_s,
typeVoie: dossier.etablissement.type_voie.to_s,
entreprise: { entreprise: {
siren: dossier.etablissement.entreprise_siren, siren: dossier.etablissement.entreprise_siren,
dateCreation: dossier.etablissement.entreprise_date_creation.iso8601, dateCreation: dossier.etablissement.entreprise_date_creation.iso8601,
capitalSocial: dossier.etablissement.entreprise_capital_social.to_s, capitalSocial: dossier.etablissement.entreprise_capital_social.to_s,
codeEffectifEntreprise: dossier.etablissement.entreprise_code_effectif_entreprise.to_s codeEffectifEntreprise: dossier.etablissement.entreprise_code_effectif_entreprise.to_s,
} }
} }
}) })
@ -434,7 +438,8 @@ describe API::V2::GraphqlController do
context "when there are missing data" do context "when there are missing data" do
before do before do
dossier.etablissement.update!(entreprise_code_effectif_entreprise: nil, entreprise_capital_social: nil) dossier.etablissement.update!(entreprise_code_effectif_entreprise: nil, entreprise_capital_social: nil,
numero_voie: nil, type_voie: nil)
end end
it "should be returned" do it "should be returned" do
@ -450,6 +455,8 @@ describe API::V2::GraphqlController do
id: dossier.etablissement.to_typed_id, id: dossier.etablissement.to_typed_id,
siret: dossier.etablissement.siret, siret: dossier.etablissement.siret,
siegeSocial: dossier.etablissement.siege_social, siegeSocial: dossier.etablissement.siege_social,
numeroVoie: nil,
typeVoie: nil,
entreprise: { entreprise: {
siren: dossier.etablissement.entreprise_siren, siren: dossier.etablissement.entreprise_siren,
dateCreation: dossier.etablissement.entreprise_date_creation.iso8601, dateCreation: dossier.etablissement.entreprise_date_creation.iso8601,