Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5565] LBX en TBX extra kostprijs weghalen
    • -1
    • +4
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 46 more files in changeset.
[ICT-5565] LBX en TBX extra kostprijs weghalen
    • -4
    • +1
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 46 more files in changeset.
[ICT] Aanpassing door PM aan masterdata
    • -7
    • +7
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 1 more file in changeset.
[ICT] Aanpassing door PM aan masterdata
    • -4
    • +4
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 19 more files in changeset.
[ICT-4241] Aanpassen schroef spacestep in bouwstenen

- Unittests zijn aangepast na deze wijziging

    • -7
    • +7
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 2 more files in changeset.
[ICT] Wijziging plaatmateriaal door PM
    • -1
    • +1
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 39 more files in changeset.
[ICT] Updata data plaatmateriaal
    • -1
    • +1
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 9 more files in changeset.
[ICT-3531] Proboxinhoudetiket - Interne breedte vermelden

- InterneBreedte wordt nu wel opgevuld door uitbreiding convertor

    • -0
    • +1
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 46 more files in changeset.
[ICT-3368] Testrunner Maatwerkcalculator

- Afronding op 6 getallen brengen

    • -6
    • +6
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 48 more files in changeset.
[ICT] Enumeraties in .Net uitgebreid
    • -2
    • +3
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
[ICT] PM neemt geleidelijk aan het plaatmateriaal van LBX en MVX samen
    • -8
    • +8
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 35 more files in changeset.
[ICT] PM neemt geleidelijk aan het plaatmateriaal van LBX en MVX samen
    • -7
    • +7
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 25 more files in changeset.
Hier staat nog een regel commentaar dat waarschijnlijk weg mag

Hier staat nog een regel commentaar dat waarschijnlijk weg mag

Geen idee of dit van u is, maar hier staat een schrijffout http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Geen idee of dit van u is, maar hier staat een schrijffout

[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator

- Testen updaten

    • -0
    • +1
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 59 more files in changeset.
[ICT-2836][rvPVR] LBX BomBol .Net klaarzetten en testen vanuit AX:

- End2End bestanden LBX GeleiderTechnologie na omleiden naar .Net, enkel volgorde gewijzigd

- End2End bestanden LBX met uitsparing na omleiden naar .Net, enkel volgorde gewijzigd

    • -107
    • +98
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 4 more files in changeset.
[ICT-2873][rvNVT] ICT: De Grote Prijswijziging van 01/05/2022 : ICT automatische testen aanpassen

- UnitTests End2End LBX

    • -111
    • +111
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 32 more files in changeset.
[ICT-2522] Update testbestanden voor prijswijzigingen 1 oktober 2021
    • -71
    • +71
    ./TestLadeMetSpacestepUitvoeringVolledig/Vandaag.txt
  1. … 43 more files in changeset.
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.

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

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

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

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

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?