Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2494][rvNVT] HX: Connecties van/naar visidot01 (server) uitschakelen in caché

- APPS.VisiDot.impl.CleanUp : gescheduleded cleanup uitschakelen

[ICT-2022] PA-UB-203: Vhoss EWMS geen onderscheid kant A en B
[ICT-2022] PA-UB-203: Vhoss EWMS geen onderscheid kant A en B
[ICT-2022] PA-UB-203: Vhoss EWMS geen onderscheid kant A en B
[ICT-2022] PA-UB-203: Vhoss EWMS geen onderscheid kant A en B
[ICT-2022] PA-UB-203: Vhoss EWMS geen onderscheid kant A en B

- method op VisiDot/impl/Tech toegevoegd om achterkant van een frameid terug te geven

[ICT-1243] Vhosskasten recepteren kuist de database niet proper op

- extra queries om locations en carriers op te kuisen

[ICT-1057] Nog niet alle vhosskasten worden geregistreerd

- while ipv if om oude records ook te verwijderen

[ICT-1057] Nog niet alle vhosskasten worden geregistreerd

- oneliner gemaakt

Dees klopt niet (sorry, niet gezien bij orginele review) De method CheckVhossOp... is geen classmethod. Aangezien de klasse ...VHossService nog niet geïnstantieerd werd, geeft het een fout at runti...

Dees klopt niet (sorry, niet gezien bij orginele review)
De method CheckVhossOp... is geen classmethod. Aangezien de klasse ...VHossService nog niet geïnstantieerd werd, geeft het een fout at runtime.
Oplossing: ##class(APPS.VisiDot.impl.VHossService).%New().CheckVhossOpMeerdereLocaties()

nog een kleintje: als je #dim doet, zet er dan een "As %Boolean" bij (in dit geval). Op zich zal het even goed werken want het is een untyped language, maar het is een betere manier van werken.

[ICT-999] LBX Vhoss Kasten
[ICT-999] LBX Vhoss Kasten
naamgevingnitpicking: een IndienMethodCall wordt enkel voor "sturing" van het SUT gebruikt, dus is het eigenlijk een EwmsLocationsStub ipv ...Mock. Bij een VerwachtMethodCall is het wel een ...Mock...

naamgevingnitpicking: een IndienMethodCall wordt enkel voor "sturing" van het SUT gebruikt, dus is het eigenlijk een EwmsLocationsStub ipv ...Mock. Bij een VerwachtMethodCall is het wel een ...Mock gelijk hieronder. In principe is dat ook niet 100% correct, maar dat is een beetje de conventie zoals ze aangehouden wordt in deze gevallen.
Waarom hier ook geen VerwachtMethodCall gebruikt trouwens? In principe is het hetzelfde gedrag als de test hieronder, enkel de returnvalue is anders

Opgelet met de indexen van de data, wanneer je rechtstreeks in de globals gaat aanpassen. In dit geval is het geen probleem. Anderszijds zie ik echt geen zinvolle reden waarom je hier niet gewoon ...

Opgelet met de indexen van de data, wanneer je rechtstreeks in de globals gaat aanpassen.
In dit geval is het geen probleem.

Anderszijds zie ik echt geen zinvolle reden waarom je hier niet gewoon het object opent (het ObjectId heb je al), property wijzigt en een propere .Save() uitvoert.
Wijzigen via globals zou ik alleen "toelaten" wanneer een andere methode echt (te) complex zou zijn. Wat hier zeker niet het geval is, hé.

TIP : ivm $Listbuild : om enkel het 3e list-item te wijzigen Set $LI(varOfGlobal, 3) = NewValue Dit moet je hier zeker (leren) gebruiken!

TIP : ivm $Listbuild : om enkel het 3e list-item te wijzigen
Set $LI(varOfGlobal, 3) = NewValue

Dit moet je hier zeker (leren) gebruiken!

##class(APPS.VisiDot.impl.Tech).IsVhossID( ) oproepen

##class(APPS.VisiDot.impl.Tech).IsVhossID( ) oproepen

[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen

Extra test met mock of EWMS/Locations

  1. … 2 more files in changeset.
[ICT-974]MAG: Voorraad: zbcheck VHOSS op meerdere plaatsen

Edit VhossService to use mock and alter test

  1. … 2 more files in changeset.
mocks gebruiken

mocks gebruiken

TECH.Mockable maken en de gegenereerde mocks in de testvhossservice gebruiken

TECH.Mockable maken en de gegenereerde mocks in de testvhossservice gebruiken

zie opmerking andere test

zie opmerking andere test

01... gebruiken ipv TT....

01... gebruiken ipv TT....

data best opkuisen in OnAfterAllTests

data best opkuisen in OnAfterAllTests

misschien XX in een aparte variabele steken voor de leesbaarheid

misschien XX in een aparte variabele steken voor de leesbaarheid

Ge moogt gerust spaties gebruiken in testnamen

Ge moogt gerust spaties gebruiken in testnamen