Skip to content

Commit

Permalink
chore: Made test for sql serve schema issue sql-server specific. I su…
Browse files Browse the repository at this point in the history
…spect the failing tests indicate the bug is still present on other DBMS's, but I don't know enough to go chasing that right now.
  • Loading branch information
wokket authored and erikbra committed Dec 19, 2023
1 parent 6f0c140 commit f7da997
Showing 1 changed file with 0 additions and 28 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -95,32 +95,4 @@ public async Task Creates_a_new_version_with_status_InProgress()
version.status.Should().Be(MigrationStatus.InProgress);
}

[Test]
public async Task Bug230_Uses_Server_Casing_Rules_For_Schema()
{
//for bug #230 - when targeting an existing schema use the servers casing rules, not .Net's
var db = TestConfig.RandomDatabase();
var parent = CreateRandomTempDirectory();
var knownFolders = FoldersConfiguration.Default(null);

CreateDummySql(parent, knownFolders[Sprocs]); // make sure there's something that could be logged...

await using (var migrator = Context.GetMigrator(db, parent, knownFolders))
{
await migrator.Migrate();
Assert.True(await migrator.DbMigrator.Database.VersionTableExists()); // we migrated into the `grate` schema.
}

// Now we'll run again with the same name but different cased schema
var grateConfig = Context.GetConfiguration(db, parent, knownFolders) with
{
SchemaName = "GRATE"
};

await using (var migrator = Context.GetMigrator(grateConfig))
{
await migrator.Migrate(); // should either reuse the existing schema if a case-insensitive server, or create a new second schema for use if case-sensitive.
Assert.True(await migrator.DbMigrator.Database.VersionTableExists()); // we migrated into the `GRATE` schema, which may be the same as 'grate' depending on server settings.
}
}
}

0 comments on commit f7da997

Please sign in to comment.