CacheAdminA_trunk2010/vhUnitTest/APPS/EDI/common/impl

Validator

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 12 Dec 16
Commits this week: 0
Total Lines of Code (LoC): 197
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 09 Nov 2018
11:42 am

KurtC

[UW621] ZijkantLogoId - afwerken voor ECON - vhisie4 validatie en Default.

- fix voor data personality en voor klant. (getest op accept.)

- fix converter - vhintra geeft logoids, vhisie4 geeft logoid.

- validators + testen op klant aangepast.

Options
    • -2
    • +2
    ./ZijkantLogoValidator/IsValid/Test.cls.xml
  2. … 11 more files in changeset.
Monday 22 Oct 2018
12:51 pm

Wim Vermeulen

Wednesday 17 Oct 2018
1:39 pm

Tom Vermeulen

Tuesday 16 Oct 2018
4:25 pm

Wim Vermeulen

Niet alleen de "moveclass" maar ook de RunTestsOnCompile : die gaat ook op zoek naar ClassNames (maar niet als ze in ne String zitten) Dit is voor mij een belangrijkere reden dan moveclass.

Niet alleen de "moveclass" maar ook de RunTestsOnCompile : die gaat ook op zoek naar ClassNames (maar niet als ze in ne String zitten)
Dit is voor mij een belangrijkere reden dan moveclass.

8:12 am

Tom Vermeulen

een gewone vraag, geen opmerking: gebruik je bovenstaande omdat de moveclass deze dan ook zou vinden? of is er een andere reden om niet gewoon %Extends("APPS.blablabla") te doen?

een gewone vraag, geen opmerking:
gebruik je bovenstaande omdat de moveclass deze dan ook zou vinden? of is er een andere reden om niet gewoon %Extends("APPS.blablabla") te doen?

8:11 am

Tom Vermeulen

"RughHoogte" zou ik anders schrijven (met enkel een kleine h) 't is precies alsof iemand eerst een grote gebruikt heeft en die dan veranderd heeft naar een kleine http://subversion02.vanhoecke.be/s...

"RughHoogte" zou ik anders schrijven (met enkel een kleine h)
't is precies alsof iemand eerst een grote gebruikt heeft en die dan veranderd heeft naar een kleine

Friday 05 Oct 2018
12:10 pm

Wim Vermeulen

[UST4038] TAX-SY: RugHoogteValidator aanpassen voor TAX-Sifon
[UST4038] TAX-SY: RugHoogteValidator aanpassen voor TAX-Sifon
Thursday 04 Oct 2018
8:54 am

Wim Vermeulen

TAX-SY: RugHoogteValidator aanpassen voor TAX-Sifon

- UT uitgebreid met 2 testmethods voor TAX

- constantie: in de implementatie zijn niet alle mogelijken onmogelijke combinaties van rughoogte gecoverd (lees: geblocked).

Options
    • -0
    • +26
    ./RugHoogteValidator/IsValid/Test.cls.xml
Thursday 20 Sep 2018
3:49 pm

Wim Vermeulen

3:48 pm

Wim Vermeulen

Opgekuist (tijdens development gaf dit nog problemen, nu blijkbaar ni meer, dus terug leeg http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

Opgekuist (tijdens development gaf dit nog problemen, nu blijkbaar ni meer, dus terug leeg )

Friday 14 Sep 2018
12:19 pm

Martijn Vierstraete

Thursday 13 Sep 2018
1:55 pm

Wim Vermeulen

1:52 pm

Tom Vermeulen

12:43 pm

Wim Vermeulen

Er zijn (in een tamelijk recent verleden) helaas 2 versies van de "StandaardLade" ontstaan. En blijkbaar stonden nog niet alle converters aan elkaar geschakeld in de UT's voor die converters. Om éé...

Er zijn (in een tamelijk recent verleden) helaas 2 versies van de "StandaardLade" ontstaan. En blijkbaar stonden nog niet alle converters aan elkaar geschakeld in de UT's voor die converters.
Om één grote ketting van UT's te kunnen maken, moest de StandaardLade wel overal dezelfde kenmerken hebben.

Daarom heb ik in deze test de bestaande "StandaardLade" gedegradeerd tot een "GewoneCLadeOriongrijs", zodat duidelijk is dat dit NIET meer de standaardlade is.
Deze method wordt echter nog wel gebruikt in een aantal andere testen --> dus ook vhTestFiles, dus volledig aanpassen naar de nieuwe StandaardLade zou veel meer werk zijn t.o.v. alleen de methodnaam te wijzigen en bestaande tests+testresults te behouden.
De nieuwe "StandaardLade" in deze klasse is nu wel conform met de rest "APPS - DOM - EXT - EDI - ECON - ..."

11:12 am

Wim Vermeulen

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-? Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 . De review heb ik wel hier toegevoegd, w...

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-?
Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 .
De review heb ik wel hier toegevoegd, want gij hebt er achter gevraagd :-D :-D

Tuesday 11 Sep 2018
3:36 pm

Wim Vermeulen

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven! *vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: ...

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven!

  • vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: TAX LadeMetVeelExtras)
    Maar ik maak een nieuwe testmethod bij, die controleert of verpakking "" wordt geconverteerd naar V1.
