Convert remaining declarative config factory tests to parameterized test

[jack-berg]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.02%. Comparing base (53fc695) to head (95c98af).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #8460      +/-   ##
============================================
+ Coverage     91.00%   91.02%   +0.01%     
- Complexity     7817     7819       +2     
============================================
  Files           893      893              
  Lines         23721    23721              
  Branches       2364     2364              
============================================
+ Hits          21588    21592       +4     
+ Misses         1410     1408       -2     
+ Partials        723      721       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jkwatson]

subtract 600 lines of code and not drop the test coverage? I approve.

[psx95]

A small, optional suggestion.

[psx95]

A small improvement that could be made to the parameterized tests is the use of named arguments.

They increase readability (especially when the test arguments are similar) by associating a string with the argument payload. This would help determine the initial purpose of the different test cases (especially when the arguments are complex objects) when re-visiting the tests later to add/edit test cases.

Arguments.of(
    Named.of(
        "SpanProcessorModel with OTLP",
        new SpanProcessorModel()
                .withBatch(
                    new BatchSpanProcessorModel()
                        .withExporter(
                            new SpanExporterModel().withOtlpHttp(new OtlpHttpExporterModel()))),
            BatchSpanProcessor.builder(
                    OtlpHttpSpanExporter.builder().setComponentLoader(context).build())
                .build()),
 ...

[jack-berg]

Yeah! I should get in the habit of using named arguments more often. I always forget they're there.

[jack-berg]

Went with this new junit API, argumentSet, which doesn't require an another layer of wrapping. Will followup with guidance in https://github.com/open-telemetry/opentelemetry-java/blob/main/docs/knowledge/testing-patterns.md

Arguments.argumentSet(
            "otlp_grpc default",