CacheAdminA_trunk2010/vhTestFiles/.../Test/Test2lijnenzelfdeleverancierZelfdeProduct

Journaal2lijnenZelfdeProduct.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 21 Sep 17
Commits this week: 0
Total Lines of Code (LoC): 0
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 05 Oct 2017
2:20 pm

Jo Willems

Tuesday 03 Oct 2017
8:54 am

Jo Claes

  • More
  • CR-497
  • finished reviewing
Friday 22 Sep 2017
3:47 pm

Jo Willems

Moeilijk te doen zonder complete integratie test op te zetten. commentaar in InventoryOut convertor gezet met alle converters in het zelfde geval.

Moeilijk te doen zonder complete integratie test op te zetten. commentaar in InventoryOut convertor gezet met alle converters in het zelfde geval.

Thursday 21 Sep 2017
6:00 pm

Jo Willems

In standaard gedrag word getest of de converter van de terug te geven response opgeroepen wordt. De mock geeft een voorgedefinieerd antwoord terug. Asserten van dit lijkt mij niet direct zinvol hie...

In standaard gedrag word getest of de converter van de terug te geven response opgeroepen wordt. De mock geeft een voorgedefinieerd antwoord terug.
Asserten van dit lijkt mij niet direct zinvol hier, maar wel in de test van die converter.

5:55 pm

Jo Willems

is al gebeurd, wsl outdated

is al gebeurd, wsl outdated

5:54 pm

Jo Willems

anders opgelost.

anders opgelost.

12:52 pm

Jo Claes

Gelieve ook de response te asserten

Gelieve ook de response te asserten

12:51 pm

Jo Claes

'k zou hier opteren om de LijnId en ProductId mee te geven zodat je duidelijk aan toont dat het over het zelfde product gaat met verschillende LijnId's

'k zou hier opteren om de LijnId en ProductId mee te geven zodat je duidelijk aan toont dat het over het zelfde product gaat met verschillende LijnId's

12:48 pm

Jo Claes

Gelieve het resultaat duidelijk te maken in de response

Gelieve het resultaat duidelijk te maken in de response

12:46 pm

Jo Claes

Gelieve in uw test benamingen ook uw gedrag te beschrijven maw het resultaat bvb Test: 2 Lijnen voor zelfde leverancier worden toegevoegd aan de receptie

Gelieve in uw test benamingen ook uw gedrag te beschrijven maw het resultaat

bvb Test: 2 Lijnen voor zelfde leverancier worden toegevoegd aan de receptie

12:44 pm

Jo Claes

Op een stub wordt er geen Verifieer() uitgevoerd, die maakt het zeer onduidelijk waar je eigenlijk de nadruk wil op leggen. Het principe is zo : voor het basis gedrag werk je met verwachts , vanaf...

Op een stub wordt er geen Verifieer() uitgevoerd, die maakt het zeer onduidelijk waar je eigenlijk de nadruk wil op leggen.

Het principe is zo : voor het basis gedrag werk je met verwachts , vanaf dat je de nadruk wil leggen op iets dat afwijkt van het standaard gedrag dan is enkel hetgeen je wil testen verwacht en de rest worden dan allemaal stubs of dummy of fakes afhankelijk van welke situatie.

Het belangrijkste is de intentie duidelijk maken en door alles te behandelen op dezelfde manier vergaat hetgeen eigenlijk aandacht nodig heeft in de massa.

12:37 pm

Jo Claes

ToeleveringId ipv ToeleveringNummer

ToeleveringId ipv ToeleveringNummer

Tuesday 19 Sep 2017
4:23 pm

Jo Claes

ValideerProductId

ValideerProductId

4:23 pm

Jo Claes

API overbodig

API overbodig

4:22 pm

Jo Claes

Array mag niet opgezet worden in de constructor -> moet per call opnieuw geset worden -> meerdere malen callen kan voor raar gedrag zorgen

Array mag niet opgezet worden in de constructor -> moet per call opnieuw geset worden -> meerdere malen callen kan voor raar gedrag zorgen

4:19 pm

Jo Claes

Als je LeverancierId doorgeeft tot het laagste niveau -> maw verlaten de encapsulatie en dan gaan we over naar de specifieke types die de ge-encapsuleerde code nodig heeft dan moet je tussendoor ni...

Als je LeverancierId doorgeeft tot het laagste niveau -> maw verlaten de encapsulatie en dan gaan we over naar de specifieke types die de ge-encapsuleerde code nodig heeft dan moet je tussendoor niet al teveel datatypes en id wisselen van hot naar her

