From 429989b13c65e705082d18837d717c7ac79e345c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20van=20Paassen?= <rene.vanpaassen@gmail.com>
Date: Mon, 11 May 2020 19:45:59 +0200
Subject: [PATCH] add more tests of the exception messages

---
 slycot/analysis.py             |  2 +-
 slycot/synthesis.py            |  2 +-
 slycot/tests/CMakeLists.txt    |  2 ++
 slycot/tests/test_analysis.py  | 44 ++++++++++++++++++++++++++++++++++
 slycot/tests/test_mb.py        | 17 ++++++++++++-
 slycot/tests/test_transform.py | 21 ++++++++++++++++
 6 files changed, 85 insertions(+), 3 deletions(-)
 create mode 100755 slycot/tests/test_analysis.py
 create mode 100755 slycot/tests/test_transform.py

diff --git a/slycot/analysis.py b/slycot/analysis.py
index aacc0c8f..4ae19b27 100644
--- a/slycot/analysis.py
+++ b/slycot/analysis.py
@@ -1620,7 +1620,7 @@ def ab13fd(n, A, tol = 0.0):
                 'dwork' + hidden, 'ldwork' + hidden, 'cwork' + hidden,
                 'lcwork' + hidden, 'info' + hidden]
     out = _wrapper.ab13fd(n, A, tol)
-    raise_if_slycot_error(out[-1], arg_list, ab13fd.__dict__)
+    raise_if_slycot_error(out[-1], arg_list, ab13fd.__doc__)
     return out[0], out[1]
 
 def ag08bd(l,n,m,p,A,E,B,C,D,equil='N',tol=0.0,ldwork=None):
diff --git a/slycot/synthesis.py b/slycot/synthesis.py
index f00281fd..cfbc6cba 100644
--- a/slycot/synthesis.py
+++ b/slycot/synthesis.py
@@ -1904,7 +1904,7 @@ def sg03ad(dico,job,fact,trans,uplo,N,A,E,Q,Z,X,ldwork=None):
 
     out = _wrapper.sg03ad(dico,job,fact,trans,uplo,N,A,E,Q,Z,X,ldwork)
 
-    raise_if_slycot_error(out[-1], arg_list, sg03ad.__dict__)
+    raise_if_slycot_error(out[-1], arg_list, sg03ad.__doc__)
 
     return out[:-1]
 
