Fix developed by Matthias Behr

volkszaehler · Jun 29, 2024 · 81c9649 · 81c9649
1 parent 25f9ab3
commit 81c9649
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/protocols/MeterOMS.cpp b/src/protocols/MeterOMS.cpp
@@ -601,7 +601,7 @@ double MeterOMS::get_record_value(mbus_data_record *record) const {
 			t.tm_year =
 				100 + (((record->data[3] & 0xe0) >> 5) |
 					   ((record->data[4] & 0xf0) >> 1)); // tm_year is number of years since 1900.
-			t.tm_isdst = ((record->data[0] & 0x40) == 0x40) ? 1 : 0;
+			t.tm_isdst = -1;                             // We ignore data[0] & 0x40
 			// check for time invalid at bit 16 (1-based)
 			if ((record->data[1] & 0x80) == 0x80) {
 				// time invalid!

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -88,7 +88,7 @@ IF(GCOV_PATH AND LCOV_PATH AND GENHTML_PATH)
         COMMAND vzlogger_unit_tests
         COMMAND mock_metermap
         COMMAND mock_MeterW1therm
-        COMMAND env TZ="Europe/Berlin" mock_MeterOMS
+        COMMAND mock_MeterOMS
         COMMAND mock_MeterS0
 
         # Capturing lcov counters and generating report

diff --git a/tests/mocks/mock_MeterOMS.cpp b/tests/mocks/mock_MeterOMS.cpp
@@ -136,7 +136,7 @@ TEST(mock_MeterOMS, first_packets) {
 	t.tm_mday = 20;
 	t.tm_mon = 6 - 1;
 	t.tm_year = 2015 - 1900;
-	t.tm_isdst = 1;
+	t.tm_isdst = -1;
 	ASSERT_EQ(rds[7].time_s(), mktime(&t));
 }