Rename helper method to be more explicit

Author: mwcraig

src/astro_image_display_api/widget_api_test.py

@@ -49,7 +49,7 @@ class variable does the trick.
         """
         self.image = self.image_widget_class(image_width=250, image_height=100)
 
-    def _check_marker_table_return_properties(self, table):
+    def _check_empty_marker_table_return_properties(self, table):
         assert isinstance(table, Table)
         assert len(table) == 0
         assert sorted(table.colnames) == sorted(['x', 'y', 'coord', 'marker name'])
@@ -119,7 +119,7 @@ def test_zoom(self):
 
     def test_marking_operations(self):
         marks = self.image.get_markers(marker_name="all")
-        self._check_marker_table_return_properties(marks)
+        self._check_empty_marker_table_return_properties(marks)
         assert not self.image.is_marking
 
         # Ensure you cannot set it like this.
@@ -160,7 +160,7 @@ def test_marking_operations(self):
             warnings.simplefilter("error")
             t = self.image.get_markers(marker_name='markymark')
 
-        self._check_marker_table_return_properties(t)
+        self._check_empty_marker_table_return_properties(t)
 
         self.image.click_drag = True
         self.image.start_marking()
@@ -177,7 +177,7 @@ def test_marking_operations(self):
         self.image.stop_marking(clear_markers=True)
 
         assert self.image.is_marking is False
-        self._check_marker_table_return_properties(self.image.get_markers(marker_name="all"))
+        self._check_empty_marker_table_return_properties(self.image.get_markers(marker_name="all"))
 
         # Hate this, should add to public API
         marknames = self.image._marktags
@@ -250,15 +250,15 @@ def test_add_markers(self):
         self.image.reset_markers()
         marknames = self.image._marktags
         assert len(marknames) == 0
-        self._check_marker_table_return_properties(self.image.get_markers(marker_name="all"))
+        self._check_empty_marker_table_return_properties(self.image.get_markers(marker_name="all"))
         # Check that no markers remain after clearing
-        tab = self.image.get_markers(marker_name=self.image._default_mark_tag_name)
-        self._check_marker_table_return_properties(tab)
+        tab = self.image.get_markers(marker_name=self.image.DEFAULT_MARKER_NAME)
+        self._check_empty_marker_table_return_properties(tab)
 
         # Check that retrieving a marker set that doesn't exist returns
         # an empty table with the right columns
         tab = self.image.get_markers(marker_name='test1')
-        self._check_marker_table_return_properties(tab)
+        self._check_empty_marker_table_return_properties(tab)
 
     def test_get_markers_accepts_list_of_names(self):
         # Check that the get_markers method accepts a list of marker names
@@ -288,7 +288,7 @@ def test_remove_markers_name_all(self):
         self.image.add_markers(tab, marker_name='test2')
 
         self.image.remove_markers(marker_name='all')
-        self._check_marker_table_return_properties(self.image.get_markers(marker_name='all'))
+        self._check_empty_marker_table_return_properties(self.image.get_markers(marker_name='all'))
 
     def test_remove_marker_accepts_list(self):
         data = np.arange(10).reshape(5, 2)
@@ -298,7 +298,7 @@ def test_remove_marker_accepts_list(self):
 
         self.image.remove_markers(marker_name=['test1', 'test2'])
         marks = self.image.get_markers(marker_name='all')
-        self._check_marker_table_return_properties(marks)
+        self._check_empty_marker_table_return_properties(marks)
 
     def test_adding_markers_as_world(self, data, wcs):
         ndd = NDData(data=data, wcs=wcs)