CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Verkoop/impl/Externe/VerkoopService

OfferteVoorBestellingValidator

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 27 Sep 16
Commits this week: 0
Total Lines of Code (LoC): 305
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
Friday 24 Nov 2017
7:56 am

KurtC

[DEF626r] OfferteBestellingValidator - foutmelding Personality ID op..

- indien met personality en klantid

Options
    • -1
    • +1
    ./TestVH.cls.xml
7:55 am

Kurt Callebaut

Thursday 23 Nov 2017
4:00 pm

Wim Vermeulen

  • More
  • CR-751
  • finished reviewing
3:57 pm

Wim Vermeulen

Waarom gebruik je $$$ElkeWaarde bij de IndienMethodcall ? Dat wil je toch juist testen, want daar zat de bug in de code, hé? Je krijgt 'em van mij cadeau http://subversion02.vanhoecke.be/static/ogd...

Waarom gebruik je $$$ElkeWaarde bij de IndienMethodcall ?
Dat wil je toch juist testen, want daar zat de bug in de code, hé?
Je krijgt 'em van mij cadeau
.IndienMethodCall("Bepaal",PersonalityID, ..#KlantID)

Tuesday 21 Nov 2017
4:41 pm

Kurt Callebaut

[DEF626] OfferteBestellingValidator - foutmelding Personality ID op IsStandaardTransport
[DEF626] OfferteBestellingValidator - foutmelding Personality ID op IsStandaardTransport
4:36 pm

KurtC

[DEF626] OfferteBestellingValidator - foutmelding Personality ID op IsStandaardTransport

- Personality.GeefID meegeven.

Options
    • -6
    • +14
    ./TestVH.cls.xml
  2. … 1 more file in changeset.
Tuesday 12 Sep 2017
9:00 am

Kurt Callebaut

8:59 am

Kurt Callebaut

excepties van gemaakt.

excepties van gemaakt.

Monday 11 Sep 2017
5:22 pm

Jo Claes

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ??? Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist. Ofwel krij...

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ???

Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist.
Ofwel krijgt de init een %String binnen en gaat deze de datum omvormen ofwel gaat iedere testcase de init correct aanroepen.

5:08 pm

Jo Claes

Een VanHoeckeOfferte is toch altijd een vhisie4 versie. Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

Een VanHoeckeOfferte is toch altijd een vhisie4 versie.

Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

5:05 pm

Jo Claes

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

4:05 pm

Kurt Callebaut

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere softw...

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere software op deze exceptiecode. ???

4:01 pm

Kurt Callebaut

excepties gebruikt ipv tekst of codes.

excepties gebruikt ipv tekst of codes.

11:15 am

Jo Claes

  • More
  • CR-511
  • finished reviewing
11:15 am

Jo Claes

Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin ee...

Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin een code en een string en kan er per code makkelijker gemapped worden welke exceptie te gooien.
Als code kunnen we misschien de exceptie klasse naam gebruiken zo kunnen we iets generiek maken om de exceptie te throwen

10:31 am

Jo Claes

We er voor zorgen dat je een andere foutcode naar buiten gooit

We er voor zorgen dat je een andere foutcode naar buiten gooit

10:31 am

Jo Claes

TECH.Interface ipv TECH.Mockable

TECH.Interface ipv TECH.Mockable

Wednesday 06 Sep 2017
3:51 pm

Kurt Callebaut

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
1:56 pm

KurtC

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar

- Test bestond blijkbaar al in testbase.

Options
    • -71
    • +0
    ./TestValideer.cls.xml
    • -1
    • +1
    ./TestBase.cls.xml
1:13 pm

KurtC

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar

- Beetje refactoring.

Options
    • -8
    • +5
    ./TestValideer.cls.xml
    • -8
    • +5
    ./TestVH.cls.xml
1:09 pm

KurtC

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar

- Testje voor de nieuwe exceptie.

Options
    • -0
    • +74
    ./TestValideer.cls.xml
11:04 am

KurtC

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar

- fixen falende testen.

Options
    • -1
    • +12
    ./TestBase.cls.xml
10:16 am

KurtC

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar

- validatie vervallenprijzen enkel op winkelkarren die maatwerklijnen hebben.

Options
    • -8
    • +44
    ./TestVH.cls.xml
  2. … 1 more file in changeset.
Tuesday 05 Sep 2017
9:58 am

Kurt Callebaut

9:57 am

Kurt Callebaut

Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

Monday 04 Sep 2017
1:51 pm

Jo Claes

  • More
  • CR-506
  • finished reviewing
1:50 pm

Jo Claes

Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen

Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

12:02 pm

Jo Claes

IsVerschillend lijkt mij een betere naam.

IsVerschillend lijkt mij een betere naam.

11:59 am

Jo Claes

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

11:23 am

Kurt Callebaut

[UST2682] vh4 : vervallen offertes
[UST2682] vh4 : vervallen offertes