CacheAdminA_trunk2010/vhTest/Fake/APPS/Halux/PPS

Document

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 30 Apr 19
Commits this week: 0
Total Lines of Code (LoC): 135
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 11 Jan 2024
10:03 am

Jelle Baele

Wednesday 10 Jan 2024
4:51 pm

Jelle Baele

Monday 23 Oct 2023
3:24 pm

Peter Vercoutter

3:20 pm

Peter Vercoutter

3:16 pm

Jelle Baele

3:15 pm

Jelle Baele

3:15 pm

Jelle Baele

[ICT-3795] VHLOG MES M1 - (4) trigger voor verzenden productie-orders naar OBJT:
[ICT-3795] VHLOG MES M1 - (4) trigger voor verzenden productie-orders naar OBJT:
Thursday 07 Sep 2023
10:45 am

Jelle Baele

10:44 am

Jelle Baele

[ICT-3795] VHLOG MES M1 - (4) trigger voor verzenden productie-orders naar OBJT:
[ICT-3795] VHLOG MES M1 - (4) trigger voor verzenden productie-orders naar OBJT:
10:41 am

Jelle Baele

10:39 am

Jelle Baele

[ICT-3794] VHLOG MES M1 - (3) nieuwe aansturingactie 'batch naar OBJT':
[ICT-3794] VHLOG MES M1 - (3) nieuwe aansturingactie 'batch naar OBJT':
Monday 11 May 2020
8:12 am

Peter Vercoutter

Friday 08 May 2020
1:53 pm

Tom Vermeulen

Monday 04 May 2020
9:34 am

Peter Vercoutter

9:34 am

Peter Vercoutter

[ICT-1459] HLX: aanpassing sortering pickingblad zijkanten LBX
[ICT-1459] HLX: aanpassing sortering pickingblad zijkanten LBX
Wednesday 29 Apr 2020
3:56 pm

Peter Vercoutter

3:52 pm

Peter Vercoutter

Dit is nog een restantje van TBX picking en boeit niet meer volgens NBO.

Dit is nog een restantje van TBX picking en boeit niet meer volgens NBO.

1:57 pm

Tom Vermeulen

1:52 pm

Tom Vermeulen

De Bak-informatie is verdwenen, is dat juist? (ook nog in andere testbestanden)

De Bak-informatie is verdwenen, is dat juist? (ook nog in andere testbestanden)

1:16 pm

Peter Vercoutter

[ICT-1459] HLX: aanpassing sortering pickingblad zijkanten LBX
[ICT-1459] HLX: aanpassing sortering pickingblad zijkanten LBX
Tuesday 25 Feb 2020
11:58 am

Peter Vercoutter

Monday 24 Feb 2020
4:12 pm

Tommy Hebb

4:12 pm

Tommy Hebb

Mag gerust ingekort worden naar: "Test: Zet 0 karakters van korttekst in bold indien TBX" Idem voor die van TAX hierna.

Mag gerust ingekort worden naar:
"Test: Zet 0 karakters van korttekst in bold indien TBX"

Idem voor die van TAX hierna.

4:11 pm

Tommy Hebb

