Parameter alwaysRun=true for before-methods forces execution of those methods

[krmahadevan]

Closes #1622

Fixes #1622 .

Did you remember to?

• Add test case(s)
• Update CHANGES.txt

We encourage pull requests that:

• Add new features to TestNG (or)
• Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.

[juherr]

@krmahadevan Did you plan to update this PR according to #1622 (comment) ?
You can revert #683 if needed (but keep the test case)

[krmahadevan]

@juherr - I will come back to this hopefully during this weekend and see what I can do with respect to this issue.

[aTan-aka-Xellos]

@krmahadevan @juherr Hello! Any updates on this PR? Also having troubles because of #1622 issue :(

[krmahadevan]

@aTan-aka-Xellos - Thanks for reminding me on this. Let me see if I can get around to this over the weekend.

[juherr]

Closing in favor of #3453.

The underlying #1622 regression is still present on master, but this PR ties the behavior to configFailurePolicy, which does not match the documented semantics or the pre-6.9.5 behavior. Its regression test also remains green without the production change.

#3453 addresses the root cause by separating alwaysRun group semantics from configuration-failure bypassing, with red → green → red regression coverage for both SKIP and CONTINUE.