cucumber / cucumber-js

Cucumber for JavaScript
https://cucumber.io
MIT License
5.04k stars 1.09k forks source link

Address deprecation warning in Unit Tests #2267

Closed michael-lloyd-morris closed 1 year ago

michael-lloyd-morris commented 1 year ago

šŸ¤” What's changed?

Fixing a deprecation warning in the unit tests.

> mocha 'src/**/*_spec.ts'

  ........(node:38244) [DEP0147] DeprecationWarning: In future versions of Node.js, fs.rmdir(path, { recursive: true }) will be removed. Use fs.rm(path, { recursive: true }) instead
(Use `node --trace-deprecation ...` to show where the warning was created)
...........................................................................................................
  ...................................................................................................................
  ................................................................................................

  326 passing (4s)

āš”ļø What's your motivation?

Bugfix - well, a potential bug when the Node team finally follows up on the deprecation.

šŸ·ļø What kind of change is this?

šŸ“‹ Checklist:

coveralls commented 1 year ago

Coverage Status

Coverage: 98.561% (+0.0006%) from 98.561% when pulling 3b94338f4e3fb5a22c1867e027204f2a4fea1524 on michael-lloyd-morris:fixDeprecation into 627030d4d657b7efc19b1343d3d77d77c774139e on cucumber:main.