Browse Source

Bump test dependencies (#1693)

* Bump actions/checkout from 4 to 5
* Bump actions/upload-pages-artifact from 3 to 4
* Bump actions/download-artifact from 4 to 5
* Bump the dependencies group with 5 updates
* fix
* fix locally failing portforwarding tests

port already in use

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Rob Hague <rob.hague00@gmail.com>
dependabot[bot] 1 month ago
parent
commit
2f0ae3190c

+ 4 - 4
.github/workflows/build.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-24.04
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
       with:
         fetch-depth: 0 # needed for Nerdbank.GitVersioning
 
@@ -57,7 +57,7 @@ jobs:
     runs-on: windows-2025
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
       with:
         fetch-depth: 0 # needed for Nerdbank.GitVersioning
 
@@ -108,7 +108,7 @@ jobs:
     runs-on: windows-2025
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
       with:
         fetch-depth: 0 # needed for Nerdbank.GitVersioning
 
@@ -155,7 +155,7 @@ jobs:
       - Linux
     steps:
       - name: Download NuGet Package
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v5
         with:
           name: NuGet Package
 

+ 2 - 2
.github/workflows/docs.yml

@@ -22,7 +22,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Setup Pages
         uses: actions/configure-pages@v5
@@ -37,7 +37,7 @@ jobs:
         run: docfx ./docfx/docfx.json
 
       - name: Upload documentation
-        uses: actions/upload-pages-artifact@v3
+        uses: actions/upload-pages-artifact@v4
         with:
           path: './docfx/_site'
 

+ 6 - 6
Directory.Packages.props

@@ -4,16 +4,16 @@
     <CentralPackageVersionOverrideEnabled>false</CentralPackageVersionOverrideEnabled>
   </PropertyGroup>
   <ItemGroup>
-    <PackageVersion Include="BenchmarkDotNet" Version="0.15.2" />
+    <PackageVersion Include="BenchmarkDotNet" Version="0.15.3" />
     <PackageVersion Include="BouncyCastle.Cryptography" Version="2.6.2" />
     <PackageVersion Include="coverlet.collector" Version="6.0.4" />
     <PackageVersion Include="coverlet.msbuild" Version="6.0.4" />
     <PackageVersion Include="GitHubActionsTestLogger" Version="2.4.1" />
-    <PackageVersion Include="Meziantou.Analyzer" Version="2.0.210" />
+    <PackageVersion Include="Meziantou.Analyzer" Version="2.0.220" />
     <!-- Should stay on LTS .NET releases. -->
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
-    <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="9.0.7" />
-    <PackageVersion Include="MSTest" Version="3.10.0" />
+    <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="9.0.9" />
+    <PackageVersion Include="MSTest" Version="3.9.3" />
     <PackageVersion Include="Moq" Version="4.20.72" />
     <PackageVersion Include="Nerdbank.GitVersioning" Version="3.7.115" />
     <PackageVersion Include="PolySharp" Version="1.15.0" />
@@ -21,6 +21,6 @@
     <PackageVersion Include="StyleCop.Analyzers" Version="1.2.0-beta.556" />
     <!-- Should stay on LTS .NET releases. -->
     <PackageVersion Include="System.Formats.Asn1" Version="8.0.2" />
-    <PackageVersion Include="Testcontainers" Version="4.6.0" />
+    <PackageVersion Include="Testcontainers" Version="4.7.0" />
   </ItemGroup>
-</Project>
+</Project>

+ 6 - 10
test/Renci.SshNet.IntegrationTests/SshTests.cs

@@ -527,12 +527,9 @@ namespace Renci.SshNet.IntegrationTests
                 {
                     client.Connect();
 
-                    var localEndPoint = new IPEndPoint(IPAddress.Loopback, 1225);
-
                     for (var i = 0; i < (connectionInfo.MaxSessions + 1); i++)
                     {
-                        var forwardedPort = new ForwardedPortLocal(localEndPoint.Address.ToString(),
-                                                                   (uint)localEndPoint.Port,
+                        var forwardedPort = new ForwardedPortLocal(IPAddress.Loopback.ToString(),
                                                                    hostNameAlias,
                                                                    80);
                         client.AddForwardedPort(forwardedPort);
@@ -547,7 +544,8 @@ namespace Renci.SshNet.IntegrationTests
 
                             using HttpClient httpClient = new(handler);
 
-                            using HttpResponseMessage httpResponse = httpClient.GetAsync("http://" + localEndPoint).Result;
+                            using HttpResponseMessage httpResponse = httpClient.GetAsync(
+                                $"http://{forwardedPort.BoundHost}:{forwardedPort.BoundPort}").Result;
 
                             Assert.AreEqual(HttpStatusCode.MovedPermanently, httpResponse.StatusCode);
                         }
@@ -583,10 +581,7 @@ namespace Renci.SshNet.IntegrationTests
                 {
                     client.Connect();
 
-                    var localEndPoint = new IPEndPoint(IPAddress.Loopback, 1225);
-
-                    var forwardedPort = new ForwardedPortLocal(localEndPoint.Address.ToString(),
-                                                               (uint)localEndPoint.Port,
+                    var forwardedPort = new ForwardedPortLocal(IPAddress.Loopback.ToString(),
                                                                hostNameAlias,
                                                                80);
                     forwardedPort.Exception +=
@@ -603,7 +598,8 @@ namespace Renci.SshNet.IntegrationTests
 
                         using HttpClient httpClient = new(handler);
 
-                        using HttpResponseMessage httpResponse = httpClient.GetAsync("http://" + localEndPoint).Result;
+                        using HttpResponseMessage httpResponse = httpClient.GetAsync(
+                            $"http://{forwardedPort.BoundHost}:{forwardedPort.BoundPort}").Result;
 
                         Assert.AreEqual(HttpStatusCode.MovedPermanently, httpResponse.StatusCode);
                     }