Codecov Report

Merging #1076 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1076      +/-   ##
==========================================
+ Coverage   97.59%   97.59%   +<.01%     
==========================================
  Files          25       25              
  Lines        2456     2457       +1     
  Branches      223      223              
==========================================
+ Hits         2397     2398       +1     
  Misses         45       45              
  Partials       14       14

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 f5b1e2e...a18e836. Read the comment docs.

Thanks! Adding a unit test to check clean() works with all enum values would be good (see other enum field tests), I can maybe take a look at that before I merge.

Maybe this is the time when we should bump most/all enum fields to eg 64 chars to avoid migrations like this.

I've added a test, which also hit some unrelated issues.

Resolves #1075