Add Align lawful tests for Option instance

[gagandeepkalra]

while back-porting Align to Scala 2.11, noticed we are missing tests for Option instance.

[travisbrown]

Can you rebase or merge master and add the import? Thanks for catching this!

[travisbrown]

I think it still needs the cats.Align import (we recently changed the tests to use imports instead of package nesting in order to match usage more directly).

[gagandeepkalra]

Sorry, I did that in a hurry.

[travisbrown]

Thanks, @gagandeepkalra. The failure in CI was unrelated, so I restarted the build and opened #3353 to track it. +1 on green.

[codecov-io]

Codecov Report

Merging #3350 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3350      +/-   ##
==========================================
+ Coverage   93.32%   93.34%   +0.01%     
==========================================
  Files         378      378              
  Lines        7689     7692       +3     
  Branches      206      203       -3     
==========================================
+ Hits         7176     7180       +4     
+ Misses        513      512       -1

Flag	Coverage Δ
#scala_version_212	93.4% <ø> (+0.04%)	⬆️
#scala_version_213	93.12% <ø> (+0.01%)	⬆️

Impacted Files	Coverage Δ
core/src/main/scala/cats/instances/either.scala	100% <0%> (ø)	⬆️
core/src/main/scala/cats/syntax/either.scala	83.33% <0%> (+0.24%)	⬆️
core/src/main/scala/cats/instances/option.scala	100% <0%> (+1.69%)	⬆️

---

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 fc23079...f95677e. Read the comment docs.

[gagandeepkalra]

Thank you, Sir 🙂