CacheAdminA_trunk2010/vhTest/Fake/APPS/Halux/Docs

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 14 May 18
Commits this week: 0
Total Lines of Code (LoC): 167
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
Tuesday 28 May 2024
3:00 pm

Joannes Lemmens

Thursday 28 Sep 2023
3:40 pm

Jelle Baele

Friday 01 Sep 2023
3:08 pm

Peter Vercoutter

3:08 pm

Peter Vercoutter

Voor mij oke! Ik zou misschien hebben gekozen voor een andere variabele naam dan 'IsSpecialeLade' want dit geeft niet meteen aan waarvoor het zal worden gebruikt.

Voor mij oke! Ik zou misschien hebben gekozen voor een andere variabele naam dan 'IsSpecialeLade' want dit geeft niet meteen aan waarvoor het zal worden gebruikt.

2:26 pm

Jelle Baele

1:31 pm

Jelle Baele

[ICT-3849] [rvPVR] Koppelen orgalux sticker aan lade code in het bewerkingscenter:
[ICT-3849] [rvPVR] Koppelen orgalux sticker aan lade code in het bewerkingscenter:
Wednesday 24 Aug 2022
2:04 pm

Jelle Baele

1:58 pm

Jelle Baele

8:54 am

Peter Vercoutter

Tuesday 23 Aug 2022
8:41 am

Jelle Baele

8:39 am

Jelle Baele

[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
Tuesday 07 Sep 2021
2:06 pm

Tommy Hebb

2:05 pm

Tommy Hebb

Ik vermoed dat vhUnitTest.APPS.Halux.Docs.TestDataEtiketten nergens anders gebruikt wordt en dus mag opgekuist worden hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emo...

Ik vermoed dat vhUnitTest.APPS.Halux.Docs.TestDataEtiketten nergens anders gebruikt wordt en dus mag opgekuist worden hé

1:49 pm

Tommy Hebb

Info2 wordt wel geassert, maar doet niet echt ter zake in deze UT. Toch niet volgens de naam ervan.

Info2 wordt wel geassert, maar doet niet echt ter zake in deze UT. Toch niet volgens de naam ervan.

1:48 pm

Tommy Hebb

Ik zou de UT-naam iets duidelijker maken, want "Klant OrderNummer" houdt eigenlijk pas steek als voorgaande UT in acht wordt genomen en mettertijd staan die 2 UT misschien niet meer vlak bij elkaar...

Ik zou de UT-naam iets duidelijker maken, want "Klant OrderNummer" houdt eigenlijk pas steek als voorgaande UT in acht wordt genomen en mettertijd staan die 2 UT misschien niet meer vlak bij elkaar. Ik zou ook, net zoals hierboven, vermelden dat het over het veld Info1 gaat.

1:46 pm

Tommy Hebb

Misschien nog bijzetten "in Info1";

Misschien nog bijzetten "in Info1";

1:40 pm

Tommy Hebb

Ofwel klopt de naam van de UT niet, ofwel de implementatie. Er is geen assert op ProductImg en als ik die toevoeg (met een verwachte waarde anders dan leeg), dan slaagt de test, terwijl ProductImg ...

Ofwel klopt de naam van de UT niet, ofwel de implementatie. Er is geen assert op ProductImg en als ik die toevoeg (met een verwachte waarde anders dan leeg), dan slaagt de test, terwijl ProductImg volgens de UT-naam leeg zou moeten zijn!
In de "oude" UT ontbrak die assert blijkbaar ook en de UT-naam is ongewijzigd volgens deze review, dus zou er eens nagevraagd/nagekeken moeten worden wat eigenlijk de bedoeling is, zodat het juiste kan aangepast worden.

12:06 pm

Tommy Hebb

Ik zou de projectreferentie hier en in de volgende Unit Test vooraf in een variabele steken met een duidende naam, zodat het wat meer opvalt. Duurde even voordat ik doorhad dat in de volgende UnitT...

Ik zou de projectreferentie hier en in de volgende Unit Test vooraf in een variabele steken met een duidende naam, zodat het wat meer opvalt. Duurde even voordat ik doorhad dat in de volgende UnitTest het 2-tje weg was en aangezien dàt het enige verschil is mag het wel wat opvallen

10:32 am

Tommy Hebb

Is het niet een beetje raar dat deze properties nu leeg zijn voor de assert? Worden blijkbaar niet mee uitgemockt?

Is het niet een beetje raar dat deze properties nu leeg zijn voor de assert? Worden blijkbaar niet mee uitgemockt?

Thursday 12 Aug 2021
4:59 pm

Jelle Baele

4:57 pm

Jelle Baele

[ICT-449] UST3816 - [rvTHB] Deel 2 van Tests die klanten maken: schijfruimte-probl: tests onafh van...
[ICT-449] UST3816 - [rvTHB] Deel 2 van Tests die klanten maken: schijfruimte-probl: tests onafh van...
Tuesday 22 Sep 2020
3:50 pm

Peter Vercoutter

3:47 pm

Peter Vercoutter

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

Tuesday 08 Sep 2020
11:57 am

Tom Vermeulen

11:54 am

Tom Vermeulen

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True a...

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True als laatste parameter hebben.

11:53 am

Tom Vermeulen

{$$$False} ipv 0

{$$$False} ipv 0

11:48 am

Tom Vermeulen

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier...

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier houden we de interface(s) proper (zeker als er ooit nog andere parameters bij moeten komen).

Dit geldt ook voor sommige andere methods/klasses in deze review

11:47 am

Tom Vermeulen

comments weg

comments weg

Wednesday 02 Sep 2020
3:26 pm

Peter Vercoutter

[IC-1732] Implementatie SpaceStep Klantetiket
[IC-1732] Implementatie SpaceStep Klantetiket
Thursday 29 Nov 2018
1:07 pm

Tom Vermeulen