start working on tests

Author: hjwp

chapter_22_outside_in.asciidoc

@@ -11,9 +11,6 @@ some things may not be quite right yet.
 *******************************************************************************
 
 
-
-
-
 ((("Test-Driven Development (TDD)", "outside-in technique", id="TTDoutside22")))
 In this chapter I'd like to talk about a technique called Outside-In TDD.
 It's pretty much what we've been doing all along.
@@ -88,29 +85,29 @@ write our FT to look for a "My Lists" page:
 
 
 [role="sourcecode"]
-.functional_tests/test_my_lists.py (ch19l001-1)
+.src/functional_tests/test_my_lists.py (ch22l001)
 ====
 [source,python]
 ----
     def test_logged_in_users_lists_are_saved_as_my_lists(self):
         # Edith is a logged-in user
-        self.create_pre_authenticated_session('[email protected]')
+        self.create_pre_authenticated_session("[email protected]")
 
         # She goes to the home page and starts a list
         self.browser.get(self.live_server_url)
-        self.add_list_item('Reticulate splines')
-        self.add_list_item('Immanentize eschaton')
+        self.add_list_item("Reticulate splines")
+        self.add_list_item("Immanentize eschaton")
         first_list_url = self.browser.current_url
 
         # She notices a "My lists" link, for the first time.
-        self.browser.find_element_by_link_text('My lists').click()
+        self.browser.find_element_by_link_text("My lists").click()
 
         # She sees that her list is in there, named according to its
         # first list item
         self.wait_for(
-            lambda: self.browser.find_element_by_link_text('Reticulate splines')
+            lambda: self.browser.find_element_by_link_text("Reticulate splines")
         )
-        self.browser.find_element_by_link_text('Reticulate splines').click()
+        self.browser.find_element_by_link_text("Reticulate splines").click()
         self.wait_for(
             lambda: self.assertEqual(self.browser.current_url, first_list_url)
         )
@@ -130,7 +127,7 @@ appear on the My Lists page as well.  The FT continues, and while we're at it,
 we check that only logged-in users can see the "My Lists" page:
 
 [role="sourcecode"]
-.functional_tests/test_my_lists.py (ch19l001-2)
+.src/functional_tests/test_my_lists.py (ch22l002)
 ====
 [source,python]
 ----
@@ -141,24 +138,24 @@ we check that only logged-in users can see the "My Lists" page:
 
         # She decides to start another list, just to see
         self.browser.get(self.live_server_url)
-        self.add_list_item('Click cows')
+        self.add_list_item("Click cows")
         second_list_url = self.browser.current_url
 
         # Under "my lists", her new list appears
-        self.browser.find_element_by_link_text('My lists').click()
+        self.browser.find_element_by_link_text("My lists").click()
         self.wait_for(
-            lambda: self.browser.find_element_by_link_text('Click cows')
+            lambda: self.browser.find_element_by_link_text("Click cows")
         )
-        self.browser.find_element_by_link_text('Click cows').click()
+        self.browser.find_element_by_link_text("Click cows").click()
         self.wait_for(
             lambda: self.assertEqual(self.browser.current_url, second_list_url)
         )
 
         # She logs out.  The "My lists" option disappears
-        self.browser.find_element_by_link_text('Log out').click()
+        self.browser.find_element_by_link_text("Log out").click()
         self.wait_for(lambda: self.assertEqual(
-            self.browser.find_elements_by_link_text('My lists'),
-            []
+            self.browser.find_elements_by_link_text("My lists"),
+            [],
         ))
 ----
 ====
@@ -168,7 +165,7 @@ text into the right input box.  We define it in 'base.py':
 
 
 [role="sourcecode small-code"]
-.functional_tests/base.py (ch22l003)
+.src/functional_tests/base.py (ch22l003)
 ====
 [source,python]
 ----
@@ -189,7 +186,7 @@ And while we're at it we can use it in a few of the other FTs, like this:
 
 
 [role="sourcecode currentcontents dofirst-ch22l004"]
-.functional_tests/test_list_item_validation.py
+.src/functional_tests/test_list_item_validation.py
 ====
 [source,python]
 ----
@@ -226,7 +223,7 @@ Here's the minimal code change:
 * TODO: update this link for latest bootstrap / style nicely
 
 [role="sourcecode small-code"]
-.lists/templates/base.html (ch22l005)
+.src/lists/templates/base.html (ch22l005)
 ====
 [source,html]
 ----
@@ -262,7 +259,7 @@ Again, we can go outside-in, starting at the presentation layer with just the
 URL and nothing else:
 
 [role="sourcecode"]
