test: remove usage of inherits

[AugustinMauroy]

Description

Remove useage of utils.intherits on test. this legacy api is still tested test-util-inherits.js

[juanarbol]

Could you please add the Signed-off-by trailer as the linter wants? I do it by using the -s option to the git commit command.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.03%. Comparing base (4f44c6f) to head (dd7f554).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #63178      +/-   ##
==========================================
- Coverage   90.04%   90.03%   -0.02%     
==========================================
  Files         713      713              
  Lines      224645   224699      +54     
  Branches    42448    42480      +32     
==========================================
+ Hits       202288   202297       +9     
- Misses      14167    14183      +16     
- Partials     8190     8219      +29     

see 39 files 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.

[AugustinMauroy]

Could you please add the Signed-off-by trailer as the linter wants? I do it by using the -s option to the git commit command.

I ran git commit --amend -s and poof it's seem working. but strange thing is that gh desktop not automatically do it ...

[aduh95]

Looks like that PR removes this check, seems important

[AugustinMauroy]

wich check are you talking about ?

[aduh95]

See the comment I've highlighted: having an object that inherits from EE without a call to the EE constructor (relevant section of the spec is https://tc39.es/ecma262/#sec-runtime-semantics-classdefinitionevaluation)