Merge branch 'unit-test/rest' of https://github.com/RevealBi/Reveal.S…

…dk.Dom into unit-test/rest
RevealBi · Dec 9, 2024 · f4e8ca4 · f4e8ca4
2 parents d6d4a58 + be195b7
commit f4e8ca4
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/Reveal.Sdk.Dom.Tests/Data/DataSources/MicrosoftSqlServerDataSourceFixture.cs b/src/Reveal.Sdk.Dom.Tests/Data/DataSources/MicrosoftSqlServerDataSourceFixture.cs
@@ -67,9 +67,9 @@ public void Host_SetsAndGetsValue_WithDifferentInputs(string host)
         }
 
         [Theory]
-        [InlineData("1234")]
+        [InlineData(3306)]
         [InlineData(null)]
-        public void Port_SetsAndGetsValue_WithDifferentInputs(string port)
+        public void Port_SetsAndGetsValue_WithDifferentInputs(int port)
         {
             // Arrange
             var dataSource = new MicrosoftSqlServerDataSource();

diff --git a/src/Reveal.Sdk.Dom.Tests/Data/HostDataSourceFixture.cs b/src/Reveal.Sdk.Dom.Tests/Data/HostDataSourceFixture.cs
@@ -27,14 +27,14 @@ public void GetPort_ReturnSameValue_WithSetPort()
         {
             // Arrange
             var hostDataSource = new HostDataSource();
-            var expectedPort = "8080";
+            var expectedPort = 8080;
 
             // Act
             hostDataSource.Port = expectedPort;
 
             // Assert
             Assert.Equal(expectedPort, hostDataSource.Port);
-            Assert.Equal(expectedPort, hostDataSource.Properties.GetValue<string>("Port"));
+            Assert.Equal(expectedPort, hostDataSource.Properties.GetValue<int>("Port"));
         }
     }
 }
diff --git a/src/Reveal.Sdk.Dom/Data/HostDataSource.cs b/src/Reveal.Sdk.Dom/Data/HostDataSource.cs
@@ -13,9 +13,9 @@ public string Host
         }
 
         [JsonIgnore]
-        public string Port
+        public int Port
         {
-            get => Properties.GetValue<string>("Port");
+            get => Properties.GetValue<int>("Port");
             set => Properties.SetItem("Port", value);
         }
     }