CacheAdminA_trunk2010/vhTest/Fake/APPS/Halux/Docs/impl

LadeKlantEtiketBuilder

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 15 Jul 21
Commits this week: 0
Total Lines of Code (LoC): 92
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

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