Closed krisgesling closed 2 years ago
Voight Kampff Integration Test Succeeded (Results)
Merging #3022 (f8f640e) into dev (dd710a6) will not change coverage. The diff coverage is
0.00%
.
@@ Coverage Diff @@
## dev #3022 +/- ##
=======================================
Coverage 53.04% 53.04%
=======================================
Files 123 123
Lines 11170 11170
=======================================
Hits 5925 5925
Misses 5245 5245
Impacted Files | Coverage Δ | |
---|---|---|
mycroft/client/enclosure/base.py | 0.00% <0.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update dd710a6...f8f640e. Read the comment docs.
Description
When inserting a new namespace for the GUI, bus events could cause the dict to change while it is being iterated over.
How to test
Unsure - saw it in OVOS and can see how that could happen. Change shouldn't have a negative impact.
Contributor license agreement signed?
Thanks to @JarbasAI for identifying and fixing it