search

migrating-ravens / RavenMigrations

A small migrations framework to help you manage your RavenDB Instance.
MIT License
53 stars 24 forks source link

AlterTests AlterCollectionMigration doesn't have MigrationAttribute which let's the RunnerTests fail #15

Closed danielmarbach closed 9 years ago

danielmarbach commented 10 years ago

Closes #13

rogersillito commented 9 years ago

I'm guessing that the intention was for AlterTests to be atomic in relation to those contained in RunnerTests? Assuming I'm right, would it be better to assign a higher number and profile to the migration attribute to fix this, so it wouldn't interfere with any tests added in future? e.g:

[Migration(1000000, "for-AlterTests-only")]

Either way your fix works for me - hopefully it will get accepted soon.

danielmarbach commented 9 years ago

I think this project is no longer maintained. You can use my fork if you want.

rogersillito commented 9 years ago

OK - thanks, I created my own fork to fix another issue before seeing your reply.

dportzline83 commented 9 years ago

This issue is fixed by 7c420c7568eaa65074a16d32cc63d974ab49a17a