-
Notifications
You must be signed in to change notification settings - Fork 164
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
Fix issue 70: isTest looks at superclass methods #97
Fix issue 70: isTest looks at superclass methods #97
Conversation
aledsage
commented
Sep 11, 2013
- classes with superclasses with test methods now detected as tests
Fix for #70 |
Ah, I just found |
I've refactored to use |
@aledsafe let's revisit this PR, do you still have issue #70 with latest vesion of testng eclipse plugin. if so, could you update this PR so that we can have a look then |
Classes with superclasses with test methods now detected as tests
72be17d
to
6c499b8
Compare
@missedone Thanks for looking at this! I confirmed this issue #70 was still an issue on master, and have re-applied my changes again master. PR is again ready for review. |
} | ||
|
||
private static boolean doIsTest(IType iType) { | ||
ITestContent testContent = TypeParser.parseType(iType); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's have a check if iType
is abstract or not at the beginning of this method:
if (Flags.isAbstract(iType.getFlags())) {
return false;
}
Thanks @missedone - added check for |
…rclass Fix issue 70: isTest looks at superclass methods