Skip to content

Commit 24dc0e3

Browse files
Update unit tests to check the type of unique constraint / primary keys.
1 parent aec24bc commit 24dc0e3

File tree

3 files changed

+17
-8
lines changed

3 files changed

+17
-8
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
ALTER TABLE [dbo].[TableDifference]
22
ADD CONSTRAINT [PrimaryKeyDifference]
3-
PRIMARY KEY ([Type], [MaxLength])
3+
PRIMARY KEY NONCLUSTERED ([Type], [MaxLength])
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
ALTER TABLE [dbo].[TableDifference]
22
ADD CONSTRAINT [UniqueConstraintDifference]
3-
UNIQUE ([Type], [MaxLength])
3+
UNIQUE CLUSTERED ([Type], [MaxLength])

tests/UnitTests.Databases.SqlServer.Tests/SqlServerDatabaseComparerTest.cs

+15-6
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,7 @@ public void Compare()
310310
differences.Tables[0].Source.Indexes[1].IncludedColumns.Should().HaveCount(0);
311311
differences.Tables[0].Source.Indexes[1].IsUnique.Should().BeTrue();
312312
differences.Tables[0].Source.Indexes[1].Name.Should().Be("PrimaryKeyDifference");
313-
differences.Tables[0].Source.Indexes[1].Type.Should().Be("CLUSTERED");
313+
differences.Tables[0].Source.Indexes[1].Type.Should().Be("NONCLUSTERED");
314314

315315
differences.Tables[0].Target.Indexes.Should().HaveCount(2);
316316
differences.Tables[0].Target.Indexes[0].Columns.Should().HaveCount(2);
@@ -344,14 +344,17 @@ public void Compare()
344344
differences.Tables[0].Indexes[0].Target.Should().BeSameAs(differences.Tables[0].Target.Indexes[0]);
345345
differences.Tables[0].Indexes[0].Type.Should().Be(SqlObjectDifferenceType.Different);
346346

347-
differences.Tables[0].Indexes[1].Properties.Should().HaveCount(0);
347+
differences.Tables[0].Indexes[1].Properties.Should().HaveCount(1);
348+
differences.Tables[0].Indexes[1].Properties[0].Name.Should().Be("Type");
349+
differences.Tables[0].Indexes[1].Properties[0].Source.Should().Be("NONCLUSTERED");
350+
differences.Tables[0].Indexes[1].Properties[0].Target.Should().Be("CLUSTERED");
348351
differences.Tables[0].Indexes[1].Source.Should().BeSameAs(differences.Tables[0].Source.Indexes[1]);
349352
differences.Tables[0].Indexes[1].Target.Should().BeSameAs(differences.Tables[0].Target.Indexes[1]);
350353
differences.Tables[0].Indexes[1].Type.Should().Be(SqlObjectDifferenceType.Different);
351354

352355
// Tables / Primary keys
353356
differences.Tables[0].Source.PrimaryKey.Name.Should().Be("PrimaryKeyDifference");
354-
differences.Tables[0].Source.PrimaryKey.Type.Should().Be("CLUSTERED");
357+
differences.Tables[0].Source.PrimaryKey.Type.Should().Be("NONCLUSTERED");
355358
differences.Tables[0].Source.PrimaryKey.Columns.Should().HaveCount(2);
356359
differences.Tables[0].Source.PrimaryKey.Columns[0].Name.Should().Be("Type");
357360
differences.Tables[0].Source.PrimaryKey.Columns[0].Position.Should().Be(1);
@@ -374,7 +377,10 @@ public void Compare()
374377
differences.Tables[0].PrimaryKey.Columns[1].Target.Should().BeSameAs(differences.Tables[0].Target.PrimaryKey.Columns[1]);
375378
differences.Tables[0].PrimaryKey.Columns[1].Type.Should().Be(SqlObjectDifferenceType.Different);
376379

377-
differences.Tables[0].PrimaryKey.Properties.Should().BeEmpty();
380+
differences.Tables[0].PrimaryKey.Properties.Should().HaveCount(1);
381+
differences.Tables[0].PrimaryKey.Properties[0].Name.Should().Be("Type");
382+
differences.Tables[0].PrimaryKey.Properties[0].Source.Should().Be("NONCLUSTERED");
383+
differences.Tables[0].PrimaryKey.Properties[0].Target.Should().Be("CLUSTERED");
378384
differences.Tables[0].PrimaryKey.Source.Should().Be(differences.Tables[0].Source.PrimaryKey);
379385
differences.Tables[0].PrimaryKey.Target.Should().Be(differences.Tables[0].Target.PrimaryKey);
380386
differences.Tables[0].PrimaryKey.Type.Should().Be(SqlObjectDifferenceType.Different);
@@ -412,7 +418,7 @@ public void Compare()
412418
differences.Tables[0].Source.UniqueConstraints[0].Columns[1].Name.Should().Be("MaxLength");
413419
differences.Tables[0].Source.UniqueConstraints[0].Columns[1].Position.Should().Be(2);
414420
differences.Tables[0].Source.UniqueConstraints[0].Name.Should().Be("UniqueConstraintDifference");
415-
differences.Tables[0].Source.UniqueConstraints[0].Type.Should().Be("NONCLUSTERED");
421+
differences.Tables[0].Source.UniqueConstraints[0].Type.Should().Be("CLUSTERED");
416422

417423
differences.Tables[0].Target.UniqueConstraints.Should().HaveCount(1);
418424
differences.Tables[0].Target.UniqueConstraints[0].Columns.Should().HaveCount(2);
@@ -424,7 +430,10 @@ public void Compare()
424430
differences.Tables[0].Target.UniqueConstraints[0].Type.Should().Be("NONCLUSTERED");
425431

426432
differences.Tables[0].UniqueConstraints.Should().HaveCount(1);
427-
differences.Tables[0].UniqueConstraints[0].Properties.Should().HaveCount(0);
433+
differences.Tables[0].UniqueConstraints[0].Properties.Should().HaveCount(1);
434+
differences.Tables[0].UniqueConstraints[0].Properties[0].Name.Should().Be("Type");
435+
differences.Tables[0].UniqueConstraints[0].Properties[0].Source.Should().Be("CLUSTERED");
436+
differences.Tables[0].UniqueConstraints[0].Properties[0].Target.Should().Be("NONCLUSTERED");
428437
differences.Tables[0].UniqueConstraints[0].Source.Should().BeSameAs(differences.Tables[0].Source.UniqueConstraints[0]);
429438
differences.Tables[0].UniqueConstraints[0].Target.Should().BeSameAs(differences.Tables[0].Target.UniqueConstraints[0]);
430439
differences.Tables[0].UniqueConstraints[0].Type.Should().Be(SqlObjectDifferenceType.Different);

0 commit comments

Comments
 (0)