Completed the unit test of wrapper

Author: suikan4github

test/test_picowrapper.cpp

@@ -204,7 +204,7 @@ TEST(PicoWrapper, i2c_init) {
 
   // Check the data from test spy. Call order.
   ASSERT_EQ(fff.call_history[0], (void *)i2c_init);
-  ASSERT_EQ(fff.call_history[0], (void *)i2c_init);
+  ASSERT_EQ(fff.call_history[1], (void *)i2c_init);
 
   // Check the data from test spy. DrawRelative Parameters.
   ASSERT_EQ(i2c_init_fake.arg0_history[0], &i2c);
@@ -218,28 +218,74 @@ TEST(PicoWrapper, i2c_init) {
 TEST(PicoWrapper, i2c_read_blocking) {
   PicoWrapper pico;
   i2c_inst_t i2c = 17;
-  const uint baud[] = {3, 100000};
+  uint8_t buf[10];
+  uint8_t addrs[8] = {3, 3, 3, 3, 5, 5, 5, 5};
+  size_t bufsize[8] = {6, 6, 7, 7, 6, 6, 7, 7};
+  bool nostop[8] = {true, false, true, false, true, false, true, false};
+  int myReturnVals[] = {1, 2, 3, 4, 5, 6, 7, 8};
 
   FFF_RESET_HISTORY();
   RESET_FAKE(i2c_read_blocking);
 
-  uint myReturnVals[] = {1, 2};
   SET_RETURN_SEQ(i2c_read_blocking, myReturnVals,
                  sizeof(myReturnVals) / sizeof(bool));
 
-  ASSERT_EQ(pico.i2c_read_blocking(&i2c, baud[0]), myReturnVals[0]);
-  ASSERT_EQ(pico.i2c_read_blocking(&i2c, baud[1]), myReturnVals[1]);
+  for (int i = 0; i < 8; i++) {
+    ASSERT_EQ(
+        pico.i2c_read_blocking(&i2c, addrs[i], buf, bufsize[i], nostop[i]),
+        myReturnVals[i]);
+  }
 
   // Check the data from test spy. How many time called?
-  ASSERT_EQ(i2c_read_blocking_fake.call_count, 2);
+  ASSERT_EQ(i2c_read_blocking_fake.call_count, 8);
+
+  for (int i = 0; i < 8; i++) {
+    // Check the data from test spy. Call order.
+    ASSERT_EQ(fff.call_history[i], (void *)i2c_read_blocking);
+    // Check the data from test spy. DrawRelative Parameters.
+    ASSERT_EQ(i2c_read_blocking_fake.arg0_history[i], &i2c);
+    ASSERT_EQ(i2c_read_blocking_fake.arg1_history[i], addrs[i]);
+    ASSERT_EQ(i2c_read_blocking_fake.arg2_history[i], buf);
+    ASSERT_EQ(i2c_read_blocking_fake.arg3_history[i], bufsize[i]);
+    ASSERT_EQ(i2c_read_blocking_fake.arg4_history[i], nostop[i]);
+  }
+}
 
-  // Check the data from test spy. Call order.
-  ASSERT_EQ(fff.call_history[0], (void *)i2c_read_blocking);
-  ASSERT_EQ(fff.call_history[0], (void *)i2c_read_blocking);
+// FAKE_VALUE_FUNC(int, i2c_write_blocking, i2c_inst_t *, uint8_t, const uint8_t
+// *,
+//                size_t, bool);
+TEST(PicoWrapper, i2c_write_blocking) {
+  PicoWrapper pico;
+  i2c_inst_t i2c = 17;
+  uint8_t buf[10];
+  uint8_t addrs[8] = {3, 3, 3, 3, 5, 5, 5, 5};
+  size_t bufsize[8] = {6, 6, 7, 7, 6, 6, 7, 7};
+  bool nostop[8] = {true, false, true, false, true, false, true, false};
+  int myReturnVals[] = {1, 2, 3, 4, 5, 6, 7, 8};
 
-  // Check the data from test spy. DrawRelative Parameters.
-  ASSERT_EQ(i2c_read_blocking_fake.arg0_history[0], &i2c);
-  ASSERT_EQ(i2c_read_blocking_fake.arg1_history[0], baud[0]);
-  ASSERT_EQ(i2c_read_blocking_fake.arg0_history[1], &i2c);
-  ASSERT_EQ(i2c_read_blocking_fake.arg1_history[1], baud[1]);
+  FFF_RESET_HISTORY();
+  RESET_FAKE(i2c_write_blocking);
+
+  SET_RETURN_SEQ(i2c_write_blocking, myReturnVals,
+                 sizeof(myReturnVals) / sizeof(bool));
+
+  for (int i = 0; i < 8; i++) {
+    ASSERT_EQ(
+        pico.i2c_write_blocking(&i2c, addrs[i], buf, bufsize[i], nostop[i]),
+        myReturnVals[i]);
+  }
+
+  // Check the data from test spy. How many time called?
+  ASSERT_EQ(i2c_write_blocking_fake.call_count, 8);
+
+  for (int i = 0; i < 8; i++) {
+    // Check the data from test spy. Call order.
+    ASSERT_EQ(fff.call_history[i], (void *)i2c_write_blocking);
+    // Check the data from test spy. DrawRelative Parameters.
+    ASSERT_EQ(i2c_write_blocking_fake.arg0_history[i], &i2c);
+    ASSERT_EQ(i2c_write_blocking_fake.arg1_history[i], addrs[i]);
+    ASSERT_EQ(i2c_write_blocking_fake.arg2_history[i], buf);
+    ASSERT_EQ(i2c_write_blocking_fake.arg3_history[i], bufsize[i]);
+    ASSERT_EQ(i2c_write_blocking_fake.arg4_history[i], nostop[i]);
+  }
 }

test/todo/test_picowrapper.md

@@ -1,12 +1,12 @@
 # test_picowrapper TDD
 
-- [] Create PicoWrapper.i2c_write_blocking() to fail. 
-- [] Make it success. 
-- [] Remove the duplication. 
+- [x] Create PicoWrapper.i2c_write_blocking() to fail. 
+- [x] Make it success. 
+- [x] Remove the duplication. 
 
-- [] Create PicoWrapper.i2c_read_blocking() to fail. 
-- [] Make it success. 
-- [] Remove the duplication. 
+- [x] Create PicoWrapper.i2c_read_blocking() to fail. 
+- [x] Make it success. 
+- [x] Remove the duplication. 
 
 - [x] Create PicoWrapper.i2c_init() to fail. 
 - [x] Make it success.