Projet

Général

Profil

Development #72638

dpark: ne pas forcer le ServiceBinding

Ajouté par Benjamin Dauvergne il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Benjamin Dauvergne
Version cible:
-
Début:
20 décembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Il varie entre les instances de DPark (intégration/recette/prod).

Fichiers

0001-dpark-remove-hardcoding-of-the-service-binding-72638.patch (39,7 ko) 0001-dpark-remove-hardcoding-of-the-service-binding-72638.patch Benjamin Dauvergne, 20 décembre 2022 12:01
0001-dpark-remove-hardcoding-of-the-service-binding-72638.patch (39,8 ko) 0001-dpark-remove-hardcoding-of-the-service-binding-72638.patch Benjamin Dauvergne, 20 décembre 2022 12:01
0002-dpark-remove-hardcoding-of-the-service-binding-72638.patch (56,1 ko) 0002-dpark-remove-hardcoding-of-the-service-binding-72638.patch Benjamin Dauvergne, 20 décembre 2022 15:24
0001-tests-improve-ResponsesSoap-72638.patch (5,32 ko) 0001-tests-improve-ResponsesSoap-72638.patch Benjamin Dauvergne, 20 décembre 2022 15:24
0002-dpark-remove-hardcoding-of-the-service-binding-72638.patch (56,1 ko) 0002-dpark-remove-hardcoding-of-the-service-binding-72638.patch Benjamin Dauvergne, 20 décembre 2022 15:43
0001-tests-improve-ResponsesSoap-72638.patch (5,48 ko) 0001-tests-improve-ResponsesSoap-72638.patch Benjamin Dauvergne, 20 décembre 2022 15:43
0002-dpark-remove-hardcoding-of-the-service-binding-72638.patch (56,1 ko) 0002-dpark-remove-hardcoding-of-the-service-binding-72638.patch Benjamin Dauvergne, 21 décembre 2022 13:03
0001-tests-improve-ResponsesSoap-72638.patch (9,33 ko) 0001-tests-improve-ResponsesSoap-72638.patch Benjamin Dauvergne, 21 décembre 2022 13:03

Révisions associées

Révision c8155b70 (diff)
Ajouté par Benjamin Dauvergne il y a plus d'un an

tests: improve ResponsesSoap (#72638)

  • add support for overriden service binding address
  • encode wsdl_content when it is not bytes() but str()
  • keep deserialized SOAP requests bodies in mock.soap_requests for
    inspection
  • allow nesting ResponsesSoap context manager by using the same
    RequestsMock context manager.

Révision 654e2f22 (diff)
Ajouté par Benjamin Dauvergne il y a plus d'un an

dpark: remove hardcoding of the service binding (#72638)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a plus d'un an

#4

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Il faut que je teste ça directement en recette, les test unitaires ne sont pas adaptés à vérifier le code dans get_service() (ou alors je pourrait voir de tester ça avec le nouveau code de test SOAP introduit pour maelis, je vais voir).

#7

Mis à jour par Nicolas Roche il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

Je valide.
Je note qu'il y aura un conflit avec le second patch de #72563.
J'y ai posé un 4ème patch pour expliquer ce qui risque de coincer si je pousse le patch avant.

#8

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Voilà, j'ai ajouté la possibilité d'imbriquer les context manager ResponsesSoap en passant un requests_mock explicite au constructeur qui pourra être partagé.

#9

Mis à jour par Benjamin Dauvergne il y a plus d'un an

warning corrigé sur la branche.

#10

Mis à jour par Nicolas Roche il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

Ack.

#11

Mis à jour par Benjamin Dauvergne il y a plus d'un an

Je vais pousser la partie sur les tests, le reste concernant dpark vendredi.

#12

Mis à jour par Benjamin Dauvergne il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 654e2f22a9a455649b13546530118976f212ad01
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Dec 20 12:00:57 2022 +0100

    dpark: remove hardcoding of the service binding (#72638)
#13

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#14

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF