tests passing maybe

Author: hjwp

chapter_20_fixtures_and_wait_decorator.asciidoc

@@ -43,33 +43,36 @@ so this isn't an unrealistic cheat at all.
 Here's how you can set it up:
 
 [role="sourcecode"]
-.src/functional_tests/test_my_lists.py
+.src/functional_tests/test_my_lists.py (ch20l001)
 ====
 [source,python]
 ----
 from django.conf import settings
 from django.contrib.auth import BACKEND_SESSION_KEY, SESSION_KEY, get_user_model
 from django.contrib.sessions.backends.db import SessionStore
+
 from .base import FunctionalTest
+
 User = get_user_model()
 
 
 class MyListsTest(FunctionalTest):
-
     def create_pre_authenticated_session(self, email):
         user = User.objects.create(email=email)
         session = SessionStore()
-        session[SESSION_KEY] = user.pk #<1>
+        session[SESSION_KEY] = user.pk # <1>
         session[BACKEND_SESSION_KEY] = settings.AUTHENTICATION_BACKENDS[0]
         session.save()
         ## to set a cookie we need to first visit the domain.
         ## 404 pages load the quickest!
         self.browser.get(self.live_server_url + "/404_no_such_url/")
-        self.browser.add_cookie(dict(
-            name=settings.SESSION_COOKIE_NAME,
-            value=session.session_key, #<2>
-            path='/',
-        ))
+        self.browser.add_cookie(
+            dict(
+                name=settings.SESSION_COOKIE_NAME,
+                value=session.session_key,  # <2>
+                path="/",
+            )
+        )
 ----
 ====
 
@@ -260,7 +263,7 @@ a good place for a commit:
 
 [subs="specialcharacters,quotes"]
 ----
-$ *git add functional_tests*
+$ *git add src/functional_tests*
 $ *git commit -m "test_my_lists: precreate sessions, move login checks into base"*
 ----
 
@@ -305,8 +308,14 @@ TIP: Once you have more than a handful of fields on a model, and/or several
 Our Final Explicit Wait Helper: A Wait Decorator
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-((("decorators", "wait decorator", id="Dwait20")))((("explicit and implicit waits", id="exp20")))((("implicit and explicit waits", id="imp20")))((("helper methods", id="help20")))((("wait_for_row_in_list_table helper method")))((("self.wait_for helper method")))((("wait_to_be_logged_in/out")))We've
-used decorators a few times in our code so far, but it's time to learn
+((("decorators", "wait decorator", id="Dwait20")))
+((("explicit and implicit waits", id="exp20")))
+((("implicit and explicit waits", id="imp20")))
+((("helper methods", id="help20")))
+((("wait_for_row_in_list_table helper method")))
+((("self.wait_for helper method")))
+((("wait_to_be_logged_in/out")))
+We've used decorators a few times in our code so far, but it's time to learn
 how they actually work by making one of our own.
 
 First, let's imagine how we might want our decorator to work.  It would be
@@ -398,7 +407,8 @@ That's 'almost' right, but not quite;  try running it?
 $ pass:quotes[*python src/manage.py test functional_tests.test_my_lists*]
 [...]
     self.wait_to_be_logged_out(email)
-TypeError: modified_fn() takes 0 positional arguments but 2 were given
+TypeError: wait.<locals>.modified_fn() takes 0 positional arguments but 2 were
+given
 ----
 
 
@@ -414,7 +424,7 @@ that have [keep-together]#arguments#:
 ----
     @wait
     def wait_to_be_logged_in(self, email):
-        self.browser.find_element(By.LINK_TEXT, "Log out").click()
+        self.browser.find_element(By.LINK_TEXT, "Log out")
 ----
 ====
 

source/chapter_20_fixtures_and_wait_decorator/superlists

@@ -353,7 +353,7 @@ def prep_database(self):
 
     def assertLineIn(self, line, lines):
         if "\t" in line or "\t" in "\n".join(lines):
-            print('tabz')
+            print("tabz")
         if line not in lines:
             raise AssertionError(
                 "%s not found in:\n%s" % (repr(line), "\n".join(repr(l) for l in lines))
@@ -537,7 +537,8 @@ def run_test_and_check_result(self, bdd=False):
     def run_js_tests(self, tests_path: Path):
         p = subprocess.run(
             ["python", str(JASMINE_RUNNER), str(tests_path)],
-            stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+            stdout=subprocess.PIPE,
+            stderr=subprocess.STDOUT,
             # env={**os.environ, "OPENSSL_CONF": "/dev/null"},
             check=False,
         )
@@ -556,7 +557,12 @@ def check_current_contents(self, listing, actual_contents):
         for block in split_blocks(listing_contents):
             stripped_block = [line.strip() for line in block.strip().split("\n")]
             for line in stripped_block:
-                self.assertIn(line, stripped_actual_lines)
+                self.assertIn(
+                    line,
+                    stripped_actual_lines,
+                    f"{line!r} not found in\n"
+                    + "\n".join(repr(l) for l in stripped_actual_lines),
+                )
             self.assertTrue(
                 contains(stripped_actual_lines, stripped_block),
                 "\n{}\n\nnot found in\n\n{}".format(

tests/book_tester.py

@@ -13,7 +13,7 @@ def test_listings_and_commands_and_output(self):
         self.parse_listings()
 
         # sanity checks
-        self.assertEqual(self.listings[0].type, "code listing")
+        self.assertEqual(self.listings[0].type, "code listing with git ref")
         self.assertEqual(self.listings[1].type, "other command")
         self.assertEqual(self.listings[2].type, "output")