Activity CacheAdminA_trunk2010/vhUnitTest/ECON/impl/PM/Maatwerk

11:07 am

Tom Vermeulen

More
38901
changed 3 files
root:

[UST2040] Vh4 TBX B.E.: e-Con naar Edi mapping van ingegeven kenmerken TBX

- Moveclass van ECON-EDI converter, testjes deel 2

- -35
- +0
./EdiConverter/ConverteerEconNaarEdi/TestTBX.cls.xml
- History
- Source
- Diff
./EdiConverter/.../TestTBX.cls.xml
… 2 more files in changeset.

9:45 am

Tom Vermeulen

More
38899
changed 8 files
root:

[UST2040] Vh4 TBX B.E.: e-Con naar Edi mapping van ingegeven kenmerken TBX

- Moveclass van ECON-EDI converter, testjes deel 1

- -3
- +3
./EdiConverter/ConverteerEconNaarEdi/TestTBX.cls.xml
- History
- Source
- Diff
./EdiConverter/.../TestTBX.cls.xml
- -3
- +3
./EdiConverter/ConverteerEdiNaarEcon/TestTBX.cls.xml
- History
- Source
- Diff
./EdiConverter/.../TestTBX.cls.xml
… 6 more files in changeset.

7:50 am

Dacht dat over een paar dagen/weken de converter ook LBX en TAX ging moeten verwerken, daarom geen TBXKenmerken in naam. De tests hebben wel TBX in hun naam. Andere convertoren behandelen nu ook al...

Dacht dat over een paar dagen/weken de converter ook LBX en TAX ging moeten verwerken, daarom geen TBXKenmerken in naam. De tests hebben wel TBX in hun naam. Andere convertoren behandelen nu ook alle types schuifkes.

5:18 pm

Jo Claes

More
CR-9
finished reviewing

5:18 pm

Jo Claes

More
Re: CR-9
commented on review

In de naamgeving van de klasse ontbreekt het feit dat het over TbxKenmerken gaat en ECONimpl is een root package cfr AXimpl en WSimpl Dus eigenlijk zou ECONimpl.PM.Maatwerk.TbxKenmerkenEdiConverte...

In de naamgeving van de klasse ontbreekt het feit dat het over TbxKenmerken gaat en ECONimpl is een root package
cfr AXimpl en WSimpl

Dus eigenlijk zou ECONimpl.PM.Maatwerk.TbxKenmerkenEdiConverter een correctere naam zijn --> natuurlijk is het dan noodzakelijk om de UT's hun naam te wijzigen

5:12 pm

Jo Claes

More
Re: CR-9
commented on review

Is eigenlijk een beetje muggenziften maar het zou nog dat tikkeltje beter zijn mocht de code er ongeveer zo uitzien do ..AssertEdiNaarEcon(##class(vhTest.Utils.APPS.EDI.common.dto.LadeTBX).Standaa...

Is eigenlijk een beetje muggenziften maar het zou nog dat tikkeltje beter zijn mocht de code er ongeveer zo uitzien

do ..AssertEdiNaarEcon(##class(vhTest.Utils.APPS.EDI.common.dto.LadeTBX).StandaardLade(),##class(vhTest.Utils.ECON.PM.Maatwerk.dto.TbxKenmerken).StandaardLade())

2:49 pm