Fix Alerts migration.

immense · Mar 25, 2020 · e6f32c3 · e6f32c3
1 parent a7e8673
commit e6f32c3
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 42 deletions.
diff --git a/...rations/20200325144430_Alerts.Designer.cs → ...rations/20200325145606_Alerts.Designer.cs b/...rations/20200325144430_Alerts.Designer.cs → ...rations/20200325145606_Alerts.Designer.cs
diff --git a/Server/Migrations/20200325144430_Alerts.cs → Server/Migrations/20200325145606_Alerts.cs b/Server/Migrations/20200325144430_Alerts.cs → Server/Migrations/20200325145606_Alerts.cs
@@ -15,8 +15,7 @@ protected override void Up(MigrationBuilder migrationBuilder)
                     DeviceID = table.Column<string>(nullable: true),
                     Message = table.Column<string>(nullable: true),
                     OrganizationID = table.Column<string>(nullable: true),
-                    UserId = table.Column<string>(nullable: true),
-                    UserIDId = table.Column<string>(nullable: true)
+                    UserID = table.Column<string>(nullable: true)
                 },
                 constraints: table =>
                 {
@@ -34,14 +33,8 @@ protected override void Up(MigrationBuilder migrationBuilder)
                         principalColumn: "ID",
                         onDelete: ReferentialAction.Restrict);
                     table.ForeignKey(
-                        name: "FK_Alerts_RemotelyUsers_UserIDId",
-                        column: x => x.UserIDId,
-                        principalTable: "RemotelyUsers",
-                        principalColumn: "Id",
-                        onDelete: ReferentialAction.Restrict);
-                    table.ForeignKey(
-                        name: "FK_Alerts_RemotelyUsers_UserId",
-                        column: x => x.UserId,
+                        name: "FK_Alerts_RemotelyUsers_UserID",
+                        column: x => x.UserID,
                         principalTable: "RemotelyUsers",
                         principalColumn: "Id",
                         onDelete: ReferentialAction.Restrict);
@@ -58,14 +51,9 @@ protected override void Up(MigrationBuilder migrationBuilder)
                 column: "OrganizationID");
 
             migrationBuilder.CreateIndex(
-                name: "IX_Alerts_UserIDId",
-                table: "Alerts",
-                column: "UserIDId");
-
-            migrationBuilder.CreateIndex(
-                name: "IX_Alerts_UserId",
+                name: "IX_Alerts_UserID",
                 table: "Alerts",
-                column: "UserId");
+                column: "UserID");
         }
 
         protected override void Down(MigrationBuilder migrationBuilder)

diff --git a/Server/Migrations/ApplicationDbContextModelSnapshot.cs b/Server/Migrations/ApplicationDbContextModelSnapshot.cs
@@ -236,10 +236,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                     b.Property<string>("OrganizationID")
                         .HasColumnType("TEXT");
 
-                    b.Property<string>("UserIDId")
-                        .HasColumnType("TEXT");
-
-                    b.Property<string>("UserId")
+                    b.Property<string>("UserID")
                         .HasColumnType("TEXT");
 
                     b.HasKey("ID");
@@ -248,9 +245,7 @@ protected override void BuildModel(ModelBuilder modelBuilder)
 
                     b.HasIndex("OrganizationID");
 
-                    b.HasIndex("UserIDId");
-
-                    b.HasIndex("UserId");
+                    b.HasIndex("UserID");
 
                     b.ToTable("Alerts");
                 });
@@ -631,13 +626,9 @@ protected override void BuildModel(ModelBuilder modelBuilder)
                         .WithMany("Alerts")
                         .HasForeignKey("OrganizationID");
 
-                    b.HasOne("Remotely.Shared.Models.RemotelyUser", "UserID")
-                        .WithMany()
-                        .HasForeignKey("UserIDId");
-
                     b.HasOne("Remotely.Shared.Models.RemotelyUser", "User")
                         .WithMany("Alerts")
-                        .HasForeignKey("UserId");
+                        .HasForeignKey("UserID");
                 });
 
             modelBuilder.Entity("Remotely.Shared.Models.ApiToken", b =>

diff --git a/Tests/DataServiceTests.cs b/Tests/DataServiceTests.cs
@@ -151,5 +151,19 @@ public async Task UpdateServerAdmins()
             Assert.AreEqual(1, currentAdmins.Count);
             Assert.AreEqual(TestData.Admin2.UserName, currentAdmins[0]);
         }
+
+        [TestMethod]
+        [DoNotParallelize]
+        public async Task AddAlert()
+        {
+            var alert = new Alert()
+            {
+                DeviceID = TestData.Device1.ID,
+                OrganizationID = TestData.OrganizationID,
+                Message = "Test Message",
+                UserID = TestData.Admin1.Id
+            };
+            await DataService.AddAlert(alert);
+        }
     }
 }