GROOVY-12117: adding tests for some edge cases#2646
Conversation
There was a problem hiding this comment.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.
| Benchmark suite | Current: f27a508 | Previous: 93f3d16 | Ratio |
|---|---|---|---|
org.apache.groovy.perf.grails.DynamicDispatchBench.methodMissingSavePattern |
38.83550699247537 ms/op |
23.4392629811486 ms/op |
1.66 |
This comment was automatically generated by workflow using github-action-benchmark.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2646 +/- ##
==================================================
+ Coverage 68.5485% 68.5524% +0.0039%
- Complexity 33744 33746 +2
==================================================
Files 1523 1523
Lines 127940 127940
Branches 23254 23254
==================================================
+ Hits 87701 87706 +5
+ Misses 32459 32455 -4
+ Partials 7780 7779 -1 🚀 New features to boost your workflow:
|
|
JMH summary — classic (commit
|
| Group | Speedup | n |
|---|---|---|
| bench | 1.007 × | 26 |
| core | 1.063 × | 77 |
| grails | 0.977 × | 80 |
Baseline: dev/bench/jmh/<part>/classic/data.js on gh-pages, trailing 90 days. Daily dashboard · Per-suite raw data
JMH summary — indy (commit
|
| Group | Speedup | n |
|---|---|---|
| bench | 0.967 × | 26 |
| core | 3.568 × | 77 |
| grails | 2.271 × | 80 |
Baseline: dev/bench/jmh/<part>/indy/data.js on gh-pages, trailing 90 days. Daily dashboard · Per-suite raw data
✅ All tests passed ✅🏷️ Commit: f27a508 Learn more about TestLens at testlens.app. |
|
@blackdrag I tried to merge these into the existing test hierarchy. Let me know if you spot any problems. |



This test is an attempt to clarify some semantic gaps in our tests for traits