Skip to content

Pep8 #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
fpietka wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Pep8 #27

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ee30a51
Remove unused packages
fpietka Oct 16, 2016
26e02ae
Unalign assignment
fpietka Oct 16, 2016
01905e4
Remove or add spaces where needed
fpietka Oct 16, 2016
2994a7a
Comparison with None must use 'is'
fpietka Oct 16, 2016
34f67eb
Correct separation
fpietka Oct 16, 2016
4302930
Fix comments lenght
fpietka Oct 16, 2016
14ca7a3
Use 'not in' instead of 'not ... in'
fpietka Oct 16, 2016
2698c89
Remove one useless if level
fpietka Oct 16, 2016
3731801
Fix long lines
fpietka Oct 16, 2016
2395f31
Fix indentation
fpietka Oct 16, 2016
340ddfa
Remove unwanted extra lines
fpietka Oct 16, 2016
01e00aa
Comma not needed in list with only one element
fpietka Oct 16, 2016
727a54e
Remove comma after last element of list
fpietka Oct 16, 2016
c5218f2
Close parenthesis on the last element level
fpietka Oct 16, 2016
21ef148
Remove unused parameters
fpietka Oct 16, 2016
e10bb50
Remove unused variable
fpietka Oct 16, 2016
f8be397
Fix display of server response
fpietka Oct 16, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
186 changes: 125 additions & 61 deletions TestDailymotion.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,25 @@
import time
import os


class TestA(unittest.TestCase):

@classmethod
def setUpClass(self):
self.api_base_url = config.BASE_URL or 'http://api.dailymotion.com'
self.api_key = config.CLIENT_ID
self.api_secret = config.CLIENT_SECRET
self.username = config.USERNAME
self.password = config.PASSWORD
self.scope = ['manage_videos', 'manage_playlists', 'userinfo']
self.redirect_uri = config.REDIRECT_URI
self.oauth_authorize_endpoint_url = config.OAUTH_AUTHORIZE_URL or 'https://api.dailymotion.com/oauth/authorize'
self.oauth_token_endpoint_url = config.OAUTH_TOKEN_URL or 'https://api.dailymotion.com/oauth/token'
self.session_file_directory = './data'
self.api_base_url = config.BASE_URL or 'http://api.dailymotion.com'
self.api_key = config.CLIENT_ID
self.api_secret = config.CLIENT_SECRET
self.username = config.USERNAME
self.password = config.PASSWORD
self.scope = ['manage_videos', 'manage_playlists', 'userinfo']
self.redirect_uri = config.REDIRECT_URI
self.oauth_authorize_endpoint_url = (
config.OAUTH_AUTHORIZE_URL or
'https://api.dailymotion.com/oauth/authorize')
self.oauth_token_endpoint_url = (
config.OAUTH_TOKEN_URL or
'https://api.dailymotion.com/oauth/token')
self.session_file_directory = './data'
if not os.path.exists(self.session_file_directory):
os.makedirs(self.session_file_directory)

Expand All @@ -31,7 +36,9 @@ def test_init(self):
d = dailymotion.Dailymotion()
self.assertEqual(d.api_base_url, 'https://api.dailymotion.com')

d = dailymotion.Dailymotion(api_base_url='http://api.stage.dailymotion.com', timeout=10, debug=True)
d = dailymotion.Dailymotion(
api_base_url='http://api.stage.dailymotion.com', timeout=10,
debug=True)
self.assertEqual(d.api_base_url, 'http://api.stage.dailymotion.com')
self.assertEqual(d.timeout, 10)
self.assertEqual(d.debug, True)
Expand All @@ -46,62 +53,103 @@ def test_get(self):

def test_set_grant_type(self):
d = dailymotion.Dailymotion()
self.assertRaises(dailymotion.DailymotionClientError, d.set_grant_type, 'password', api_secret=self.api_secret, scope=self.scope,
info={'username': self.username, 'password': self.password})
self.assertRaises(dailymotion.DailymotionClientError, d.set_grant_type, 'password', api_secret=self.api_secret, scope=self.scope)
self.assertRaises(dailymotion.DailymotionClientError, d.set_grant_type, 'password', api_secret=self.api_secret, scope=None)
self.assertRaises(dailymotion.DailymotionClientError, d.set_grant_type,
'password', api_secret=self.api_secret,
scope=self.scope, info={
'username': self.username,
'password': self.password
})
self.assertRaises(dailymotion.DailymotionClientError, d.set_grant_type,
'password', api_secret=self.api_secret,
scope=self.scope)
self.assertRaises(dailymotion.DailymotionClientError, d.set_grant_type,
'password', api_secret=self.api_secret, scope=None)

def test_get_authorization_url(self):
d = dailymotion.Dailymotion(api_base_url=self.api_base_url, oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url)
d.set_grant_type('authorization', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'redirect_uri' : self.redirect_uri})
authorization_url = d.get_authorization_url(redirect_uri=self.redirect_uri, scope=self.scope)
self.assertEqual(re.match('https?://(?:www)?(?:[\w-]{2,255}(?:\.\w{2,6}){1,2})(?:/[\w&%?#-]{1,300})?',authorization_url) == None, False)
d = dailymotion.Dailymotion(
api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url)
d.set_grant_type('authorization', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={'redirect_uri': self.redirect_uri})
authorization_url = d.get_authorization_url(
redirect_uri=self.redirect_uri, scope=self.scope)
self.assertEqual(re.match(
'https?://(?:www)?(?:[\w-]{2,255}(?:\.\w{2,6}){1,2})'
'(?:/[\w&%?#-]{1,300})?',
authorization_url) is None, False)

