In principe is het niet nodig om zulke testen bij te maken. Het is voldoende om één test te hebben die goed overweg kan met de KlantInstellingService:GeefMinimaleBestelhoeveelheid
Ik zou echter alles testen door de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser ipv expliciet de APPS.EC.Winkelkar.impl.ExterneWinkelkarService.KenmerkenAanpasser.LBX aan te roepen. Nu ontbreken we de stap dat de KenmerkenAanpasser de juiste aanpassing doet indien het om LBX gaat.
Vraagje : Vanwaar de relatie dat 1 leverdatum niet asap is ? Kan 1 leverdatum geen "onmiddellijk leverbaar" betekenen? Of heeft asap een andere betekenis
awel, dat ik het eerlijk gezegd zelf totaal niet zou weten. Ik had eerst wat extra wijzigingen in deze klasse (niets van doen met deze lijn, maar soit), maar die zijn allemaal ongedaan gemaakt (behalve die on-the-flight natuurlijk). Zonder die typfout had ik trouwens gewoon een revert gedaan en was er van bovenstaande niets meer te zien. Het was een mysterie, het is een mysterie en zal ten eeuwige dagen een mysterie blijven denk ik.
Misschien moeten we het iets explicieter maken dat het gaat over "EenBelg" of misschien nog beter "DenBelg" --> natuurlijk zal dat wel het gevolg hebben dat we "DenBelg" als begrip overal moeten gebruiken. Dan lijkt het mij nog beter om dat "DeBelgischeOnderneming" te nemen
Geen logica gebruiken in testen .. beter hiervoor een hardcoded waarde voor gebruiken . zal wel te vinden zijn in de vhTest.Utils Als de ToArray faalt dan zal dit negatieve inpakt hebben op de testen terwijl het er los van staat.
Op het eerste zicht lijkt mij dit toch wel een beetje clever code.
En proberen om de $extract in te korten om zo de maximale lengte van een lijn niet te overschrijven dat zal ook wel niet pakken.
Het is beter indien de UItvoeringInfo.DueOutTijdstip een %TimeStamp om de FormatAPI:FormatTimeStamp te gebruiken kwestie van het allemaal toch een beetje leesbaarder te hebben voor de gewone sterveling
Classification:
Improvement desirable
Ranking:
Minor
En waar staat dat ???
Men moet het gaan zoeken en wat blijkt in de den Assert staat er ergens dat de InterneBreedte 600 is .. Gelieve de info zo dicht mogelijk te zetten bij de test zelf
Volgens dat ik kan zien wordt er enkel getest dat Afmeting 75 is .. Geen enkele verwijzing te vinden naar de kleine aangepaste lade. ---> Heb het uiteindelijke gevonden het gaat om die 164 gebruikt onderaan in de test..
Gelieve de nodige aandacht te vestigen om de dingen die er toe doen.
Ik zou dan voorstellen dat je zelf ook geen .ToArray gebruikt in je testen. Commit 39782 Joc en commit 39550 TomV -> vhUnitTest.WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Test
Beste de code eens bekijken. Deze werking is eigen aan de werking van de builders. De aanpasser bepaalt of het om een spoelbaklade gaat en past de kenmerken enkel in dat geval aan.
Zoals besproken : 1. Aantal in halffab is een algemene waarde die de lengte representeert en is niet direct gerelateerd aan aantaldwarsverdelingen. Dit veld testen zal dan thuis horen in een andere test die algemener moet zijn. 2. Aangepaste kenmerken, method PasAan geeft geen 2 lades weer, maar 1 lade waardoor de berekening en deze test correct is.