Merge pull request #51 from Sensirion/new_test_setup

New test setup

.clang-format

@@ -10,4 +10,5 @@ IndentCaseLabels: true
 SpacesBeforeTrailingComments: 2
 PointerAlignment: Left
 AlignEscapedNewlines: Left
+ForEachMacros: ['TEST_GROUP', 'TEST']
 ...

.gitignore

@@ -3,3 +3,6 @@
 /sps30-i2c/sps30_example_usage
 /sps30-i2c/user_config.inc
 /sps-common/sps_git_version.c
+/i2c-mux-testbed/
+/tests/sps30-test-hw_i2c
+/tests/sps30-test-sw_i2c

.gitlab-ci.yml

@@ -1,20 +1,33 @@
+include: 'https://raw.githubusercontent.com/Sensirion/ci-config-github/master/github-status-gitlab-ci-template-v1.yml'
+
 stages:
+  - publish_build_pending
   - build
+  - publish_build_status
+
+variables:
+  GIT_SUBMODULE_STRATEGY: recursive
+  GITHUB_PROJECT_NAME: embedded-sps
+
+publish_build_pending:
+  extends: .github_publish_pending
+  stage: publish_build_pending
+
+publish_success:
+  extends: .github_publish_success
+  stage: publish_build_status
+
+publish_failure:
+  extends: .github_publish_failure
+  stage: publish_build_status
 
-trigger_testbed:
+test:
   stage: build
   image:
-    name: registry.gitlab.sensirion.lokal/sensirion/docker/docker-ubuntu:18.04-1.2.0
+    name: registry.gitlab.sensirion.lokal/sensirion/docker/docker-rpi-testbed:1.1.0
   tags:
-    - docker
-    - linux
+    - mso-sw-testbed-2
   script:
-  - >
-    curl
-    --request POST
-    --form ref=master
-    --form token=$CI_JOB_TOKEN
-    --form "variables[DRIVER_NAMES]=embedded-sps"
-    --form "variables[DRIVER_BRANCH]=$CI_COMMIT_REF_NAME"
-    --form "variables[DRIVER_COMMIT_SHA]=$CI_COMMIT_SHA"
-    https://gitlab/api/v4/projects/801/trigger/pipeline
+    - git clone -b 2.0.0 https://gitlab-ci-token:${CI_JOB_TOKEN}@gitlab/MSO-SW/drivers/testbed/i2c-mux-testbed.git
+    - cd tests
+    - make test

tests/Makefile

@@ -0,0 +1,25 @@
+include ../embedded-common/test-config/base_config.inc
+sps_driver_dir := ${driver_dir}/embedded-sps
+include ${sps_driver_dir}/sps30-i2c/default_config.inc
+
+sps30_test_binaries := sps30-test-hw_i2c sps30-test-sw_i2c
+
+.PHONY: all clean prepare test
+
+all: clean prepare test
+
+prepare:
+	cd ${sps_driver_dir} && $(MAKE) prepare
+
+sps30-test-hw_i2c: sps30-test.cpp ${sps30_i2c_sources} ${hw_i2c_sources} ${sensirion_test_sources}
+	$(CXX) $(CXXFLAGS) -o $@ $^ $(LDFLAGS)
+
+sps30-test-sw_i2c: CONFIG_I2C_TYPE := sw_i2c
+sps30-test-sw_i2c: sps30-test.cpp ${sps30_i2c_sources} ${sw_i2c_sources} ${sensirion_test_sources}
+	$(CXX) $(CXXFLAGS) -o $@ $^ $(LDFLAGS)
+
+clean:
+	$(RM) ${sps30_test_binaries}
+
+test: prepare ${sps30_test_binaries}
+	set -ex; for test in ${sps30_test_binaries}; do echo $${test}; ./$${test}; echo; done;

tests/sps30-test.cpp

