fix(@opentelemetry/instrumentation-dataloader): fix instrumentation of ESM-imported dataloader

[TheDadi]

Which problem is this PR solving?

• The ESM imported top-level object is a Module Namespace Object per ECMA262 spec. The ".default" holds the equivalent of require('dataloader')

TypeError: Cannot read properties of undefined (reading 'load') at DataloaderInstrumentation._patchLoad

Contributes to #1942

[linux-foundation-easycla]

• ✅login: TheDadi / (ffdf6c8)

The committers listed above are authorized under a signed CLA.

[codecov]

Codecov Report

❌ Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.49%. Comparing base (a99ba69) to head (ffdf6c8).
⚠️ Report is 90 commits behind head on main.

Files with missing lines	Patch %	Lines
.../instrumentation-dataloader/src/instrumentation.ts	92.30%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2813      +/-   ##
==========================================
- Coverage   89.51%   89.49%   -0.02%     
==========================================
  Files         180      180              
  Lines        8735     8741       +6     
  Branches     1778     1780       +2     
==========================================
+ Hits         7819     7823       +4     
- Misses        916      918       +2     

Files with missing lines	Coverage Δ
.../instrumentation-dataloader/src/instrumentation.ts	98.14% <92.30%> (-0.88%)	⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[david-luna]

Hi @TheDadi

Thanks for your contribution to OpenTelemetry. I think it would be good to add a test case for the ESM usage.

#2807 fixed ESM support for pg and included such test. You can take it as a reference

[dyladan]

Related: #1942

This needs tests and fix the conflicts and we can get this merged

[david-luna]

@TheDadi

This package was relocated in #2928. Please sync with the main branch and add the missing test so we can proceed to review again and merge the changes

Thank you

[pichlermarc]

Closing due to inactivity - if anyone is interested in picking this up again, this is only missing tests for us to be able to merge this.