CacheAdminA_trunk2010

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 11
Total Committers: 21
Last Commit: 04 Jul 23
Commits this week: 11
Total Lines of Code (LoC): 1.974.989
Net change in LoC this week: 527

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

Subversion Details

3 properties
  • svn:externals = /svn/CacheAdminACommon/trunk2010/TECH TECH /svn/CacheAdminACommon/trunk2010/vhLib vhLib /svn/CacheAdminACommon/trunk2010/Tools Tools /svn/CacheAdminACommon/trunk2010/vhUnitTest/TECH vhUnitTest/TECH /svn/CacheAdminACommon/trunk2010/vhUnitTest/vhLib vhUnitTest/vhLib /svn/CacheAdminACommon/trunk2010/vhUnitTest/Tools vhUnitTest/Tools /svn/CacheAdminASourceControl/trunk2010/svn svn /svn/CacheAdminAStories/trunk2010 vhStory /svn/CacheAdminACommon/trunk2010/vhTest/Mock/TECH vhTest/Mock/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Fake/TECH vhTest/Fake/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Dummy/TECH vhTest/Dummy/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Integratie/TECH vhTest/Integratie/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Spy/TECH vhTest/Spy/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Utils/TECH vhTest/Utils/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Mock/vhLib vhTest/Mock/vhLib /svn/CacheAdminACommon/trunk2010/vhTest/Fake/Tools vhTest/Fake/Tools /svn/CacheAdminACommon/trunk2010/vhTestFiles/UnitTests/TECH vhTestFiles/UnitTests/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Data/TECH vhTest/Data/TECH
  • svn:ignore = .git ClassExplorer Default_KurtC.prj.xml
  • svn:mergeinfo = /deploy:2316,2391,2429,2461,2790,2922,3203,4080,4105,4814,38942 /deploy2010:3001-3010,3420,3443,3457,3491,3525,3620,3642,3646,3650,3677,3680-3683,3686-3687,3689,3703-3707,3709-3711,3716-3718,3726-3728,3733,3735,3743,3747-3749,3751-3752,3762-3764,3766-3767,3770,3772-3773,3775,3782,3784,3794,3839,3847,3861,3910,3945,4008-4009,4030,4145,4151,4217,4231,4268,4437,4448,4477,4507,4519,4612-4613,4622-4624,4723,4749,4870,4922,5121,5177-5178,5183,5210,5762-5763,5980,5997,6077,6084,6086,6287,6299,6336-6338,6506-6507,6637,6706,6916,6928,7057,7059,7067,7070,7137,7220,7250,7330,7346,7720,7758,7792,7914,7952-7953,8018,8024,8027,8102,8135,8176,8214,8250,8360,8362-8363,8406,8409,8484,8491,8561,8659,8716,8733-8734,8736,8799-8800,8802,8859,9035,9207,9216,9236,9423,10263,10286,10392,10409,10559,10571,10714,10927,10961,11543,11629,11872,11927,12065,12106,12170,12391-12393,12713,12792-12794,12803,12925,12952,12962,13047,13066,13192,13202,13654,14017,14027-14028,14071,14344,14439,14557-14565,14569,14600,14903,15035,15047,15087,15188-15190,15192,15265,15411,15902,16164-16165,16287,16384-16385,16408,16691,16721,16853,16935,16959,16983,17141,17494,17662,18100,18108-18109,18278-18286,18580,18679,18746,19120-19121,19578,20111,21545,22164,22436,22482,22832,22838-22839,23286,23416,23418,23421,23483,23562,23616,23636,29622,30300,30320,31795,31797,31799,34956,34962,35175-35176,35751,35965,36123,36181,36462,37123,37148-37149,38067-38072,38498,38583,38927-38930,39142,39203,39205-39206,39367,39464,39466,39482,39603-39604,39773,39902,39960,40332-40334,40390,40515-40516,40557-40558,40653,40849,40851,41111-41116,41548,41875,41879,42461,42556,42971-42973,44286-44288,44375-44376,46402,46479-46483,46490,46521,46864,46891-46896,47159-47160,47228-47229,47234,47505-47506,47508,47545-47551,48541,48604-48609,51918,52329,52348-52350,53211-53212,53261,53462,54030-54035,54521-54525,54962-54966,55339-55342,55524-55525,55596,55607,56283,56731-56732,56826,56981-56982,57290-57291,57294,57298-57299,57445,57542-57546,58227-58230,58805-58806,58966-58969,60346-60347,60367,60423,60434,61619,61621-61622,62122-62124,62527,63268-63273,65629,65638-65642,65650-65653 /trunk:2977,2979,2985,2987,2990-2996,2999,3002-3005,3007-3009,3011-3015,3019-3045,3049-3050,3052-3053,3055,3060-3062,3067-3068,3081-3091,3094-3095,3097-3099,3104-3108,3113,3115,3120,3123-3126,3128,3134,3142,3145-3148,3157,3164,3173-3174,3183,3185,3189,3199,3202,3204,3211,3213-3215,3222-3223,3225,3247,3252,3264,3289-3292,3298,3309,3356-3357,3362,3374,3389,3391,3402,3405,3409,3430-3431,3448,3451-3452,3463,3467,3477,3479,3481-3482,3484,3486,3500,3526,3535,3539,3551,3561,3571,3573,3614,3616,3636,3638,3640,3647,3654,3664,3668,3672,3674,3678,3690-3692,3699,3701,3717,3724,3729,3737,3739,3741-3742,3744,3787,3789-3790,3792,3803,3833,3837,3844,3846,3850-3851,3857,3862,3868,3872-3873,3879,3909,3914,3927,3930,3932-3933,3941,3957,3966,3968,3970,3972-3973,3982,3986,3990,3994,3996,4000,4006,4019,4022,4034,4037-4040,4045,4056,4079,4104,4134,4156,4159,4173,4397,4817,4990,5059-5060,5093-5094,5108-5137,5144,5222-5223,5446,5651,7918
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 18 Jul 2019
9:44 am

