Merge pull request #508 from ddobrinskiy/docs/test_update_docs

refactor `00_test.ipynb`

Author: jph00

fastcore/test.py

@@ -12,16 +12,16 @@
 
 # %% ../nbs/00_test.ipynb 6
 def test_fail(f, msg='', contains='', args=None, kwargs=None):
-    args, kwargs = args or [], kwargs or {}
     "Fails with `msg` unless `f()` raises an exception and (optionally) has `contains` in `e.args`"
+    args, kwargs = args or [], kwargs or {}
     try: f(*args, **kwargs)
     except Exception as e:
         assert not contains or contains in str(e)
         return
     assert False,f"Expected exception but none raised. {msg}"
 
 # %% ../nbs/00_test.ipynb 10
-def test(a, b, cmp,cname=None):
+def test(a, b, cmp, cname=None):
     "`assert` that `cmp(a,b)`; display inputs and `cname or cmp.__name__` if it fails"
     if cname is None: cname=cmp.__name__
     assert cmp(a,b),f"{cname}:\n{a}\n{b}"
@@ -34,7 +34,7 @@ def nequals(a,b):
 # %% ../nbs/00_test.ipynb 20
 def test_eq(a,b):
     "`test` that `a==b`"
-    test(a,b,equals, '==')
+    test(a,b,equals, cname='==')
 
 # %% ../nbs/00_test.ipynb 25
 def test_eq_type(a,b):
@@ -78,14 +78,14 @@ def test_stdout(f, exp, regex=False):
     "Test that `f` prints `exp` to stdout, optionally checking as `regex`"
     s = io.StringIO()
     with redirect_stdout(s): f()
-    if regex: assert re.search(exp, s.getvalue()) is not None
+    if regex: assert re.search(exp, s.getvalue()) is not None, f"regex '{exp}' did not not match stdout '{s.getvalue()}'"
     else: test_eq(s.getvalue(), f'{exp}\n' if len(exp) > 0 else '')
 
 # %% ../nbs/00_test.ipynb 40
 def test_warns(f, show=False):
     with warnings.catch_warnings(record=True) as w:
         f()
-        test_ne(len(w), 0)
+        assert w, "No warnings raised"
         if show:
             for e in w: print(f"{e.category}: {e.message}")