Add test for case of map_name

Author: mwcraig

src/astro_image_display_api/dummy_viewer.py

@@ -160,13 +160,13 @@ def colormap_options(self) -> list[str]:
     colormap_options.__doc__ = ImageViewerInterface.colormap_options.__doc__
 
     def set_colormap(self, map_name: str, image_label: str | None = None) -> None:
-        if map_name not in self.colormap_options:
+        if map_name.lower() not in self.colormap_options:
             raise ValueError(f"Invalid colormap '{map_name}'. Must be one of {self.colormap_options}.")
 
         image_label = self._resolve_image_label(image_label)
         if image_label not in self._images:
             raise ValueError(f"Image label '{image_label}' not found. Please load an image first.")
-        self._images[image_label].colormap = map_name
+        self._images[image_label].colormap = map_name.lower()
 
     set_colormap.__doc__ = ImageViewerInterface.set_colormap.__doc__
 

src/astro_image_display_api/widget_api_test.py

@@ -735,6 +735,19 @@ def test_set_get_colormap(self, data):
         self.image.set_colormap(new_cmap, image_label='test')
         assert self.image.get_colormap(image_label='test') == new_cmap
 
+    def test_set_get_colormap_map_name_case_insensitive(self, data):
+        # Check that the colormap can be set with a name that is not
+        # case-sensitive.
+        self.image.load_image(data, image_label='test')
+        cmap_desired = 'GrAy'
+        self.image.set_colormap(cmap_desired)
+        assert self.image.get_colormap() == cmap_desired.lower()
+
+        # Set the colormap with a different case
+        new_cmap = "Viridis"
+        self.image.set_colormap(new_cmap, image_label='test')
+        assert self.image.get_colormap(image_label='test') == 'viridis'
+
     def test_set_colormap_errors(self, data):
         # Check that setting a colormap raises an error if the colormap
         # is not in the list of allowed colormaps.