add period for clarity

chanzuckerberg · Oct 12, 2023 · c7b0f62 · c7b0f62
1 parent 2168d6a
commit c7b0f62
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cellxgene_schema_cli/cellxgene_schema/validate.py b/cellxgene_schema_cli/cellxgene_schema/validate.py
@@ -755,7 +755,7 @@ def _validate_colors_in_uns_dict(self, uns_dict: dict) -> None:
                 if not obs_unique_values:
                     error_message = f"Colors field uns[{key}] does not have a corresponding categorical field in obs"
                     if column_name in df.columns:
-                        error_message += f" {column_name} is present but is dtype {df[column_name].dtype.name}"
+                        error_message += f". {column_name} is present but is dtype {df[column_name].dtype.name}"
                     self.errors.append(error_message)
                     continue
                 # 2. Verify that the value is a numpy array

diff --git a/cellxgene_schema_cli/tests/test_schema_compliance.py b/cellxgene_schema_cli/tests/test_schema_compliance.py
@@ -1600,7 +1600,7 @@ def test_colors_bool_obs_counterpart(self, validator_with_adata):
         validator.adata.uns["is_primary_data_colors"] = numpy.array(["green", "purple"])
         validator.validate_adata()
         assert validator.errors == [
-            "ERROR: Colors field uns[is_primary_data_colors] does not have a corresponding categorical field in obs is_primary_data is present but is dtype bool"
+            "ERROR: Colors field uns[is_primary_data_colors] does not have a corresponding categorical field in obs. is_primary_data is present but is dtype bool"
         ]
 
     def test_colors_without_obs_counterpart(self, validator_with_adata):