CacheAdminA_trunk2010/vhUnitTest/DOM/PM/Maatwerk/Calc/Kost

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 6
Last Commit: 08 Mar 16
Commits this week: 0
Total Lines of Code (LoC): 759
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 27 Oct 2021
1:42 pm

Peter Vercoutter

Tuesday 05 Oct 2021
12:49 pm

Tommy Hebb

12:21 pm

Tom Vermeulen

11:09 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
Thursday 23 Sep 2021
4:17 pm

Tommy Hebb

Zoals besproken blijkbaar ongewild in deze review terechtgekomen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zoals besproken blijkbaar ongewild in deze review terechtgekomen

4:06 pm

Tom Vermeulen

is deze wel nodig?

is deze wel nodig?

Tuesday 21 Sep 2021
2:51 pm

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Uitzondering i.g.v. OudZwart dekte de hele lading niet, er was meer aan de hand (zie volgend punt...)

- Correctie => Front, Rug en Bodem => zijn nooit bicolor (enkel zijkant kan dat zijn), dus geen opcentiemen in dat geval

- Een volledige lade bevat zijkanten, dus dan eventueel wel bicolor met bijhorende opcentiemen

Options
    • -21
    • +76
    ./TAOR/SpecKostv004/PostProcessTotaalKostItem/Test.cls.xml
  2. … 1 more file in changeset.
Monday 20 Sep 2021
10:41 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Kleine refactor en afzondering van Bicolor-percentage-verhoging

- Uitzondering voorzien indien LadeKleur OudZwart => Toch normaal unicode-percentage van 2

Options
    • -0
    • +16
    ./TAOR/SpecKostv004/PostProcessTotaalKostItem/Test.cls.xml
  2. … 1 more file in changeset.
Wednesday 01 Sep 2021
5:05 pm

Wim Vermeulen

Als kritische reviewer ben ik verplicht om te melden dat bovenstaande If - Else {} nogal slordige code is. Wat je zou kunnen doen is iets in de aard van If (...) { Set ProcentueelKostItem = .....

Als kritische reviewer ben ik verplicht om te melden dat bovenstaande If - Else {} nogal slordige code is.
Wat je zou kunnen doen is iets in de aard van 

If (...) {
    Set ProcentueelKostItem = ..GeefProcentueelKostItemVoorLbxMetSpaceStep( Naam, DecimaalPercentage, TotaalKostItem) 
}
Else {
    Set ProcentueelKostItem = ..GeefProcentueelKostItemVoorLbxLade( Naam, DecimaalPercentage, TotaalKostItem) 
}
Do TotaalKostItem.VoegToe(ProcentueelkostItem)
}

+ de 2 private methods, waarin je dan proper het verschil kan zien tussen SSTProcentueelKostItem met de ExcludeFilter versus het gewone ProcentueelKostItem.

5:05 pm

Wim Vermeulen

Deze klasse mag rechstreeks afleiden van MaatwerkProductAlsHFKostItemBuilder (i.p.v. MaatwerkProductAlsHFKostItemBuilderSST) Zie ook opmerking bij klasse MaatwerkProductAlsHFKostItemBuilderSST

Deze klasse mag rechstreeks afleiden van MaatwerkProductAlsHFKostItemBuilder (i.p.v. MaatwerkProductAlsHFKostItemBuilderSST)
Zie ook opmerking bij klasse MaatwerkProductAlsHFKostItemBuilderSST

5:02 pm

Wim Vermeulen

5:00 pm

Wim Vermeulen

Als de 3 Test-methods alle 3 dezelfde SST-kenmerken gebruiken, dan zal de test meteen veel duidelijker worden wanneer je deze in een private method afzondert, en enkel de variatie als parameter doo...

Als de 3 Test-methods alle 3 dezelfde SST-kenmerken gebruiken, dan zal de test meteen veel duidelijker worden wanneer je deze in een private method afzondert,
en enkel de variatie als parameter doorgeeft. In dit geval is dat property Uitvoering en Bewegingstechnologie, denk.

Een andere insteek van Unittest :
Vul enkel die waarden in die noodzakelijk zijn. M.a.w. Hoogte, Breedte, Diepte, Kleur, stekkerType, ... zijn allemaal niet relevant voor deze UT.
Je zou dus kunnen opteren voor een GeefLegeSSTKenmerken() en nadien enkel Uitvoering en technologie in te vullen (or whatever is needed).

4:26 pm

Wim Vermeulen

Volgens mijn wiskundegevoel is dit gewoon gelijk aan   KostLade * ..Percentage of anders heb ik hier net een bug ontdekt? :-P

Volgens mijn wiskundegevoel is dit gewoon gelijk aan 

    KostLade * ..Percentage


