Skip to content

Integration test fix#7460

Merged
auvipy merged 20 commits intocelery:masterfrom
dobosevych:integration-test-fix
Apr 14, 2022
Merged

Integration test fix#7460
auvipy merged 20 commits intocelery:masterfrom
dobosevych:integration-test-fix

Conversation

@dobosevych
Copy link
Copy Markdown
Contributor

Note: Before submitting this pull request, please review our contributing
guidelines
.

Description

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2022

Codecov Report

Merging #7460 (78a19eb) into master (55c8ca1) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #7460      +/-   ##
==========================================
- Coverage   89.26%   89.26%   -0.01%     
==========================================
  Files         138      138              
  Lines       16775    16779       +4     
  Branches     2453     2452       -1     
==========================================
+ Hits        14974    14977       +3     
- Misses       1568     1569       +1     
  Partials      233      233              
Flag Coverage Δ
unittests 89.25% <100.00%> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
celery/worker/request.py 96.69% <ø> (ø)
celery/app/task.py 94.70% <100.00%> (ø)
celery/canvas.py 94.98% <100.00%> (+0.02%) ⬆️
celery/backends/asynchronous.py 66.66% <0.00%> (-0.44%) ⬇️

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 55c8ca1...78a19eb. Read the comment docs.

@auvipy auvipy self-requested a review April 14, 2022 13:17
@auvipy
Copy link
Copy Markdown
Member

auvipy commented Apr 14, 2022

#6649 related

@auvipy auvipy added the CI label Apr 14, 2022
@auvipy auvipy merged commit 9e324ca into celery:master Apr 14, 2022
@auvipy
Copy link
Copy Markdown
Member

auvipy commented Apr 14, 2022

merged as the redis integration tests passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants