CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Winkelkar/impl/MaatwerkLijnBuilder

Test.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 22 Jan 20
Commits this week: 0
Total Lines of Code (LoC): 221
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 30 Jan 2020
8:47 am

Tom Vermeulen

8:47 am

Tom Vermeulen

is niet volledig duplicate, setup is beetje anders. aangezien het de bedoeling was dat dat alpnachverhaal vrij snel van situatie 1 naar 2 geswitched ging worden, heb ik het niet liggen afzonderen

is niet volledig duplicate, setup is beetje anders. aangezien het de bedoeling was dat dat alpnachverhaal vrij snel van situatie 1 naar 2 geswitched ging worden, heb ik het niet liggen afzonderen

Monday 27 Jan 2020
11:04 am

Kurt Callebaut

10:59 am

Kurt Callebaut

duplicate code. kan deze via assert met personality en klantid ?

duplicate code. kan deze via assert met personality en klantid ?

10:25 am

Kurt Callebaut

Kan het zijn dat hier vroeger al een stukje code ontbrak ? de controle op sfs personality ? (zoals in de controle IsAlpnach() . mogelijks geeft dat hier geen probleem. 't is maar een vaststelling.

Kan het zijn dat hier vroeger al een stukje code ontbrak ? de controle op sfs personality ?
(zoals in de controle IsAlpnach() . mogelijks geeft dat hier geen probleem. 't is maar een vaststelling.

10:19 am

Kurt Callebaut

alpnach Vanhoecke klant ipv rechtstreeks ?

alpnach Vanhoecke klant ipv rechtstreeks ?

10:18 am

Kurt Callebaut

gemeenschappelijke code. code op de APPS laag gebruikt toch al de DOM laag.

gemeenschappelijke code. code op de APPS laag gebruikt toch al de DOM laag.

10:15 am

Kurt Callebaut

hier ook : is vanhoecke klant. ipv rechtstreeks. heeft iets meer betekenis.

hier ook : is vanhoecke klant. ipv rechtstreeks. heeft iets meer betekenis.

10:14 am

Kurt Callebaut

rechtstreeks betekent vanhoecke klant. misschien benamingen : sfs klant of vanhoecke klant gebruiken dat geeft iets meer betekenis.eventueel via personality enum.

rechtstreeks betekent vanhoecke klant.
misschien benamingen : sfs klant of vanhoecke klant gebruiken dat geeft iets meer betekenis.eventueel via personality enum.

10:10 am

Kurt Callebaut

els if om instructie te sparen in geval van sfs- alpnach ?

els if om instructie te sparen in geval van sfs- alpnach ?

Thursday 23 Jan 2020
2:46 pm

Tom Vermeulen

[ICT-1100] Alpnach : van klant van SFS naar rechtstreekse klant
[ICT-1100] Alpnach : van klant van SFS naar rechtstreekse klant
Wednesday 12 Jun 2019
1:03 pm

Tom Vermeulen

Tuesday 11 Jun 2019
10:45 am

Kurt Callebaut

8:19 am

Tom Vermeulen

[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)
[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)
Monday 28 Jan 2019
10:21 am

Tom Vermeulen

Thursday 24 Jan 2019
11:23 am

Tom Vermeulen

ja, maar code vliegt er over twee weken toch weer uit

ja, maar code vliegt er over twee weken toch weer uit

11:22 am

Tom Vermeulen

nope, code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

10:08 am

Kurt Callebaut

10:07 am

Kurt Callebaut

Eén instantie maken van legacypartij lijkt me voldoende.

Eén instantie maken van legacypartij lijkt me voldoende.

10:06 am

Kurt Callebaut

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

10:01 am

Kurt Callebaut

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

9:51 am

Kurt Callebaut

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvr...

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvragen van klantsessiedata.

9:48 am

Kurt Callebaut

Duplicate testcode op klantid na ?

Duplicate testcode op klantid na ?

8:56 am

Kurt Callebaut

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

8:56 am

Kurt Callebaut

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

8:54 am

Kurt Callebaut

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevest...

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevestigingschroevenVoorKlantID().

8:48 am

Kurt Callebaut

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

8:43 am

Kurt Callebaut

equals() ?

equals() ?

8:42 am

Kurt Callebaut

stringutils.equals ?

stringutils.equals ?

8:42 am

Kurt Callebaut

2 ifs samenvoegen om duplicate code te vermijden.

2 ifs samenvoegen om duplicate code te vermijden.