Skip to content

Fix mock assertions #1730

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.

Sign up for GitHub

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

Jump to bottom
Merged
ajdawson merged 3 commits into from
Jul 21, 2015
Merged

Fix mock assertions #1730

ajdawson merged 3 commits into from
Jul 21, 2015

Conversation

@rhattersley
Copy link
Member

@rhattersley rhattersley commented Jul 21, 2015

Fixes #1724.

@ajdawson
Copy link
Member

ajdawson commented Jul 21, 2015

Missed one in iris.tests.unit.fileformats.pp.test_PPField.Test_save.test_float64 by the looks of it, and introduced a new issue with the renaming of CurvilinearRegridder...

@rhattersley rhattersley mentioned this pull request Jul 21, 2015
Closed
@rhattersley
Copy link
Member Author

rhattersley commented Jul 21, 2015

Missed one in iris.tests.unit.fileformats.pp.test_PPField.Test_save.test_float64 by the looks of it, and introduced a new issue with the renaming of CurvilinearRegridder...

Thanks @ajdawson - fixed now.

@cpelley
Copy link

cpelley commented Jul 21, 2015

Cheers both, I'll rebase my PR's once this goes in :)

ajdawson added a commit that referenced this pull request Jul 21, 2015
@ajdawson
Merge pull request #1730 from rhattersley/mock-assertions
ddd6add 
Fix mock assertions
@ajdawson ajdawson merged commit ddd6add into SciTools:master Jul 21, 2015
@ajdawson ajdawson mentioned this pull request Aug 27, 2015
Merged
@rhattersley rhattersley deleted the mock-assertions branch April 28, 2016 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rhattersley @ajdawson @cpelley