Skip to content

Conversation

@blueyed
Copy link
Contributor

@blueyed blueyed commented Oct 10, 2018

`expect()` expects an regular expression, so "Pdb" is equivalent to
"(Pdb)".

But instead of escaping the parenthesis this patch removes them, to
allow for matching "(Pdb++)", too.
With pdb++ this additional `expect` is required, otherwise `sendeof()`
will block forever.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.009%) to 93.801% when pulling be511c1 on blueyed:pdb into 4b164d9 on pytest-dev:master.

@codecov
Copy link

codecov bot commented Oct 11, 2018

Codecov Report

Merging #4110 into master will increase coverage by 0.03%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4110      +/-   ##
==========================================
+ Coverage    94.5%   94.53%   +0.03%     
==========================================
  Files         109      109              
  Lines       23810    23813       +3     
  Branches     2349     2349              
==========================================
+ Hits        22502    22512      +10     
+ Misses       1000      997       -3     
+ Partials      308      304       -4
Flag Coverage Δ
#doctesting 29.24% <0%> (+0.83%) ⬆️
#linux 94.38% <0%> (+0.03%) ⬆️
#nobyte 0% <0%> (ø) ⬆️
#numpy 28.15% <0%> (+0.06%) ⬆️
#pexpect 0% <0%> (ø) ⬆️
#py27 92.66% <0%> (+0.03%) ⬆️
#py34 92.11% <0%> (+0.02%) ⬆️
#py35 92.12% <0%> (+0.02%) ⬆️
#py36 92.69% <0%> (+0.03%) ⬆️
#py37 92.33% <0%> (+0.02%) ⬆️
#trial 31.38% <0%> (+0.06%) ⬆️
#windows 93.84% <0%> (+0.03%) ⬆️
#xdist 18.57% <0%> (+0.01%) ⬆️
Impacted Files Coverage Δ
testing/test_pdb.py 52.7% <0%> (-0.44%) ⬇️
src/_pytest/terminal.py 91.62% <0%> (+1.74%) ⬆️

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 4b164d9...be511c1. Read the comment docs.

@nicoddemus nicoddemus merged commit e20987c into pytest-dev:master Oct 11, 2018
@blueyed blueyed deleted the pdb branch October 11, 2018 11:54
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