commit 194c5e100619fbbd1e3ace7347aaa647c39e143f
author Wyatt Hepler <hepler@google.com> Wed Dec 01 10:53:12 2021 -0800
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Sat Mar 26 01:12:35 2022 +0000
tree 3d5c9b3c37be1bd07779ec0da29dc780205247cb
parent a8a1dbd62fb2b7badd7fc08c593fb1895776ae46

pw_transfer: Expand integration test

- Increase iterations to 5.
- Add 8192 B test that sometimes failed in presubmit.

Change-Id: I3b72d6a8acd326c1a2ac02f4b65916cd60cc43ad
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/71562
Reviewed-by: Erik Gilling <konkers@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>

pw_transfer/integration_test.cc

diff --git a/pw_transfer/integration_test.cc b/pw_transfer/integration_test.cc
index 28cba43..3f6f829 100644
--- a/pw_transfer/integration_test.cc
+++ b/pw_transfer/integration_test.cc
@@ -35,9 +35,7 @@
 
 using namespace std::chrono_literals;
 
-// TODO(hepler): Use more iterations when the pw_transfer synchronization issues
-//     that make this flaky are fixed.
-constexpr int kIterations = 1;
+constexpr int kIterations = 5;
 
 constexpr auto kData512 = bytes::Initialized<512>([](size_t i) { return i; });
 constexpr auto kData8192 = bytes::Initialized<8192>([](size_t i) { return i; });
@@ -206,6 +204,7 @@
 PW_TRANSFER_TEST_READ(SingleByte_2, "?");
 PW_TRANSFER_TEST_READ(SmallData, "hunter2");
 PW_TRANSFER_TEST_READ(LargeData, kData512);
+PW_TRANSFER_TEST_READ(VeryLargeData, kData8192);
 
 TEST_F(TransferIntegration, Write_UnknownId) {
   constexpr std::byte kData[] = {std::byte{0}, std::byte{1}, std::byte{2}};
@@ -237,6 +236,7 @@
 PW_TRANSFER_TEST_WRITE(SmallData, "hunter2");
 PW_TRANSFER_TEST_WRITE(LargeData, kData512);
 PW_TRANSFER_TEST_WRITE(HdlcEscape, kDataHdlcEscape);
+PW_TRANSFER_TEST_WRITE(VeryLargeData, kData8192);
 
 }  // namespace
 }  // namespace pw::transfer