From 7bbf780875ea3e16d0bdff2ec4489a8659aea6da Mon Sep 17 00:00:00 2001 From: microzchang <110015819+microzchang@users.noreply.github.com> Date: Wed, 19 Jul 2023 10:13:04 +0800 Subject: [PATCH] fix live test failure (#4800) --- .../test/ut/share_client_test.cpp | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/sdk/storage/azure-storage-files-shares/test/ut/share_client_test.cpp b/sdk/storage/azure-storage-files-shares/test/ut/share_client_test.cpp index 664c11339..bc9172d76 100644 --- a/sdk/storage/azure-storage-files-shares/test/ut/share_client_test.cpp +++ b/sdk/storage/azure-storage-files-shares/test/ut/share_client_test.cpp @@ -462,7 +462,10 @@ namespace Azure { namespace Storage { namespace Test { EXPECT_NO_THROW(properties = shareClient.GetProperties().Value); EXPECT_EQ(Files::Shares::Models::AccessTier::Hot, properties.AccessTier.Value()); EXPECT_FALSE(properties.AccessTierTransitionState.HasValue()); - EXPECT_EQ(properties.LastModified, properties.AccessTierChangedOn.Value()); + auto timeBefore = properties.LastModified - std::chrono::seconds(1); + auto timeAfter = properties.LastModified + std::chrono::seconds(1); + auto accessTierChangedOn = properties.AccessTierChangedOn.Value(); + EXPECT_TRUE(timeBefore <= accessTierChangedOn && accessTierChangedOn <= timeAfter); shareClients.emplace(std::move(shareName), std::move(shareClient)); } { @@ -474,7 +477,10 @@ namespace Azure { namespace Storage { namespace Test { EXPECT_NO_THROW(properties = shareClient.GetProperties().Value); EXPECT_EQ(Files::Shares::Models::AccessTier::Cool, properties.AccessTier.Value()); EXPECT_FALSE(properties.AccessTierTransitionState.HasValue()); - EXPECT_EQ(properties.LastModified, properties.AccessTierChangedOn.Value()); + auto timeBefore = properties.LastModified - std::chrono::seconds(1); + auto timeAfter = properties.LastModified + std::chrono::seconds(1); + auto accessTierChangedOn = properties.AccessTierChangedOn.Value(); + EXPECT_TRUE(timeBefore <= accessTierChangedOn && accessTierChangedOn <= timeAfter); shareClients.emplace(std::move(shareName), std::move(shareClient)); } @@ -500,7 +506,10 @@ namespace Azure { namespace Storage { namespace Test { { EXPECT_EQ(Files::Shares::Models::AccessTier::Hot, properties.AccessTier.Value()); } - EXPECT_EQ(properties.LastModified, properties.AccessTierChangedOn.Value()); + auto timeBefore = properties.LastModified - std::chrono::seconds(1); + auto timeAfter = properties.LastModified + std::chrono::seconds(1); + auto accessTierChangedOn = properties.AccessTierChangedOn.Value(); + EXPECT_TRUE(timeBefore <= accessTierChangedOn && accessTierChangedOn <= timeAfter); } // List shares works.