Add more compile checks and resolve warnings

Jan Breuer

2015-04-22 cb9a2ff1daf017a3aee23c70c5210608e19df681

 libscpi/test/test_lib.c

@@ -15,7 +15,7 @@
 * CUnit Test Suite
 */

scpi_result_t text_function(scpi_t* context) {
static scpi_result_t text_function(scpi_t* context) {
    const char* param;
    size_t param_len;

@@ -32,6 +32,20 @@
    test[param_len] = 0;

    SCPI_ResultText(context, test);

    return SCPI_RES_OK;
}

static scpi_result_t test_treeA(scpi_t* context) {

    SCPI_ResultInt(context, 10);

    return SCPI_RES_OK;
}

static scpi_result_t test_treeB(scpi_t* context) {

    SCPI_ResultInt(context, 20);

    return SCPI_RES_OK;
}
@@ -63,7 +77,10 @@

    { .pattern = "STATus:PRESet", .callback = SCPI_StatusPreset,},
    
    { .pattern = "TEXTfunction", .callback = text_function,},
    { .pattern = "TEXTfunction?", .callback = text_function,},

    { .pattern = "TEST:TREEA?", .callback = test_treeA,},
    { .pattern = "TEST:TREEB?", .callback = test_treeB,},

    SCPI_CMD_LIST_END
};
@@ -170,17 +187,17 @@
};


int init_suite(void) {
static int init_suite(void) {
    SCPI_Init(&scpi_context);

    return 0;
}

int clean_suite(void) {
static int clean_suite(void) {
    return 0;
}

void testCommandsHandling(void) {
static void testCommandsHandling(void) {
#define TEST_INPUT(data, output) {                              \
    SCPI_Input(&scpi_context, data, strlen(data));              \
    CU_ASSERT_STRING_EQUAL(output, output_buffer);              \
@@ -206,13 +223,15 @@
    TEST_INPUT("", "MA, IN, 0, VER\r\n");
    output_buffer_clear();
    
    /* Test ctree traversal */
    TEST_INPUT("TEST:TREEA?;TREEB?\r\n", "10\r\n20\r\n");
    output_buffer_clear();

    CU_ASSERT_EQUAL(err_buffer_pos, 0);
    error_buffer_clear();
    
    // TODO: Compound commands A:B;C -> A:B; A:C
}

void testErrorHandling(void) {
static void testErrorHandling(void) {
    output_buffer_clear();
    error_buffer_clear();

@@ -228,7 +247,7 @@
    TEST_ERROR("IDN?\r\n", "", SCPI_ERROR_UNDEFINED_HEADER);
    TEST_ERROR("*ESE\r\n", "", SCPI_ERROR_MISSING_PARAMETER);
    TEST_ERROR("*IDN? 12\r\n", "MA, IN, 0, VER\r\n", SCPI_ERROR_PARAMETER_NOT_ALLOWED);
    TEST_ERROR("TEXT \"PARAM1\", \"PARAM2\"\r\n", "\"PARAM2\"\r\n", 0);
    TEST_ERROR("TEXT? \"PARAM1\", \"PARAM2\"\r\n", "\"PARAM2\"\r\n", 0);

    // TODO: SCPI_ERROR_INVALID_SEPARATOR
    // TODO: SCPI_ERROR_INVALID_SUFFIX
@@ -240,7 +259,7 @@
    error_buffer_clear();
}

void testIEEE4882(void) {
static void testIEEE4882(void) {
#define TEST_IEEE4882(data, output) {                           \
    SCPI_Input(&scpi_context, data, strlen(data));              \
    CU_ASSERT_STRING_EQUAL(output, output_buffer);              \
@@ -288,7 +307,7 @@
    TEST_IEEE4882("SYSTem:VERSion?\r\n", "1999.0\r\n");
}

void testParameters(void) {
static void testParameters(void) {
    // TODO: test parsin parameters
    
    // TODO: Int
@@ -299,7 +318,7 @@
    // TODO: Choice
}

void testResults(void) {
static void testResults(void) {
    // TODO: test producing results
    
    // TODO: String