Skip to content

Commit

Permalink
Fix build under Linux.
Browse files Browse the repository at this point in the history
  • Loading branch information
lanierd-adsk committed Nov 13, 2023
1 parent b2d5779 commit 2748c35
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ TEST(FlowViewportAPI, viewportInformationWithHydra)
//Get all Hydra viewports information
Fvp::ViewportInformationSet viewportInformationSet;
informationInterface.GetViewportsInformation(viewportInformationSet);
ASSERT_EQ(viewportInformationSet.size(), 1);//We should have 1 hydra viewport
ASSERT_EQ(viewportInformationSet.size(), (size_t)1);//We should have 1 hydra viewport

//Check renderer name
auto it = viewportInformationSet.cbegin();
Expand All @@ -105,7 +105,7 @@ TEST(FlowViewportAPI, viewportInformationWithoutHydra)
//Get all Hydra viewports information
Fvp::ViewportInformationSet viewportInformationSet;
informationInterface.GetViewportsInformation(viewportInformationSet);
ASSERT_EQ(viewportInformationSet.size(), 0); //we should have no Hydra viewports
ASSERT_EQ(viewportInformationSet.size(), (size_t)0); //we should have no Hydra viewports

//Only SceneIndexRemoved should have been called once
ASSERT_EQ(_infoClientTest.GetSceneIndexAdded(), 0);
Expand All @@ -123,7 +123,7 @@ TEST(FlowViewportAPI, viewportInformationWithHydraAgain)
//Get all Hydra viewports information
Fvp::ViewportInformationSet viewportInformationSet;
informationInterface.GetViewportsInformation(viewportInformationSet);
ASSERT_EQ(viewportInformationSet.size(), 1);//We should have 1 hydra viewport
ASSERT_EQ(viewportInformationSet.size(), (size_t)1);//We should have 1 hydra viewport

//Check renderer name
auto it = viewportInformationSet.cbegin();
Expand Down

0 comments on commit 2748c35

Please sign in to comment.