avoid multiple registrations of the same method

jpaniagualaconich commented 1 year ago

Fixes #657

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

codecov[bot] commented 1 year ago

Codecov Report

Merging #658 (f11ba39) into master (d6d8b63) will increase coverage by 0.05%. Report is 2 commits behind head on master. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #658      +/-   ##
==========================================
+ Coverage   95.44%   95.50%   +0.05%     
==========================================
  Files          76       77       +1     
  Lines        3293     3335      +42     
  Branches      377      379       +2     
==========================================
+ Hits         3143     3185      +42     
  Misses        119      119              
  Partials       31       31

Files Changed	Coverage Δ
tests/test_registry.py	`100.00% <100.00%> (ø)`
two_factor/plugins/registry.py	`100.00% <100.00%> (ø)`

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

claudep commented 1 year ago

Once again, it would be great to add a small regression test.