casa / casa6 / 2ca402c80bf

Commits

David Mehringer authored and Takeshi Nakazato committed 2ca402c80bf Merge 06 Sep 2022

Pull request #625: CAS-13654 add more useful error message

Merge in CASA/casa6 from CAS-13654 to master

* commit 'e753e56b4b4f1e157b65f4699eb4c0f2d40dca7f':
  Added a missing * to identify the bullet element.
  add another hdkey value to test for verfication of this issue
  fix error message
  fix test
  test more useful error message
  CAS-13654 add more useful error message

casatasks/tests/tasks/test_task_imhead.py

Modified

-423
+                try:
-424
+                    self.assertEqual(np.all(x, exp_dict[k]), msg)
-425
+                except:
-426
+                    # string arrays can throw "Cannot reduce flexible type" exceptions
-427
+                    self.assertEqual(list(x), list(exp_dict[k]), msg)
-428
+            else:
-429
+                self.assertEqual(x, exp_dict[k], msg)
-430
+        # test bogus key fails
-431
+        k = 'boguskey'
-432
+        self.assertRaises(
--
+                Exception, imhead, datacopy, mode='get', hdkey=k,
--
+                msg='Incorrectly found bogus key'
-+
+            Exception, imhead, datacopy, mode='get', hdkey=k,
-+
+            msg='Incorrectly found bogus key'
-435
+        )
-+
+        # CAS-13654
-+
+        for k in ('incr', 'axisnames'):
-+
+            with self.assertRaises(RuntimeError) as cm:
-+
+                imhead(datacopy, mode='get', hdkey=k)
-+
+            exc = cm.exception
-+
+            expected_msg = (
-+
+                f'Unknown keyword {k}. If you are trying to use a key name '
-+
+                'from the imhead summary dictionary, note that some keys in '
-+
+                'mode=\'put\'/\'get\' are different from mode=\'summary\'. '
-+
+                'Please see imhead description in the CASA online '
-+
+                'documentation for complete details.'
-+
+            )
-+
+            pos = str(exc).find(expected_msg)
-+
+            self.assertNotEqual(
-+
+                pos, -1, msg=f'Unexpected exception was thrown: {exc}'
-+
+            )
-+
-453
-454
+    def test_summary(self):
-455
+        '''
-456
+            test_summary
-457
+            -----------------
-458
+        '''
-459
+        ###NOTE: to test the function of the verbose parameter an image with multiple beams is required
-460
+        summaryDictnoV = {
-461
+            'axisnames': np.array(['Right Ascension', 'Declination', 'Stokes', 'Frequency'], dtype='<U16'),
-462
+            'axisunits': np.array(['rad', 'rad', '', 'Hz'], dtype='<U16'), 'defaultmask': '',

casa6

Commits

Jira Issues

Add shortcut