Skip to content

Commit

Permalink
EXPECT_GTだと見辛いので反転させてLEを使う
Browse files Browse the repository at this point in the history
コメントで混乱しないように「~より大きくなる」の表現を書き替える。
  • Loading branch information
berryzplus committed Apr 23, 2020
1 parent 03f7c79 commit 01e070b
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions tests/unittests/test-cnative.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ TEST(CNativeW, ConstructWithoutParam)
* @brief コンストラクタ(文字列と文字列長指定)の仕様
* @remark バッファが確保される
* @remark 文字列長は指定した文字列長になる
* @remark バッファサイズは指定した文字列長より大きくなる
* @remark バッファサイズは指定した文字列を格納できるサイズになる
*/
TEST(CNativeW, ConstructWithStringWithLength)
{
Expand All @@ -54,7 +54,7 @@ TEST(CNativeW, ConstructWithStringWithLength)
CNativeW value(sz, cch);
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());
}

/*!
Expand All @@ -70,7 +70,7 @@ TEST(CNativeW, ConstructWithString)
CNativeW value(sz);
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());
}

/*!
Expand All @@ -83,7 +83,7 @@ TEST(CNativeW, ConstructWithStringEmpty)
CNativeW value(sz);
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(0, value.GetStringLength());
EXPECT_GT(value.capacity(), 0);
EXPECT_LE(0, value.capacity());
}

/*!
Expand Down Expand Up @@ -133,7 +133,7 @@ TEST(CNativeW, ConstructFromOtherByMove)
CNativeW value(std::move(other));
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());

// ムーブ元は抜け殻になる
ASSERT_EQ(NULL, other.GetStringPtr());
Expand All @@ -157,7 +157,7 @@ TEST(CNativeW, CopyFromOther)
value = other;
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());

// コピー元バッファとは別に新しいバッファが確保される
ASSERT_NE(other.GetStringPtr(), value.GetStringPtr());
Expand All @@ -179,7 +179,7 @@ TEST(CNativeW, MoveFromOther)
value = std::move(other);
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());

// ムーブ元は抜け殻になる
ASSERT_EQ(NULL, other.GetStringPtr());
Expand Down Expand Up @@ -220,7 +220,7 @@ TEST(CNativeW, AssignString)
value = sz;
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());
}

/*!
Expand Down Expand Up @@ -253,7 +253,7 @@ TEST(CNativeW, AssignStringNullLiteral)
* @brief 加算代入演算子(文字指定)の仕様
* @remark バッファが確保される
* @remark 文字列長は演算子呼出前の文字列長+1になる
* @remark バッファサイズは2以上になる
* @remark バッファサイズは連結された文字列を格納できるサイズになる
*/
TEST(CNativeW, AppendChar)
{
Expand All @@ -263,7 +263,7 @@ TEST(CNativeW, AppendChar)
value += sz[0];
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(1, value.GetStringLength());
EXPECT_GT(value.capacity(), 1);
EXPECT_LE(1, value.capacity());
}

/*!
Expand All @@ -280,7 +280,7 @@ TEST(CNativeW, AppendString)
value += sz;
ASSERT_STREQ(sz, value.GetStringPtr());
EXPECT_EQ(cch, value.GetStringLength());
EXPECT_GT(value.capacity(), cch);
EXPECT_LE(cch, value.capacity());
}

/*!
Expand Down Expand Up @@ -540,15 +540,15 @@ TEST(CNativeW, operatorNotEqualStringNull)
* @brief 独自関数Replaceの仕様
* @remark バッファが確保される
* @remark 文字列長は0になる
* @remark バッファサイズは1以上になる
* @remark バッファサイズは空文字列を格納できるサイズになる
*/
TEST(CNativeW, ReplaceOfNullString)
{
CNativeW value;
value.Replace(L"置換前", L"置換後");
ASSERT_STREQ(L"", value.GetStringPtr());
EXPECT_EQ(0, value.GetStringLength());
EXPECT_GT(value.capacity(), 0);
EXPECT_LE(0, value.capacity());
}

/*!
Expand Down Expand Up @@ -600,7 +600,7 @@ TEST(CNativeW, Clear)

// 1-3. バッファの状態をチェックする

EXPECT_GT(orgCapacity, 0); // データ追加後のバッファサイズを確認する
EXPECT_LE(0, orgCapacity); // データ追加後のバッファサイズを確認する
EXPECT_EQ(orgLength, fixedPatternLen); // データ追加後のデータサイズを確認する

// 2-1. CNativeW をクリアする
Expand Down

0 comments on commit 01e070b

Please sign in to comment.