Skip to content

Commit

Permalink
Update version to 1.0.53 and fix warning in config test (microsoft#947)
Browse files Browse the repository at this point in the history
* version bump

* Fix warning with unused variable
  • Loading branch information
Mathos1432 authored Jan 23, 2025
1 parent 51ee0f0 commit cadb6af
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion Version.props
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<Project>
<!-- Versioning property for builds and packages -->
<PropertyGroup>
<VersionPrefix>1.0.52</VersionPrefix>
<VersionPrefix>1.0.53</VersionPrefix>
</PropertyGroup>
</Project>
8 changes: 4 additions & 4 deletions test/Garnet.test.cluster/ClusterConfigTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ public void ClusterConfigInitializesUnassignedWorkerTest()
public void ClusterForgetAfterNodeRestartTest()
{
int nbInstances = 4;
context.CreateInstances(4);
context.CreateInstances(nbInstances);
context.CreateConnection();
var (shards, slots) = context.clusterTestUtils.SimpleSetupCluster(logger: context.logger);

Expand All @@ -77,7 +77,7 @@ public void ClusterForgetAfterNodeRestartTest()

var firstNode = context.nodes[0];
var nodesResult = context.clusterTestUtils.ClusterNodes(0);
Assert.That(nodesResult.Nodes.Count == 4);
Assert.That(nodesResult.Nodes.Count == nbInstances);

try
{
Expand All @@ -96,13 +96,13 @@ public void ClusterForgetAfterNodeRestartTest()
}

nodesResult = context.clusterTestUtils.ClusterNodes(0);
Assert.That(nodesResult.Nodes.Count == 4, "No node should've been removed from the cluster after an invalid id was passed.");
Assert.That(nodesResult.Nodes.Count == nbInstances, "No node should've been removed from the cluster after an invalid id was passed.");
Assert.That(nodesResult.Nodes.ElementAt(0).IsMyself);
Assert.That(nodesResult.Nodes.ElementAt(0).EndPoint.ToIPEndPoint().Port == 7000, "Expected the node to be replying to be the one with port 7000.");

context.clusterTestUtils.ClusterForget(0, nodesResult.Nodes.Last().NodeId, 0);
nodesResult = context.clusterTestUtils.ClusterNodes(0);
Assert.That(nodesResult.Nodes.Count == 3, "A node should've been removed from the cluster.");
Assert.That(nodesResult.Nodes.Count == nbInstances - 1, "A node should've been removed from the cluster.");
Assert.That(nodesResult.Nodes.ElementAt(0).IsMyself);
Assert.That(nodesResult.Nodes.ElementAt(0).EndPoint.ToIPEndPoint().Port == 7000, "Expected the node to be replying to be the one with port 7000.");
}
Expand Down

0 comments on commit cadb6af

Please sign in to comment.