Skip to content

Conversation

@yufeiminds
Copy link
Contributor

ENHANCEMENTS:

  • Improve testing with requests_mock
  • Add InvalidResponseException and HTTPStatusException
  • Add request_uuid to response and exception logging
  • Add default exception middileware
  • Change documentation url

BUG FIXES:

  • Fix pytest version compatible

@codecov-commenter
Copy link

Codecov Report

Merging #56 into master will increase coverage by 3.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
+ Coverage   93.88%   96.89%   +3.01%     
==========================================
  Files          19       19              
  Lines         572      612      +40     
==========================================
+ Hits          537      593      +56     
+ Misses         35       19      -16     
Impacted Files Coverage Δ
ucloud/core/exc/__init__.py 100.00% <ø> (ø)
ucloud/core/client/_client.py 100.00% <100.00%> (+3.79%) ⬆️
ucloud/core/exc/_exc.py 100.00% <100.00%> (ø)
ucloud/core/transport/_requests.py 93.75% <100.00%> (+11.78%) ⬆️
ucloud/core/transport/http.py 95.45% <100.00%> (+5.79%) ⬆️
ucloud/core/auth/_cfg.py 100.00% <0.00%> (+3.44%) ⬆️
ucloud/core/utils/middleware.py 100.00% <0.00%> (+14.28%) ⬆️

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 d29b51f...9039397. Read the comment docs.

@yufeiminds yufeiminds merged commit e2e3315 into ucloud:master Jul 31, 2020
ucloud-bot added a commit to ucloud-bot/ucloud-sdk-python3 that referenced this pull request Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants