Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3136] TAX - only's - geen V1-doos meer berekenen (productiewijze <> volledig)

- Niet alles van karton uitschakelen uiteraard ;)

  1. … 2 more files in changeset.
[ICT-2129] PM: Maatwerk: dev MERIVOBOX voor De Decker (MVX2/DEV)

- Testen updaten sinds spoelbakwijzigingen bij matten enkel voor LBX zijn, niet voor MVX, fake LBX gebruiken zodat de de testen voor spoelbak nog slagen

    • -0
    • +67
    ./impl/FakeKenmerken/Lbx.cls.xml
  1. … 2 more files in changeset.
Goed opgemerkt, Jelle. Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan. @ iedereen (vooral de jonkies) : Bovenal is de vraag : is er geen UT die dit aan het licht ...

Goed opgemerkt, Jelle.
Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan.

@ iedereen (vooral de jonkies) :
Bovenal is de vraag : is er geen UT die dit aan het licht brengt?
Zal zelf al deels antwoorden :
Deze klasse BL.MB.UP.Receptie.cls bevat stokoude implementatie, en er is nauwelijks sprake van "scheiding van dependencies".
Dit laatste maakt het dan ook zeer moeilijk om UT hiervoor te schrijven.
@Tom : misschien toch een systeemke bedenken om de Dna-complexiteit af te zonderen in een helper-klasse, zodat die alsnog kan ge-UT-ed worden :
bvb. DnaCodeSoortBepaler .IsVoorKad() .IsVoorVhoss(), .IsVoorSpaceStep()
( *) woordkeuze "...Soort..." kan vast beter!

Geen Asserts na een "VerwachteExceptie". Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen. @iedereen : deze opmerking a...

Geen Asserts na een "VerwachteExceptie".
Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen.

@iedereen : deze opmerking altijd maken als reviewer, wanneer iemand hiertegen zondigt. En als developer hier best zelf proberen op te letten :-P

Vind dit persoonlijk wat moeilijk om allemaal te begrijpen wat er juist gebeurt met de globals

Vind dit persoonlijk wat moeilijk om allemaal te begrijpen wat er juist gebeurt met de globals

Waarvoor wordt DnaCode4 gebruikt?

Waarvoor wordt DnaCode4 gebruikt?

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest" #dim IngegevenKenmerken As DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken = ##class(vhTest.Dummy.DOM.PM.Maatwerk.Calc.Common.impl.LBXKenm...

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

#dim IngegevenKenmerken As DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken = ##class(vhTest.Dummy.DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken).DummyLade()
Heb hier ook over zitten peinzen. Probleem is dat voor dit kaartje heel het systeem uitging van 1 DnaCode per Product. Nu maken we 2 DnaCodes voor 1 Product. Was niet van zin om op dit moment dat p...

Heb hier ook over zitten peinzen. Probleem is dat voor dit kaartje heel het systeem uitging van 1 DnaCode per Product. Nu maken we 2 DnaCodes voor 1 Product. Was niet van zin om op dit moment dat principe overal te gaan overhoop halen en daarom heb ik het hier maar in deze method gestoken. Op zich blijven de oproepers van MaakDnaCode-methods dan in het ongewisse.

Staan al bovenaan gedefinieerd in 'OneBeforeOneTest'         Do ..ProductTypeApiStub.IndienMethodCall("IsLegraboxProduct", 1000).DanReturn($$$False)

Staan al bovenaan gedefinieerd in 'OneBeforeOneTest'

        Do ..ProductTypeApiStub.IndienMethodCall("IsLegraboxProduct", 1000).DanReturn($$$False)
heb dat in volgende kaartje aangepast. dingen liepen tegelijk (yesyes, kleine kaartjes en rap committen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

heb dat in volgende kaartje aangepast. dingen liepen tegelijk (yesyes, kleine kaartjes en rap committen )

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

Productnummer wordt vergeleken met boolean?

Productnummer wordt vergeleken met boolean?

Toevoegen dat de method een 'boolean' teruggeeft

Toevoegen dat de method een 'boolean' teruggeeft

Ik denk dat je nog eens moet denken over het verwijderen van het IsKlantEtiketSpaceStep argument. Je verwijdert hem uit de argumenten maar niet uit de binnenkomende parameters in deze method. Ook i...

Ik denk dat je nog eens moet denken over het verwijderen van het IsKlantEtiketSpaceStep argument. Je verwijdert hem uit de argumenten maar niet uit de binnenkomende parameters in deze method. Ook in de method MagDnaCodeBijgewerktWorden is die niet opgekuist en wordt die wel nog gebruikt?

Door het toevoegen van deze method is er wel wat duplicated code ontstaan om een dna code te maken. Ik laat de keuze aan jou of je een soort van dnaCodeInvullen method maakt of niet. Voor mij kan d...

Door het toevoegen van deze method is er wel wat duplicated code ontstaan om een dna code te maken. Ik laat de keuze aan jou of je een soort van dnaCodeInvullen method maakt of niet. Voor mij kan dit er nog mee door omdat het maar 2 methods zijn uiteindelijk

[ICT-2115] pa-ub-213: spacestep en dnacodes
[ICT-2115] pa-ub-213: spacestep en dnacodes
[ICT-2115] pa-ub-213: spacestep en dnacodes

- IngegevenKenmerkenSpaceStepHelper mockable gemaakt

    • -0
    • +67
    ./impl/IngegevenKenmerkenSpaceStepHelper.cls.xml
  1. … 2 more files in changeset.
[EPIC52][UST2878] Conversie VoorWandType (DOM <> APP):

- Conversie dto -> dom

    • -0
    • +18
    ./impl/LBXKenmerkenFactory.cls.xml
  1. … 12 more files in changeset.
[UST1820] Hein: LBX en TAX: Afmeting op maatwerklijn bewaren

- Testen voor LBX V1 en TAX V1

- Halffabboom meegeven vanuit winkelkar zodat deze niet in elke klasse opgehaald moet worden.

- Halffabboom en ingegevenkenmerken worden via contractuor meegegeven daarvoor refactoring.

  1. … 18 more files in changeset.
[UST1557] TAX export : Verpakking afmeting van een maatwerklijn bepalen.

- Fake klassen ipv Dummy.

- Extra test voor niet TAX Maatwerklijnen (LBX)

    • -0
    • +29
    ./impl/TAORKenmerken.cls.xml
    • -0
    • +29
    ./impl/LBXKenmerken.cls.xml
  1. … 2 more files in changeset.