Skip to content

Fix RDS incorrect statuses#4930

Merged
bblommers merged 1 commit intogetmoto:masterfrom
kazanzhy:fix_incorrect_statuses
Mar 11, 2022
Merged

Fix RDS incorrect statuses#4930
bblommers merged 1 commit intogetmoto:masterfrom
kazanzhy:fix_incorrect_statuses

Conversation

@kazanzhy
Copy link
Contributor

@kazanzhy kazanzhy commented Mar 11, 2022

Fixed RDS incorrect statuses that were added by me in previous PRs

@codecov-commenter
Copy link

codecov-commenter commented Mar 11, 2022

Codecov Report

Merging #4930 (7b6a7b0) into master (29d01c3) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4930   +/-   ##
=======================================
  Coverage   95.54%   95.54%           
=======================================
  Files         618      618           
  Lines       65477    65477           
=======================================
  Hits        62558    62558           
  Misses       2919     2919           
Flag Coverage Δ
servertests 36.10% <0.00%> (ø)
unittests 95.47% <100.00%> (ø)

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

Impacted Files Coverage Δ
moto/rds/models.py 89.73% <100.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 29d01c3...7b6a7b0. Read the comment docs.

Copy link
Collaborator

@bblommers bblommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix @kazanzhy

@bblommers bblommers merged commit e03ec43 into getmoto:master Mar 11, 2022
@bblommers bblommers added this to the 3.1.0 milestone Mar 11, 2022
@github-actions
Copy link
Contributor

This is now part of moto >= 3.1.0.dev7

sahilshah6196 pushed a commit to EBSCOIS/moto that referenced this pull request Jul 1, 2022
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