o Use grid.dims and create a new function test_ugrid_compliance

Author: Rajeev Jain

test/test_io_common.py

@@ -86,21 +86,23 @@ def test_return_type(self, grid_from_format):
 
         # Basic validation
         assert isinstance(grid, ux.Grid)
+
+    def test_ugrid_compliance(self, grid_from_format):
+        """Test that grids from all formats meet basic UGRID standards."""
+        grid = grid_from_format
+
+        # Basic topology and coordinate presence
         assert 'face_node_connectivity' in grid.connectivity
         assert 'node_lon' in grid.coordinates
         assert 'node_lat' in grid.coordinates
 
-        # Check required dimensions
+        # Required dimensions
         assert 'n_node' in grid.dims
         assert 'n_face' in grid.dims
 
-        # Validate grid
+        # Validate grid structure
         grid.validate()
 
-    def test_ugrid_compliance(self, grid_from_format):
-        """Test that grids from all formats meet basic UGRID standards."""
-        grid = grid_from_format
-
         # Check UGRID compliance
         # 1. Connectivity should use proper fill values
         assert grid.face_node_connectivity._FillValue == INT_FILL_VALUE
@@ -120,8 +122,8 @@ def test_grid_properties_consistency(self, grid_from_format):
         grid = grid_from_format
 
         # Check that all grids have the essential properties
-        assert hasattr(grid, 'n_node')
-        assert hasattr(grid, 'n_face')
+        assert 'n_node' in grid.dims
+        assert 'n_face' in grid.dims
         assert 'face_node_connectivity' in grid.connectivity
         assert 'node_lon' in grid.coordinates
         assert 'node_lat' in grid.coordinates