caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
10.94k stars 385 forks source link

Test Coverage and Code Tidy #1015

Closed caronc closed 6 months ago

caronc commented 6 months ago

Description:

Related issue (if applicable): n/a

Just some code cleanup; inline documentation updates, spelling errors and more test coverage

Checklist

Testing

Anyone can help test this source code as follows:

# Create a virtual environment to work in as follows:
python3 -m venv apprise

# Change into our new directory
cd apprise

# Activate our virtual environment
source bin/activate

# Install the branch
pip install git+https://github.com/caronc/apprise.git@code-cleanup

# Test out the changes with the following command:
apprise -t "Test Title" -b "Test Message" \
  <apprise url related to ticket>
codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (524eba6) 99.52% compared to head (693b66c) 99.55%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1015 +/- ## ========================================== + Coverage 99.52% 99.55% +0.02% ========================================== Files 129 129 Lines 17189 17185 -4 Branches 3510 3508 -2 ========================================== + Hits 17107 17108 +1 + Misses 70 68 -2 + Partials 12 9 -3 ```

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