def test_get_access_token(self):
d = dailymotion.Dailymotion(api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url)
d.set_grant_type('password', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'username': self.username, 'password': self.password})
d = dailymotion.Dailymotion(
api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url)
d.set_grant_type('password', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={
'username': self.username,
'password': self.password
})
access_token = d.get_access_token()
self.assertEqual(isinstance (access_token, str) or isinstance(access_token, unicode), True)
self.assertEqual(isinstance(access_token, str) or
isinstance(access_token, unicode), True)
d.logout()

def test_set_access_token(self):
d = dailymotion.Dailymotion()
d.set_grant_type('password', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'username': self.username, 'password': self.password})
d.set_grant_type('password', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={
'username': self.username,
'password': self.password
})
d.set_access_token(d.get_access_token())
response = d.get('/me/?fields=fullname')
self.assertEqual(isinstance (response.get('fullname'), str) or isinstance(response.get('fullname'), unicode), True)
self.assertEqual(isinstance(response.get('fullname'), str) or
isinstance(response.get('fullname'), unicode), True)
d.logout()

def test_auth_call(self):
d = dailymotion.Dailymotion(api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True)

d.set_grant_type('password', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'username': self.username, 'password': self.password})
d = dailymotion.Dailymotion(
api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True)

d.set_grant_type('password', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={
'username': self.username,
'password': self.password
})
response = d.get('/me/?fields=fullname')
self.assertEqual(isinstance (response.get('fullname'), str) or isinstance(response.get('fullname'), unicode), True)
self.assertEqual(isinstance(response.get('fullname'), str) or
isinstance(response.get('fullname'), unicode), True)
d.logout()

def test_upload(self):
d = dailymotion.Dailymotion(api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True)

d.set_grant_type('password', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'username': self.username, 'password': self.password})
d = dailymotion.Dailymotion(
api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True)

d.set_grant_type('password', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={
'username': self.username,
'password': self.password
})
url = d.upload('./examples/video.mp4')
self.assertEqual(re.match('https?://(?:www)?(?:[\w-]{2,255}(?:\.\w{2,6}){1,2})(?:/[\w&%?#-]{1,300})?',url) == None, False)
d.post('/videos', {'url' : url,
'title' : 'my_test_upload_%s' % time.strftime("%c"),
'published' : 'true',
'channel' : 'news'
})
self.assertEqual(re.match(
'https?://(?:www)?(?:[\w-]{2,255}(?:\.\w{2,6}){1,2})'
'(?:/[\w&%?#-]{1,300})?', url) is None, False)
d.post('/videos', {'url': url,
'title': 'my_test_upload_%s' % time.strftime("%c"),
'published': 'true',
'channel': 'news'})
d.logout()


def test_session_store_option(self):
d = dailymotion.Dailymotion(session_store_enabled=False)
self.assertFalse(d._session_store_enabled)
Expand All @@ -113,29 +161,45 @@ def test_session_store_option(self):
self.assertEqual(d.DEFAULT_SESSION_STORE, d._session_store_enabled)

def test_in_memory_session(self):
d = dailymotion.Dailymotion(api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True)
d.set_grant_type('password', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'username': self.username, 'password': self.password})
d = dailymotion.Dailymotion(
api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True)
d.set_grant_type('password', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={
'username': self.username,
'password': self.password
})
access_token = d.get_access_token()
self.assertEqual(isinstance (access_token, str) or isinstance(access_token, unicode), True)
self.assertEqual(isinstance(access_token, str) or
isinstance(access_token, unicode), True)
second_access_token = d.get_access_token()
self.assertEqual(isinstance (second_access_token, str) or isinstance(second_access_token, unicode), True)
self.assertEqual(isinstance(second_access_token, str) or
isinstance(second_access_token, unicode), True)
self.assertEqual(second_access_token, access_token)
d.logout()

def test_file_storage_session(self):
fs = dailymotion.FileSessionStore(self.session_file_directory)
d = dailymotion.Dailymotion(api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True,
session_store=fs)
d.set_grant_type('password', api_key=self.api_key, api_secret=self.api_secret, scope=self.scope, info={'username': self.username, 'password': self.password})
d = dailymotion.Dailymotion(
api_base_url=self.api_base_url,
oauth_authorize_endpoint_url=self.oauth_authorize_endpoint_url,
oauth_token_endpoint_url=self.oauth_token_endpoint_url,
session_store_enabled=True,
session_store=fs)
d.set_grant_type('password', api_key=self.api_key,
api_secret=self.api_secret, scope=self.scope,
info={
'username': self.username,
'password': self.password
})
access_token = d.get_access_token()
self.assertEqual(isinstance (access_token, str) or isinstance(access_token, unicode), True)
self.assertEqual(isinstance(access_token, str) or
isinstance(access_token, unicode), True)
second_access_token = d.get_access_token()
self.assertEqual(isinstance (second_access_token, str) or isinstance(second_access_token, unicode), True)
self.assertEqual(isinstance(second_access_token, str) or
isinstance(second_access_token, unicode), True)
self.assertEqual(second_access_token, access_token)
d.logout()
Loading