3:02 pm

Wim Vermeulen

in volgende fase wordt dit opgekuist http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

in volgende fase wordt dit opgekuist

2:56 pm

Wim Vermeulen

Bij volgende update mag dit definitief verdwijnen.

Bij volgende update mag dit definitief verdwijnen.

2:55 pm

Wim Vermeulen

Klopt, maar is in een ander kanban-kaartje, dus ook aparte review

Klopt, maar is in een ander kanban-kaartje, dus ook aparte review

Wednesday 29 Aug 2018
11:10 am

Tom Vermeulen

moet de omgekeerde in APPS.PM.Maatwerk.impl.NaarObjectConverterTAX ook niet nog aangepast worden?

moet de omgekeerde in APPS.PM.Maatwerk.impl.NaarObjectConverterTAX ook niet nog aangepast worden?

11:05 am

Tom Vermeulen

idem

idem

11:05 am

Tom Vermeulen

mag dit niet gewoon weg?

mag dit niet gewoon weg?

10:59 am

Tom Vermeulen

die comments mogen weg zeker?

die comments mogen weg zeker?

10:40 am

Tom Vermeulen

in principe zouden hier nog wat testjes rond moeten komen

in principe zouden hier nog wat testjes rond moeten komen

10:37 am

Tom Vermeulen

oude logica laten staan in dit geval. alpnach stuurt nl die tag niet door en dat zou dus voor verrassingen kunnen zorgen. en in principe is het een sifon-gerelateerd kaartje en zou de werking van v...

oude logica laten staan in dit geval. alpnach stuurt nl die tag niet door en dat zou dus voor verrassingen kunnen zorgen.
en in principe is het een sifon-gerelateerd kaartje en zou de werking van verpakkingtypes niet aangepast moeten worden (ander kaartje nodig, kleine kaartjes enzo)

Wednesday 22 Aug 2018
11:25 am

Martijn Vierstraete

Zie opmerkingen vhTest.Utils.APPS.EDI.common.dto.LadeLBX

Zie opmerkingen vhTest.Utils.APPS.EDI.common.dto.LadeLBX

11:25 am

Martijn Vierstraete

Commentaar verwijderen

Commentaar verwijderen

11:23 am

Martijn Vierstraete

Waarom wordt dit hier ingevuld en niet bij de (nieuwe) standaardlade. Beiden zijn sowieso geen binnenlade...

Waarom wordt dit hier ingevuld en niet bij de (nieuwe) standaardlade. Beiden zijn sowieso geen binnenlade...

11:22 am

Martijn Vierstraete

naamgeving "gewone..." *ik zou deze lade niet voorzien als "standaard", in veel gevallen kan de standaardlade gebruikt worden. Indien toch deze variant vereist is kan de standaardlade in de test ...
  • naamgeving "gewone..."
  • ik zou deze lade niet voorzien als "standaard", in veel gevallen kan de standaardlade gebruikt worden. Indien toch deze variant vereist is kan de standaardlade in de test zelf nog overschreven worden
Tuesday 21 Aug 2018
5:51 pm

Wim Vermeulen

[UST3954] TAX-SY: herschikken van de LBX-testen (converters)
[UST3954] TAX-SY: herschikken van de LBX-testen (converters)