Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2278] [rvWV] PM: Maatwerk: TAX: End2End-testen aanmaken, ter validatie van de builder(s):

- IngegevenKenmerken worden nu meegegeven aan GeefHalffabItemsBoom op de MaatwerkDetailsConvertor, om op de juiste plek aan de Familie en Variant te geraken => Ook in de UnitTesten toevoegen graag => En deze ook nog => En dan dees 2 nog

[ICT-1990] BOM BOL omleiden naar .NET voor MAT

- Testen updaten omwille van omschakeling bombol van cache naar .NET

[ICT-1437] TBX/LBX/TAX uitfasering matmateriaal = antislip - Cache gedeelte
  1. … 61 more files in changeset.
Private methods onderaan plaatsen. (zie zelfde opm bij andere UT)

Private methods onderaan plaatsen.
(zie zelfde opm bij andere UT)

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken. Let's skype http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken.
Let's skype

Private methods onderaan plaatsen. Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant) Dus toch private en onderaan...

Private methods onderaan plaatsen.

Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant)

Dus toch private en onderaan
En dan zou ik er een Function van maken i.p.v. "Do ..." van maken,
Bijvoorbeeld : GeefDataMMatCreator() met dezelfde args.

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven. Da's 4 lijnen code-duplicatie minder. Anderszijds wou ik opmerken dat je de kleur-variat...

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven.
Da's 4 lijnen code-duplicatie minder.

Anderszijds wou ik opmerken dat je de kleur-variaties beter in de UT zelf implementeert.
De Utils.MatKenmerken klasse is niet bedoeld om alle mogelijke variaties van kleuren, die we ooit ergens (in een UT) zouden nodig hebben, hier te gaan toevoegen.
1 kleur-variatie lijkt me nog wel te kunnen in deze klasse (is al schemerzone), maar 3 toch liever niet.

Ter illustratie : in deze klasse kan bijvoorbeeld wel : TaorIndelingMat , ExtraBredeMat, ... (die zijn nu uiteraard niet nodig)

ICT-1497: Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
ICT-1497: Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor

Extra Assert toevoegen bij Unittesten voor methode GeefDimensieVoorKorttekst

[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor

Bugfix copy paste fout in verkeerde klasse die test deed falen nu dus terugdraaien

[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor

Unittesten voor nieuw methode GeefKleurCodeVoorKorttekst + /vhTest/Utils/DOM/PM/Maatwerk/Calc/Common/impl/MATKenmerken uitbreiden

  1. … 1 more file in changeset.
[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor

Refactor Unittesten voor methode GeefDimensieVoorKorttekst

[ICT-1497] Matten: kleur wordt niet weergegeven bij type Orgalux en Taor

Unittesten voor nieuwe methodes GeefDimensieVoorKorttekst en GeefKleurCodeVoorKorttekst

  1. ./DataMMATCreator/GeefKleurCodeVoorKorttekst
  2. ./DataMMATCreator/GeefDimensieVoorKorttekst
[UST3860] VHIP184: Sync Prod: Wissel leverancier:
[UST3860] VHIP184: Sync Prod: Wissel leverancier:
[UST3860] VHIP184: Sync Prod: Wissel leverancier:

- UnitTests => Het al dan niet gebruik van de ZetLeverancierId bepaalt of huidige J node behouden blijft of een nieuwe gemaakt wordt en de huidige verwijderd wordt

[UST3710] TAX LC: Calc: generisch product per type onderdeel, voor telbaarheid in AX:

- ZetGenerischProductId bijgezet

  1. … 7 more files in changeset.
[DEF672] VHIP184: Sync Prod: BtwKode op product nodig (MAG: Leverbon maken: CalcBon error):
[DEF672] VHIP184: Sync Prod: BtwKode op product nodig (MAG: Leverbon maken: CalcBon error):
[DEF672] VHIP184: Sync Prod: BtwKode op product nodig (MAG: Leverbon maken: CalcBon error):

- ZetBTWCode op DataMProduct

  1. … 3 more files in changeset.
[UST3673] VHIP504: ORDER2AX: Pakketreferentie meesturen

- Kunnen opvragen van pakket naam adhv de pakketid (pakketid)

  1. … 3 more files in changeset.
[ICT] Openstaande wijziging Cache01

- extra logging bij Korttekst > 25, test vergeten mee te committen.

  • More
  • CR-877
  • finished reviewing
?

?

[UW420] VHIP184 : validatie op ident nummer inbouwen
[UW420] VHIP184 : validatie op ident nummer inbouwen