Browse Source

test(sentry10): Update email tests for Sentry 10 (#13033)

Update email tests for Sentry 10. This will allow mainlining
Sentry 10 in future without breaking these tests.
Lyn Nagara 5 years ago
parent
commit
d0554a17ce

+ 13 - 12
tests/acceptance/test_emails.py

@@ -63,17 +63,18 @@ class EmailTestCase(AcceptanceTestCase):
         )
 
     def test_emails(self):
-        for url, name in EMAILS:
-            # HTML output is captured as a snapshot
-            self.browser.get(self.build_url(url, 'html'))
-            self.browser.wait_until('#preview')
-            self.browser.snapshot(u'{} email html'.format(name))
+        with self.feature('organizations:sentry10'):
+            for url, name in EMAILS:
+                # HTML output is captured as a snapshot
+                self.browser.get(self.build_url(url, 'html'))
+                self.browser.wait_until('#preview')
+                self.browser.snapshot(u'{} email html'.format(name))
 
-            # Text output is asserted against static fixture files
-            self.browser.get(self.build_url(url, 'txt'))
-            self.browser.wait_until('#preview')
-            elem = self.browser.find_element_by_css_selector('#preview pre')
-            text_src = elem.get_attribute('innerHTML')
+                # Text output is asserted against static fixture files
+                self.browser.get(self.build_url(url, 'txt'))
+                self.browser.wait_until('#preview')
+                elem = self.browser.find_element_by_css_selector('#preview pre')
+                text_src = elem.get_attribute('innerHTML')
 
-            fixture_src = read_txt_email_fixture(name)
-            assert fixture_src == text_src
+                fixture_src = read_txt_email_fixture(name)
+                assert fixture_src == text_src

+ 1 - 1
tests/fixtures/emails/assigned.txt

@@ -7,7 +7,7 @@ foo@example.com assigned an issue to foo@example.com
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/?referrer=AssignedActivityEmail
+http://testserver/organizations/organization/issues/1/?project=1&referrer=AssignedActivityEmail
 
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

+ 1 - 1
tests/fixtures/emails/assigned_self.txt

@@ -7,7 +7,7 @@ foo@example.com assigned an issue to themselves
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/?referrer=AssignedActivityEmail
+http://testserver/organizations/organization/issues/1/?project=1&referrer=AssignedActivityEmail
 
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

+ 6 - 6
tests/fixtures/emails/digest.txt

@@ -4,22 +4,22 @@ June 8, 2016, 7:25 a.m. UTC to July 4, 2016, 2:11 p.m. UTC
 Rule #1
 
 * CursusError: nibh curabitur (9729 events, 5901 users)
-  http://testserver/example/example/issues/9/?referrer=digest_email
+  http://testserver/organizations/example/issues/9/?referrer=digest_email
 
 * CuraeError: pulvinar fames suspendisse erat integer ad magna (7555 events, 4549 users)
-  http://testserver/example/example/issues/15/?referrer=digest_email
+  http://testserver/organizations/example/issues/15/?referrer=digest_email
 
 * erat pellentesque enim imperdiet tortor (3180 events, 276 users)
-  http://testserver/example/example/issues/16/?referrer=digest_email
+  http://testserver/organizations/example/issues/16/?referrer=digest_email
 
 * NecVivamusError: consequat ipsum proin gravida inceptos scelerisque inceptos iaculis libero vulputate (2824 events, 1188 users)
-  http://testserver/example/example/issues/14/?referrer=digest_email
+  http://testserver/organizations/example/issues/14/?referrer=digest_email
 
 * LectusDapibusTacitiError: semper porta curae luctus fermentum molestie praesent ac auctor (2358 events, 7906 users)
-  http://testserver/example/example/issues/2/?referrer=digest_email
+  http://testserver/organizations/example/issues/2/?referrer=digest_email
 
 * VelTorquentError: porttitor praesent ridiculus (562 events, 8950 users)
-  http://testserver/example/example/issues/13/?referrer=digest_email
+  http://testserver/organizations/example/issues/13/?referrer=digest_email
 
 
 

+ 1 - 1
tests/fixtures/emails/note.txt

@@ -9,6 +9,6 @@ metus volutpat
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/activity/?referrer=NoteActivityEmail
+http://testserver/organizations/organization/issues/1/activity/?project=1&referrer=NoteActivityEmail
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

+ 1 - 1
tests/fixtures/emails/regression.txt

@@ -7,7 +7,7 @@ Sentry marked an issue as a regression
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/?referrer=RegressionActivityEmail
+http://testserver/organizations/organization/issues/1/?project=1&referrer=RegressionActivityEmail
 
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

+ 1 - 1
tests/fixtures/emails/regression_with_version.txt

@@ -7,7 +7,7 @@ Sentry marked an issue as a regression in abcdef
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/?referrer=RegressionActivityEmail
+http://testserver/organizations/organization/issues/1/?project=1&referrer=RegressionActivityEmail
 
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

+ 3 - 3
tests/fixtures/emails/release.txt

@@ -1,9 +1,9 @@
 Version 6c998f7 was deployed to production on Oct. 12, 2016, 3:39 p.m.
 
 
-    http://testserver/organization/project/releases/6c998f755f304593a4713abd123eaf8833a2de5e/
+    http://testserver/organizations/organization/releases/6c998f755f304593a4713abd123eaf8833a2de5e/?project=1
 
-    http://testserver/organization/another-project/releases/6c998f755f304593a4713abd123eaf8833a2de5e/
+    http://testserver/organizations/organization/releases/6c998f755f304593a4713abd123eaf8833a2de5e/?project=2
 
-    http://testserver/organization/yet-another-project/releases/6c998f755f304593a4713abd123eaf8833a2de5e/
+    http://testserver/organizations/organization/releases/6c998f755f304593a4713abd123eaf8833a2de5e/?project=3
 

+ 1 - 1
tests/fixtures/emails/resolved.txt

@@ -7,7 +7,7 @@ Sentry marked an issue as resolved
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/?referrer=ResolvedActivityEmail
+http://testserver/organizations/organization/issues/1/?project=1&referrer=ResolvedActivityEmail
 
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

+ 1 - 1
tests/fixtures/emails/resolved_in_release.txt

@@ -7,7 +7,7 @@ Sentry marked an issue as resolved in abcdef
 
 ornare elit pulvinar nisl integer cum ad massa
 
-http://testserver/organization/project/issues/1/?referrer=ResolvedInReleaseActivityEmail
+http://testserver/organizations/organization/issues/1/?project=1&referrer=ResolvedInReleaseActivityEmail
 
 
 Unsubscribe: javascript:alert("This is a preview page, what did you expect to happen?");

Some files were not shown because too many files changed in this diff