edg-l/ddnet
1
0
Fork 0
mirror of https://github.com/ddnet/ddnet.git synced 2024-11-10 01:58:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use EXPECT_* rather than ASSERT_* in tests

Browse source 
`EXPECT_*` allows the test to continue even in the case of failure. Use
this where it makes sense to continue.
This commit is contained in:
heinrich5991 2019-03-12 02:06:54 +01:00
parent 5c1dfe6b65
commit 8b3f0246a9
2 changed files with 14 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/test/git_revision.cpp
View file

@ -7,6 +7,6 @@ TEST(GitRevision, ExistsOrNull)
{
if(GIT_SHORTREV_HASH)
{
ASSERT_STRNE(GIT_SHORTREV_HASH, "");
EXPECT_STRNE(GIT_SHORTREV_HASH, "");
}
}

26
src/test/name_ban.cpp
View file

@ -4,26 +4,26 @@
TEST(NameBan, Empty)
{
ASSERT_FALSE(IsNameBanned("", 0, 0));
ASSERT_FALSE(IsNameBanned("abc", 0, 0));
EXPECT_FALSE(IsNameBanned("", 0, 0));
EXPECT_FALSE(IsNameBanned("abc", 0, 0));
}
TEST(NameBan, Equality)
{
CNameBan Abc0("abc", 0, 0);
ASSERT_TRUE(IsNameBanned("abc", &Abc0, 1));
ASSERT_TRUE(IsNameBanned(" abc", &Abc0, 1));
ASSERT_TRUE(IsNameBanned("abc ", &Abc0, 1));
ASSERT_TRUE(IsNameBanned("abc foo", &Abc0, 1)); // Maximum name length.
ASSERT_TRUE(IsNameBanned("äbc", &Abc0, 1)); // Confusables
ASSERT_FALSE(IsNameBanned("def", &Abc0, 1));
ASSERT_FALSE(IsNameBanned("abcdef", &Abc0, 1));
EXPECT_TRUE(IsNameBanned("abc", &Abc0, 1));
EXPECT_TRUE(IsNameBanned(" abc", &Abc0, 1));
EXPECT_TRUE(IsNameBanned("abc ", &Abc0, 1));
EXPECT_TRUE(IsNameBanned("abc foo", &Abc0, 1)); // Maximum name length.
EXPECT_TRUE(IsNameBanned("äbc", &Abc0, 1)); // Confusables
EXPECT_FALSE(IsNameBanned("def", &Abc0, 1));
EXPECT_FALSE(IsNameBanned("abcdef", &Abc0, 1));
}
TEST(NameBan, Substring)
{
CNameBan Xyz("xyz", 0, 1);
ASSERT_TRUE(IsNameBanned("abcxyz", &Xyz, 1));
ASSERT_TRUE(IsNameBanned("abcxyzdef", &Xyz, 1));
ASSERT_FALSE(IsNameBanned("abcdef", &Xyz, 1));
}
EXPECT_TRUE(IsNameBanned("abcxyz", &Xyz, 1));
EXPECT_TRUE(IsNameBanned("abcxyzdef", &Xyz, 1));
EXPECT_FALSE(IsNameBanned("abcdef", &Xyz, 1));
}