PizziPayment / PizziAPIDB

API's BDD
0 stars 0 forks source link

Add transaction workflow #22

Closed Harmos274 closed 2 years ago

Harmos274 commented 2 years ago

Inclus:

@agarof @Krapaince

Harmos274 commented 2 years ago

oui ça passe pas les tests CI c'est assez logique

Krapaince commented 2 years ago

oui ça passe pas les tests CI c'est assez logique

Et pourquoi ça d'abord?

Harmos274 commented 2 years ago

oui ça passe pas les tests CI c'est assez logique

Et pourquoi ça d'abord?

bah les tests de pizziDB n'initialisent pas une DB ils s'y connectent juste tu veux que je change ça ?

Krapaince commented 2 years ago

oui ça passe pas les tests CI c'est assez logique

Et pourquoi ça d'abord?

bah les tests de pizziDB n'initialisent pas une DB ils s'y connectent juste tu veux que je change ça ?

Ha si pour le coup ça initialize une DB cependant vu que DeployDB n'est pas à jour sur master par rapport à ici les transactions doivent échouées je suppose.

Harmos274 commented 2 years ago

Ah oui ça fait toujours sens du coup, ça veut dire qu'à chaque fois qu'on met à jour les modèles sequelize ça passera jamais les tests, faudrait changer ça