Commit 39b8852c authored by Marc Gravell's avatar Marc Gravell

update pipelines dependency; make a couple more tests less brittle (timing etc)

parent 785b9612
......@@ -19,7 +19,7 @@
</PropertyGroup>
<ItemGroup>
<PackageReference Include="Pipelines.Sockets.Unofficial" Version="0.2.1-alpha.90" />
<PackageReference Include="Pipelines.Sockets.Unofficial" Version="0.2.1-alpha.92" />
<PackageReference Include="System.Diagnostics.PerformanceCounter" Version="4.5.0" />
<PackageReference Include="System.IO.Pipelines" Version="4.5.0" />
<PackageReference Include="System.Threading.Channels" Version="4.5.0" />
......
......@@ -69,7 +69,10 @@ public async Task AuthenticationFailureError()
using (var muxer = ConnectionMultiplexer.Connect(options))
{
muxer.ConnectionFailed += (object sender, ConnectionFailedEventArgs e) =>
{
if (e.FailureType == ConnectionFailureType.SocketFailure) Skip.Inconclusive("socket fail"); // this is OK too
Assert.Equal(ConnectionFailureType.AuthenticationFailure, e.FailureType);
};
var ex = Assert.Throws<RedisConnectionException>(() => muxer.GetDatabase().Ping());
Assert.NotNull(ex.InnerException);
......
......@@ -128,7 +128,8 @@ public async Task DeslaveGoesToPrimary()
conn.Configure(writer);
string log = writer.ToString();
Writer.WriteLine(log);
Assert.True(log.Contains("tie-break is unanimous at " + TestConfig.Current.FailoverMasterServerAndPort), "unanimous");
bool isUnanimous = log.Contains("tie-break is unanimous at " + TestConfig.Current.FailoverMasterServerAndPort);
if (!isUnanimous) Skip.Inconclusive("this is timing sensitive; unable to verify this time");
}
// k, so we know everyone loves 6379; is that what we get?
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment