Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1269)

Unified Diff: appengine/findit/handlers/crash/test/crash_handler_test.py

Issue 2556463002: [Predator] Add https:// in front of feedback link. (Closed)
Patch Set: Fix tests. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « appengine/findit/crash/findit_for_chromecrash.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/handlers/crash/test/crash_handler_test.py
diff --git a/appengine/findit/handlers/crash/test/crash_handler_test.py b/appengine/findit/handlers/crash/test/crash_handler_test.py
index ee59333fb658abea16c575bd5872d540037d0ff1..e0311d0347a63c54ab57bea980e1c1c693f1e777 100644
--- a/appengine/findit/handlers/crash/test/crash_handler_test.py
+++ b/appengine/findit/handlers/crash/test/crash_handler_test.py
@@ -188,7 +188,7 @@ class CrashHandlerTest(CrashTestCase):
self.mock(crash_pipeline.pubsub_util, 'PublishMessagesToTopic',
Mocked_PublishMessagesToTopic)
- MOCK_HOST = 'https://host.com'
+ MOCK_HOST = 'host.com'
self.mock(app_identity, 'get_default_version_hostname', lambda: MOCK_HOST)
testcase = self
@@ -258,7 +258,8 @@ class CrashHandlerTest(CrashTestCase):
processed_analysis_result = copy.deepcopy(analysis_result)
processed_analysis_result['feedback_url'] = (
- '%s/crash/fracas-result-feedback?key=%s' % (MOCK_HOST, MOCK_KEY))
+ 'https://%s/crash/fracas-result-feedback?key=%s' % (MOCK_HOST,
+ MOCK_KEY))
for cl in processed_analysis_result.get('suspected_cls', []):
cl['confidence'] = round(cl['confidence'], 2)
« no previous file with comments | « appengine/findit/crash/findit_for_chromecrash.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698