Fix tests that rely on the ordering of values in a dictionary.
Reviewed in https://codereview.appspot.com/7311096/.
This commit is contained in:
@@ -863,7 +863,7 @@ class Discovery(unittest.TestCase):
|
||||
self.assertTrue(hasattr(new_zoo, 'scopedAnimals'))
|
||||
self.assertTrue(callable(new_zoo.scopedAnimals))
|
||||
|
||||
self.assertEqual(zoo._dynamic_attrs, new_zoo._dynamic_attrs)
|
||||
self.assertEqual(sorted(zoo._dynamic_attrs), sorted(new_zoo._dynamic_attrs))
|
||||
self.assertEqual(zoo._baseUrl, new_zoo._baseUrl)
|
||||
self.assertEqual(zoo._developerKey, new_zoo._developerKey)
|
||||
self.assertEqual(zoo._requestBuilder, new_zoo._requestBuilder)
|
||||
|
||||
@@ -19,6 +19,7 @@ import unittest
|
||||
from apiclient import push
|
||||
from apiclient import model
|
||||
from apiclient import http
|
||||
from test_discovery import assertUrisEqual
|
||||
|
||||
|
||||
class ClientTokenGeneratorTest(unittest.TestCase):
|
||||
@@ -92,13 +93,15 @@ class WebhookChannelTest(unittest.TestCase):
|
||||
|
||||
def test_creation_no_appengine(self):
|
||||
c = push.WebhookChannel('http://example.org')
|
||||
self.assertEqual('web_hook?url=http%3A%2F%2Fexample.org&app_engine=false',
|
||||
c.as_header_value())
|
||||
assertUrisEqual(self,
|
||||
'web_hook?url=http%3A%2F%2Fexample.org&app_engine=false',
|
||||
c.as_header_value())
|
||||
|
||||
def test_creation_appengine(self):
|
||||
c = push.WebhookChannel('http://example.org', app_engine=True)
|
||||
self.assertEqual('web_hook?url=http%3A%2F%2Fexample.org&app_engine=true',
|
||||
c.as_header_value())
|
||||
assertUrisEqual(self,
|
||||
'web_hook?url=http%3A%2F%2Fexample.org&app_engine=true',
|
||||
c.as_header_value())
|
||||
|
||||
|
||||
class HeadersTest(unittest.TestCase):
|
||||
@@ -146,15 +149,17 @@ class SubscriptionTest(unittest.TestCase):
|
||||
c = push.WebhookChannel('http://example.org')
|
||||
s = push.Subscription.for_channel(c)
|
||||
self.assertTrue(s.client_token)
|
||||
self.assertEqual('web_hook?url=http%3A%2F%2Fexample.org&app_engine=false',
|
||||
s.subscribe)
|
||||
assertUrisEqual(self,
|
||||
'web_hook?url=http%3A%2F%2Fexample.org&app_engine=false',
|
||||
s.subscribe)
|
||||
|
||||
def test_create_for_channel_client_token(self):
|
||||
c = push.WebhookChannel('http://example.org')
|
||||
s = push.Subscription.for_channel(c, client_token='my_token')
|
||||
self.assertEqual('my_token', s.client_token)
|
||||
self.assertEqual('web_hook?url=http%3A%2F%2Fexample.org&app_engine=false',
|
||||
s.subscribe)
|
||||
assertUrisEqual(self,
|
||||
'web_hook?url=http%3A%2F%2Fexample.org&app_engine=false',
|
||||
s.subscribe)
|
||||
|
||||
def test_subscribe(self):
|
||||
s = push.Subscription()
|
||||
|
||||
648
upload-diffs.py
648
upload-diffs.py
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user