Yes, the return values seems correct for your test cases. But we don't have a test case for running a migration against a database which already has migrations run against (we only run an "empty" migration against it), nor do we test any of the exceptions.