tweaked a condition so an if statement could be reached, and added a test for value0_nonnans array testing. hopefully coverage will increase

Author: chrispycheng

nibabel/cmdline/diff.py

@@ -93,7 +93,7 @@ def are_values_different(*values):
                 raise
 
     for value in values[1:]:
-        if type(value0) != type(value):  # if types are different, then we consider them different
+        if value0 is not np.nan and type(value0) != type(value):  # if types different, then they different
             return True
         elif isinstance(value0, np.ndarray):
             if value0.dtype != value.dtype or \

nibabel/tests/test_diff.py

@@ -12,8 +12,6 @@
 
 from nibabel.cmdline.diff import are_values_different
 
-# TODO: MAJOR TO DO IS TO FIGURE OUT HOW TO USE HYPOTHESIS FOR LONGER LIST LENGTHS WHILE STILL CONTROLLING FOR OUTCOMES
-
 
 def test_diff_values_int():
     large = 10**30
@@ -68,6 +66,7 @@ def test_diff_values_array():
     assert not are_values_different(array([nan, 1]), array([nan, 1]))
     assert are_values_different(array([nan, nan]), array([nan, 1]))
     assert are_values_different(array([0, nan]), array([nan, 0]))
+    assert are_values_different(array([1, 2, 3, nan]), array([nan, 3, 5, 4]))
     # and some inf should not be a problem
     assert not are_values_different(array([0, inf]), array([0, inf]))
     assert are_values_different(array([0, inf]), array([inf, 0]))