From ca75b995ff20bb9049d912eae403526ffa2c0d59 Mon Sep 17 00:00:00 2001 From: Anton Karpov Date: Tue, 4 Aug 2026 18:12:34 +0300 Subject: [PATCH] Add the two missing asserts in test_columnaliases The two statements were evaluated and discarded, so the test did not check what its name says. The behaviour is already correct, this is about the test guarding it. Verified both ways: the asserts pass as written, and flipping one to 'is not None' makes the test fail, so they are now comparing something. --- pyvo/dal/tests/test_query.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pyvo/dal/tests/test_query.py b/pyvo/dal/tests/test_query.py index 4ad30282..6b69e58b 100644 --- a/pyvo/dal/tests/test_query.py +++ b/pyvo/dal/tests/test_query.py @@ -583,8 +583,8 @@ def test_columnaliases(self): truth = 'Illuminatus' assert record.getbyutype('foobar') == truth - record.getbyucd('baz') is None - record.getbyutype('foobaz') is None + assert record.getbyucd('baz') is None + assert record.getbyutype('foobaz') is None def test_datasets(self): records = DALResults.from_result_url(