Changeset - 4086efa27652
[Not reviewed]
default
0 2 0
Thomas De Schampheleire - 8 years ago 2018-02-07 09:24:14
thomas.de_schampheleire@nokia.com
tests: notification: clarify that cs_target_repo is a URL

In analogy with the fake URLs used for pullrequest notification testing, use
an URL for cs_target_repo. When testing the resulting HTML in a real email
client, it will become obvious that the mail client transforms URLs to
something clickable, regardless of any present <a> tag.
2 files changed with 9 insertions and 9 deletions:
0 comments (0 inline, 0 general)
kallithea/tests/models/test_dump_html_mails.ref.html
Show inline comments
 
@@ -22,7 +22,7 @@ This is the new comment.
 
 - and here it ends indented.
 

	
 

	
 
Changeset on repo_target branch brunch:
 
Changeset on http://example.com/repo_target branch brunch:
 
"This changeset did something clever which is hard to explain" by u2 u3 (u2).
 

	
 

	
 
@@ -112,7 +112,7 @@ View Comment: http://comment.org
 
            <div>
 
                Changeset on
 
                <a style="color:#395fa0;text-decoration:none"
 
                   href="repo_target">repo_target</a>
 
                   href="http://example.com/repo_target">http://example.com/repo_target</a>
 
                branch
 
                <span style="color:#395fa0">brunch</span>:
 
            </div>
 
@@ -181,7 +181,7 @@ This is the new comment.
 
 - and here it ends indented.
 

	
 

	
 
Changeset on repo_target branch brunch:
 
Changeset on http://example.com/repo_target branch brunch:
 
"This changeset did something clever which is hard to explain" by u2 u3 (u2).
 

	
 

	
 
@@ -271,7 +271,7 @@ View Comment: http://comment.org
 
            <div>
 
                Changeset on
 
                <a style="color:#395fa0;text-decoration:none"
 
                   href="repo_target">repo_target</a>
 
                   href="http://example.com/repo_target">http://example.com/repo_target</a>
 
                branch
 
                <span style="color:#395fa0">brunch</span>:
 
            </div>
 
@@ -342,7 +342,7 @@ This is the new comment.
 
 - and here it ends indented.
 

	
 

	
 
Changeset on repo_target branch brunch:
 
Changeset on http://example.com/repo_target branch brunch:
 
"This changeset did something clever which is hard to explain" by u2 u3 (u2).
 

	
 

	
 
@@ -448,7 +448,7 @@ View Comment: http://comment.org
 
            <div>
 
                Changeset on
 
                <a style="color:#395fa0;text-decoration:none"
 
                   href="repo_target">repo_target</a>
 
                   href="http://example.com/repo_target">http://example.com/repo_target</a>
 
                branch
 
                <span style="color:#395fa0">brunch</span>:
 
            </div>
 
@@ -519,7 +519,7 @@ This is the new comment.
 
 - and here it ends indented.
 

	
 

	
 
Changeset on repo_target branch brunch:
 
Changeset on http://example.com/repo_target branch brunch:
 
"This changeset did something clever which is hard to explain" by u2 u3 (u2).
 

	
 

	
 
@@ -625,7 +625,7 @@ View Comment: http://comment.org
 
            <div>
 
                Changeset on
 
                <a style="color:#395fa0;text-decoration:none"
 
                   href="repo_target">repo_target</a>
 
                   href="http://example.com/repo_target">http://example.com/repo_target</a>
 
                branch
 
                <span style="color:#395fa0">brunch</span>:
 
            </div>
kallithea/tests/models/test_notifications.py
Show inline comments
 
@@ -220,7 +220,7 @@ class TestNotifications(TestController):
 
                        message='This changeset did something clever which is hard to explain',
 
                        message_short='This changeset did something cl...',
 
                        status_change=[None, 'Approved'],
 
                        cs_target_repo='repo_target',
 
                        cs_target_repo='http://example.com/repo_target',
 
                        cs_url='http://changeset.com',
 
                        cs_author=User.get(self.u2))),
 
                    (Notification.TYPE_MESSAGE,
0 comments (0 inline, 0 general)