Skip to content

Wire api/incubator testConvertToModel suite into check #8690

Description

@thswlsqls

Describe the bug

api/incubator/build.gradle.kts registers a testConvertToModel JvmTestSuite but never wires it into check, so the suite's 4 tests do not run in CI.

Steps to reproduce

Run ./gradlew :api:incubator:check --dry-run. The graph contains compileTestConvertToModelJava and checkstyleTestConvertToModel, but not the test task :api:incubator:testConvertToModel.

What did you expect to see?

The suite executed by check, and so by the ./gradlew build CI runs on every PR — as in sdk/trace, sdk/logs, context and seven other modules, which use tasks { check { dependsOn(testing.suites) } }.

What did you see instead?

Nothing looks broken, but the 4 tests in src/testConvertToModel/.../InstrumentationConfigUtilTest.java are silently skipped, and have been since the suite was added in #7453 (2025-07-10). Nothing in .github/ runs it separately.

What version and what artifacts are you using?

Artifacts: opentelemetry-api-incubator (build configuration only)
Version: main @ dd811c4
How did you reference these artifacts? N/A — build configuration in this repository.

Environment

Compiler: Temurin 21
OS: N/A

Additional context

docs/knowledge/testing-patterns.md states this rule, but it was added later, in #8303 (2026-06), so the module was never brought in line. Of the 14 modules registering a JvmTestSuite, 11 wire it into check.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions