CacheAdminA_trunk2010/vhUnitTest/BL/MB/UGLYPicking/Receptie/BuildBufferViaDnaCode

Test.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 27 Sep 21
Commits this week: 0
Total Lines of Code (LoC): 267
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 27 Sep 2021
3:14 pm

Wim Vermeulen

[ICT] Bugfix : Falende UT "TestGeefToeleveringenNietInBatch" veroorzaakt door slechte cleanup in andere UT

- cleanup in OnAfterOneTest bij BuildBufferViaDnaCode.Test aangepast : index ^KTO1(...) alleen opkuisen indien de volledige KTO(ToeNr) niet meer bestaat.

Options
    • -1
    • +3
    ./Test.cls.xml
Thursday 11 Feb 2021
10:27 am

Tom Vermeulen

Wednesday 27 Jan 2021
7:25 pm

Wim Vermeulen

7:24 pm

Wim Vermeulen

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!

6:59 pm

Wim Vermeulen

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

Tuesday 26 Jan 2021
3:14 pm

Peter Vercoutter and Jelle Baele

11:31 am

Jelle Baele

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

11:23 am

Jelle Baele

Waarvoor wordt DnaCode4 gebruikt?

Waarvoor wordt DnaCode4 gebruikt?

11:11 am

Jelle Baele

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()
11:08 am

Tom Vermeulen

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.

11:05 am

Jelle Baele

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)
11:03 am

Tom Vermeulen

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 )

10:58 am

Jelle Baele

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

10:57 am

Tom Vermeulen

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

- foutje in StringUtils.Equals gebruik

- deze keer met testje

Options
    • -0
    • +49
    ./Test.cls.xml
  2. … 1 more file in changeset.
10:40 am

Jelle Baele

Productnummer wordt vergeleken met boolean?

Productnummer wordt vergeleken met boolean?

9:58 am

Jelle Baele

Toevoegen dat de method een 'boolean' teruggeeft

Toevoegen dat de method een 'boolean' teruggeeft

9:06 am

Peter Vercoutter

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?

9:00 am

Peter Vercoutter

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

8:32 am

Tom Vermeulen

Tuesday 19 Jan 2021
1:35 pm

Tom Vermeulen

[ICT-2115] pa-ub-213: spacestep en dnacodes
[ICT-2115] pa-ub-213: spacestep en dnacodes
Friday 08 Jan 2021
8:38 am

Tom Vermeulen

Tuesday 21 Apr 2020
2:36 pm

Tom Vermeulen

Die zal niet meer getoond worden in het antwoord

Die zal niet meer getoond worden in het antwoord

1:42 pm

Tom Vermeulen

had ik bewust niet gedaan omdat al de rest van deze klasse procedureblock=0 is en ik geen extra can of worms wou openen

had ik bewust niet gedaan omdat al de rest van deze klasse procedureblock=0 is en ik geen extra can of worms wou openen

10:00 am

Wim Vermeulen

10:00 am

Wim Vermeulen

Wat als de tweede toelevering wel reeds volledig gescand is? Kan het zijn dat deze dan nog in de buffer zit, en getoond wordt met aantalGescand : 1/1 ? Of is de toelevering dan weg, als in "niet me...

Wat als de tweede toelevering wel reeds volledig gescand is? Kan het zijn dat deze dan nog in de buffer zit, en getoond wordt met aantalGescand : 1/1 ?
Of is de toelevering dan weg, als in "niet meer vermeld in het Antwoord-object ?
--> geen idee of het zinvol is om deze situatie als TestMethod toe te voegen.

Monday 20 Apr 2020
5:45 pm

Wim Vermeulen

Zie opmerking i.v.m. de methodnaam aan de Caché-kant. Hier is de methodnaam al iets dichter bij de werkelijkheid

Zie opmerking i.v.m. de methodnaam aan de Caché-kant.
Hier is de methodnaam al iets dichter bij de werkelijkheid

5:04 pm

Wim Vermeulen

VH-naming conventies : ook in Delphi ? --> RondAfKaderdeurDag

VH-naming conventies : ook in Delphi ?
--> RondAfKaderdeurDag

5:02 pm

Wim Vermeulen

Olaahh ... dit is een randgeval, kwestie van de code in UI of in de Backend te schrijven.

Olaahh ...
dit is een randgeval, kwestie van de code in UI of in de Backend te schrijven.

4:57 pm

Wim Vermeulen

"ZoakGebr.... " is dat een sint-niklaases dialect? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Bij de WSimpl heb ik tevens een opmerking gemaakt over...

"ZoakGebr.... " is dat een sint-niklaases dialect?
Bij de WSimpl heb ik tevens een opmerking gemaakt over de naam van deze method. Zie WSimpl.

4:54 pm

Wim Vermeulen

Deze naam is eerder verwarrend, vind ik : "Zoek..." en je geeft geen zoek-criteria op, er wordt dus ook niks gefilterd (lees: gezocht) Wat dacht je van "GeefAlleAangemeldeGebruikers" of zoiets? Al ...

Deze naam is eerder verwarrend, vind ik : "Zoek..." en je geeft geen zoek-criteria op, er wordt dus ook niks gefilterd (lees: gezocht)
Wat dacht je van "GeefAlleAangemeldeGebruikers" of zoiets?
Al dan niet met gevolgen voor de klasse WS.Halux.AAP.GebruikerService en het WSDL-gedoe.

Eens effe overleggen?
.