Skip to content

Commit b01d937

Browse files
committed
Rename method so copilot likes it
1 parent 804af60 commit b01d937

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

src/astro_image_display_api/widget_api_test.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ class variable does the trick.
4949
"""
5050
self.image = self.image_widget_class(image_width=250, image_height=100)
5151

52-
def _check_empty_marker_table_return_properties(self, table):
52+
def _assert_empty_marker_table(self, table):
5353
assert isinstance(table, Table)
5454
assert len(table) == 0
5555
assert sorted(table.colnames) == sorted(['x', 'y', 'coord', 'marker name'])
@@ -127,7 +127,7 @@ def test_zoom(self):
127127

128128
def test_marking_operations(self):
129129
marks = self.image.get_markers(marker_name="all")
130-
self._check_empty_marker_table_return_properties(marks)
130+
self._assert_empty_marker_table(marks)
131131
assert not self.image.is_marking
132132

133133
# Ensure you cannot set it like this.
@@ -169,7 +169,7 @@ def test_marking_operations(self):
169169
warnings.simplefilter("error")
170170
t = self.image.get_markers(marker_name='markymark')
171171

172-
self._check_empty_marker_table_return_properties(t)
172+
self._assert_empty_marker_table(t)
173173

174174
self.image.click_drag = True
175175
self.image.start_marking()
@@ -186,7 +186,7 @@ def test_marking_operations(self):
186186
self.image.stop_marking(clear_markers=True)
187187

188188
assert self.image.is_marking is False
189-
self._check_empty_marker_table_return_properties(self.image.get_markers(marker_name="all"))
189+
self._assert_empty_marker_table(self.image.get_markers(marker_name="all"))
190190

191191
# Hate this, should add to public API
192192
marknames = self._get_marker_names_as_set()
@@ -258,15 +258,15 @@ def test_add_markers(self):
258258
self.image.reset_markers()
259259
marknames = self._get_marker_names_as_set()
260260
assert len(marknames) == 0
261-
self._check_empty_marker_table_return_properties(self.image.get_markers(marker_name="all"))
261+
self._assert_empty_marker_table(self.image.get_markers(marker_name="all"))
262262
# Check that no markers remain after clearing
263263
tab = self.image.get_markers(marker_name=self.image.DEFAULT_MARKER_NAME)
264-
self._check_empty_marker_table_return_properties(tab)
264+
self._assert_empty_marker_table(tab)
265265

266266
# Check that retrieving a marker set that doesn't exist returns
267267
# an empty table with the right columns
268268
tab = self.image.get_markers(marker_name='test1')
269-
self._check_empty_marker_table_return_properties(tab)
269+
self._assert_empty_marker_table(tab)
270270

271271
def test_get_markers_accepts_list_of_names(self):
272272
# Check that the get_markers method accepts a list of marker names
@@ -296,7 +296,7 @@ def test_remove_markers_name_all(self):
296296
self.image.add_markers(tab, marker_name='test2')
297297

298298
self.image.remove_markers(marker_name='all')
299-
self._check_empty_marker_table_return_properties(self.image.get_markers(marker_name='all'))
299+
self._assert_empty_marker_table(self.image.get_markers(marker_name='all'))
300300

301301
def test_remove_marker_accepts_list(self):
302302
data = np.arange(10).reshape(5, 2)
@@ -306,7 +306,7 @@ def test_remove_marker_accepts_list(self):
306306

307307
self.image.remove_markers(marker_name=['test1', 'test2'])
308308
marks = self.image.get_markers(marker_name='all')
309-
self._check_empty_marker_table_return_properties(marks)
309+
self._assert_empty_marker_table(marks)
310310

311311
def test_adding_markers_as_world(self, data, wcs):
312312
ndd = NDData(data=data, wcs=wcs)

0 commit comments

Comments
 (0)