From ca33af45434be98d3c412e93fb6768aafa418158 Mon Sep 17 00:00:00 2001
From: Jan Breuer <jan.breuer@jaybee.cz>
Date: 周六, 20 2月 2016 23:59:50 +0800
Subject: [PATCH] Optimize #49 memory consumption of double to string conversion

---
 libscpi/test/test_fifo.c |  110 ++++++++++++++++++++++++++++++++++++++++++-------------
 1 files changed, 84 insertions(+), 26 deletions(-)

diff --git a/libscpi/test/test_fifo.c b/libscpi/test/test_fifo.c
index fd6fa51..40c65c5 100644
--- a/libscpi/test/test_fifo.c
+++ b/libscpi/test/test_fifo.c
@@ -9,79 +9,136 @@
 #include <stdlib.h>
 #include "CUnit/Basic.h"
 
-#include "scpi/fifo.h"
+#include "../src/fifo_private.h"
 
 /*
  * CUnit Test Suite
  */
 
-int init_suite(void) {
+static int init_suite(void) {
     return 0;
 }
 
-int clean_suite(void) {
+static int clean_suite(void) {
     return 0;
 }
 
-void testFifo() {
-    fifo_t fifo;
-    fifo_init(&fifo);
+static void testFifo() {
+    scpi_fifo_t fifo;
+    int16_t fifo_data[4];
+    fifo_init(&fifo, fifo_data, 4);
     int16_t value;
-
-    fifo.size = 4;
 
 #define TEST_FIFO_COUNT(n)                      \
     do {                                        \
         fifo_count(&fifo, &value);              \
         CU_ASSERT_EQUAL(value, n);              \
     } while(0)                                  \
-    
+
 
     TEST_FIFO_COUNT(0);
-    CU_ASSERT_TRUE(fifo_push(&fifo, 1));
+    CU_ASSERT_TRUE(fifo_is_empty(&fifo));
+    CU_ASSERT_FALSE(fifo_is_full(&fifo));
+
+    CU_ASSERT_TRUE(fifo_add(&fifo, 1));
     TEST_FIFO_COUNT(1);
-    CU_ASSERT_TRUE(fifo_push(&fifo, 2));
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_FALSE(fifo_is_full(&fifo));
+
+    CU_ASSERT_TRUE(fifo_add(&fifo, 2));
     TEST_FIFO_COUNT(2);
-    CU_ASSERT_TRUE(fifo_push(&fifo, 3));
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_FALSE(fifo_is_full(&fifo));
+
+    CU_ASSERT_TRUE(fifo_add(&fifo, 3));
     TEST_FIFO_COUNT(3);
-    CU_ASSERT_TRUE(fifo_push(&fifo, 4));
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_FALSE(fifo_is_full(&fifo));
+
+    CU_ASSERT_TRUE(fifo_add(&fifo, 4));
     TEST_FIFO_COUNT(4);
-    CU_ASSERT_FALSE(fifo_push(&fifo, 1));
-    TEST_FIFO_COUNT(4);
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_TRUE(fifo_is_full(&fifo));
 
     CU_ASSERT_EQUAL(fifo.data[0], 1);
     CU_ASSERT_EQUAL(fifo.data[1], 2);
     CU_ASSERT_EQUAL(fifo.data[2], 3);
     CU_ASSERT_EQUAL(fifo.data[3], 4);
 
-    CU_ASSERT_TRUE(fifo_pop(&fifo, &value));
+
+    CU_ASSERT_FALSE(fifo_add(&fifo, 5));
+    TEST_FIFO_COUNT(4);
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_TRUE(fifo_is_full(&fifo));
+
+    CU_ASSERT_EQUAL(fifo.data[0], 1);
+    CU_ASSERT_EQUAL(fifo.data[1], 2);
+    CU_ASSERT_EQUAL(fifo.data[2], 3);
+    CU_ASSERT_EQUAL(fifo.data[3], 4);
+
+    CU_ASSERT_TRUE(fifo_remove_last(&fifo, &value));
+    CU_ASSERT_EQUAL(value, 4);
+    TEST_FIFO_COUNT(3);
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_FALSE(fifo_is_full(&fifo));
+
+    CU_ASSERT_TRUE(fifo_add(&fifo, 6));
+    TEST_FIFO_COUNT(4);
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_TRUE(fifo_is_full(&fifo));
+
+    CU_ASSERT_EQUAL(fifo.data[0], 1);
+    CU_ASSERT_EQUAL(fifo.data[1], 2);
+    CU_ASSERT_EQUAL(fifo.data[2], 3);
+    CU_ASSERT_EQUAL(fifo.data[3], 6);
+
+    CU_ASSERT_TRUE(fifo_remove(&fifo, &value));
     CU_ASSERT_EQUAL(value, 1);
     TEST_FIFO_COUNT(3);
+    CU_ASSERT_FALSE(fifo_is_empty(&fifo));
+    CU_ASSERT_FALSE(fifo_is_full(&fifo));
 
-    CU_ASSERT_TRUE(fifo_push(&fifo, 5));
+    CU_ASSERT_TRUE(fifo_add(&fifo, 7));
     TEST_FIFO_COUNT(4);
 
-    CU_ASSERT_TRUE(fifo_pop(&fifo, &value));
+    CU_ASSERT_TRUE(fifo_remove(&fifo, &value));
     CU_ASSERT_EQUAL(value, 2);
     TEST_FIFO_COUNT(3);
 
-    CU_ASSERT_TRUE(fifo_pop(&fifo, &value));
+    CU_ASSERT_TRUE(fifo_remove(&fifo, &value));
     CU_ASSERT_EQUAL(value, 3);
     TEST_FIFO_COUNT(2);
 
-    CU_ASSERT_TRUE(fifo_pop(&fifo, &value));
-    CU_ASSERT_EQUAL(value, 4);
+    CU_ASSERT_TRUE(fifo_add(&fifo, 10));
+    TEST_FIFO_COUNT(3);
+
+    CU_ASSERT_TRUE(fifo_add(&fifo, 11));
+    TEST_FIFO_COUNT(4);
+
+    CU_ASSERT_TRUE(fifo_remove(&fifo, &value));
+    CU_ASSERT_EQUAL(value, 6);
+    TEST_FIFO_COUNT(3);
+
+    CU_ASSERT_TRUE(fifo_remove(&fifo, &value));
+    CU_ASSERT_EQUAL(value, 7);
+    TEST_FIFO_COUNT(2);
+
+    CU_ASSERT_TRUE(fifo_remove_last(&fifo, &value));
+    CU_ASSERT_EQUAL(value, 11);
     TEST_FIFO_COUNT(1);
 
-    CU_ASSERT_TRUE(fifo_pop(&fifo, &value));
-    CU_ASSERT_EQUAL(value, 5);
+    CU_ASSERT_TRUE(fifo_remove(&fifo, &value));
+    CU_ASSERT_EQUAL(value, 10);
     TEST_FIFO_COUNT(0);
-    
-    CU_ASSERT_FALSE(fifo_pop(&fifo, &value));
+
+    CU_ASSERT_FALSE(fifo_remove(&fifo, &value));
     TEST_FIFO_COUNT(0);
+
+    CU_ASSERT_FALSE(fifo_remove_last(&fifo, NULL));
 }
 
 int main() {
+    unsigned int result;
     CU_pSuite pSuite = NULL;
 
     /* Initialize the CUnit test registry */
@@ -104,6 +161,7 @@
     /* Run all tests using the CUnit Basic interface */
     CU_basic_set_mode(CU_BRM_VERBOSE);
     CU_basic_run_tests();
+    result = CU_get_number_of_tests_failed();
     CU_cleanup_registry();
-    return CU_get_error();
+    return result ? result : CU_get_error();
 }

--
Gitblit v1.9.1