runtime consolidation: refactoring

cucumber / cucumber-js

Cucumber for JavaScript

https://cucumber.io

MIT License

5.01k stars 1.09k forks source link

runtime consolidation: refactoring #2398

Closed davidjgoss closed 2 months ago

davidjgoss commented 2 months ago

This PR makes some non-breaking refactors around the runtime area, namely:

Inclusion of a workerId with a message is now the responsibility of the runtime-agnostic test case runner, so we don't have messages being created across many parts of the code
Use of a single coordinates block rather than passing around all three arrays of paths everywhere

Small first step to #2357.

coveralls commented 2 months ago

coverage: 98.395% (-0.002%) from 98.397% when pulling e41ba5aeca6c0ffc4164fb5fe921205ac9dbeaf3 on feat/runtime-consolidation-part-1 into 2c147a8de6f497d9d5d08db0a878af177b666cbf on main.