@@ -0,0 +1,155 @@
+#include "sensirion_test_setup.h"
+#include "sps30.h"
+
+// Measurement ranges according to datasheet
+#define SPS30_MIN_MC 0
+#define SPS30_MAX_MC 1000
+#define SPS30_MIN_NC 0
+#define SPS30_MAX_NC 3000
+
+#define SECONDS_PER_DAY 86400u  // 24h/d * 60m/h * 60s/m
+
+// Less or equal (<=) for float arithmetics with small error (1e-5)
+#define LEQ(l, u) ((l) - (u) < 1e-5)
+
+static void sps30_delayed_reset() {
+    int16_t ret = sps30_reset();
+    CHECK_ZERO_TEXT(ret, "sps30_reset in test reset");
+    sensirion_sleep_usec(SPS30_RESET_DELAY_USEC);
+}
+
+static void sps30_test(uint8_t set_interval_days) {
+    uint8_t get_interval_days;
+    int16_t ret;
+    uint16_t data_ready;
+    uint32_t get_interval_seconds;
+    uint32_t set_interval_seconds;
+    struct sps30_measurement m;
+
+    // Start measurement
+    ret = sps30_start_measurement();
+    CHECK_ZERO_TEXT(ret, "sps30_start_measurement");
+
+    // Check if there is old data available
+    ret = sps30_read_data_ready(&data_ready);
+    CHECK_ZERO_TEXT(ret, "sps30_read_data_ready for old data");
+
+    if (data_ready) {
+        // Get old data and discard it
+        ret = sps30_read_measurement(&m);
+        CHECK_ZERO_TEXT(ret, "sps30_read_measurement for old data");
+    }
+
+    // Sleep while waiting for new measurement
+    sensirion_sleep_usec(SPS30_MEASUREMENT_DURATION_USEC - 1e5);
+
+    // Wait for data to be ready
+    do {
+        sensirion_sleep_usec(1e5);  // Sleep 100ms
+        ret = sps30_read_data_ready(&data_ready);
+        CHECK_ZERO_TEXT(ret, "sps30_read_data_ready while polling");
+    } while (!data_ready);
+
+    // Read measurement
+    ret = sps30_read_measurement(&m);
+    CHECK_ZERO_TEXT(ret, "sps30_read_measurement");
+    printf("measured values:\n"
+           "\t%0.2f pm1.0\n"
+           "\t%0.2f pm2.5\n"
+           "\t%0.2f pm4.0\n"
+           "\t%0.2f pm10.0\n"
+           "\t%0.2f nc0.5\n"
+           "\t%0.2f nc1.0\n"
+           "\t%0.2f nc2.5\n"
+           "\t%0.2f nc4.5\n"
+           "\t%0.2f nc10.0\n"
+           "\t%0.2f typical particle size\n\n",
+           m.mc_1p0, m.mc_2p5, m.mc_4p0, m.mc_10p0, m.nc_0p5, m.nc_1p0,
+           m.nc_2p5, m.nc_4p0, m.nc_10p0, m.typical_particle_size);
+
+    // Check if mass concentration is rising monotonously
+    CHECK_TRUE_TEXT(LEQ(SPS30_MIN_MC, m.mc_1p0) && LEQ(m.mc_1p0, m.mc_2p5) &&
+                        LEQ(m.mc_2p5, m.mc_4p0) && LEQ(m.mc_4p0, m.mc_10p0) &&
+                        LEQ(m.mc_10p0, SPS30_MAX_MC),
+                    "Mass concentration not rising monotonously");
+
+    // Check if number concentration is rising monotonously
+    CHECK_TRUE_TEXT(LEQ(SPS30_MIN_NC, m.nc_0p5) && LEQ(m.nc_0p5, m.nc_1p0) &&
+                        LEQ(m.nc_1p0, m.nc_2p5) && LEQ(m.nc_2p5, m.nc_4p0) &&
+                        LEQ(m.nc_4p0, m.nc_10p0) &&
+                        LEQ(m.nc_10p0, SPS30_MAX_NC),
+                    "Number concentration not rising monotonously");
+
+    // Set auto cleaning interval in days, reset and get it again
+    ret = sps30_set_fan_auto_cleaning_interval_days(set_interval_days);
+    CHECK_ZERO_TEXT(ret, "sps30_set_fan_auto_cleaning_interval_days");
+    sps30_delayed_reset();
+    ret = sps30_get_fan_auto_cleaning_interval_days(&get_interval_days);
+    CHECK_ZERO_TEXT(ret, "sps30_get_fan_auto_cleaning_interval_days");
+    CHECK_EQUAL_TEXT(set_interval_days, get_interval_days,
+                     "Fan auto cleaning interval days do not match");
+
+    // Set auto cleaning interval in days, reset and set it again
+    set_interval_seconds = set_interval_days * SECONDS_PER_DAY;
+    ret = sps30_set_fan_auto_cleaning_interval(set_interval_seconds);
+    CHECK_ZERO_TEXT(ret, "sps30_set_fan_auto_cleaning_interval");
+    sps30_delayed_reset();
+    ret = sps30_get_fan_auto_cleaning_interval(&get_interval_seconds);
+    CHECK_ZERO_TEXT(ret, "sps30_get_fan_auto_cleaning_interval");
+    CHECK_EQUAL_TEXT(set_interval_seconds, get_interval_seconds,
+                     "Fan auto cleaning interval seconds do not match");
+
+    // Start manual cleaning event
+    ret = sps30_start_manual_fan_cleaning();
+    CHECK_ZERO_TEXT(ret, "sps30_start_manual_fan_cleaning");
+
+    // Stop measurement
+    ret = sps30_stop_measurement();
+    CHECK_ZERO_TEXT(ret, "sps30_stop_measurement");
+}
+
+static void sps30_test_setup() {
+    uint8_t fs_major;
+    uint8_t fs_minor;
+    int16_t ret;
+    char serial[SPS30_MAX_SERIAL_LEN];
+
+    ret = sps30_probe();
+    CHECK_ZERO_TEXT(ret, "sps30_probe in test setup");
+
+    sps30_delayed_reset();
+
+    const char* version = sps_get_driver_version();
+    printf("sps30_get_driver_version: %s\n", version);
+
+    ret = sps30_get_serial(serial);
+    CHECK_ZERO_TEXT(ret, "sps30_get_serial");
+    printf("SPS30 serial: %s\n", serial);
+
+    ret = sps30_read_firmware_version(&fs_major, &fs_minor);
+    CHECK_ZERO_TEXT(ret, "sps30_read_firmware_version");
+    printf("SPS30 Firmware version: %u.%u\n", fs_major, fs_minor);
+}
+
+TEST_GROUP (SPSTestGroup) {
+    void setup() {
+        int16_t ret;
+        sensirion_i2c_init();
+        ret = sensirion_i2c_mux_set_single_channel(0x72, 3);
+        CHECK_EQUAL_TEXT(0, ret, "sensirion_i2c_mux_select_TEXT(0x72, 3)");
+        sps30_test_setup();
+    }
+
+    void teardown() {
+        sps30_delayed_reset();
+        sensirion_i2c_release();
+    }
+};
+
+TEST (SPSTestGroup, SPS30Test_no_cleaning) { sps30_test(0); }
+
+TEST (SPSTestGroup, SPS30Test_daily_cleaning) { sps30_test(1); }
+
+TEST (SPSTestGroup, SPS30Test_weekly_cleaning) { sps30_test(7); }
+
+TEST (SPSTestGroup, SPS30Test_maximum_cleaning) { sps30_test(255); }