Fix TestNG session status reported with EFD#11087
Fix TestNG session status reported with EFD#11087daniel-mohedano wants to merge 5 commits intomasterfrom
Conversation
Test Environment - sbt-scalatestJob Status: success
|
Test Environment - nebula-release-pluginJob Status: success
|
Test Environment - netflix-zuulJob Status: success
|
Test Environment - reactive-streams-jvmJob Status: success
|
Test Environment - pass4sJob Status: success
|
Test Environment - sonar-kotlinJob Status: success
|
Test Environment - jolokiaJob Status: success
|
Test Environment - okhttpJob Status: success
|
Test Environment - spring_bootJob Status: success
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 61 metrics, 10 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.06 s) : 0, 1059732
Total [baseline] (8.867 s) : 0, 8866719
Agent [candidate] (1.062 s) : 0, 1062006
Total [candidate] (8.814 s) : 0, 8814323
section iast
Agent [baseline] (1.22 s) : 0, 1220364
Total [baseline] (9.505 s) : 0, 9504653
Agent [candidate] (1.218 s) : 0, 1217934
Total [candidate] (9.525 s) : 0, 9524547
gantt
title insecure-bank - break down per module: candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.228 ms) : 0, 1228
crashtracking [candidate] (1.238 ms) : 0, 1238
BytebuddyAgent [baseline] (633.99 ms) : 0, 633990
BytebuddyAgent [candidate] (636.664 ms) : 0, 636664
AgentMeter [baseline] (29.512 ms) : 0, 29512
AgentMeter [candidate] (29.514 ms) : 0, 29514
GlobalTracer [baseline] (248.565 ms) : 0, 248565
GlobalTracer [candidate] (249.546 ms) : 0, 249546
AppSec [baseline] (32.425 ms) : 0, 32425
AppSec [candidate] (32.487 ms) : 0, 32487
Debugger [baseline] (59.281 ms) : 0, 59281
Debugger [candidate] (59.413 ms) : 0, 59413
Remote Config [baseline] (610.1 µs) : 0, 610
Remote Config [candidate] (596.495 µs) : 0, 596
Telemetry [baseline] (8.047 ms) : 0, 8047
Telemetry [candidate] (8.079 ms) : 0, 8079
Flare Poller [baseline] (9.798 ms) : 0, 9798
Flare Poller [candidate] (8.242 ms) : 0, 8242
section iast
crashtracking [baseline] (1.237 ms) : 0, 1237
crashtracking [candidate] (1.22 ms) : 0, 1220
BytebuddyAgent [baseline] (798.668 ms) : 0, 798668
BytebuddyAgent [candidate] (797.596 ms) : 0, 797596
AgentMeter [baseline] (11.305 ms) : 0, 11305
AgentMeter [candidate] (11.32 ms) : 0, 11320
GlobalTracer [baseline] (238.247 ms) : 0, 238247
GlobalTracer [candidate] (237.937 ms) : 0, 237937
AppSec [baseline] (31.199 ms) : 0, 31199
AppSec [candidate] (31.188 ms) : 0, 31188
Debugger [baseline] (60.02 ms) : 0, 60020
Debugger [candidate] (60.661 ms) : 0, 60661
Remote Config [baseline] (1.155 ms) : 0, 1155
Remote Config [candidate] (1.724 ms) : 0, 1724
Telemetry [baseline] (12.738 ms) : 0, 12738
Telemetry [candidate] (11.177 ms) : 0, 11177
Flare Poller [baseline] (3.477 ms) : 0, 3477
Flare Poller [candidate] (3.398 ms) : 0, 3398
IAST [baseline] (25.827 ms) : 0, 25827
IAST [candidate] (25.708 ms) : 0, 25708
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.058 s) : 0, 1057571
Total [baseline] (11.007 s) : 0, 11007226
Agent [candidate] (1.053 s) : 0, 1053091
Total [candidate] (10.965 s) : 0, 10964946
section appsec
Agent [baseline] (1.244 s) : 0, 1244292
Total [baseline] (11.209 s) : 0, 11208844
Agent [candidate] (1.263 s) : 0, 1263025
Total [candidate] (11.116 s) : 0, 11116381
section iast
Agent [baseline] (1.226 s) : 0, 1225875
Total [baseline] (11.236 s) : 0, 11236165
Agent [candidate] (1.221 s) : 0, 1221082
Total [candidate] (11.128 s) : 0, 11128310
section profiling
Agent [baseline] (1.184 s) : 0, 1183654
Total [baseline] (11.081 s) : 0, 11080881
Agent [candidate] (1.18 s) : 0, 1180402
Total [candidate] (11.149 s) : 0, 11149138
gantt
title petclinic - break down per module: candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.232 ms) : 0, 1232
crashtracking [candidate] (1.213 ms) : 0, 1213
BytebuddyAgent [baseline] (631.903 ms) : 0, 631903
BytebuddyAgent [candidate] (630.407 ms) : 0, 630407
AgentMeter [baseline] (29.549 ms) : 0, 29549
AgentMeter [candidate] (29.389 ms) : 0, 29389
GlobalTracer [baseline] (249.698 ms) : 0, 249698
GlobalTracer [candidate] (247.95 ms) : 0, 247950
AppSec [baseline] (32.539 ms) : 0, 32539
AppSec [candidate] (32.206 ms) : 0, 32206
Debugger [baseline] (60.301 ms) : 0, 60301
Debugger [candidate] (59.772 ms) : 0, 59772
Remote Config [baseline] (608.179 µs) : 0, 608
Remote Config [candidate] (586.109 µs) : 0, 586
Telemetry [baseline] (8.135 ms) : 0, 8135
Telemetry [candidate] (8.071 ms) : 0, 8071
Flare Poller [baseline] (7.571 ms) : 0, 7571
Flare Poller [candidate] (7.492 ms) : 0, 7492
section appsec
crashtracking [baseline] (1.219 ms) : 0, 1219
crashtracking [candidate] (1.27 ms) : 0, 1270
BytebuddyAgent [baseline] (659.274 ms) : 0, 659274
BytebuddyAgent [candidate] (672.857 ms) : 0, 672857
AgentMeter [baseline] (12.054 ms) : 0, 12054
AgentMeter [candidate] (12.176 ms) : 0, 12176
GlobalTracer [baseline] (247.836 ms) : 0, 247836
GlobalTracer [candidate] (250.589 ms) : 0, 250589
AppSec [baseline] (184.54 ms) : 0, 184540
AppSec [candidate] (185.594 ms) : 0, 185594
Debugger [baseline] (65.868 ms) : 0, 65868
Debugger [candidate] (66.345 ms) : 0, 66345
Remote Config [baseline] (601.241 µs) : 0, 601
Remote Config [candidate] (610.359 µs) : 0, 610
Telemetry [baseline] (8.495 ms) : 0, 8495
Telemetry [candidate] (8.609 ms) : 0, 8609
Flare Poller [baseline] (3.54 ms) : 0, 3540
Flare Poller [candidate] (3.56 ms) : 0, 3560
IAST [baseline] (24.536 ms) : 0, 24536
IAST [candidate] (24.803 ms) : 0, 24803
section iast
crashtracking [baseline] (1.225 ms) : 0, 1225
crashtracking [candidate] (1.227 ms) : 0, 1227
BytebuddyAgent [baseline] (802.652 ms) : 0, 802652
BytebuddyAgent [candidate] (798.789 ms) : 0, 798789
AgentMeter [baseline] (11.391 ms) : 0, 11391
AgentMeter [candidate] (11.372 ms) : 0, 11372
GlobalTracer [baseline] (239.317 ms) : 0, 239317
GlobalTracer [candidate] (238.327 ms) : 0, 238327
AppSec [baseline] (31.198 ms) : 0, 31198
AppSec [candidate] (31.935 ms) : 0, 31935
Debugger [baseline] (64.236 ms) : 0, 64236
Debugger [candidate] (59.417 ms) : 0, 59417
Remote Config [baseline] (548.741 µs) : 0, 549
Remote Config [candidate] (1.108 ms) : 0, 1108
Telemetry [baseline] (9.858 ms) : 0, 9858
Telemetry [candidate] (13.434 ms) : 0, 13434
Flare Poller [baseline] (3.464 ms) : 0, 3464
Flare Poller [candidate] (3.504 ms) : 0, 3504
IAST [baseline] (25.73 ms) : 0, 25730
IAST [candidate] (25.73 ms) : 0, 25730
section profiling
ProfilingAgent [baseline] (94.614 ms) : 0, 94614
ProfilingAgent [candidate] (93.737 ms) : 0, 93737
crashtracking [baseline] (1.168 ms) : 0, 1168
crashtracking [candidate] (1.166 ms) : 0, 1166
BytebuddyAgent [baseline] (689.584 ms) : 0, 689584
BytebuddyAgent [candidate] (689.131 ms) : 0, 689131
AgentMeter [baseline] (9.11 ms) : 0, 9110
AgentMeter [candidate] (9.039 ms) : 0, 9039
GlobalTracer [baseline] (207.159 ms) : 0, 207159
GlobalTracer [candidate] (206.374 ms) : 0, 206374
AppSec [baseline] (32.662 ms) : 0, 32662
AppSec [candidate] (32.747 ms) : 0, 32747
Debugger [baseline] (65.944 ms) : 0, 65944
Debugger [candidate] (65.281 ms) : 0, 65281
Remote Config [baseline] (575.585 µs) : 0, 576
Remote Config [candidate] (572.171 µs) : 0, 572
Telemetry [baseline] (7.877 ms) : 0, 7877
Telemetry [candidate] (7.849 ms) : 0, 7849
Flare Poller [baseline] (3.548 ms) : 0, 3548
Flare Poller [candidate] (3.551 ms) : 0, 3551
Profiling [baseline] (95.189 ms) : 0, 95189
Profiling [candidate] (94.3 ms) : 0, 94300
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 18 metrics, 15 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section baseline
no_agent (19.232 ms) : 19038, 19427
. : milestone, 19232,
appsec (18.745 ms) : 18548, 18942
. : milestone, 18745,
code_origins (17.757 ms) : 17584, 17930
. : milestone, 17757,
iast (18.091 ms) : 17912, 18270
. : milestone, 18091,
profiling (19.43 ms) : 19236, 19624
. : milestone, 19430,
tracing (17.966 ms) : 17789, 18143
. : milestone, 17966,
section candidate
no_agent (18.023 ms) : 17841, 18204
. : milestone, 18023,
appsec (18.779 ms) : 18588, 18970
. : milestone, 18779,
code_origins (17.823 ms) : 17648, 17997
. : milestone, 17823,
iast (17.757 ms) : 17581, 17932
. : milestone, 17757,
profiling (18.423 ms) : 18237, 18608
. : milestone, 18423,
tracing (18.629 ms) : 18442, 18815
. : milestone, 18629,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section baseline
no_agent (1.258 ms) : 1247, 1270
. : milestone, 1258,
iast (3.288 ms) : 3237, 3339
. : milestone, 3288,
iast_FULL (5.758 ms) : 5700, 5816
. : milestone, 5758,
iast_GLOBAL (3.688 ms) : 3628, 3749
. : milestone, 3688,
profiling (2.272 ms) : 2249, 2296
. : milestone, 2272,
tracing (1.825 ms) : 1810, 1840
. : milestone, 1825,
section candidate
no_agent (1.24 ms) : 1227, 1252
. : milestone, 1240,
iast (3.195 ms) : 3154, 3236
. : milestone, 3195,
iast_FULL (5.91 ms) : 5850, 5970
. : milestone, 5910,
iast_GLOBAL (3.594 ms) : 3534, 3654
. : milestone, 3594,
profiling (2.331 ms) : 2311, 2352
. : milestone, 2331,
tracing (1.905 ms) : 1889, 1921
. : milestone, 1905,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section baseline
no_agent (14.942 s) : 14942000, 14942000
. : milestone, 14942000,
appsec (14.836 s) : 14836000, 14836000
. : milestone, 14836000,
iast (18.306 s) : 18306000, 18306000
. : milestone, 18306000,
iast_GLOBAL (18.189 s) : 18189000, 18189000
. : milestone, 18189000,
profiling (14.721 s) : 14721000, 14721000
. : milestone, 14721000,
tracing (14.875 s) : 14875000, 14875000
. : milestone, 14875000,
section candidate
no_agent (15.573 s) : 15573000, 15573000
. : milestone, 15573000,
appsec (14.867 s) : 14867000, 14867000
. : milestone, 14867000,
iast (18.745 s) : 18745000, 18745000
. : milestone, 18745000,
iast_GLOBAL (18.264 s) : 18264000, 18264000
. : milestone, 18264000,
profiling (14.872 s) : 14872000, 14872000
. : milestone, 14872000,
tracing (15.146 s) : 15146000, 15146000
. : milestone, 15146000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.62.0-SNAPSHOT~4fd99ee3ec, baseline=1.62.0-SNAPSHOT~693449b6f0f
dateFormat X
axisFormat %s
section baseline
no_agent (1.483 ms) : 1471, 1494
. : milestone, 1483,
appsec (3.832 ms) : 3609, 4054
. : milestone, 3832,
iast (2.271 ms) : 2201, 2341
. : milestone, 2271,
iast_GLOBAL (2.319 ms) : 2249, 2390
. : milestone, 2319,
profiling (2.106 ms) : 2051, 2162
. : milestone, 2106,
tracing (2.091 ms) : 2036, 2145
. : milestone, 2091,
section candidate
no_agent (1.488 ms) : 1476, 1500
. : milestone, 1488,
appsec (3.82 ms) : 3596, 4043
. : milestone, 3820,
iast (2.266 ms) : 2196, 2336
. : milestone, 2266,
iast_GLOBAL (2.316 ms) : 2246, 2386
. : milestone, 2316,
profiling (2.091 ms) : 2036, 2147
. : milestone, 2091,
tracing (2.078 ms) : 2023, 2132
. : milestone, 2078,
|
Test Environment - sonar-javaJob Status: success
|
| result.setStatus(ITestResult.FAILURE); | ||
| result.setThrowable( | ||
| new AssertionError( | ||
| "Datadog Early Flake Detection: test has flaky results (mixed pass/fail)")); |
There was a problem hiding this comment.
Nit: IMO explicitly mentioning "Datadog Early Flake Detection" here adds implicit coupling/breaks encapsulation. It is true that EFD is currently the only policy that can return true from shouldPropagateFailure, but this isn't enforced in any way in the code and may change in the future. I'd choose a more generic error message here
There was a problem hiding this comment.
Good point, updated it to "Datadog: propagating test failure based on aggregated execution results" to align it better with the overall functionality 👍
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in Use ⏳ Processing |
What Does This Do
passandfail, the test session was correctly failed.failandpass, the test session would incorrectly pass, while EFD should fail the session when a flaky is found.fail.Motivation
Aligns EFD behavior for TestNG with the rest of the testing frameworks.
Additional Information
Two unit tests have been introduced to verify the behavior.
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: SDTEST-3720
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.