Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Moet blijkbaar ook niet wegens deprecated.

Moet blijkbaar ook niet wegens deprecated.

op de zelfde plaats als al de andere mp's http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (nergens dus alleen VH wordt getest.)

op de zelfde plaats als al de andere mp's (nergens dus alleen VH wordt getest.)

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

Opkuis van deze klasse is wenselijk.

Opkuis van deze klasse is wenselijk.

Probleempje met de TAB toets? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

Probleempje met de TAB toets?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Is deze test overbodig?

Is deze test overbodig?

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Waar wordt dit getest?

Waar wordt dit getest?

Unishop is uitgefaseerd --> renamen

Unishop is uitgefaseerd --> renamen

BeslagTechnikk ontbreekt

BeslagTechnikk ontbreekt

Veel duplicate code, ik maak een aparte issue aan

Veel duplicate code, ik maak een aparte issue aan

[ICT-774] EC: MP: BeslagTeknikk multipersonality opzetten
[ICT-774] EC: MP: BeslagTeknikk multipersonality opzetten
zie hierboven

zie hierboven

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

  • More
  • CR-227
  • finished reviewing
Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

beter de vhTest.Fake.TECH.Mail ?

beter de vhTest.Fake.TECH.Mail ?