mqtt-tools / pytest-mqtt

pytest-mqtt supports testing systems based on MQTT.
MIT License
9 stars 2 forks source link

Improve error handling when Docker daemon is not running #5

Closed amotl closed 1 year ago

amotl commented 1 year ago

About

@horta reported at GH-4 about long error stacktraces without a specific error message, when the Docker daemon is not running.

Details

While this patch does not truncate the long exception output, at least it adds a more significant error message.

Screenshot

image

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 66.66% and project coverage change: -1.35% :warning:

Comparison is base (679bac0) 97.74% compared to head (542ce73) 96.40%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #5 +/- ## ========================================== - Coverage 97.74% 96.40% -1.35% ========================================== Files 5 5 Lines 133 139 +6 ========================================== + Hits 130 134 +4 - Misses 3 5 +2 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `96.40% <66.66%> (-1.35%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mqtt-tools#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files Changed](https://app.codecov.io/gh/mqtt-tools/pytest-mqtt/pull/5?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mqtt-tools) | Coverage Δ | | |---|---|---| | [pytest\_mqtt/mosquitto.py](https://app.codecov.io/gh/mqtt-tools/pytest-mqtt/pull/5?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mqtt-tools#diff-cHl0ZXN0X21xdHQvbW9zcXVpdHRvLnB5) | `86.84% <66.66%> (-3.79%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.