From 98fc1560b227794a7cbf26c1b00eaec8e7e51925 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Fr=C3=A9d=C3=A9ric=20Perrin?= <frederic.perrin@resel.fr>
Date: Fri, 11 Nov 2016 19:26:10 +0000
Subject: [PATCH] Fix link to work page

Add testing for broken links
---
 quotes/conftest.py                   | 19 ++++++++++++++++++-
 quotes/templates/quotes/display.html |  2 +-
 quotes/test_cloud.py                 |  1 +
 quotes/test_quotes.py                |  2 ++
 4 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/quotes/conftest.py b/quotes/conftest.py
index 9210e6a..26ac8ef 100644
--- a/quotes/conftest.py
+++ b/quotes/conftest.py
@@ -1,11 +1,13 @@
 import pytest
 
 import lxml.etree
+import html5lib
+import urlparse
 
 class ValidatingClient(object):
     def __init__(self, client):
         self.client = client
-    
+
     def request(self, url, method, exp_status=200, params={}):
         if not url.startswith('/quotes/'):
             url = '/quotes/' + url
@@ -23,6 +25,10 @@ class ValidatingClient(object):
         document = response.content.decode(response.charset)
         print 'For url %s got page:\n%s' % (url, document)
         lxml.etree.fromstring(document.replace('<br>', '<br/>'))
+
+        parser = html5lib.HTMLParser(strict=True)
+        parser.parse(document)
+
         assert '<script>' not in document
         return document
 
@@ -32,6 +38,17 @@ class ValidatingClient(object):
     def postPage(self, url, params, exp_status=200):
         return self.request(url, 'post', params=params, exp_status=exp_status)
 
+    def checkAllLinks(self, url, document):
+        tree = lxml.etree.fromstring(document)
+        links = tree.xpath("//a")
+        for link in links:
+            href = link.attrib['href']
+            if href == '#':
+                continue
+            fullurl = urlparse.urljoin(url, href)
+            print "asking for fullurl=", fullurl
+            assert self.getPage(fullurl)
+
 @pytest.fixture
 def c(client):
     return ValidatingClient(client)
diff --git a/quotes/templates/quotes/display.html b/quotes/templates/quotes/display.html
index 2db4c57..878c4dc 100644
--- a/quotes/templates/quotes/display.html
+++ b/quotes/templates/quotes/display.html
@@ -10,7 +10,7 @@
 	— <span class="name">
 	  <a href="{{ quote.work.author.get_absolute_url }}"
 	     class="author_name">{{ quote.work.author.name }}</a>,
-	  <a href="{% quote.work.get_absolute_url }}"
+	  <a href="{{ quote.work.get_absolute_url }}"
 	     class="work_name">{{ quote.work.name }}</a>
 	</span>
       </p>
diff --git a/quotes/test_cloud.py b/quotes/test_cloud.py
index 906aa5b..b7c3011 100644
--- a/quotes/test_cloud.py
+++ b/quotes/test_cloud.py
@@ -31,6 +31,7 @@ class Test_Cloud(object):
         cloud = c.getPage("cloud/")
         for size in top20:
             assert re.search("tag-%d-, %d quotes" % (size, size), cloud)
+        c.checkAllLinks('cloud/', cloud)
 
     def test_cloud(self, size_range_small, db, c):
         self.fill_db(size_range_small)
diff --git a/quotes/test_quotes.py b/quotes/test_quotes.py
index b4c7915..b194ec1 100644
--- a/quotes/test_quotes.py
+++ b/quotes/test_quotes.py
@@ -51,6 +51,8 @@ class Test_Views():
             assert q.text in content
             assert q.notes in content
 
+        c.checkAllLinks('all/', content)
+
     @pytest.mark.slow
     def test_random(self, c):
         seen = {}
-- 
2.43.0