Unpin pytest

[alimcmaster1]

Follow up from #26619. Since pytest have just done a point release for 4.6.1 we should be good to remove the pin.

pytest-dev/pytest#5358

cc @simonjayhawkins

[codecov]

Codecov Report

Merging #26625 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #26625      +/-   ##
==========================================
- Coverage   91.87%   91.87%   -0.01%     
==========================================
  Files         174      174              
  Lines       50692    50692              
==========================================
- Hits        46575    46571       -4     
- Misses       4117     4121       +4

Flag	Coverage Δ
#multiple	90.41% <ø> (ø)	⬆️
#single	41.78% <ø> (-0.09%)	⬇️

Impacted Files	Coverage Δ
pandas/io/gbq.py	78.94% <0%> (-10.53%)	⬇️
pandas/core/frame.py	97% <0%> (-0.12%)	⬇️

---

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 6354580...d2beb74. Read the comment docs.

[codecov]

Codecov Report

Merging #26625 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #26625      +/-   ##
==========================================
- Coverage   91.88%   91.87%   -0.01%     
==========================================
  Files         174      174              
  Lines       50692    50692              
==========================================
- Hits        46576    46572       -4     
- Misses       4116     4120       +4

Flag	Coverage Δ
#multiple	90.41% <ø> (ø)	⬆️
#single	41.8% <ø> (-0.09%)	⬇️

Impacted Files	Coverage Δ
pandas/io/gbq.py	78.94% <0%> (-10.53%)	⬇️
pandas/core/frame.py	97% <0%> (-0.12%)	⬇️

---

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 6354580...08c8159. Read the comment docs.

[alimcmaster1]

Interestingly I think there still might be an issue with the pytest release based on:

   def manual_compare_stacked(df, df_stacked, lev0, lev1):
>       assert all(df.loc[row, col] ==
                   df_stacked.loc[(row, col[lev0]), col[lev1]]
                   for row in df.index for col in df.columns)
E       NameError: name 'col' is not defined

will take a further look tomorrow

[alimcmaster1]

bumped up to latest pytest release which should fix this - CI running..

[gfyoung]

@jreback : How does it work when it comes to developer dependencies? Should we be bumping version requirements this high to be the latest version?

[jreback]

doesn’t make sense to bump this - we try to move very slowly in things like thus

[alimcmaster1]

Given that pandas tests will fail running with pytest version 4.6.0 and 4.6.1 doesn’t it make sense to enforce this in the dependencies? ( yes we could do this by keeping the current pin in place ).

But keeping it pinned means nothing in CI will draw our attention to any future breaking pytest changes? Potentially making it harder to update further down the road. What do you think ?

[jreback]

these are 3.5 builds only - pinning is ok

to be honest could pin them to a much earlier version
say 4.02 would be good (or for that would be ok)

all other builds float to the latest