Xarray + numpy reshape tests

[edoyango]

This brings xarray and numpy reshape tests to 100%. There were a couple if statements that were always true, so I set them as no cover.

[coveralls]

Pull Request Test Coverage Report for Build 20304891244

Details

• 54 of 56 (96.43%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.2%) to 64.983%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/pipeline/tests/operations/numpy/test_numpy_reshape.py	43	45	95.56%

Totals
Change from base Build 20304024396:	0.2%
Covered Lines:	10480
Relevant Lines:	15767

---

💛 - Coveralls

[edoyango]

hi @gemmaellen I'm adding you as a reviewer as you worked on the xarray reshape code that some of these tests are for. Some input on the applying encoders could would be helpful. Seems like the code never gets run since dataset attributes are always overwritten!

[tennlee]

Sorry for the delay here ... a few things at work meant a building of reviews, and I had some difficult refactoring to do on "scores". I'll get to this next week, maybe earlier.

[tennlee]

Thanks. Great to get the increase in test coverage. At some point it might be nice to go through the nocover areas and either remove the redundant code or figure out the requirement and developer intent to see if there is an additional case that needs testing, but getting coverage up with reasonable tests is what's important.