Mirror/betaflight
1
0
Fork 0
mirror of https://github.com/betaflight/betaflight.git synced 2025-07-26 01:35:41 +03:00
Code Issues Wiki Activity

Fix and re-enable BMP085 unit test

Browse source 
Since BMP085 is now included in some unified targets it makes sense that the unit test is active.
This commit is contained in:
Bruce Luckcuck 2020-10-14 10:39:49 -04:00
parent fa6c0e2db1
commit b9ed6b4034
2 changed files with 26 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/test/Makefile
View file

@ -51,13 +51,11 @@ atomic_unittest_SRC := \
$(USER_DIR)/build/atomic.c \ $(USER_DIR)/build/atomic.c \
$(TEST_DIR)/atomic_unittest_c.c $(TEST_DIR)/atomic_unittest_c.c
# This test is disabled due to build errors. baro_bmp085_unittest_SRC := \
# Its source code is archived in unit/baro_bmp085_unittest.cc.txt $(USER_DIR)/drivers/barometer/barometer_bmp085.c
#
#baro_bmp085_unittest_SRC := \
# $(USER_DIR)/drivers/barometer/barometer_bmp085.c \
# $(USER_DIR)/drivers/io.c
baro_bmp085_unittest_DEFINES := \
USE_BARO_BMP085=
baro_bmp280_unittest_SRC := \ baro_bmp280_unittest_SRC := \
$(USER_DIR)/drivers/barometer/barometer_bmp280.c $(USER_DIR)/drivers/barometer/barometer_bmp280.c

31
src/test/unit/baro_bmp085_unittest.cc.txt → src/test/unit/baro_bmp085_unittest.cc
View file

@ -18,9 +18,14 @@
extern "C" { extern "C" {
void bmp085_calculate(int32_t *pressure, int32_t *temperature); #include "platform.h"
extern uint32_t bmp085_up; #include "target.h"
extern uint16_t bmp085_ut; #include "drivers/barometer/barometer.h"
#include "drivers/bus.h"
void bmp085Calculate(int32_t *pressure, int32_t *temperature);
extern uint32_t bmp085_up;
extern uint16_t bmp085_ut;
typedef struct { typedef struct {
int16_t ac1; int16_t ac1;
@ -45,7 +50,7 @@ typedef struct {
int16_t oversampling_setting; int16_t oversampling_setting;
} bmp085_t; } bmp085_t;
bmp085_t bmp085; bmp085_t bmp085;
} }
@ -77,7 +82,7 @@ TEST(baroBmp085Test, TestBmp085CalculateOss0)
bmp085.oversampling_setting = 0; bmp085.oversampling_setting = 0;
// when // when
bmp085_calculate(&pressure, &temperature); bmp085Calculate(&pressure, &temperature);
// then // then
EXPECT_EQ(69964, pressure); // Datasheet says 69965 EXPECT_EQ(69964, pressure); // Datasheet says 69965
@ -108,7 +113,7 @@ TEST(baroBmp085Test, TestBmp085CalculateOss3)
bmp085.oversampling_setting = 3; bmp085.oversampling_setting = 3;
// when // when
bmp085_calculate(&pressure, &temperature); bmp085Calculate(&pressure, &temperature);
// then // then
EXPECT_EQ(99998, pressure); EXPECT_EQ(99998, pressure);
@ -139,7 +144,7 @@ TEST(baroBmp085Test, TestBmp085CalculateOss3Cold)
bmp085.oversampling_setting = 3; bmp085.oversampling_setting = 3;
// when // when
bmp085_calculate(&pressure, &temperature); bmp085Calculate(&pressure, &temperature);
// then // then
EXPECT_EQ(92251, pressure); EXPECT_EQ(92251, pressure);
@ -170,7 +175,7 @@ TEST(baroBmp085Test, TestBmp085CalculateOss3Hot)
bmp085.oversampling_setting = 3; bmp085.oversampling_setting = 3;
// when // when
bmp085_calculate(&pressure, &temperature); bmp085Calculate(&pressure, &temperature);
// then // then
EXPECT_EQ(108922, pressure); EXPECT_EQ(108922, pressure);
@ -188,5 +193,13 @@ extern "C" {
void delayMicroseconds(uint32_t) {} void delayMicroseconds(uint32_t) {}
bool busReadRegisterBuffer(const busDevice_t*, uint8_t, uint8_t*, uint8_t) {return true;} bool busReadRegisterBuffer(const busDevice_t*, uint8_t, uint8_t*, uint8_t) {return true;}
bool busWriteRegister(const busDevice_t*, uint8_t, uint8_t) {return true;} bool busWriteRegister(const busDevice_t*, uint8_t, uint8_t) {return true;}
void IOConfigGPIO() {}
void IOHi() {}
void IOLo() {}
void IOInit() {}
void IOGetByTag() {}
bool busBusy(const busDevice_t*, bool*) {return false;}
void busDeviceRegister(const busDevice_t*) {}
bool busReadRegisterBufferStart(const busDevice_t*, uint8_t, uint8_t*, uint8_t) {return true;}
bool busWriteRegisterStart(const busDevice_t*, uint8_t, uint8_t) {return true;}
} }