Skip to content

Remove Groovy v1-v2 checker, move DSL script tests into analyzer modules#13782

Merged
wu-sheng merged 1 commit intomasterfrom
remove-groovy-checker
Apr 2, 2026
Merged

Remove Groovy v1-v2 checker, move DSL script tests into analyzer modules#13782
wu-sheng merged 1 commit intomasterfrom
remove-groovy-checker

Conversation

@wu-sheng
Copy link
Copy Markdown
Member

@wu-sheng wu-sheng commented Apr 2, 2026

Remove Groovy v1-v2 checker and move DSL script tests into analyzer modules

What changed

  • Delete test/script-cases/ entirely — Groovy v1 modules and v1-v2 checker removed from build
  • Move hierarchy tests + scripts into oap-server/analyzer/hierarchy/
  • Move LAL tests + scripts into oap-server/analyzer/log-analyzer/
  • Create oap-server/analyzer/meter-analyzer-scripts-test/ for MAL tests (separate module for ProcessRegistry FQCN isolation)
  • MAL .data.yaml files reference starter configs via script: field — no duplicated production scripts
  • MalRuleLoader.loadFromDataFiles() — data-driven loader that scans .data.yaml first, resolves scripts from script: path
  • Fix checkstyle JMH exclusion patterns (jmh_generated not generated)

Test counts (matching original checker)

  • MAL: 1268 expression + 32 filter = 1300
  • LAL: 37
  • Hierarchy: 22
  • Total: 1359, all pass

@wu-sheng wu-sheng requested a review from wankai123 April 2, 2026 01:07
@wu-sheng wu-sheng added this to the 10.5.0 milestone Apr 2, 2026
@wu-sheng wu-sheng added the test Test requirements about performance, feature or before release. label Apr 2, 2026
@wu-sheng wu-sheng force-pushed the remove-groovy-checker branch from 2daa22e to 81f5143 Compare April 2, 2026 01:42
…n tests

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@wu-sheng wu-sheng force-pushed the remove-groovy-checker branch from 81f5143 to 2022017 Compare April 2, 2026 01:46
@wu-sheng wu-sheng changed the title Remove Groovy v1-v2 checker, replace with v2-only DSL script execution tests Remove Groovy v1-v2 checker, move DSL script tests into analyzer modules Apr 2, 2026
@wu-sheng wu-sheng merged commit b20e6ab into master Apr 2, 2026
411 of 419 checks passed
@wu-sheng wu-sheng deleted the remove-groovy-checker branch April 2, 2026 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Test requirements about performance, feature or before release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants