tacitknowledge/autopatch — comment on commit 22bed10a2904a743258d54d80169a8b522de353d
File:  line 0
By MexicanHacker:

I think you are right in the sense that it's not the responsibility of this method to create the table, but it is his responsibility to handle edge cases (Defensive Programming) like this. Without the unit tests for that how do we ensure we throw the right errors upon the right conditions?

For example, the contract for shouldMigrationsRun specifies that we throw a MigrationException, what if instead we are swallowing the exception? or throwing an NPE?