Hooks are executed twice if Driver class is inherited

[gucu112]

Reqnroll Version

2.2.1

Which test runner are you using?

NUnit

Test Runner Version Number

3.14.0

.NET Implementation

.NET 8.0

Test Execution Method

Visual Studio Test Explorer

Content of reqnroll.json configuration file

No response

Issue Description

I have found weird behavior when inheriting from Driver class marked with [Binding] attribute and contains [BeforeScenario] and [AfterScenario] functions. If this situation occurs then Hooks are executed twice, which in my opinion is confusing and shouldn't happen.

I have prepared simple example on my github to demonstrate:
https://github.com/gucu112/ReqnrollBits/blob/master/BindingAttributeInheritance/StepDefinitions/CalculatorStepDefinitions.cs

When I ran the test I get this output:

My first impression is that this behavior exists due to inheritance flag on [Bindings] attribute set to true (by default). However, I did not look through the code, so I am not really sure why it works like this.

Steps to Reproduce

Run the test in the project

Link to Repro Project

https://github.com/gucu112/ReqnrollBits

[clrudolphi]

Interesting. I can repro this.
The vast majority of Reqnroll users do not place binding classes into inheritance hierarchies like this and so they do not experience this.
Even so, this is not intuitive behavior.
The code in the TestExecutionEngine that fires hook bindings does attempt to filter out duplicate methods; but in this situation is not considering them as identical. We'll investigate why not and determine whether we can improve that.

[clrudolphi]

I would like to make another attempt at solving this. The approach suggested in #678 (additional run-time filtering) was agreed as NOT the correct approach.
The preferred approach would be one that attempts to identify these situations when we build the Binding Registry and error out those situations before test execution begins. If possible during compilation and/or during the editing experience via the VisualStudio Extension.

See #924 for progress and proposals and questions on how to proceed.
TIA