-.lists/templates/base.html (ch22l006)
+.src/lists/templates/base.html (ch22l006)
 ====
 [source,html]
 ----
@@ -282,7 +279,7 @@ functions.
 As always, we start with a test:
 
 [role="sourcecode"]
-.lists/tests/test_views.py (ch19l003)
+.src/lists/tests/test_views.py (ch19l003)
 ====
 [source,python]
 ----
@@ -303,7 +300,7 @@ And we fix it, still at the presentation level, in 'urls.py':
 
 
 [role="sourcecode"]
-.lists/urls.py (ch19l004)
+.src/lists/urls.py (ch19l004)
 ====
 [source,python]
 ----
@@ -333,7 +330,7 @@ We move in from the presentation layer to the views layer, and create a
 minimal placeholder:
 
 [role="sourcecode"]
-.lists/views.py (ch22l005)
+.src/lists/views.py (ch22l005)
 ====
 [source,python]
 ----
@@ -345,7 +342,7 @@ def my_lists(request, email):
 And a minimal template:
 
 [role="sourcecode"]
-.lists/templates/my_lists.html (ch22l006)
+.src/lists/templates/my_lists.html (ch22l006)
 ====
 [source,html]
 ----
@@ -389,7 +386,7 @@ Also, the "My Lists" page doesn't need the new item form,
 so we'll put that into a block too, making it optional:
 
 [role="sourcecode"]
-.lists/templates/base.html (ch19l007-1)
+.src/lists/templates/base.html (ch19l007-1)
 ====
 [source,diff]
 ----
@@ -413,7 +410,7 @@ so we'll put that into a block too, making it optional:
 ====
 
 [role="sourcecode"]
-.lists/templates/base.html (ch19l007-2)
+.src/lists/templates/base.html (ch19l007-2)
 ====
 [source,html]
 ----
@@ -445,7 +442,7 @@ Designing Our API Using the Template
 be empty...
 
 [role="sourcecode"]
-.lists/templates/my_lists.html
+.src/lists/templates/my_lists.html
 ====
 [source,html]
 ----
@@ -460,7 +457,7 @@ be empty...
 And then we can just work inside the `extra_content` block:
 
 [role="sourcecode"]
-.lists/templates/my_lists.html
+.src/lists/templates/my_lists.html
 ====
 [source,html]
 ----
@@ -530,7 +527,7 @@ Moving Down to the Next Layer: What the View Passes to the Template
 our views layer needs to respond to the requirements we've laid out in the template layer, by giving it the objects it needs.  In this case, the list owner:
 
 [role="sourcecode"]
-.lists/tests/test_views.py (ch19l011)
+.src/lists/tests/test_views.py (ch19l011)
 ====
 [source,python]
 ----
@@ -559,7 +556,7 @@ KeyError: 'owner'
 So:
 
 [role="sourcecode"]
-.lists/views.py (ch19l012)
+.src/lists/views.py (ch19l012)
 ====
 [source,python]
 ----
@@ -578,7 +575,7 @@ the previous test. We just need to add a user for it as well:
 
 
 [role="sourcecode"]
-.lists/tests/test_views.py (ch19l013)
+.src/lists/tests/test_views.py (ch19l013)
 ====
 [source,python]
 ----
@@ -609,7 +606,7 @@ Here's a first crack at writing the test:
 
 
 [role="sourcecode"]
-.lists/tests/test_views.py (ch22l014)
+.src/lists/tests/test_views.py (ch22l014)
 ====
 [source,python]
 ----
@@ -637,7 +634,7 @@ AttributeError: 'List' object has no attribute 'owner'
 To fix this, we can try writing code like this:
 
 [role="sourcecode"]
-.lists/views.py (ch22l015)
+.src/lists/views.py (ch22l015)
 ====
 [source,python]
 ----

source/chapter_22_outside_in/superlists

@@ -1,9 +1,9 @@
 #!/usr/bin/env python
-import subprocess
-import os
 import getpass
-from chapters import CHAPTERS
+import os
+import subprocess
 
+from chapters import CHAPTERS
 
 REMOTE = "local" if getpass.getuser() == "harry" else "origin"
 BASE_FOLDER = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
@@ -22,6 +22,7 @@ def fetch_if_possible(target_dir):
         if (
             "Name or service not known" in stderr.decode()
             or "Could not resolve" in stderr.decode()
+            or "github.com port 22: Undefined error" in stderr.decode()
         ):
             # no internet
             print("No Internet")