CacheAdminA_trunk2010/vhTest/Mock/APPS/Halux/Docs

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 14 May 18
Commits this week: 0
Total Lines of Code (LoC): 182
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 24 Aug 2022
2:04 pm

Jelle Baele

1:58 pm

Jelle Baele

8:54 am

Peter Vercoutter

Tuesday 23 Aug 2022
10:13 am

Joannes Lemmens

8:41 am

Jelle Baele

8:39 am

Jelle Baele

[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten
Monday 22 Aug 2022
3:14 pm

Wim Vermeulen

3:14 pm

Wim Vermeulen

In verband met het statement : "QR-code ... staat op dezelfde plaats op het etiket als de barcode, dus ze mogen niet beide ingevuld zijn" Optie 1 : in de software dit controleren/valideren en een ...

In verband met het statement : "QR-code ... staat op dezelfde plaats op het etiket als de barcode, dus ze mogen niet beide ingevuld zijn"

Optie 1 : in de software dit controleren/valideren en een exceptie smijten wanneer dit zich toch voordoet. De exceptie dan ook in UT opnemen : ZetVerwachteExceptie (je weet wel).
Optie 2 : in de UT tonen dat de software aanvaardt dat beide velden tegelijk ingevuld zijn, dus zonder exceptie te smijten. Hierbij wordt de verantwoordelijkheid dan naar Bartender (in dit geval) doorgeschoven.

Jij hebt hier gekozen voor Optie 2 (bewust of onbewust). Voor mij is dit oké.
Eventueel in die specifieke unit test method toelichten dat dit scenario problemen zal geven in Bartender.

10:28 am

Joannes Lemmens

[ICT-3055] VHOSS Etiket QRCode voor De Decker
[ICT-3055] VHOSS Etiket QRCode voor De Decker
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 30 Jan 2020
8:45 am

Tom Vermeulen

Thursday 23 Jan 2020
3:21 pm

Tom Vermeulen

is inderdaad onterecht gecommit. In de huidige versie staan de tests weer actief trouwens

is inderdaad onterecht gecommit. In de huidige versie staan de tests weer actief trouwens

Friday 17 Jan 2020
11:30 am

Tom Vermeulen

dat dit een vhossetiket is dat afleidt van een normaal etiket. De vhossvelden worden achteraan toegevoegd.

dat dit een vhossetiket is dat afleidt van een normaal etiket. De vhossvelden worden achteraan toegevoegd.