Tommy Hebb

Wednesday 17 Jul 2019
12:41 pm

Wim Vermeulen

12:41 pm

Wim Vermeulen

#dim ProductServiceMock As vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService = ##class(vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService).%New() Do ProductServiceMock.VerwachtMethodCall("...

#dim ProductServiceMock As vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService = ##class(vhTest.Mock.WSimpl.AX.CalculatedProduct.ProductService).%New()
Do ProductServiceMock.VerwachtMethodCall("GeefProduct", ..#RandomProductId, Request.ItemId).ExactAantalKeer(0)
Set ..ProductService = ProductServiceMock

Dit toevoegen, dan kan je de Do ..Verifieer() oproepen

10:12 am

Wim Vermeulen

Deze property wordt niet meer gebruikt in deze klasse en mag dus weg (is verplaatst naar productUpdater)

Deze property wordt niet meer gebruikt in deze klasse en mag dus weg (is verplaatst naar productUpdater)

Tuesday 16 Jul 2019
11:49 am

Tommy Hebb

11:06 am

Martijn Vierstraete

9:55 am

Tommy Hebb

[UST4269] PM: Maatwerk: BOMBOL VHIP481: AxNaarEconConverter:
[UST4269] PM: Maatwerk: BOMBOL VHIP481: AxNaarEconConverter:
Monday 15 Jul 2019
2:14 pm

Tommy Hebb

[UST4276] PM: Maatwerk: BOMBOL VHIP481: Refactor RequestConverter:
[UST4276] PM: Maatwerk: BOMBOL VHIP481: Refactor RequestConverter:
2:12 pm

Tommy Hebb

11:47 am

Wim Vermeulen

10:20 am

Tommy Hebb

10:20 am

Tommy Hebb

Apart kaartje: UST4276 (ICT-729)

Apart kaartje: UST4276 (ICT-729)

10:13 am

Wim Vermeulen

10:06 am

Tommy Hebb

[UST4275] PM: Maatwerk: BOMBOL VHIP481: Verfijning en bug-fixing:
[UST4275] PM: Maatwerk: BOMBOL VHIP481: Verfijning en bug-fixing:
Friday 12 Jul 2019
4:37 pm

Wim Vermeulen

ProjectSettings in UT is wat mij betreft toch redelijk hard "vloeken in de kerk" Ik weet wel dat het op andere plaatsen ook gedaan wordt, ma toch blijft het een verkrachting van UT . Op deze manier...

ProjectSettings in UT is wat mij betreft toch redelijk hard "vloeken in de kerk"
Ik weet wel dat het op andere plaatsen ook gedaan wordt, ma toch blijft het een verkrachting van UT .
Op deze manier worden een heel aantal methods niet getest. Hoe kan je dan weten of het veilig is om de projectsetting terug te switchen, indien nodig.
Beter : code afzonderen en zorgen dat ze ALTIJD werkt, ongeacht de waarde van de projectsetting. M.a.w. zowel UT voor oude impl blijft en UT voor nieuwe impl ==> properste oplossing : 2 verschillende klassen

N.B. : Dit stukje had ik nog niet bekeken voor we vandaag gesproken hebben. Dit is voor mij absoluut voldoende argument om de nieuwe code af te splitsen!
Alle methods waarin staat : If (...ProjectSetting) of Not(ProjectSetting) zijn "Afgekeurd by review". Sorry dude.

Thursday 11 Jul 2019
1:28 pm

Wim Vermeulen

idem andere while() continue in deze review

idem andere while() continue in deze review

12:42 pm

Wim Vermeulen

Goed gedaan! goede uitbreiding : uwe CheckDelControleur is nen "interface" en alles wat er achter die interface gebeurt, is hier niet relevant (single responsibility). dus door die uit te mocken he...

Goed gedaan!
goede uitbreiding :
uwe CheckDelControleur is nen "interface" en alles wat er achter die interface gebeurt, is hier niet relevant (single responsibility).
dus door die uit te mocken heb je perfect op eenvoudige wijze deze UT uitgebreid, en zo getest wat er moest getest worden.
Toppie!

12:31 pm

Wim Vermeulen

Ik zou hier toch op "SHOFTPUZBKWME" controleren i.p.v. $$$ElkeWaarde. --> motivatie : geeft de CheckDel hetzelfde resultaat als je bvb. "" (leeg) als signatuur doorgeeft? Ikzelf vermoed van niet, d...

Ik zou hier toch op "SHOFTPUZBKWME" controleren i.p.v. $$$ElkeWaarde.
--> motivatie : geeft de CheckDel hetzelfde resultaat als je bvb. "" (leeg) als signatuur doorgeeft? Ikzelf vermoed van niet, dus beter op de specifieke signatuur checken.

Ik begrijp echter wel waarom je voor $$$ElkeWaarde had gekozen

12:21 pm

Wim Vermeulen

Waarom niet gewoon : If (Bezwaar = "") { }

Waarom niet gewoon : If (Bezwaar = "") { }

11:46 am

Wim Vermeulen

Waarom niet gewoon : If (LadeVariant = "") { }

Waarom niet gewoon : If (LadeVariant = "") { }

11:38 am

Wim Vermeulen

Nog een fijne tip om de UT te verbeteren : De vier aangeduide lijnen code verplaatsen naar een private method Method AssertLadeVariant(ProductTypeApiStub, ProductId, Kenmerken As %ListOfObjects, V...

Nog een fijne tip om de UT te verbeteren :
De vier aangeduide lijnen code verplaatsen naar een private method 

 Method AssertLadeVariant(ProductTypeApiStub, ProductId, Kenmerken As %ListOfObjects, VerwachteLadeVariant As AXimpl.PM.enu.LadeVariant) [private]  

Deze techniek passen we dikwijls toe, als we telkens hetzelfde asserten (+verifieer).

ProductTypeApi hoeft dan geen property meer te zijn (bijkomstig logisch gevolg )

11:22 am

Wim Vermeulen

"Continue" op deze plaats in de While heeft geen zin. Beter : while (HasNext() ) && (AxLadeVariant = "")

"Continue" op deze plaats in de While heeft geen zin.
Beter : while (HasNext() ) && (AxLadeVariant = "")

10:38 am

Wim Vermeulen

--> %ListOfObjects ?

--> %ListOfObjects ?

10:37 am

Wim Vermeulen

Is niet fout ofzo, maar ik zou eerder het Kenmerken-object doorgeven aan de method, en de KenmerkIterator hoort eigenlijk binnen de method te zitten.

Is niet fout ofzo,
maar ik zou eerder het Kenmerken-object doorgeven aan de method, en de KenmerkIterator hoort eigenlijk binnen de method te zitten.

10:15 am

Tommy Hebb

10:14 am

Tommy Hebb

10:14 am

Tommy Hebb

Wednesday 10 Jul 2019
6:24 pm

Wim Vermeulen

6:22 pm

Wim Vermeulen

6:14 pm

Wim Vermeulen