Add libusbp_write_pipe for macOS and get the tests passing.

Author: DavidEGrayson

src/mac/generic_handle_mac.c

@@ -455,6 +455,69 @@ libusbp_error * libusbp_read_pipe(
     return error;
 }
 
+libusbp_error * libusbp_write_pipe(
+    libusbp_generic_handle * handle,
+    uint8_t pipe_id,
+    const void * buffer,
+    size_t size,
+    size_t * transferred)
+{
+    if (transferred != NULL)
+    {
+        *transferred = 0;
+    }
+
+    if (handle == NULL)
+    {
+        return error_create("Generic handle is null.");
+    }
+
+    libusbp_error * error = NULL;
+
+    if (error == NULL && size > UINT32_MAX)
+    {
+        error = error_create("Transfer size is too large.");
+    }
+
+    if (error == NULL && buffer == NULL && size)
+    {
+        error = error_create("Buffer is null.");
+    }
+
+    if (error == NULL)
+    {
+        error = check_pipe_id_out(pipe_id);
+    }
+
+    if (error == NULL)
+    {
+        uint8_t endpoint_number = pipe_id & MAX_ENDPOINT_NUMBER;
+        uint32_t no_data_timeout = 0;
+        uint32_t completion_timeout = handle->out_timeout[endpoint_number];
+        uint32_t pipe_index = handle->out_pipe_index[endpoint_number];
+        kern_return_t kr = (*handle->ioh)->WritePipeTO(handle->ioh, pipe_index,
+          (void *)buffer, size, no_data_timeout, completion_timeout);
+        if (kr != KERN_SUCCESS)
+        {
+            error = error_create_mach(kr, "");
+        }
+    }
+
+    if (error == NULL && transferred != NULL)
+    {
+        // There was no error, so just assume the entire amount was transferred.
+        // WritePipeTO does not give us a number.
+        *transferred = size;
+    }
+
+    if (error != NULL)
+    {
+        error = error_add(error, "Failed to write to pipe.");
+    }
+
+    return error;
+}
+
 #pragma pack(4)
 typedef struct
 {

test/CMakeLists.txt

@@ -11,7 +11,7 @@ if (NOT HAVE_CATCH_FRAMEWORK)
   return ()
 endif ()
 
-use_cxx11()
+use_cxx11 ()
 
 set(USE_TEST_DEVICE_A FALSE CACHE BOOL
   "Run tests that require Test Device A.")
@@ -21,8 +21,16 @@ set(USE_TEST_DEVICE_B FALSE CACHE BOOL
 
 file(GLOB test_sources *.cpp)
 
+if (APPLE)
+  set (link_flags "-framework CoreFoundation ${link_flags}")
+endif ()
+
 add_executable(run_test ${test_sources})
 
+set_target_properties(run_test PROPERTIES
+  LINK_FLAGS "${link_flags}"
+)
+
 include_directories (
   "${CMAKE_CURRENT_SOURCE_DIR}"
   "${CMAKE_SOURCE_DIR}/include"

test/write_pipe_test.cpp

@@ -133,7 +133,7 @@ TEST_CASE("write_pipe (synchronous) on a bulk endpoint ", "[wpi]")
 
     SECTION("can time out")
     {
-      // First packet causes a 150 ms delay, so this transfer will timeout after
+      // First packet causes a long delay, so this transfer will timeout after
       // a partial data transfer.  Need three packets because of double
       // buffering on the device.
       uint8_t buffer[32 * 3] = { 0xDE, 150, 0 };
@@ -145,7 +145,7 @@ TEST_CASE("write_pipe (synchronous) on a bulk endpoint ", "[wpi]")
       catch (const libusbp::error & e)
       {
         REQUIRE(e.has_code(LIBUSBP_ERROR_TIMEOUT));
-        #ifdef __linux__
+        #if defined(__linux__) || defined(__APPLE__)
         REQUIRE(transferred == 0);  // bad
         #else
         REQUIRE(transferred == 64);