test: add more test cases

[json-derulo]

No description provided.

[gund]

Hey, thanks for the PR!
If this is supposed to cover #25 then the tests should fail but why they are not failing?

[json-derulo]

Hey, thanks for the PR! If this is supposed to cover #25 then the tests should fail but why they are not failing?

Hi! The newly added tests are not supposed to cover #25. Those are just additional test scenarios which I recognized that they are not covered yet, while trying to implement a fix. The code for covering #25 would be the following:

getValidTestCase(`
  class Class1 {
    /** @deprecated */ method() {};
  }
  class Class2 extends Class1 {
    method() {};
  }
  new Class2().method();
`),

This test case fails currently. Should I add this and comment it out?

[gund]

I see, no we don't need to add broken tests =)
I would also ask you to please add to your new test cases a one line comment to each explaining the use-case like in the other cases there.

[codeclimate]

Code Climate has analyzed commit ebed6f0 and detected 0 issues on this pull request.

View more on Code Climate.

[gund]

Thanks for the fixes! Looks good!

[gund]

🎉 This PR is included in version 1.3.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