commit d5a82b72e094e64acc7c9517d0ab64c660985a59
author Don Turner <donturner@google.com> Thu Jul 12 18:09:15 2018 +0100
committer Don Turner <dturner@users.noreply.github.com> Mon Jul 16 13:38:17 2018 +0100
tree 8e7725dd0fe6eca0b534eb0e31745328296fda3b
parent ba5a01bdf8db3d5cb56b2c00881ff9532609dcbf

Adding missing tests and stream closed default values

tests/testStreamClosedMethods.cpp

diff --git a/tests/testStreamClosedMethods.cpp b/tests/testStreamClosedMethods.cpp
index e148ee5..4ca5797 100644
--- a/tests/testStreamClosedMethods.cpp
+++ b/tests/testStreamClosedMethods.cpp
@@ -14,12 +14,6 @@
  * limitations under the License.
  */
 
-/*
- * TODO:
- Create test runner APK
- Remove hardcoded ANDROID_NDK from cmake config
- */
-
 #include <gtest/gtest.h>
 #include <oboe/Oboe.h>
 
@@ -319,3 +313,38 @@
     StreamState next;
     ASSERT_EQ(mStream->waitForStateChange(StreamState::Open, &next, 0), Result::ErrorClosed);
 }
+
+TEST_F(StreamClosedReturnValues, SetBufferSizeInFramesReturnsClosed){
+
+    openAndCloseStream();
+    auto r = mStream->setBufferSizeInFrames(192);
+    ASSERT_EQ(r.error(), Result::ErrorClosed);
+}
+
+TEST_F(StreamClosedReturnValues, CalculateLatencyInMillisReturnsClosedIfSupported){
+
+    openAndCloseStream();
+
+    if (mStream->getAudioApi() == AudioApi::AAudio){
+        auto r = mStream->calculateLatencyMillis();
+        ASSERT_EQ(r.error(), Result::ErrorClosed);
+    }
+}
+
+TEST_F(StreamClosedReturnValues, ReadReturnsClosed){
+
+    openAndCloseStream();
+
+    void *buffer;
+    auto r = mStream->read(buffer, 1, 0);
+    ASSERT_EQ(r.error(), Result::ErrorClosed);
+}
+
+TEST_F(StreamClosedReturnValues, WriteReturnsClosed){
+
+    openAndCloseStream();
+
+    void *buffer;
+    auto r = mStream->write(buffer, 1, 0);
+    ASSERT_EQ(r.error(), Result::ErrorClosed);
+}