Skip to content

Commit

Permalink
Fix test comparison types.
Browse files Browse the repository at this point in the history
Formatting fix.
  • Loading branch information
abellgithub committed May 21, 2018
1 parent d44eb0a commit 4e7fe68
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion pdal/ArtifactManager.hpp
Expand Up @@ -86,7 +86,8 @@ class ArtifactManager
std::vector<std::string> keys() const
{
std::vector<std::string> ks;
for(auto e : m_storage) ks.push_back(e.first);
for (auto e : m_storage)
ks.push_back(e.first);
return ks;
}

Expand Down
4 changes: 2 additions & 2 deletions test/unit/ArtifactTest.cpp
Expand Up @@ -130,12 +130,12 @@ TEST(ArtifactTest, key_access)
EXPECT_TRUE(t.artifactManager().keys().empty());

t.artifactManager().put("MyTest", ta);
EXPECT_EQ(t.artifactManager().keys().size(), 1);
EXPECT_EQ(t.artifactManager().keys().size(), 1U);
EXPECT_EQ(t.artifactManager().keys().at(0), "MyTest");

t.artifactManager().put("MyTest2", ta);
auto keys = t.artifactManager().keys();
EXPECT_EQ(keys.size(), 2);
EXPECT_EQ(keys.size(), 2U);
EXPECT_EQ(std::find(keys.begin(), keys.end(), "Foo"), keys.end());
EXPECT_NE(std::find(keys.begin(), keys.end(), "MyTest"), keys.end());
EXPECT_NE(std::find(keys.begin(), keys.end(), "MyTest2"), keys.end());
Expand Down

0 comments on commit 4e7fe68

Please sign in to comment.