Refine module auth

[iMattPro]

The test fix should eliminate the random test failings

[iMattPro]

@paul999 What are your thoughts on the proposed changes to the two migration files? In particular, what would happen if we uncommented the erroneous line in the first migration file?

[paul999]

I would say if you umcomment, the other migration is technically direclty effectivly installed for new installs.
However, to keep in line with not changing migrations, I would say leave it out, as you already need to fix old installs. And for new installs it doesn't matter the if it is directly set correctly, or via the second migration.

[iMattPro]

I would say if you umcomment, the other migration is technically direclty effectivly installed for new installs. However, to keep in line with not changing migrations, I would say leave it out, as you already need to fix old installs. And for new installs it doesn't matter the if it is directly set correctly, or via the second migration.

Okay, I’ll leave it in but commented out. I’m doing this to prevent anyone who might copy this code for themselves from making the same mistake we did with adding modules. We forgot that line of code.

[paul999]

Yep, sounds good. Please mention it when submitting, so we don't forget we discussed it already.