fix: Add support for setConfig in ioredis instrumentation

[mustafain117]

Which problem is this PR solving?

• resolves ioredis: SetConfig not supported #624

Short description of the changes

• Override base setConfig function in IORedisInstrumentation
• Unit test added to verify setConfig function works as expected, asserts instrumentation uses the new config on future operations

[Flarna]

The fix doesn't match to the description in the issue which tells that the problem is that _config is only read during patching.

It seem to work in your test but this is just because it does an additional enable/disable.

There is no need to override setConfig as IORedisInstrumentationConfig extends InstrumentationConfig therefore the API in base class is fine.

[mustafain117]

@Flarna Thank you for the feedback, I have updated the instrumentation to use the latest config from this instead and removed the override on setConfig.

[blumamir]

Thanks for contributing and fixing this issue.
Wrote few minor optional comments. Feel free to address them or resolve if you do not want to fix them as part of this PR.

[codecov]

Codecov Report

Merging #689 (0ff652d) into main (fe85fca) will increase coverage by 0.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #689      +/-   ##
==========================================
+ Coverage   96.31%   96.86%   +0.55%     
==========================================
  Files           5       11       +6     
  Lines         515      701     +186     
  Branches       97      130      +33     
==========================================
+ Hits          496      679     +183     
- Misses         19       22       +3     

Impacted Files	Coverage Δ
...trumentation-document-load/src/enums/EventNames.ts	100.00% <0.00%> (ø)
...ages/auto-instrumentations-node/test/utils.test.ts	96.87% <0.00%> (ø)
...tapackages/auto-instrumentations-node/src/utils.ts	96.77% <0.00%> (ø)
...strumentation-document-load/src/instrumentation.ts	98.79% <0.00%> (ø)
...entation-document-load/src/enums/AttributeNames.ts	100.00% <0.00%> (ø)
...lemetry-instrumentation-document-load/src/utils.ts	100.00% <0.00%> (ø)

[blumamir]

LGTM

[vmarchaud]

@mustafain117 You'll need to rebase the PR

[mustafain117]

@vmarchaud Done

[obecny]

we can't merge this as it is outdated, please iether update to latest or allow to edit

[mustafain117]

@obecny Updated it