CacheAdminA_trunk2010/vhUnitTest/APPS/Halux/AAP/DnaCodeService

WijzigAantalDnaCodesVoorToeleveringLijn

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 29 Apr 19
Commits this week: 0
Total Lines of Code (LoC): 172
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
Thursday 07 Jul 2022
8:55 am

Joannes Lemmens

Wednesday 15 Jun 2022
8:25 am

Jelle Baele

Wednesday 08 Jun 2022
8:37 am

Joannes Lemmens

Kleine update na de fout van vorige keer. Als dit aanvaard wordt ga ik het nog zeker wel op Accept testen, deze keer met AfdrukTijdstip ook in het oog te houden! http://subversion02.vanhoecke.be/st...

Kleine update na de fout van vorige keer.
Als dit aanvaard wordt ga ik het nog zeker wel op Accept testen, deze keer met AfdrukTijdstip ook in het oog te houden!

8:35 am

Joannes Lemmens

Dit is een herschrijving van het volgende: If Resultaat { Set Resultaat = ((DnaCode.AfdrukTijdstip = "") || Herafdruk) } In beide gevallen is: *Resultaat = True -> Resultaat = ((DnaCode.AfdrukT...

Dit is een herschrijving van het volgende:

If Resultaat {
	Set Resultaat = ((DnaCode.AfdrukTijdstip = "") || Herafdruk)
}

In beide gevallen is:

  • Resultaat = True -> Resultaat = ((DnaCode.AfdrukTijdstip = "") || Herafdruk)
  • Resultaat = False -> False
Tuesday 07 Jun 2022
1:08 pm

Joannes Lemmens

[ICT-2482] [rvJBA] Uitbreiding DNA codes voor mat op maat uit meerdere stukken:
[ICT-2482] [rvJBA] Uitbreiding DNA codes voor mat op maat uit meerdere stukken:
Tuesday 31 May 2022
1:38 pm

Joannes Lemmens

Getest op Accept en het lijkt te werken! *Matten voor niet-spoelbaklades krijgen maar 1 DnaCode *Matten voor spoelbaklade TBX krijgt er 3 en voor LBX (Free/Pure) 2 *SpaceStep blijft een extra Dna...

Getest op Accept en het lijkt te werken!

  • Matten voor niet-spoelbaklades krijgen maar 1 DnaCode
  • Matten voor spoelbaklade TBX krijgt er 3 en voor LBX (Free/Pure) 2
  • SpaceStep blijft een extra DnaCode genereren met speciale productcode afhankelijk van ladetype
Friday 08 Apr 2022
9:52 am

Joannes Lemmens

Ik heb hiervoor een aantal parameters gemaakt die doorheen alle tests gebruikt worden, hopelijk is dit een beetje in de aard van wat je bedoelt

Ik heb hiervoor een aantal parameters gemaakt die doorheen alle tests gebruikt worden, hopelijk is dit een beetje in de aard van wat je bedoelt

9:14 am

Joannes Lemmens

De enige reden dat ik kan zien is omdat de %New dan bij de (default) IndienMethodCalls staat, maar eigenlijk is het beter in de OnBeforeAll inderdaad http://subversion02.vanhoecke.be/static/ogdo0b/...

De enige reden dat ik kan zien is omdat de %New dan bij de (default) IndienMethodCalls staat, maar eigenlijk is het beter in de OnBeforeAll inderdaad
Edit: bij nader inzien heb ik extra methods gemaakt voor de verschillende varianten van hoe de ProductTypeApiStub wordt opgezet. Zo wordt het ook maar 1x opgeroepen per test, i.p.v. telkens overschreven te worden.

Thursday 07 Apr 2022
3:04 pm

Jelle Baele

Ik zou dit gedrag zeker ook nog eens op accept testen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik zou dit gedrag zeker ook nog eens op accept testen

2:58 pm

Jelle Baele

Je zou deze, samen met de andere variabelen in algemene properties kunnen steken en deze overal gebruiken zodat het duidelijk wordt vanwaar deze komen. Maakt het iets duidelijker want nu lijkt het ...

Je zou deze, samen met de andere variabelen in algemene properties kunnen steken en deze overal gebruiken zodat het duidelijk wordt vanwaar deze komen. Maakt het iets duidelijker want nu lijkt het bijna 'at random'

2:56 pm

Jelle Baele

Is er een reden waarom deze in de 'OnBeforeOneTest' staat?

Is er een reden waarom deze in de 'OnBeforeOneTest' staat?

2:55 pm

Jelle Baele

Zelfde opmerking als hierboven http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zelfde opmerking als hierboven

2:54 pm

Jelle Baele

Ik zou deze dan net niet 'Stub' noemen, want deze wordt enkel teruggegeven bij een 'indienMethodCall'

Ik zou deze dan net niet 'Stub' noemen, want deze wordt enkel teruggegeven bij een 'indienMethodCall'

2:51 pm

Jelle Baele

Aangezien je hier een 'IndienMethodCall' op zal doen, kan je deze ook 'BatchStub' noemen

Aangezien je hier een 'IndienMethodCall' op zal doen, kan je deze ook 'BatchStub' noemen

Wednesday 23 Feb 2022
3:22 pm

Joannes Lemmens

3:21 pm

Joannes Lemmens

Ik had eerst van TECH.Mockable afgeleid, maar later dat ongedaan gemaakt. Vandaar dat dit er dus zo bij staat.

Ik had eerst van TECH.Mockable afgeleid, maar later dat ongedaan gemaakt. Vandaar dat dit er dus zo bij staat.

Tuesday 01 Feb 2022
2:11 pm

Joannes Lemmens

[ICT-2482] [rvJBA] Uitbreiding DNA codes voor mat op maat uit meerdere stukken:

- Unit tests aangepast ter voorbereiding (IsAntislipmatProduct -> False)

Options
    • -0
    • +2
    ./Test.cls.xml
  2. … 1 more file in changeset.
Thursday 28 Oct 2021
11:38 am

Jelle Baele

10:15 am

Tom Vermeulen

Tuesday 19 Oct 2021
12:49 pm

Tom Vermeulen

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor co...

detail, maar hier kan je gewoon Quit ##class(TECH.ListIterator).%New(DnaCodeList) gebruiken. Het dimmen in een aparte variabele heeft niet echt meerwaarde omdat de dim bij cache enkel zorgt voor code-completion

Wednesday 13 Oct 2021
3:12 pm

Jelle Baele

3:10 pm

Jelle Baele

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
3:09 pm

Jelle Baele

3:04 pm

Jelle Baele

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket
Friday 08 Oct 2021
2:41 pm

Jelle Baele

[ICT-2520] [rvTVE] Sommige referenties van ladetype ontbreken soms op ProboxinhoudEtiket

- Extra veld 'OrderlijnId' toevoegen in DnaCode

- Dit extra veld opvullen in DnaCodeService bij aanmaken van een DnaCode

Options
    • -0
    • +2
    ./Test.cls.xml
  2. … 4 more files in changeset.
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