-
-
Couldn't load subscription status.
- Fork 0
feat(test): Enhance NVS test DEMO #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
49af25a
e15e7f7
e14204c
834e85c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -1,35 +1,141 @@ | ||||||||
| #include <Arduino.h> | ||||||||
| #include <Preferences.h> | ||||||||
|
|
||||||||
|
|
||||||||
| struct TestData { | ||||||||
| uint8_t id; | ||||||||
| uint16_t value; | ||||||||
| }; | ||||||||
|
|
||||||||
| Preferences preferences; | ||||||||
|
|
||||||||
| void validate_types() { | ||||||||
| assert(preferences.getType("char") == PT_I8); | ||||||||
| assert(preferences.getType("uchar") == PT_U8); | ||||||||
| assert(preferences.getType("short") == PT_I16); | ||||||||
| assert(preferences.getType("ushort") == PT_U16); | ||||||||
| assert(preferences.getType("int") == PT_I32); | ||||||||
| assert(preferences.getType("uint") == PT_U32); | ||||||||
| assert(preferences.getType("long") == PT_I32); | ||||||||
| assert(preferences.getType("ulong") == PT_U32); | ||||||||
| assert(preferences.getType("long64") == PT_I64); | ||||||||
| assert(preferences.getType("ulong64") == PT_U64); | ||||||||
| assert(preferences.getType("float") == PT_BLOB); | ||||||||
| assert(preferences.getType("double") == PT_BLOB); | ||||||||
| assert(preferences.getType("bool") == PT_U8); | ||||||||
| assert(preferences.getType("str") == PT_STR); | ||||||||
| assert(preferences.getType("strLen") == PT_STR); | ||||||||
| assert(preferences.getType("struct") == PT_BLOB); | ||||||||
| } | ||||||||
|
|
||||||||
| // Function to increment string values | ||||||||
| void incrementStringValues(String &val_string, char *val_string_buf, size_t buf_size) { | ||||||||
| // Extract the number from string and increment it | ||||||||
| val_string = "str" + String(val_string.substring(3).toInt() + 1); | ||||||||
|
|
||||||||
| // Extract the number from strLen and increment it | ||||||||
| String strLen_str = String(val_string_buf); | ||||||||
| int strLen_num = strLen_str.substring(6).toInt(); | ||||||||
| snprintf(val_string_buf, buf_size, "strLen%d", strLen_num + 1); | ||||||||
| } | ||||||||
|
|
||||||||
| void setup() { | ||||||||
| Serial.begin(115200); | ||||||||
|
|
||||||||
| while (!Serial) { | ||||||||
| ; | ||||||||
| } | ||||||||
|
|
||||||||
| preferences.begin("my-app", false); | ||||||||
|
|
||||||||
| // Get the counter value, if the key does not exist, return a default value of 0 | ||||||||
| unsigned int counter = preferences.getUInt("counter", 0); | ||||||||
| // Get the preferences value and if not exists, use default parameter | ||||||||
| char val_char = preferences.getChar("char", 'A'); | ||||||||
| unsigned char val_uchar = preferences.getUChar("uchar", 0); | ||||||||
| int16_t val_short = preferences.getShort("short", 0); | ||||||||
| uint16_t val_ushort = preferences.getUShort("ushort", 0); | ||||||||
| int32_t val_int = preferences.getInt("int", 0); | ||||||||
| uint32_t val_uint = preferences.getUInt("uint", 0); | ||||||||
| int64_t val_long = preferences.getLong("long", 0); | ||||||||
| uint32_t val_ulong = preferences.getULong("ulong", 0); | ||||||||
| int64_t val_long64 = preferences.getLong64("long64", 0); | ||||||||
| uint64_t val_ulong64 = preferences.getULong64("ulong64", 0); | ||||||||
| float val_float = preferences.getFloat("float", 0.0f); | ||||||||
| double val_double = preferences.getDouble("double", 0.0); | ||||||||
| bool val_bool = preferences.getBool("bool", false); | ||||||||
|
|
||||||||
| // Print the counter to Serial Monitor | ||||||||
| Serial.printf("Current counter value: %u\n", counter); | ||||||||
| // Strings | ||||||||
| String val_string = preferences.getString("str", "str0"); | ||||||||
|
||||||||
| String val_string = preferences.getString("str", "str0"); | |
| char val_string[20] = "str0"; // Fixed-size buffer for stability | |
| preferences.getString("str", val_string, sizeof(val_string)); |
Copilot
AI
Jun 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All serial data may not flush before the ESP.restart() call; consider adding Serial.flush() or a short delay after printing to ensure complete transmission.
Copilot
AI
Jun 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using %lld for val_long (which comes from getLong, typically a 32-bit value) can lead to mismatched output. Either switch to %ld for 32-bit long values or consistently use 64-bit APIs/types (e.g., getLong64) if you intend 64-bit storage.
| Serial.printf("long: %lld | ulong: %lu | long64: %lld | ulong64: %llu | ", | |
| Serial.printf("long: %ld | ulong: %lu | long64: %lld | ulong64: %llu | ", |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,11 +4,11 @@ | |
| def test_nvs(dut): | ||
| LOGGER = logging.getLogger(__name__) | ||
|
|
||
| LOGGER.info("Expecting counter value 0") | ||
| dut.expect_exact("Current counter value: 0") | ||
| LOGGER.info("Expecting default values from Preferences") | ||
| dut.expect_exact("Values from Preferences: char: A | uchar: 0 | short: 0 | ushort: 0 | int: 0 | uint: 0 | long: 0 | ulong: 0 | long64: 0 | ulong64: 0 | float: 0.00 | double: 0.00 | bool: false | str: str0 | strLen: strLen0 | struct: {id:1,val:100}") | ||
|
|
||
| LOGGER.info("Expecting counter value 1") | ||
| dut.expect_exact("Current counter value: 1") | ||
| LOGGER.info("Expecting updated preferences for the first time") | ||
| dut.expect_exact("Values from Preferences: char: B | uchar: 1 | short: 1 | ushort: 1 | int: 1 | uint: 1 | long: 1 | ulong: 1 | long64: 1 | ulong64: 1 | float: 1.10 | double: 1.10 | bool: true | str: str1 | strLen: strLen1 | struct: {id:2,val:110}") | ||
|
|
||
| LOGGER.info("Expecting counter value 2") | ||
| dut.expect_exact("Current counter value: 2") | ||
| LOGGER.info("Expecting updated preferences for the second time") | ||
| dut.expect_exact("Values from Preferences: char: C | uchar: 2 | short: 2 | ushort: 2 | int: 2 | uint: 2 | long: 2 | ulong: 2 | long64: 2 | ulong64: 2 | float: 2.20 | double: 2.20 | bool: false | str: str2 | strLen: strLen2 | struct: {id:3,val:120}") | ||
|
Comment on lines
+8
to
+14
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using the C
assertmacro here may not integrate well with the Unity framework; consider usingTEST_ASSERT_EQUAL_INT(PT_I8, preferences.getType("char"))for clearer test reporting.