Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-30514: test_pop3lib asyncore with asyncio #1864

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

grzgrzgrz3
Copy link
Contributor

@grzgrzgrz3 grzgrzgrz3 commented May 30, 2017

Pop3 server stub was base on asyncore/asynchat which is deprecated, new
stub is using asyncio instead.

test_too_long_lines make sure we read remaining overlong response

cmd_user answer only with +ERR message on invalid user

#74699

Pop3 server stub was base on asyncore/asynchat which is deprecated, new
stub is using asyncio instead.

test_too_long_lines make sure we read remaining overlong response

cmd_user answer only with +ERR message on invalid user
@mention-bot
Copy link

@mention-bot mention-bot commented May 30, 2017

@grzgrzgrz3, thanks for your PR! By analyzing the history of the files in this pull request, we identified @tiran, @serhiy-storchaka and @giampaolo to be potential reviewers.

@codecov
Copy link

@codecov codecov bot commented May 30, 2017

Codecov Report

No coverage uploaded for pull request base (master@9c972b5). Click here to learn what that means.
The diff coverage is 97.67%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1864   +/-   ##
=========================================
  Coverage          ?   82.68%           
=========================================
  Files             ?     1432           
  Lines             ?   353405           
  Branches          ?        0           
=========================================
  Hits              ?   292229           
  Misses            ?    61176           
  Partials          ?        0
Impacted Files Coverage Δ
Lib/test/test_poplib.py 98.13% <97.67%> (ø)

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 9c972b5...996e0cc. Read the comment docs.

@serhiy-storchaka serhiy-storchaka added the tests label Dec 8, 2018
@matrixise matrixise self-assigned this Sep 12, 2019
Copy link

@auvipy auvipy left a comment

a rebase on top of main branch would be a good first start

@github-actions
Copy link

@github-actions github-actions bot commented Feb 19, 2022

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Feb 19, 2022
@grzgrzgrz3 grzgrzgrz3 mannequin mentioned this pull request Apr 11, 2022
@JelleZijlstra JelleZijlstra self-assigned this May 10, 2022
@JelleZijlstra
Copy link
Member

@JelleZijlstra JelleZijlstra commented May 10, 2022

I'll work on getting this up to date so it unblocks removing asyncore. Note to self: local branch name is bpo30514-grzegorz-grzywacz-poplib.

@github-actions github-actions bot removed the stale label May 11, 2022
@AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Jun 7, 2022

Triggering CLA.

@AA-Turner AA-Turner closed this Jun 7, 2022
@AA-Turner AA-Turner reopened this Jun 7, 2022
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.

None yet