switch order of setting ymin and ymax (#3371)

* switch order of setting ymin and ymax

* change log

* .

* remove test because it requires image to be rendered

CHANGES.rst

@@ -83,6 +83,8 @@ Bug Fixes
 
 - Fixed bug with Plot Options select_all when data is float32. [#3366]
 
+- Fixed an issue with back-to-back calls of set_limits and get_limits. [#3371]
+
 
 Cubeviz
 ^^^^^^^

jdaviz/configs/default/plugins/viewers.py

@@ -162,10 +162,13 @@ def set_limits(self, x_min=None, x_max=None, y_min=None, y_max=None):
                 self.state.x_min = x_min
             if x_max is not None:
                 self.state.x_max = x_max
-            if y_min is not None:
-                self.state.y_min = y_min
+            # NOTE: for some reason, setting ymax first avoids an issue
+            # where back-to-back calls of get_limits and set_limits
+            # give different results for y limits.
             if y_max is not None:
                 self.state.y_max = y_max
+            if y_min is not None:
+                self.state.y_min = y_min
 
     def get_limits(self):
         """Return current viewer axes limits.