chore(instr-koa): update instrumentation-koa tests

[david-luna]

Which problem is this PR solving?

Fixes: #2564

Short description of the changes

• skip @koa/router tests if version is not compatible with nodejs version

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.75%. Comparing base (97ebd4f) to head (8706abf).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2566   +/-   ##
=======================================
  Coverage   90.75%   90.75%           
=======================================
  Files         169      169           
  Lines        8018     8018           
  Branches     1632     1632           
=======================================
  Hits         7277     7277           
  Misses        741      741           

[trentm]

I'm curious: does this.skip() not work in async test functions?

[pichlermarc]

this.skip() does unfortunately not work when using a lambda function. I've started using

it('does thing', function() {
...
});

in all new tests for that reason. Unfortunately we lambdas it across all tests in both repos, so it'd be very difficult to change it everywhere.

[github-actions]

This package does not have an assigned component owner and is considered unmaintained. As such this package is in feature-freeze and this PR will be closed with 14 days unless a new owner or a sponsor (a member of @open-telemetry/javascript-approvers) for the feature is found. It is the responsibility of the author to find a sponsor for this feature.
Are you familiar with this package? Consider becoming a component owner.