fix(prefer-checked): don't auto-fix when 2nd argument is a non literal.

[benmonro]

What: fixes banned attribute checks with 2nd arg passed as non literal value.

Why: fixes #58

How: add check for non literal 2nd arg, return null from fixer if present.

Checklist:

• Documentation
• Tests
• Ready to be merged
  

[codecov]

Codecov Report

Merging #62 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #62   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines          131       134    +3     
  Branches        18        21    +3     
=========================================
+ Hits           131       134    +3     

Impacted Files	Coverage Δ
src/createBannedAttributeRule.js	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8497c48...cc1f0a1. Read the comment docs.

[benmonro]

@RIP21 This PR will fix the auto-fix problem you found, however the rule will still report an error. In the example you gave you can use an if(checked) or perhaps open a PR on jest-dom to allow a variable to be passed in to toBeChecked() but imo this rule should report an error in that case. If you have no objections I will merge.

[RIP21]

All good as long as this fix is no longer breaks test :)

[benmonro]

🎉 This PR is included in version 3.1.2 🎉

The release is available on:

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

Your semantic-release bot 📦🚀