Codecov Report

Patch and project coverage have no change.

Comparison is base (63e8543) 54.65% compared to head (c684edb) 54.65%.

❗ Current head c684edb differs from pull request most recent head b7cb793. Consider uploading reports for the commit b7cb793 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##            trunk   #11718   +/-   ##
=======================================
  Coverage   54.65%   54.65%           
=======================================
  Files          85       85           
  Lines        5643     5643           
  Branches      244      244           
=======================================
  Hits         3084     3084           
  Misses       2315     2315           
  Partials      244      244           

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

What is the advantage? Why not keep it the way it is? Folks can still extend Node, right?

@diemol for most of the scenarios, a user should be perfectly fine in just extending Node and then wrapping a LocalNode within this extended class (like what we show in the documentation as well) but I noticed a user does not get calls when the session is quit using this approach because:

1. A decorated node (A class that essentially extends Node but internally delegates calls to a LocalNode object) when it encounters a delete session request via executeWebDriverCommand will executeWebDriverCommand() on the local node object that it has (as seen here )
2. The implementation in LocalNode.executeWebDriverCommand() is such that, it checks if the command is a delete session and it internally calls LocalNode.stop(SessionId id). See here
3. Now this call from (2) never invokes the stop(SessionId id) on our Decorated node.

Owing to this, a user would never be able to eavesdrop on stop(Session) via a decorated node approach. So they will have to duplicate code from LocalNode completely to be able to tap into stop session events.

If we allowed users to extend LocalNode then they can easily just override the stop(Session) method in their child class and then include their custom logic as well.

Please let me know if that answers your question.