Ter verduidelijking zou je kunnen werken met 3 lokale variabelen (booleans). Dan kun je iets beschrijvends meegeven aan de method GeefProductDetailLijnen, waardoor degene die na jou komt (of jijzel...

Ter verduidelijking zou je kunnen werken met 3 lokale variabelen (booleans). Dan kun je iets beschrijvends meegeven aan de method GeefProductDetailLijnen, waardoor degene die na jou komt (of jijzelf als het lang geleden is dat je de code gezien hebt) niet naar die private method moet gaan kijken om te weten waarvoor er booleans worden doorgegeven.

4:07 pm

Tommy Hebb

Dit is de assert die er toe doet. Aangezien het hier om stringmanipulatie gaat kun je er best voor proberen zorgen dat ofwel: *deze zo waarheidsgetrouw mogelijk is door echte kortteksten te gebru...

Dit is de assert die er toe doet. Aangezien het hier om stringmanipulatie gaat kun je er best voor proberen zorgen dat ofwel:

  • deze zo waarheidsgetrouw mogelijk is door echte kortteksten te gebruiken.
  • deze zodanig ineen te steken dat datgene wat getest wordt zo snel en duidelijk mogelijk weergegeven wordt.


Je zou dit, in samenspel met de opmerking over de naamgeving van de test, zo kunnen aanpakken dat je deze kortteksten begint met een kleine nummerreeks, gevolgd door iets beschrijvends. Bv:
"123456789 frontbevestiging Exp T"
Dat zou dan resulteren in:
"123<b>4</b>56<b>7</b>89 frontbevestiging Exp T"
Zo is het direct duidelijk dat karakters 4 en 7 in bold komen te staan bij frontbevestiging Exp T.

4:02 pm

Tommy Hebb

Je moet niet testen of het V1 is, want je WEET al dat het V1 is, want je hebt in de setup van de test ervoor gezorgd dat het ZEKER V1 is, dus deze assert mag er overal uit.

Je moet niet testen of het V1 is, want je WEET al dat het V1 is, want je hebt in de setup van de test ervoor gezorgd dat het ZEKER V1 is, dus deze assert mag er overal uit.

4:00 pm

Tommy Hebb

"Zet 2 karakters" blijft te algemeen. Je test moet zeggen welke twee karakters, dus bv: "Test: Zet karakter 4 en 7 van korttekst..."

"Zet 2 karakters" blijft te algemeen. Je test moet zeggen welke twee karakters, dus bv:
"Test: Zet karakter 4 en 7 van korttekst..."

3:44 pm

Tommy Hebb

Naamgeving parameters: IsV1 i.p.v. IsV1Aanwezig HeeftFrontBevestiging i.p.v. FrontbevestigingAanwezig Uitleg: *We gebruiken elders in de code ook bv IsProbox of IsFlatpackV1. In gedachten zetten...

Naamgeving parameters:
IsV1 i.p.v. IsV1Aanwezig
HeeftFrontBevestiging i.p.v. FrontbevestigingAanwezig

Uitleg:

  • We gebruiken elders in de code ook bv IsProbox of IsFlatpackV1. In gedachten zetten we er nog het woord "Verpakking" achter Het woord "Aanwezig" biedt geen meerwaarde.
  • Booleans beginnen bij grote voorkeur met "Is" of "Heeft".
3:36 pm

Tommy Hebb

Ook bovenstaand codeblok mag wat refactoring ondergaan... Je checkt 2 keer op HeeftFrontBevestiging, dus daar kun je dan beter eerst een selectie op maken: If HeeftFrontBevestiging { ... Hier d...

Ook bovenstaand codeblok mag wat refactoring ondergaan...
Je checkt 2 keer op HeeftFrontBevestiging, dus daar kun je dan beter eerst een selectie op maken:

If HeeftFrontBevestiging {
    ... Hier dan een genestte If-Else-structuur voor al dan niet FrontbevestigingExandoT
} Else {
    ...
}


P.s.: Geen ElseIf nodig dus, maar als je hem wel gebruikt, dan een grote I alstublieft

3:30 pm

Tommy Hebb

Bovenstaand codeblok zou gerefactored moeten worden. Aangezien de helper-methods ook werken met booleans als parameter, hoef je hier geen if-else-structuur te maken. j kunt hier gewoon 2 calls doen...

Bovenstaand codeblok zou gerefactored moeten worden. Aangezien de helper-methods ook werken met booleans als parameter, hoef je hier geen if-else-structuur te maken. j kunt hier gewoon 2 calls doen en de aangeleverde booleans hieraan meegeven:
Do ProductDetailLijnen.Insert(..GeefProductDetailLijnVerpakking(IsV1Aanwezig)
Do ProductDetailLijnen.Insert(..GeefProductDetailLijnOnderdeel(FrontbevestigingAanwezig, FrontbevestigingExpandoTAanwezig)