Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Zelfde opmerkingen vorige method http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Zelfde opmerkingen vorige method

GeefDoosLade() beter in een variabele steken, want nu wordt die 7x herhaald/berekend in één testmethod.

GeefDoosLade() beter in een variabele steken, want nu wordt die 7x herhaald/berekend in één testmethod.

F-lade --> dat mag je gerust vermelden in de methodnaam; bvb. "Test: Hoge ondiepe niet-binnenlade verpakt in ... " of "Test: niet-binnenlade F-hoogte verpakt in ... "

F-lade --> dat mag je gerust vermelden in de methodnaam;
bvb. "Test: Hoge ondiepe niet-binnenlade verpakt in ... " of "Test: niet-binnenlade F-hoogte verpakt in ... "

Hier in de UT zou ik de eerder "VerwachteAfmetingK.." noemen i.p.v. DefaultAfm... Je gaat deze immers asserten. Ik denk zelfs dat "K1=100" geen default waarde is, maar effectief een berekende waar...

Hier in de UT zou ik de eerder "VerwachteAfmetingK.." noemen i.p.v. DefaultAfm...
Je gaat deze immers asserten.

Ik denk zelfs dat "K1=100" geen default waarde is, maar effectief een berekende waarden.

Beter: GeefIngegevenKenmerken() kan je direct aan MaatwerkProduct vragen.

Beter: GeefIngegevenKenmerken() kan je direct aan MaatwerkProduct vragen.

[UST4239] TAXFP1: Productie-aansturing - Kartonmachine:
[UST4239] TAXFP1: Productie-aansturing - Kartonmachine:
[UST4239] TAXFP1: Productie-aansturing - Kartonmachine:

- Uitbreiding enum met Flatpack

    • -0
    • +8
    ./BenamingVerpakkingsOnderdeel/Verwerker.cls.xml
    • -0
    • +7
    ./BenamingVerpakkingsOnderdeel/AutoComplete.cls.xml
    • -2
    • +9
    ./BenamingVerpakkingsOnderdeel.cls.xml
Ter info : 1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls 2) Testklasse bevat meestal ook de naam van de publi...

Ter info :
1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls
2) Testklasse bevat meestal ook de naam van de public method die je aan 't testen bent. Maar misschien werd de naam dan te lang.
Je hoeft dit hier niet te veranderen, maar is taak van reviewer om dit op te merken

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk() Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan d...

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk()
Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan dubbel? --> dubbel onderhoud.
Misschien kan je hier (BOG) de implementatie van LosCompInBulkk laten oproepen.

[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket...
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket...
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- Aanpassing verpakkingsonderdeelnamen => nu mee op enum

    • -2
    • +37
    ./BenamingVerpakkingsOnderdeel.cls.xml
    • -0
    • +40
    ./BenamingVerpakkingsOnderdeel/Verwerker.cls.xml
    • -12
    • +42
    ./BenamingVerpakkingsOnderdeel/AutoComplete.cls.xml
  1. … 9 more files in changeset.
no_comment
    • -0
    • +5
    ./BenamingVerpakkingsOnderdeel/AutoComplete.cls.xml
[UST1326] TAX-PAS-LC: Wraparound-doos

- SnijDoosNaam voor wraparaound doos toevoegen en in gebruik nemen

    • -1
    • +9
    ./BenamingVerpakkingsOnderdeel/Verwerker.cls.xml
    • -0
    • +7
    ./BenamingVerpakkingsOnderdeel/AutoComplete.cls.xml
    • -2
    • +9
    ./BenamingVerpakkingsOnderdeel.cls.xml
  1. … 4 more files in changeset.
[TAX-BOG] - Verpakking naar DOM
    • -5
    • +0
    ./BenamingVerpakkingsOnderdeel/AutoComplete.cls.xml
    • -0
    • +72
    ./BenamingVerpakkingsOnderdeel/Verwerker.cls.xml
  1. … 17 more files in changeset.
[TAX-BOG] - Verpakking naar DOM
    • -0
    • +61
    ./BenamingVerpakkingsOnderdeel/AutoComplete.cls.xml
    • -0
    • +59
    ./BenamingVerpakkingsOnderdeel.cls.xml
  1. … 56 more files in changeset.