CacheAdminA_trunk2010/vhUnitTest/APPS/PM/Maatwerk/End2End

SST

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 13 Aug 20
Commits this week: 0
Total Lines of Code (LoC): 65
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
Friday 21 Mar
10:23 am

Peter Vercoutter

[ICT] Aanpassing door PM aan masterdata
Options
    • -1
    • +1
    ./TestSSTBasisPrijzen.cls.xml
  2. … 1 more file in changeset.
Tuesday 03 May 2022
5:20 pm

Wim Vermeulen

[ICT-2873][rvNVT] ICT: De Grote Prijswijziging van 01/05/2022 : ICT automatische testen aanpassen

- UnitTests End2End : Test MAT prijzen en Lade-met-Mat prijzen

- UnitTests End2End : Test SST basisprijzen

Options
    • -3
    • +3
    ./TestSSTBasisPrijzen.cls.xml
  2. … 3 more files in changeset.
Wednesday 27 Oct 2021
1:42 pm

Peter Vercoutter

1:42 pm

Peter Vercoutter

Friday 01 Oct 2021
10:53 am

Peter Vercoutter

[ICT-2522] Update testbestanden voor prijswijzigingen 1 oktober 2021
Options
    • -3
    • +3
    ./TestSSTBasisPrijzen.cls.xml
  2. … 3 more files 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?

3:07 pm

Wim Vermeulen

Tuesday 31 Aug 2021
5:45 pm

Wim Vermeulen

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() ) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builde...

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() )
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

5:44 pm

Wim Vermeulen

Deze method komt vele malen voor in de Builders (idem voor =Servodrive) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd ...

Deze method komt vele malen voor in de Builders (idem voor =Servodrive)
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

5:21 pm

Wim Vermeulen

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt. Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en B...

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt.
Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en BuilderBasis vallen samen.
Maar wat mij betreft hoeft dit niet weggewerkt te worden in caché. Misschien wel bij het porten naar .NET

FYI: het was uiteindelijk de dubbele definitie van de method GeefGekoppeldIKType() die voor enige verwarring zorgde. Vandaar de review opm.

Thursday 01 Jul 2021
4:03 pm

Peter Vercoutter

[ICT-2437] Update testbestanden voor prijswijzigingen 1 juli 2021
Options
    • -3
    • +3
    ./TestSSTBasisPrijzen.cls.xml
  2. … 10 more files in changeset.
Monday 21 Sep 2020
10:13 am

Peter Vercoutter

[ICT-1796] SST LBX - wijziging e-con + wijziging bouwsteen (mail MCL 14/9/2020)

- Wijziging bouwsteen: aantal set systeemschroeven moet van 1 naar 2

Options
    • -2
    • +2
    ./TestSSTBasisPrijzen.cls.xml
  2. … 4 more files in changeset.
Wednesday 02 Sep 2020
4:05 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep
[ICT-1634] Implementatie kostenbuilders SpaceStep
4:03 pm

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
Thursday 13 Aug 2020
5:38 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep

- De correctie van boekjaar 2018 moet enkel op de lade gebeuren en niet op de spacestep, voorgaand implementatie was niet 100% correct. De toeslag werd berekend op het bedrag waarvan de korting voor spacestep al gebeurd was

Options
    • -3
    • +3
    ./TestSSTBasisPrijzen.cls.xml
  2. … 5 more files in changeset.
9:06 am

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Toevoegen missende superklasse

Options
    • -0
    • +22
    ./Basis.cls.xml
8:48 am

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Alle halffabbuilders actief maken omdat vanaf nu alle data beshcikbaar is voor de selectiekenmerken

- End2EndTesten toevoegen die de prijs bepalen van verschillende uitvoeringen van SpaceStep

Options
    • -0
    • +43
    ./TestSSTBasisPrijzen.cls.xml
  2. ./SST
  3. … 4 more files in changeset.