4:08 pm

Jo Claes

Je kan in testen rekenen op een standaard FakeDateTime Api zodat je er geen aandacht moet aan geven in uw testen -> code simpeler houden

Je kan in testen rekenen op een standaard FakeDateTime Api zodat je er geen aandacht moet aan geven in uw testen -> code simpeler houden

4:06 pm

Jo Claes

64524.57298 lijkt mij niet echt een juiste timestamp -> als we datums doorgeven dan is het %Date als we tijdstippen doorgeven dan zijn het %TimeStamps

64524.57298 lijkt mij niet echt een juiste timestamp -> als we datums doorgeven dan is het %Date als we tijdstippen doorgeven dan zijn het %TimeStamps

4:03 pm

Jo Claes

aangezien we in onze code met leverancierid werken gelieve dan hier ook leverancierid te gebruiken -> resultaat zal automatisch leiden tot simpelere code

aangezien we in onze code met leverancierid werken gelieve dan hier ook leverancierid te gebruiken -> resultaat zal automatisch leiden tot simpelere code

3:49 pm

Jo Claes

$$INTDATE^vhLib.DataTypes gebruiken kwestie van het duidelijker te hebben welke datum het is .. datum is makkelijker terug te vinden dan een nummerke

$$INTDATE^vhLib.DataTypes gebruiken kwestie van het duidelijker te hebben welke datum het is .. datum is makkelijker terug te vinden dan een nummerke

3:46 pm

Jo Claes

ValideerProductId

ValideerProductId

2:59 pm

Tommy Hebb

2:13 pm

Tom Vermeulen

  • More
  • CR-542
  • finished reviewing
10:55 am

Tommy Hebb

[UST2830] VHIP904+VHIP903: Dummy antwoord op goederenontvangst-request (= InventSync-lijst met 1 hardcoded record):

- Aanpassingen na review -> Aantal InventSyncs wordt nu getest

Options
    • -0
    • +19
    ./Journaal2lijnenZelfdeProduct.xml
  2. … 2 more files in changeset.
Monday 18 Sep 2017
2:55 pm

Tom Vermeulen

En toch zou het geen kwaad kunnen om de inhoud van de Response af te toetsen, zeker omdat je er nu een hoop extra data toegevoegd hebt

En toch zou het geen kwaad kunnen om de inhoud van de Response af te toetsen, zeker omdat je er nu een hoop extra data toegevoegd hebt

12:24 pm

Tommy Hebb

Da's code van JoW, heb ik niets aan veranderd/mee te maken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Ik steek in dat objectje dat teruggegeven wor...

Da's code van JoW, heb ik niets aan veranderd/mee te maken
Ik steek in dat objectje dat teruggegeven wordt (Response) enkel, indien nodig, iets extra in.

11:17 am

Tom Vermeulen

Als ge niets gaat nakijken in de "response", kan je evengoed Do Converter.Convert() doen. Beter is om, toch voor het standaardgedrag, te verifiëren dat het antwoord wel degelijk bevat wat je verwac...

Als ge niets gaat nakijken in de "response", kan je evengoed Do Converter.Convert() doen. Beter is om, toch voor het standaardgedrag, te verifiëren dat het antwoord wel degelijk bevat wat je verwacht dat het zou bevatten.

Friday 15 Sep 2017
6:05 pm

Tommy Hebb

[UST2830] VHIP904+VHIP903: Dummy antwoord op goederenontvangst-request (= InventSync-lijst met 1...
[UST2830] VHIP904+VHIP903: Dummy antwoord op goederenontvangst-request (= InventSync-lijst met 1...
Tuesday 05 Sep 2017
7:54 am

Jo Claes

Is er iets specifiek aan die toelevering of zou het mogelijk zijn om een standaard ToeleveringHoofding te pakken en daar wat lijnen aan toe te voegen , kwestie van niet teveel herhalingen in versch...

Is er iets specifiek aan die toelevering of zou het mogelijk zijn om een standaard ToeleveringHoofding te pakken en daar wat lijnen aan toe te voegen , kwestie van niet teveel herhalingen in verschillende code files te hebben.

Monday 04 Sep 2017
4:34 pm

Jo Claes

KlantId ipv KlantNummer at runtime is dat dus met K|| prefix

KlantId ipv KlantNummer at runtime is dat dus met K|| prefix