diff --git a/slycot/tests/CMakeLists.txt b/slycot/tests/CMakeLists.txt
index cdf3d65f..55aee8df 100644
--- a/slycot/tests/CMakeLists.txt
+++ b/slycot/tests/CMakeLists.txt
@@ -8,6 +8,8 @@ set(PYSOURCE
   test_mb.py
   test_mc.py
   test_sb.py
+  test_analysis.py
+  test_transform.py
   test_sg02ad.py
   test_sg03ad.py
   test_tb05ad.py
diff --git a/slycot/tests/test_analysis.py b/slycot/tests/test_analysis.py
new file mode 100755
index 00000000..f7577791
--- /dev/null
+++ b/slycot/tests/test_analysis.py
@@ -0,0 +1,44 @@
+#!/usr/bin/env python
+#
+# test_ab.py - generic tests for analysis programs
+# repagh <rene.vanpaassen@gmail.com, May 2020
+
+import pytest
+from .test_exceptions import assert_docstring_parse
+from slycot import analysis
+from slycot.exceptions import SlycotArithmeticError, SlycotResultWarning
+
+
+@pytest.mark.parametrize(
+    'fun,              exception_class,       erange,         checkvars',
+    ((analysis.ab05nd, SlycotArithmeticError, 1,              {'p1': 1}),
+     (analysis.ab07nd, SlycotResultWarning,   2,              {'m': 1}),
+     (analysis.ab09ad, SlycotArithmeticError, 3,              {'dico': 'C'}),
+     (analysis.ab09ad, SlycotArithmeticError, (2,),           {'dico': 'D'}),
+     (analysis.ab09ad, SlycotResultWarning,   ((1, 0), ),     {'nr': 3,
+                                                               'Nr': 2}),
+     (analysis.ab09ax, SlycotArithmeticError, 2,              {'dico': 'C'}),
+     (analysis.ab09ax, SlycotResultWarning,   ((1, 0), ),     {'nr': 3,
+                                                               'Nr': 2}),
+     (analysis.ab09ad, SlycotArithmeticError, 3,              {'dico': 'C'}),
+     (analysis.ab09ad, SlycotResultWarning,   ((1, 0), ),     {'nr': 3,
+                                                               'Nr': 2}),
+     (analysis.ab09md, SlycotArithmeticError, 3,              {'alpha': -0.1}),
+     (analysis.ab09md, SlycotResultWarning,   ((1, 0), (2, 0)), {'nr': 3,
+                                                               'Nr': 2,
+                                                               'alpha': -0.1}),
+     (analysis.ab09nd, SlycotArithmeticError, 3,              {'alpha': -0.1}),
+     (analysis.ab09nd, SlycotResultWarning,   ((1, 0), (2, 0)), {'nr': 3,
+                                                               'Nr': 2,
+                                                               'alpha': -0.1}),
+     (analysis.ab13bd, SlycotArithmeticError, 6,              {'dico': 'C'}),
+     (analysis.ab13bd, SlycotResultWarning,   ((1, 0),),      {}),
+     (analysis.ab13dd, SlycotArithmeticError, 4,              {}),
+     (analysis.ab13ed, SlycotArithmeticError, 1,              {}),
+     (analysis.ab13fd, SlycotArithmeticError, (2,),           {}),
+     (analysis.ab13fd, SlycotResultWarning,   (1,),           {})))
+def test_ab_docparse(fun, exception_class, erange, checkvars):
+    assert_docstring_parse(fun.__doc__,  exception_class, erange, checkvars)
+
+
+
diff --git a/slycot/tests/test_mb.py b/slycot/tests/test_mb.py
index 34ff79a1..f426dcc3 100755
--- a/slycot/tests/test_mb.py
+++ b/slycot/tests/test_mb.py
@@ -5,9 +5,12 @@
 
 import sys
 import unittest
+import pytest
 
+from slycot import math
 from slycot import mb03rd, mb03vd, mb03vy, mb03wd, mb05md, mb05nd
-from slycot.exceptions import SlycotResultWarning
+from slycot.exceptions import SlycotResultWarning, SlycotArithmeticError
+from .test_exceptions import assert_docstring_parse
 
 import numpy as np
 from scipy.linalg import schur
@@ -306,5 +309,17 @@ def test_mb05nd(self):
         assert_allclose(H, H_ref, atol=0.0001)
 
 
+@pytest.mark.parametrize(
+    'fun,          exception_class,       erange,         checkvars',
+    ((math.mb03wd, SlycotResultWarning,   (4,),           {'ilo': 2,
+                                                           'ihi': 5}),
+     (math.mb05md, SlycotResultWarning,   (2, 3, 4),      {'n': 2}),
+     (math.mb05nd, SlycotArithmeticError, (2, 3),         {'n': 2,
+                                                           'delta': 1000}),
+     (math.mc01td, SlycotResultWarning,   (1, 2, (1, 0)), {'dico': 'C'})))
+def test_mb_docparse(fun, exception_class, erange, checkvars):
+    assert_docstring_parse(fun.__doc__,  exception_class, erange, checkvars)
+
+
 if __name__ == "__main__":
     unittest.main()
diff --git a/slycot/tests/test_transform.py b/slycot/tests/test_transform.py
new file mode 100755
index 00000000..f8191140
--- /dev/null
+++ b/slycot/tests/test_transform.py
@@ -0,0 +1,21 @@
+#!/usr/bin/env python
+#
+# test_transform.py - generic tests for transform routines
+# repagh <rene.vanpaassen@gmail.com, May 2020
+
+import pytest
+from .test_exceptions import assert_docstring_parse
+from slycot import transform as tf
+from slycot.exceptions import SlycotArithmeticError
+
+
+@pytest.mark.parametrize(
+    'fun,        exception_class,       erange,         checkvars',
+    ((tf.tb03ad, SlycotArithmeticError, 2,              {}),
+     (tf.tb05ad, SlycotArithmeticError, 2,              {'n30': 90,
+                                                         'jomega': 2.0,
+                                                         'rcond': 1e-12}),
+     (tf.td04ad, SlycotArithmeticError, (3,),           {}),
+     (tf.tc04ad, SlycotArithmeticError, 1,              {'leri': 'L'})))
+def test_transform_docparse(fun, exception_class, erange, checkvars):
+    assert_docstring_parse(fun.__doc__,  exception_class, erange, checkvars)