Skip to content

Adds missing timestep equality tests#3388

Merged
richardjgowers merged 2 commits intodevelopfrom
issue-3329
Aug 16, 2021
Merged

Adds missing timestep equality tests#3388
richardjgowers merged 2 commits intodevelopfrom
issue-3329

Conversation

@IAlibay
Copy link
Member

@IAlibay IAlibay commented Aug 16, 2021

Fixes #3329

Changes made in this Pull Request:

  • Adds a couple extra tests for ts.__eq__ coverage

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG updated?
  • Issue raised/referenced?

@codecov
Copy link

codecov bot commented Aug 16, 2021

Codecov Report

Merging #3388 (9024167) into develop (f8bb8ec) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3388      +/-   ##
===========================================
+ Coverage    93.65%   93.66%   +0.01%     
===========================================
  Files          177      177              
  Lines        22869    22869              
  Branches      3229     3229              
===========================================
+ Hits         21417    21421       +4     
+ Misses        1401     1397       -4     
  Partials        51       51              
Impacted Files Coverage Δ
package/MDAnalysis/coordinates/base.py 95.35% <0.00%> (+0.47%) ⬆️

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 f8bb8ec...9024167. Read the comment docs.

@richardjgowers richardjgowers merged commit 7ce532e into develop Aug 16, 2021
@richardjgowers richardjgowers deleted the issue-3329 branch August 16, 2021 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tests for Timestep's __eq__

2 participants