Skip to content

Commit 97b20aa

Browse files
committed
Review change: correct test comment.
1 parent 794cab5 commit 97b20aa

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

lib/iris/tests/unit/fileformats/nc_load_rules/actions/test__time_coords.py

+10-6
Original file line numberDiff line numberDiff line change
@@ -365,12 +365,16 @@ def test_aux_not_in_phenom_coords(self):
365365

366366
def test_aux_fails_typeident(self):
367367
# We provide a non-dimension coord variable, identified as a
368-
# CFAuxiliaryCoordinate by cf.py, but we also give it the wrong units,
369-
# which causes it to fail both 'is_time' and 'is_period' tests, so the
370-
# 'provides_coord' rule fails to trigger.
371-
# As in 'test_dim_fails_typeident', the coord is built as a
372-
# 'miscellaneous' rather than a specific coord type (recognised from its
373-
# name), but this makes absolutely no practical difference.
368+
# CFAuxiliaryCoordinate by cf.py, but we also give it "wrong" units,
369+
# unsuitable for a time or period coord.
370+
# Because it fails both 'is_time' and 'is_period' tests, it then does
371+
# not trigger 'fc_build_auxiliary_coordinate_time[[_period]]'.
372+
# As in the above testcase 'test_dim_fails_typeident', the routine
373+
# 'action_build_auxiliary_coordinate' therefore builds this as a
374+
# 'miscellaneous' rather than a specific coord type (time or period).
375+
# However, also as in that other case, this makes absolutely no
376+
# practical difference -- unlike for latitude or longitutude coords,
377+
# where it may affect the standard-name.
374378
#
375379
# Rules Triggered:
376380
# 001 : fc_default

0 commit comments

Comments
 (0)