CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Winkelkar/impl

DoorHandelMeeleverenBepaler

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 24 Oct 18
Commits this week: 0
Total Lines of Code (LoC): 557
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
Wednesday 28 Aug 2019
6:44 am

Jo Willems

6:44 am

Jo Willems

[UST4222] EC: MP: Daro: ook mailen van door handel toe te voegen losse componenten (naast de http post!)

- review opm

Options
    • -2
    • +0
    ./NietBenelux/ZetDoorHandelMeeleverens.cls.xml
  2. … 10 more files in changeset.
Monday 19 Aug 2019
11:47 am

Martijn Vierstraete

11:43 am

Martijn Vierstraete

Is er een reden waarom je tests toevoegt en uitschakelt?

Is er een reden waarom je tests toevoegt en uitschakelt?

11:42 am

Martijn Vierstraete

#dim zonder type is even erg als een Set

#dim zonder type is even erg als een Set

11:40 am

Martijn Vierstraete

Method mag private (en bij voorkeur geen strings als methodnaam, maar UpperCamelCased) *Personality specifieke httpverzender meegeven i.p.v. extra logica in test (bepaling van httpverzender via o...
  • Method mag private (en bij voorkeur geen strings als methodnaam, maar UpperCamelCased)
  • Personality specifieke httpverzender meegeven i.p.v. extra logica in test (bepaling van httpverzender via objectapi)
11:28 am

Martijn Vierstraete

Deze klasse test enkel of de dependencies opgeroepen worden: veel setup en weinig (duidelijke) asserts. Dit geeft aan dat hier logica opgesplitst zou moeten worden.

Deze klasse test enkel of de dependencies opgeroepen worden: veel setup en weinig (duidelijke) asserts. Dit geeft aan dat hier logica opgesplitst zou moeten worden.

11:25 am

Martijn Vierstraete

Kan je ook omschrijven in de testnaam wat het standaard gedrag is?

Kan je ook omschrijven in de testnaam wat het standaard gedrag is?

11:24 am

Martijn Vierstraete

Injecteren en testen

Injecteren en testen

11:23 am

Martijn Vierstraete

Wat is de meerwaarde van deze getter-methode?

Wat is de meerwaarde van deze getter-methode?

11:21 am

Martijn Vierstraete

Afzonderen en testen van TAX en LBX logica

Afzonderen en testen van TAX en LBX logica

11:20 am

Martijn Vierstraete

naamgeving: ProjectSettingsApi

naamgeving: ProjectSettingsApi

11:18 am

Martijn Vierstraete

Else (inconsistente casing)

Else (inconsistente casing)

11:16 am

Martijn Vierstraete

Injecteren en testen

Injecteren en testen

11:16 am

Martijn Vierstraete

$$$False ipv 0

$$$False ipv 0

11:15 am

Martijn Vierstraete

Gebruik property IsBinnenlade

Gebruik property IsBinnenlade

11:15 am

Martijn Vierstraete

Deze method heeft geen meerwaarde, je kan net zowel de property opvragen

Deze method heeft geen meerwaarde, je kan net zowel de property opvragen

11:11 am

Martijn Vierstraete

Injecteren en testen

Injecteren en testen

11:11 am

Martijn Vierstraete

OfferteGenerator overschrijven eerder dan onderstaande inject

OfferteGenerator overschrijven eerder dan onderstaande inject

11:09 am

Martijn Vierstraete

Api injecteren en testen

Api injecteren en testen

11:08 am

Martijn Vierstraete

Injecteren en testen

Injecteren en testen

11:07 am

Martijn Vierstraete

Commentaar mag weg

Commentaar mag weg

11:06 am

Martijn Vierstraete

Injecteren en testen

Injecteren en testen

11:04 am

Martijn Vierstraete

If ipv if (consistente casing)

If ipv if (consistente casing)

11:04 am

Martijn Vierstraete

Voorwaarden omwisselen: leesbaarheid en performantie

Voorwaarden omwisselen: leesbaarheid en performantie

11:03 am

Martijn Vierstraete

ProjectSettingsApi injecteren en gebruik ervan testen

ProjectSettingsApi injecteren en gebruik ervan testen

10:58 am

Martijn Vierstraete

If ... Else (casing)

If ... Else (casing)

10:58 am

Martijn Vierstraete

RolTAX wordt niet gebruikt -> opkuisen

RolTAX wordt niet gebruikt -> opkuisen

Tuesday 04 Jun 2019
3:21 pm

Jo Willems

Sorry voor deze review. Veel te veel revisies en meerdere stories, maar het was nog al dringend en alles liep door elkaar, en zie niet goed hoe ik dit in meerdere reviews kan steken. Desnoods lopen...

Sorry voor deze review. Veel te veel revisies en meerdere stories, maar het was nog al dringend en alles liep door elkaar, en zie niet goed hoe ik dit in meerdere reviews kan steken.
Desnoods lopen we er samen eens door.
UST4222 , UST4223, en nog een stukje aanvulling op BUS734

3:20 pm

Jo Willems

[UST4222] EC: MP: Daro: ook mailen van door handel toe te voegen losse componenten (naast de http...
[UST4222] EC: MP: Daro: ook mailen van door handel toe te voegen losse componenten (naast de http...