Migrate JsonMapperTest to Java#10716
Migrate JsonMapperTest to Java#10716gh-worker-dd-mergequeue-cf854d[bot] merged 7 commits intomasterfrom
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 63 metrics, 8 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.063 s) : 0, 1062561
Total [baseline] (11.014 s) : 0, 11013906
Agent [candidate] (1.057 s) : 0, 1056966
Total [candidate] (11.009 s) : 0, 11009058
section appsec
Agent [baseline] (1.249 s) : 0, 1248947
Total [baseline] (11.158 s) : 0, 11158450
Agent [candidate] (1.25 s) : 0, 1249736
Total [candidate] (11.081 s) : 0, 11081196
section iast
Agent [baseline] (1.225 s) : 0, 1224824
Total [baseline] (11.341 s) : 0, 11341037
Agent [candidate] (1.229 s) : 0, 1229193
Total [candidate] (11.287 s) : 0, 11286744
section profiling
Agent [baseline] (1.189 s) : 0, 1188656
Total [baseline] (11.043 s) : 0, 11043004
Agent [candidate] (1.189 s) : 0, 1189062
Total [candidate] (11.051 s) : 0, 11050904
gantt
title petclinic - break down per module: candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.194 ms) : 0, 1194
crashtracking [candidate] (1.183 ms) : 0, 1183
BytebuddyAgent [baseline] (629.669 ms) : 0, 629669
BytebuddyAgent [candidate] (628.555 ms) : 0, 628555
AgentMeter [baseline] (29.199 ms) : 0, 29199
AgentMeter [candidate] (29.014 ms) : 0, 29014
GlobalTracer [baseline] (258.038 ms) : 0, 258038
GlobalTracer [candidate] (257.22 ms) : 0, 257220
AppSec [baseline] (31.562 ms) : 0, 31562
AppSec [candidate] (31.407 ms) : 0, 31407
Debugger [baseline] (59.6 ms) : 0, 59600
Debugger [candidate] (59.221 ms) : 0, 59221
Remote Config [baseline] (587.585 µs) : 0, 588
Remote Config [candidate] (593.779 µs) : 0, 594
Telemetry [baseline] (8.651 ms) : 0, 8651
Telemetry [candidate] (8.638 ms) : 0, 8638
Flare Poller [baseline] (8.01 ms) : 0, 8010
Flare Poller [candidate] (5.052 ms) : 0, 5052
section appsec
crashtracking [baseline] (1.196 ms) : 0, 1196
crashtracking [candidate] (1.204 ms) : 0, 1204
BytebuddyAgent [baseline] (657.88 ms) : 0, 657880
BytebuddyAgent [candidate] (659.634 ms) : 0, 659634
AgentMeter [baseline] (12.1 ms) : 0, 12100
AgentMeter [candidate] (12.123 ms) : 0, 12123
GlobalTracer [baseline] (259.873 ms) : 0, 259873
GlobalTracer [candidate] (259.251 ms) : 0, 259251
AppSec [baseline] (178.498 ms) : 0, 178498
AppSec [candidate] (178.319 ms) : 0, 178319
Debugger [baseline] (65.884 ms) : 0, 65884
Debugger [candidate] (64.819 ms) : 0, 64819
Remote Config [baseline] (573.704 µs) : 0, 574
Remote Config [candidate] (584.391 µs) : 0, 584
Telemetry [baseline] (8.921 ms) : 0, 8921
Telemetry [candidate] (9.852 ms) : 0, 9852
Flare Poller [baseline] (3.618 ms) : 0, 3618
Flare Poller [candidate] (3.557 ms) : 0, 3557
IAST [baseline] (24.129 ms) : 0, 24129
IAST [candidate] (24.027 ms) : 0, 24027
section iast
crashtracking [baseline] (1.196 ms) : 0, 1196
crashtracking [candidate] (1.194 ms) : 0, 1194
BytebuddyAgent [baseline] (794.603 ms) : 0, 794603
BytebuddyAgent [candidate] (797.571 ms) : 0, 797571
AgentMeter [baseline] (11.297 ms) : 0, 11297
AgentMeter [candidate] (11.353 ms) : 0, 11353
GlobalTracer [baseline] (246.639 ms) : 0, 246639
GlobalTracer [candidate] (247.794 ms) : 0, 247794
AppSec [baseline] (26.256 ms) : 0, 26256
AppSec [candidate] (26.384 ms) : 0, 26384
Debugger [baseline] (63.407 ms) : 0, 63407
Debugger [candidate] (63.461 ms) : 0, 63461
Remote Config [baseline] (531.918 µs) : 0, 532
Remote Config [candidate] (524.768 µs) : 0, 525
Telemetry [baseline] (14.919 ms) : 0, 14919
Telemetry [candidate] (14.841 ms) : 0, 14841
Flare Poller [baseline] (4.928 ms) : 0, 4928
Flare Poller [candidate] (4.894 ms) : 0, 4894
IAST [baseline] (25.05 ms) : 0, 25050
IAST [candidate] (25.096 ms) : 0, 25096
section profiling
ProfilingAgent [baseline] (94.708 ms) : 0, 94708
ProfilingAgent [candidate] (94.564 ms) : 0, 94564
crashtracking [baseline] (1.185 ms) : 0, 1185
crashtracking [candidate] (1.178 ms) : 0, 1178
BytebuddyAgent [baseline] (686.254 ms) : 0, 686254
BytebuddyAgent [candidate] (686.979 ms) : 0, 686979
AgentMeter [baseline] (8.62 ms) : 0, 8620
AgentMeter [candidate] (8.69 ms) : 0, 8690
GlobalTracer [baseline] (216.641 ms) : 0, 216641
GlobalTracer [candidate] (216.602 ms) : 0, 216602
AppSec [baseline] (32.006 ms) : 0, 32006
AppSec [candidate] (31.956 ms) : 0, 31956
Debugger [baseline] (62.539 ms) : 0, 62539
Debugger [candidate] (62.314 ms) : 0, 62314
Remote Config [baseline] (587.177 µs) : 0, 587
Remote Config [candidate] (582.612 µs) : 0, 583
Telemetry [baseline] (11.3 ms) : 0, 11300
Telemetry [candidate] (10.597 ms) : 0, 10597
Flare Poller [baseline] (3.519 ms) : 0, 3519
Flare Poller [candidate] (4.29 ms) : 0, 4290
Profiling [baseline] (95.279 ms) : 0, 95279
Profiling [candidate] (95.126 ms) : 0, 95126
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.06 s) : 0, 1060071
Total [baseline] (8.842 s) : 0, 8841860
Agent [candidate] (1.058 s) : 0, 1058055
Total [candidate] (8.812 s) : 0, 8812046
section iast
Agent [baseline] (1.226 s) : 0, 1225805
Total [baseline] (9.575 s) : 0, 9574552
Agent [candidate] (1.237 s) : 0, 1237490
Total [candidate] (9.537 s) : 0, 9537384
gantt
title insecure-bank - break down per module: candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.187 ms) : 0, 1187
crashtracking [candidate] (1.19 ms) : 0, 1190
BytebuddyAgent [baseline] (628.585 ms) : 0, 628585
BytebuddyAgent [candidate] (628.158 ms) : 0, 628158
AgentMeter [baseline] (29.249 ms) : 0, 29249
AgentMeter [candidate] (29.117 ms) : 0, 29117
GlobalTracer [baseline] (257.347 ms) : 0, 257347
GlobalTracer [candidate] (257.039 ms) : 0, 257039
AppSec [baseline] (31.594 ms) : 0, 31594
AppSec [candidate] (31.421 ms) : 0, 31421
Debugger [baseline] (58.653 ms) : 0, 58653
Debugger [candidate] (58.495 ms) : 0, 58495
Remote Config [baseline] (616.316 µs) : 0, 616
Remote Config [candidate] (585.01 µs) : 0, 585
Telemetry [baseline] (8.656 ms) : 0, 8656
Telemetry [candidate] (8.662 ms) : 0, 8662
Flare Poller [baseline] (8.031 ms) : 0, 8031
Flare Poller [candidate] (7.353 ms) : 0, 7353
section iast
crashtracking [baseline] (1.194 ms) : 0, 1194
crashtracking [candidate] (1.201 ms) : 0, 1201
BytebuddyAgent [baseline] (794.776 ms) : 0, 794776
BytebuddyAgent [candidate] (804.286 ms) : 0, 804286
AgentMeter [baseline] (11.28 ms) : 0, 11280
AgentMeter [candidate] (11.618 ms) : 0, 11618
GlobalTracer [baseline] (248.357 ms) : 0, 248357
GlobalTracer [candidate] (248.867 ms) : 0, 248867
IAST [baseline] (25.32 ms) : 0, 25320
IAST [candidate] (25.462 ms) : 0, 25462
AppSec [baseline] (26.451 ms) : 0, 26451
AppSec [candidate] (26.653 ms) : 0, 26653
Debugger [baseline] (62.282 ms) : 0, 62282
Debugger [candidate] (62.922 ms) : 0, 62922
Remote Config [baseline] (521.899 µs) : 0, 522
Remote Config [candidate] (548.316 µs) : 0, 548
Telemetry [baseline] (14.798 ms) : 0, 14798
Telemetry [candidate] (14.799 ms) : 0, 14799
Flare Poller [baseline] (4.853 ms) : 0, 4853
Flare Poller [candidate] (4.88 ms) : 0, 4880
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 17 metrics, 16 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section baseline
no_agent (18.926 ms) : 18732, 19120
. : milestone, 18926,
appsec (18.496 ms) : 18311, 18681
. : milestone, 18496,
code_origins (17.661 ms) : 17483, 17839
. : milestone, 17661,
iast (17.926 ms) : 17747, 18105
. : milestone, 17926,
profiling (19.651 ms) : 19456, 19846
. : milestone, 19651,
tracing (17.814 ms) : 17639, 17989
. : milestone, 17814,
section candidate
no_agent (18.003 ms) : 17823, 18184
. : milestone, 18003,
appsec (19.288 ms) : 19092, 19483
. : milestone, 19288,
code_origins (17.641 ms) : 17466, 17816
. : milestone, 17641,
iast (17.83 ms) : 17653, 18007
. : milestone, 17830,
profiling (18.711 ms) : 18523, 18899
. : milestone, 18711,
tracing (17.779 ms) : 17605, 17953
. : milestone, 17779,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section baseline
no_agent (1.179 ms) : 1167, 1190
. : milestone, 1179,
iast (3.195 ms) : 3149, 3240
. : milestone, 3195,
iast_FULL (5.822 ms) : 5763, 5882
. : milestone, 5822,
iast_GLOBAL (3.444 ms) : 3389, 3499
. : milestone, 3444,
profiling (2.136 ms) : 2117, 2156
. : milestone, 2136,
tracing (1.828 ms) : 1812, 1844
. : milestone, 1828,
section candidate
no_agent (1.183 ms) : 1172, 1195
. : milestone, 1183,
iast (3.272 ms) : 3226, 3318
. : milestone, 3272,
iast_FULL (5.954 ms) : 5894, 6014
. : milestone, 5954,
iast_GLOBAL (3.496 ms) : 3443, 3549
. : milestone, 3496,
profiling (2.128 ms) : 2108, 2148
. : milestone, 2128,
tracing (1.751 ms) : 1737, 1765
. : milestone, 1751,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section baseline
no_agent (15.014 s) : 15014000, 15014000
. : milestone, 15014000,
appsec (15.07 s) : 15070000, 15070000
. : milestone, 15070000,
iast (17.953 s) : 17953000, 17953000
. : milestone, 17953000,
iast_GLOBAL (17.738 s) : 17738000, 17738000
. : milestone, 17738000,
profiling (14.825 s) : 14825000, 14825000
. : milestone, 14825000,
tracing (15.082 s) : 15082000, 15082000
. : milestone, 15082000,
section candidate
no_agent (14.78 s) : 14780000, 14780000
. : milestone, 14780000,
appsec (15.019 s) : 15019000, 15019000
. : milestone, 15019000,
iast (18.136 s) : 18136000, 18136000
. : milestone, 18136000,
iast_GLOBAL (17.709 s) : 17709000, 17709000
. : milestone, 17709000,
profiling (14.733 s) : 14733000, 14733000
. : milestone, 14733000,
tracing (15.375 s) : 15375000, 15375000
. : milestone, 15375000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.60.0-SNAPSHOT~5661257a75, baseline=1.61.0-SNAPSHOT~77900be502
dateFormat X
axisFormat %s
section baseline
no_agent (1.473 ms) : 1462, 1485
. : milestone, 1473,
appsec (2.524 ms) : 2468, 2580
. : milestone, 2524,
iast (2.262 ms) : 2192, 2332
. : milestone, 2262,
iast_GLOBAL (2.316 ms) : 2246, 2387
. : milestone, 2316,
profiling (2.094 ms) : 2038, 2149
. : milestone, 2094,
tracing (2.08 ms) : 2026, 2134
. : milestone, 2080,
section candidate
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.535 ms) : 2479, 2590
. : milestone, 2535,
iast (2.275 ms) : 2205, 2345
. : milestone, 2275,
iast_GLOBAL (2.299 ms) : 2229, 2369
. : milestone, 2299,
profiling (2.102 ms) : 2045, 2159
. : milestone, 2102,
tracing (2.066 ms) : 2012, 2119
. : milestone, 2066,
|
PerfectSlayer
left a comment
There was a problem hiding this comment.
👏 praise: Thanks for update the JSON component and running the skill on a small example easy to review! 👍
❔ question: Did you check there was no hascode() in parametrized test names? If there are, better use:
- test case description as first value of the arguments,
- test case index otherwise
🎯 suggestion: About the various comments, feel free to add them to the skills too so it will get better with time 🙏
Ah good point. Addressed in ecf9f02 (and added to Claude skill :)) |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
2f04ccb
into
master
What Does This Do
Migrate
JsonMapperTestfrom Groovy to Java, using the/migrate-groovy-to-javaClaude skill. Add a recommendation to the Claude skill for the AI to check that the test count remained the same after migration.Motivation
This is part of a broader initiative to migrate all testing in this repo to JUnit
Additional Notes
We can see in the logs and artifacts of this job that the
JsonMapperTestwas run and all tests are still present.Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]
Note: Once your PR is ready to merge, add it to the merge queue by commenting
/merge./merge -ccancels the queue request./merge -f --reason "reason"skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.