of anders heb ik hier net een bug ontdekt?
:-P

4:18 pm

Wim Vermeulen

In .NET moeten we wel de "nieuwe standaarden" van VoegToeBuilders() volgen, kwestie van geen KostItemBuilders in de %OnNew toe te voegen. Hier in caché mag dat wel zo blijven :-P

In .NET moeten we wel de "nieuwe standaarden" van VoegToeBuilders() volgen, kwestie van geen KostItemBuilders in de %OnNew toe te voegen.
Hier in caché mag dat wel zo blijven :-P

3:22 pm

Wim Vermeulen

Deze klasse is een exacte kopie van "DOM.PM.Maatwerk.Calc.Kost.MaatwerkProductAlsHFKostItemBuilder" Ik zie geen enkele reden of specifieke dependency om een specifieke versie voor SST te maken. Dus...

Deze klasse is een exacte kopie van "DOM.PM.Maatwerk.Calc.Kost.MaatwerkProductAlsHFKostItemBuilder"
Ik zie geen enkele reden of specifieke dependency om een specifieke versie voor SST te maken. Dus wat mij betreft mag deze klasse verwijderd worden.
Of, zie ik iets over t hoofd?

Thursday 18 Feb 2021
12:17 pm

Tommy Hebb

[ICT-2207] [rvWV] PM: Maatwerk: TAX: SFS TAX actie kleuren S en XG (identiek aan kleur WI = wit; nu voor zwart en grijs):

- IsWitteLadeVoorSfs losgetrokken in 2 voorwaarden => IsVoorSfs en IsSfsKortingNodig => Korting voor witte laden moet onafhankelijk blijven lopen van nieuwe korting voor grijze en zwarte laden, maar is beiden enkel igv SFS

- Soortgelijke korting voor SFS als die voor witte laden geldt nu ook voor grijze en zwarte laden => afhankelijk van ConfigSettings => SfsTaxKortingActie_IsActief en SfsTaxKortingActie_StartMoment en SfsTaxKortingActie_EindMoment

- TestCase uitgebreid

Options
    • -4
    • +175
    ./TAOR/SpecKostv004/PostProcessTotaalKostItem/Test.cls.xml
  2. … 1 more file in changeset.
Wednesday 02 Sep 2020
4:05 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep
[ICT-1634] Implementatie kostenbuilders SpaceStep
Monday 03 Aug 2020
3:21 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep

- Toevoegen test voor ko

Options
    • -0
    • +105
    ./SST/TestSpecKostv001.cls.xml
  2. ./SST
Friday 13 Sep 2019
3:46 pm

Kurt Callebaut

3:46 pm

KurtC

ICT-813: Bouwstenen moeten toegevoegd worden in alle bovenstaande gevallen van smalle lades. De kostenplaatsen zijn gekend.
Options
    • -2
    • +4
    ./Evaluator/IsSmalleLadeEvaluator/Test.cls.xml
3:38 pm

Wim Vermeulen

3:38 pm

Wim Vermeulen

Object niet als string-value doorgeven aub. Set IngegevenKenmerken = vhTestUtils.LBX .GeefLegeKenmerken() of zoiets

Object niet als string-value doorgeven aub.
Set IngegevenKenmerken = vhTestUtils.LBX .GeefLegeKenmerken() of zoiets

2:45 pm

Kurt Callebaut

ICT-813: Bouwstenen moeten toegevoegd worden in alle bovenstaande gevallen van smalle lades. De kostenplaatsen zijn gekend.
ICT-813: Bouwstenen moeten toegevoegd worden in alle bovenstaande gevallen van smalle lades. De kostenplaatsen zijn gekend.
2:43 pm

KurtC

[ICT-813] Bouwstenen moeten toegevoegd worden in alle bovenstaande gevallen van smalle lades. De kostenplaatsen zijn gekend.

test evaluator

Options
  1. ./Evaluator/IsSmalleLadeEvaluator
    • -0
    • +20
    ./Evaluator/IsSmalleLadeEvaluator/Test.cls.xml
Wednesday 31 Jul 2019
10:05 am

Tommy Hebb

Wednesday 24 Jul 2019
6:45 pm

Wim Vermeulen

12:07 pm

Wim Vermeulen

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is. Graag effe samen bespreken.

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is.
Graag effe samen bespreken.

11:26 am

Wim Vermeulen

Beter :  Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken) en de method dan laten staan op de basisklasse, en dan moet ge de method in de LBX-...

Beter : 

 Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken)


en de method dan laten staan op de basisklasse,
en dan moet ge de method in de LBX-klasse niet overriden.

9:59 am

Wim Vermeulen

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

Monday 22 Jul 2019
4:58 pm

Tommy Hebb

[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen: