Skip to content

Commit

Permalink
fixed test issue
Browse files Browse the repository at this point in the history
  • Loading branch information
ypujante committed Sep 13, 2021
1 parent 8f90411 commit 14e6b03
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions RELEASE.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ Check the [Requirements](https://jamba.dev/requirements/) documentation for info
proper version
* Use latest version of GoogleTest (get rid of CMake warnings)
* Use `gtest_discover_tests` instead of `gtest_add_tests` to minimize CMake invocation when building tests
* Make sure that `.rc` file gets regenerated properly when resource files change (Windows)
* Fixed test issue

> #### Potential breaking changes
>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ TEST(TestFastWriteMemoryStream, test_read)

auto read = [&vs](int32 numBytes, std::vector<int8> iExpected) {
std::vector<int8> buf{};
buf.reserve(numBytes + 1);
buf.resize(numBytes + 1);
int32 numBytesRead = 0;
ASSERT_EQ(kResultOk, vs.read(buf.data(), numBytes, &numBytesRead));
ASSERT_EQ(iExpected.size(), numBytesRead);
Expand Down
2 changes: 1 addition & 1 deletion test/cpp/pongasoft/VST/Utils/test-ReadOnlyMemoryStream.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ TEST(TestReadOnlyMemoryStream, test_read)

auto read = [&ms](int32 numBytes, std::vector<int8> iExpected) {
std::vector<int8> buf{};
buf.reserve(numBytes + 1);
buf.resize(numBytes + 1);
int32 numBytesRead = 0;
ASSERT_EQ(kResultOk, ms.read(buf.data(), numBytes, &numBytesRead));
ASSERT_EQ(iExpected.size(), numBytesRead);
Expand Down

0 comments on commit 14e6b03

Please sign in to comment.