CacheAdminA_trunk2010/vhUnitTest/APPS/Halux

Docs

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 7
Last Commit: 19 Aug 15
Commits this week: 0
Total Lines of Code (LoC): 3.538
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

Tuesday 16 Jan 2024
4:17 pm

Joannes Lemmens

12:20 pm

Peter Vercoutter

8:31 am

Joannes Lemmens

[ICT-4250] Klantlabel Klant Neves - Altijd barcode
[ICT-4250] Klantlabel Klant Neves - Altijd barcode
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 01 Mar 2023
8:57 am

Joannes Lemmens

8:18 am

Tom Vermeulen

Thursday 16 Feb 2023
11:46 am

Joannes Lemmens

[ICT-3418] KlantEtiketten: foute volgorde bij het afprinten door QRCode
[ICT-3418] KlantEtiketten: foute volgorde bij het afprinten door QRCode
Tuesday 17 Jan 2023
8:17 am

Joannes Lemmens

Monday 16 Jan 2023
1:37 pm

Jelle Baele

Thursday 12 Jan 2023
10:12 am

Joannes Lemmens

[ICT-3317] Klantetiket - 2D barcode vermelden voor alle klanten
[ICT-3317] Klantetiket - 2D barcode vermelden voor alle klanten
Wednesday 24 Aug 2022
2:04 pm

Jelle Baele

Tuesday 23 Aug 2022
10:13 am

Joannes Lemmens

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...