Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dependsOnMethods with alwaysRun = true and inheritance fails to find method #949

Closed
vvolkova opened this issue Jan 5, 2016 · 2 comments · Fixed by #1529
Closed

dependsOnMethods with alwaysRun = true and inheritance fails to find method #949

vvolkova opened this issue Jan 5, 2016 · 2 comments · Fixed by #1529

Comments

@vvolkova
Copy link

vvolkova commented Jan 5, 2016

We have 2 classes:

  1. Parent class ClassA has methodA and methodB, methodA depends on methodB with alwaysRun = true.
  2. Child class: ClassB extends ClassA and overrides methodB.

When we run ClassB tests they do not run with the message:

Method "ClassA.methodA()[pri:0, instance:ClassB@64cf5b3]" depends on nonexistent method "ClassA.methodB"

Note: this case was working in testng 4.6

vvolkova pushed a commit to vvolkova/testng that referenced this issue Jan 5, 2016
@juherr
Copy link
Member

juherr commented Jan 5, 2016

I'm not sure to understand: Is it working with alwaysRun = false?

@vvolkova
Copy link
Author

vvolkova commented Jan 5, 2016

With alwaysRun = false it doesn't work as well but with different exception, and didn't work in testng 6.4

vvolkova pushed a commit to vvolkova/testng that referenced this issue Jan 6, 2016
krmahadevan added a commit to krmahadevan/testng that referenced this issue Sep 6, 2017
dependsOnMethods with alwaysRun = true/false and 
inheritance fails to find method

Closes testng-team#949
@krmahadevan krmahadevan mentioned this issue Sep 6, 2017
2 tasks
krmahadevan added a commit to krmahadevan/testng that referenced this issue Sep 6, 2017
dependsOnMethods with alwaysRun = true/false and 
inheritance fails to find method

Closes testng-team#949
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants