Remove unused assertEqualSpanInstrumentationInfo

[tammy-baylis-swi]

Description

It doesn't look like this repo uses assertEqualSpanInstrumentationInfo (grep).

All contrib repo uses were removed in open-telemetry/opentelemetry-python-contrib#3037.

This will break testing for any users that do from opentelemetry.test.test_base import TestBase and use assertEqualSpanInstrumentationInfo. However, there should have been warnings logged that lookup of instrumentation info has been deprecated since version 1.11.1 via calls to that helper instead of assertEqualSpanInstrumentationScope (see contrib issue OP)

Alternatively: We could add a logger to test_base or use print / sys.stdout to show a warning that assertEqualSpanInstrumentationInfo will be removed in a future release. Is that the approach we prefer?

Fixes #4301

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Run all unit tests.

Does This PR Require a Contrib Repo Change?

• Yes. - Link to PR: Replace assertEqualSpanInstrumentationInfo with assertEqualSpanInstrumentationScope opentelemetry-python-contrib#3037
• No.

Checklist:

• Followed the style guidelines of this project
• Changelogs have been updated
• Unit tests have been added
• Documentation has been updated

[lzchen]

I am not sure if our stability guarantees extend to testing utils so I am fine with removing it outright. I am also not against adding a warning about the removal and remove it in a subsequent release. Feel free to bring it up in the SIG so we can get a concensus.

[xrmx]

I am not sure if our stability guarantees extend to testing utils so I am fine with removing it outright. I am also not against adding a warning about the removal and remove it in a subsequent release. Feel free to bring it up in the SIG so we can get a concensus.

There were warnings at runtime already when using it because the helper was accessing a deprecated property so it shouldn't come as a surpise 😅