Merge pull request #23 from hughesjs/upgrade-deps

Upgrade deps
hughesjs · Jun 4, 2023 · 55b6bcb · 55b6bcb
2 parents 6e7fcb7 + b3f48c2
commit 55b6bcb
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 13 deletions.
diff --git a/...endencyInjection.Tests/CelesTrakSdk.Microsoft.Extensions.DependencyInjection.Tests.csproj b/...endencyInjection.Tests/CelesTrakSdk.Microsoft.Extensions.DependencyInjection.Tests.csproj
@@ -9,22 +9,22 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="GitHubActionsTestLogger" Version="2.0.1">
+        <PackageReference Include="GitHubActionsTestLogger" Version="2.3.2">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
         <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="7.0.0" />
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.5.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.6.1" />
         <PackageReference Include="xunit" Version="2.4.2" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>
-        <PackageReference Include="coverlet.collector" Version="3.2.0">
+        <PackageReference Include="coverlet.collector" Version="6.0.0">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>
-        <PackageReference Include="Shouldly" Version="4.1.0" />
+        <PackageReference Include="Shouldly" Version="4.2.1" />
     </ItemGroup>
 
     <ItemGroup>

diff --git a/src/CelesTrakSdk.Tests/CelesTrakSdk.Tests.csproj b/src/CelesTrakSdk.Tests/CelesTrakSdk.Tests.csproj
@@ -11,18 +11,18 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="GitHubActionsTestLogger" Version="2.0.1">
+        <PackageReference Include="GitHubActionsTestLogger" Version="2.3.2">
           <PrivateAssets>all</PrivateAssets>
           <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
         </PackageReference>
-        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.5.0" />
-        <PackageReference Include="Shouldly" Version="4.1.0" />
+        <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.6.1" />
+        <PackageReference Include="Shouldly" Version="4.2.1" />
         <PackageReference Include="xunit" Version="2.4.2" />
         <PackageReference Include="xunit.runner.visualstudio" Version="2.4.5">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>
-        <PackageReference Include="coverlet.collector" Version="3.2.0">
+        <PackageReference Include="coverlet.collector" Version="6.0.0">
             <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
             <PrivateAssets>all</PrivateAssets>
         </PackageReference>

diff --git a/src/CelesTrakSdk.Tests/Clients/CelesTrakClientTests.cs b/src/CelesTrakSdk.Tests/Clients/CelesTrakClientTests.cs
@@ -42,18 +42,38 @@ public async Task CanGetSatCatRecord()
 										 OrbitCentre    = OrbitCentre.Earth,
 										 OrbitType      = OrbitType.Orbit
 									 };
-		
+
 		SatCatRecord? zarya = await _client.GetRecord(SatCatRecordQueryType.Name, "ZARYA");
 
 		zarya.ShouldNotBeNull();
-		zarya.ShouldBe(zaryaExpected);
+
+		// Testing individual properties because orbit properties change over time
+		zarya.ObjectName.ShouldBe(zaryaExpected.ObjectName);
+		zarya.ObjectId.ShouldBe(zaryaExpected.ObjectId);
+		zarya.NoradCatId.ShouldBe(zaryaExpected.NoradCatId);
+		zarya.ObjectType.ShouldBe(zaryaExpected.ObjectType);
+		zarya.OpsStatusCode.ShouldBe(zaryaExpected.OpsStatusCode);
+		zarya.Owner.ShouldBe(zaryaExpected.Owner);
+		zarya.LaunchDate.ShouldBe(zaryaExpected.LaunchDate);
+		zarya.LaunchSite.ShouldBe(zaryaExpected.LaunchSite);
+		zarya.DecayDate.ShouldBe(zaryaExpected.DecayDate);
+		zarya.DataStatusCode.ShouldBe(zaryaExpected.DataStatusCode);
+		zarya.OrbitCentre.ShouldBe(zaryaExpected.OrbitCentre);
+		zarya.OrbitType.ShouldBe(zaryaExpected.OrbitType);
+
+		// These should do until the ISS is deorbited
+		(zarya.Period      - zaryaExpected.Period)?.ShouldBeLessThan(5.0);
+		(zarya.Inclination - zaryaExpected.Inclination)?.ShouldBeLessThan(5.0);
+		(zarya.Apogee      - zaryaExpected.Apogee)?.ShouldBeLessThan(5);
+		(zarya.Perigee     - zaryaExpected.Perigee)?.ShouldBeLessThan(5);
+		(zarya.Rcs         - zaryaExpected.Rcs)?.ShouldBeLessThan(5.0);
 	}
 
 	[Fact]
 	public async Task CanGetSatCatGroup()
 	{
 		List<SatCatRecord> stations = await _client.GetRecords(SatCatRecordQueryType.Group, "STATIONS");
-		
+
 		stations.SingleOrDefault(s => s.ObjectName == "ISS (ZARYA)").ShouldNotBeNull();
 		stations.SingleOrDefault(s => s.ObjectName == "CSS (MENGTIAN)").ShouldNotBeNull();
 	}
@@ -62,9 +82,9 @@ public async Task CanGetSatCatGroup()
 	public async Task CanGetAllSatCatRecords()
 	{
 		List<SatCatRecord> allSatCats = await _client.GetAllRecords();
-		
+
 		allSatCats.Find(s => s.ObjectId == "1997-067A").ShouldNotBeNull();
 		allSatCats.Count.ShouldBeGreaterThan(7500);
 	}
-	
+
 }