dacabdi / xhoundpi

RaspberryPi-based high precision GPS point surveyor
Other
0 stars 0 forks source link

patch/add msg ids + spans and logs to gnss service + improve testing + cleanups #18

Closed dacabdi closed 3 years ago

dacabdi commented 3 years ago
github-actions[bot] commented 3 years ago

Unit Test Statistics

  1 files  ±0  29 suites  +2   0s :stopwatch: ±0s 57 tests +5  57 :heavy_check_mark: +5  0 :zzz: ±0  0 :x: ±0 

Results for commit d09ce2df. ± Comparison against base commit 15f99af1.

This pull request removes 2 and adds 7 tests. Note that renamed tests count towards both. ``` test.test_unit_proto_parser.test_ProtocolReaderProvider ‑ test_provide test.test_unit_proto_parser.test_StubProtocolClassifier ‑ test_classify ``` ``` test.test_unit_proto_classifier.test_StubProtocolClassifier ‑ test_classify test.test_unit_proto_serializer.test_NMEAProtocolSerializer ‑ test_serialize test.test_unit_proto_serializer.test_NMEAProtocolSerializer ‑ test_serialize_error test.test_unit_proto_serializer.test_ProtocolSerializerProvider ‑ test_provide test.test_unit_proto_serializer.test_ProtocolSerializerProvider ‑ test_provide_no_match test.test_unit_proto_serializer.test_UBXProtocolSerializer ‑ test_serialize test.test_unit_proto_serializer.test_UBXProtocolSerializer ‑ test_serialize_error ```
codecov[bot] commented 3 years ago

Codecov Report

Merging #18 (efebe19) into master (15f99af) will increase coverage by 0.33%. The diff coverage is 97.31%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   95.47%   95.81%   +0.33%     
==========================================
  Files          19       25       +6     
  Lines         442      597     +155     
==========================================
+ Hits          422      572     +150     
- Misses         20       25       +5     
Flag Coverage Δ
unittests 95.81% <97.31%> (+0.33%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
xhoundpi/gnss_service.py 94.28% <91.66%> (-5.72%) :arrow_down:
xhoundpi/gnss_service_decorators.py 92.20% <94.73%> (+3.74%) :arrow_up:
xhoundpi/events/__init__.py 100.00% <100.00%> (ø)
xhoundpi/events/app_event.py 100.00% <100.00%> (ø)
xhoundpi/events/common.py 100.00% <100.00%> (ø)
xhoundpi/events/gnss_service_action.py 100.00% <100.00%> (ø)
xhoundpi/gnss_service_iface.py 100.00% <100.00%> (ø)
xhoundpi/gnss_service_runner.py 100.00% <100.00%> (ø)
xhoundpi/message.py 100.00% <100.00%> (ø)
xhoundpi/proto_serializer.py 100.00% <100.00%> (ø)
... and 7 more
github-actions[bot] commented 3 years ago

Unit Test Statistics

  1 files  ±  0  32 suites  +5   0s :stopwatch: ±0s 71 tests +19  71 :heavy_check_mark: +19  0 :zzz: ±0  0 :x: ±0 

Results for commit 831effd1. ± Comparison against base commit 15f99af1.

This pull request removes 2 and adds 21 tests. Note that renamed tests count towards both. ``` test.test_unit_proto_parser.test_ProtocolReaderProvider ‑ test_provide test.test_unit_proto_parser.test_StubProtocolClassifier ‑ test_classify ``` ``` test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_empty_exception test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_enums_are_valued_and_names_are_added test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_exception test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_mixed_event test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_primitives_are_not_serialized test.test_status.test_Status ‑ test_equality test.test_status.test_Status ‑ test_error test.test_status.test_Status ‑ test_ok_default test.test_status.test_Status ‑ test_ok_factory test.test_status.test_Status ‑ test_ok_with_metadata … ```
github-actions[bot] commented 3 years ago

Unit Test Statistics

  1 files  ±  0  32 suites  +5   0s :stopwatch: ±0s 71 tests +19  71 :heavy_check_mark: +19  0 :zzz: ±0  0 :x: ±0 

Results for commit efebe197. ± Comparison against base commit 15f99af1.

This pull request removes 2 and adds 21 tests. Note that renamed tests count towards both. ``` test.test_unit_proto_parser.test_ProtocolReaderProvider ‑ test_provide test.test_unit_proto_parser.test_StubProtocolClassifier ‑ test_classify ``` ``` test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_empty_exception test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_enums_are_valued_and_names_are_added test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_exception test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_mixed_event test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_primitives_are_not_serialized test.test_status.test_Status ‑ test_equality test.test_status.test_Status ‑ test_error test.test_status.test_Status ‑ test_ok_default test.test_status.test_Status ‑ test_ok_factory test.test_status.test_Status ‑ test_ok_with_metadata … ```
github-actions[bot] commented 3 years ago

Unit Test Statistics

  1 files  ±  0  32 suites  +5   0s :stopwatch: ±0s 71 tests +19  71 :heavy_check_mark: +19  0 :zzz: ±0  0 :x: ±0 

Results for commit 4935f2ab. ± Comparison against base commit 15f99af1.

This pull request removes 2 and adds 21 tests. Note that renamed tests count towards both. ``` test.test_unit_proto_parser.test_ProtocolReaderProvider ‑ test_provide test.test_unit_proto_parser.test_StubProtocolClassifier ‑ test_classify ``` ``` test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_empty_exception test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_enums_are_valued_and_names_are_added test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_exception test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_mixed_event test.test_bound_logger_event.test_BoundLoggerEvent ‑ test_primitives_are_not_serialized test.test_status.test_Status ‑ test_equality test.test_status.test_Status ‑ test_error test.test_status.test_Status ‑ test_ok_default test.test_status.test_Status ‑ test_ok_factory test.test_status.test_Status ‑ test_ok_with_metadata … ```