ValideerPalletCodeResponse.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST4201] PA-UB-021: Exceptions-handling backend

- property mag weg, want zit in superclass

heb ineens nog een test toegevoegd voor het geval de ValidateHeader zelf een exception zou raisen

heb ineens nog een test toegevoegd voor het geval de ValidateHeader zelf een exception zou raisen

Ik had ook deze twijfel, maar heb toch voor een warning gekozen omdat er geen 'harde fout' optreedt naar de gebruiker toe. Ze zullen daar wel een melding krijgen, maar niet "vast" zitten. In dat ge...

Ik had ook deze twijfel, maar heb toch voor een warning gekozen omdat er geen 'harde fout' optreedt naar de gebruiker toe. Ze zullen daar wel een melding krijgen, maar niet "vast" zitten. In dat geval zullen we wel gecontacteerd worden en in de logs gaan zoeken.

GeneriekeResponse OF SpecifiekeMethodResponse ? Kunnen we hier eens over babbelen? Ik ben persoonlijk evenveel voor als tegen! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ico...

GeneriekeResponse OF SpecifiekeMethodResponse ?
Kunnen we hier eens over babbelen?
Ik ben persoonlijk evenveel voor als tegen!

Assert OmschrijvingVoorLogging ook nog.

Assert OmschrijvingVoorLogging ook nog.

Ik twijfel tussen "Warning" en "Error", of zelfs ErrorMail. Er is ten slotte iets foutgelopen, dat wellicht moet opgelost worden. Dus ik vrees dat bij Warning de boodschap zal verloren gaan.

Ik twijfel tussen "Warning" en "Error", of zelfs ErrorMail.
Er is ten slotte iets foutgelopen, dat wellicht moet opgelost worden. Dus ik vrees dat bij Warning de boodschap zal verloren gaan.

Mag hier ook weg, want zit in superklasse.

Mag hier ook weg, want zit in superklasse.

[UST4204] PA-UB-021: Exceptions-handling backend
[UST4204] PA-UB-021: Exceptions-handling backend
[UST4201] PA-UB-021: Exceptions-handling backend

- Antwoorden van gemeenschappelijke klasse laten afleiden

  1. … 5 more files in changeset.
Deze gaat binnenkort toch veranderd worden, aangezien het ook nodig gaat zijn om vhoss-kasten in te scannen

Deze gaat binnenkort toch veranderd worden, aangezien het ook nodig gaat zijn om vhoss-kasten in te scannen

Ik weet niet of het een stap te ver is, maar het lijkt me wel handig om het gemarkeerde stuk te verplaatsen naar een PalletCodeValidator-klasse Dan kan je de UT van de service opsplitsen, en daarb...

Ik weet niet of het een stap te ver is, maar het lijkt me wel handig om het gemarkeerde stuk te verplaatsen naar een PalletCodeValidator-klasse

Dan kan je de UT van de service opsplitsen, en daarbij heel proper met een mock de 3 situaties testen : PalletCodeValidator returns True , returns False en Return Exception.
En een aparte UT voor de PalletCodeValidator, die niks te maken heeft met HeaderValidator en Request-dto, gewoon PalletCode as %String

PalletCode toevoegen aan melding. Ook in de UT controleren dat de PalletCode aanwezig is.

PalletCode toevoegen aan melding. Ook in de UT controleren dat de PalletCode aanwezig is.

[UST4173] PA-UB-005: DnaCode service opzetten
[UST4173] PA-UB-005: DnaCode service opzetten
[UST4173] PA-UB-005: DnaCode service opzetten

- Al wat skeletoncode + wsdl impact

    • -0
    • +13
    ./ValideerPalletCodeResponse.cls.xml
  1. … 4 more files in changeset.