rusefi
/
rusefi-1
mirror of https://github.com/rusefi/rusefi-1.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

reducing global state in unit tests

This commit is contained in:
rusefi 2018-03-04 21:32:48 -05:00
parent defb52a6e0
commit e6d368d6f7
4 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
firmware/controllers/algo/engine_configuration.cpp
View File

@ -137,7 +137,7 @@ engine_configuration_s activeConfiguration;
extern engine_configuration_s *engineConfiguration;
void rememberCurrentConfiguration(void) {
void rememberCurrentConfiguration(DECLARE_ENGINE_PARAMETER_SIGNATURE) {
memcpy(&activeConfiguration, engineConfiguration, sizeof(engine_configuration_s));
}
@ -149,12 +149,17 @@ int getGlobalConfigurationVersion(void) {
return globalConfigurationVersion;
}
extern LoggingWithStorage sharedLogger;
/**
* this is the top-level method which should be called in case of any changes to engine configuration
* online tuning of most values in the maps does not count as configuration change, but 'Burn' command does
*/
void incrementGlobalConfigurationVersion(DECLARE_ENGINE_PARAMETER_SIGNATURE) {
globalConfigurationVersion++;
#if EFI_DEFAILED_LOGGING
scheduleMsg(&sharedLogger, "set globalConfigurationVersion=%d", globalConfigurationVersion);
#endif /* EFI_DEFAILED_LOGGING */
/**
* All these callbacks could be implemented as listeners, but these days I am saving RAM
*/
@ -182,7 +187,7 @@ void incrementGlobalConfigurationVersion(DECLARE_ENGINE_PARAMETER_SIGNATURE) {
onConfigurationChangeFsioCallback(&activeConfiguration PASS_ENGINE_PARAMETER_SUFFIX);
rememberCurrentConfiguration();
rememberCurrentConfiguration(PASS_ENGINE_PARAMETER_SIGNATURE);
}
/**

2
firmware/controllers/algo/engine_configuration.h
View File

@ -67,7 +67,7 @@ void setDefaultBasePins(DECLARE_ENGINE_PARAMETER_SIGNATURE);
void setDefaultSerialParameters(DECLARE_ENGINE_PARAMETER_SIGNATURE);
void setDefaultSdCardParameters(DECLARE_ENGINE_PARAMETER_SIGNATURE);
void rememberCurrentConfiguration(void);
void rememberCurrentConfiguration(DECLARE_ENGINE_PARAMETER_SIGNATURE);
void incrementGlobalConfigurationVersion(DECLARE_ENGINE_PARAMETER_SIGNATURE);
int getGlobalConfigurationVersion(void);

4
unit_tests/efifeatures.h
View File

@ -14,6 +14,10 @@
#define EFI_GPIO_HARDWARE FALSE
#define FUEL_MATH_EXTREME_LOGGING FALSE
#define EFI_DEFAILED_LOGGING FALSE
#define EFI_UNIT_TEST TRUE
#define EFI_FSIO TRUE

6
unit_tests/main.cpp
View File

@ -35,15 +35,11 @@
#include "test_engine_math.h"
#include "test_trigger_decoder.h"
typedef int32_t msg_t;
#include "hal_streams.h"
#include "memstreams.h"
static engine_configuration_s ec;
engine_configuration_s *engineConfiguration = &ec;
int timeNowUs = 0;
efitimeus_t getTimeNowUs(void) {
@ -54,6 +50,8 @@ efitick_t getTimeNowNt(void) {
return getTimeNowUs() * US_TO_NT_MULTIPLIER;
}
LoggingWithStorage sharedLogger("main");
extern int revolutionCounterSinceBootForUnitTest;
int getRevolutionCounter(void) {