improv(ioredis): require parent span

[vmarchaud]

I've also fixed a missing dep in test-utils which was making my local install fail.

[vmarchaud]

cc @naseemkullah since you wrote the plugin

[naseemkullah]

Would this prevent background queue processing tasks from emitting spans? Not that that's necessarily bad

[codecov]

Codecov Report

Merging #42 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   94.49%   94.51%   +0.01%     
==========================================
  Files          62       62              
  Lines        3509     3520      +11     
  Branches      371      372       +1     
==========================================
+ Hits         3316     3327      +11     
  Misses        193      193              

Impacted Files	Coverage Δ
.../opentelemetry-plugin-ioredis/test/ioredis.test.ts	95.30% <0.00%> (+0.14%)	⬆️
node/opentelemetry-plugin-ioredis/src/utils.ts	85.36% <0.00%> (+0.75%)	⬆️

[vmarchaud]

Would this prevent background queue processing tasks from emitting spans? Not that that's necessarily bad

Well yes but if there aren't created inside a trace it doesn't make sense to trace these i believe.
If you want to trace a background execution, you would need to create a span and then launch your task inside a withSpan.

[naseemkullah]

LGTM

[naseemkullah]

@open-telemetry/javascript-approvers bump

@vmarchaud Could you kindly resolve conflicts?

[vmarchaud]

I've rebased the PR

[naseemkullah]

@open-telemetry/javascript-maintainers would it be possible to include this one in the next cut?

[dyladan]

In the next contrib cut sure, but we're going to wait for 0.9.0 of the core first

[naseemkullah]

In the next contrib cut sure, but we're going to wait for 0.9.0 of the core first

Great, thanks!

[mayurkale22]

LGTM

[vmarchaud]

I've rebased and bumped again the types, i think its good to merge cc @dyladan @mayurkale22