8000 50% test coverage. · n1k0/github3.py@3ceb856 · GitHub
[go: up one dir, main page]
Skip to content

Commit 3ceb856

Browse files
sigmavirus24sigmavirus24
committed
50% test coverage.
1 parent 615da2d commit 3ceb856

File tree

1 file changed

+93
-4
lines changed

1 file changed

+93
-4
lines changed

‎tests/test_github.py

Lines changed: 93 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import github3
22
from mock import patch
3+
from contextlib import nested
34
from tests.utils import (generate_response, expect, BaseCase, load)
45

56

@@ -369,13 +370,13 @@ def test_iter_starred(self):
369370
github3.repos.Repository) # nopep8
370371
self.mock_assertions()
371372

372-
def test_iter_subscribed(self):
373+
def test_iter_subscriptions(self):
373374
self.request.return_value = generate_response('repo', _iter=True)
374375
self.args = ('get', 'https://api.github.com/user/subscriptions')
375376
self.conf.update(params=None)
376377

377378
self.login()
378-
expect(next(self.g.iter_subscribed())).isinstance(
379+
expect(next(self.g.iter_subscriptions())).isinstance(
379380
github3.repos.Repository) # nopep8
380381
self.mock_assertions()
381382

@@ -385,7 +386,7 @@ def test_iter_subscribed(self):
385386
'https://api.github.com/users/sigmavirus24/'
386387
'subscriptions'
387388
)
388-
expect(next(self.g.iter_subscribed('sigmavirus24'))).isinstance(
389+
expect(next(self.g.iter_subscriptions('sigmavirus24'))).isinstance(
389390
github3.repos.Repository) # nopep8
390391
self.mock_assertions()
391392

@@ -501,7 +502,7 @@ def test_star(self):
501502
self.args = ('put',
502503
'https://api.github.com/user/starred/'
503504
'sigmavirus24/github3.py')
504-
self.conf = dict(headers={'Content-Length': '0'}, data=None)
505+
self.conf = {'headers': {'Content-Length': '0'}, 'data': None}
505506

506507
with expect.githuberror():
507508
self.g.star('foo', 'bar')
@@ -510,3 +511,91 @@ def test_star(self):
510511
expect(self.g.star(None, None)).is_False()
511512
expect(self.g.star('sigmavirus24', 'github3.py')).is_True()
512513
self.mock_assertions()
514+
515+
def test_subscribe(self):
516+
self.request.return_value = generate_response('', 204)
517+
self.args = ('put',
518+
'https://api.github.com/user/subscriptions/'
519+
'sigmavirus24/github3.py')
520+
self.conf = {'headers': {'Content-Length': '0'}, 'data': None}
521+
522+
with expect.githuberror():
523+
self.g.subscribe('foo', 'bar')
524+
525+
self.login()
526+
expect(self.g.subscribe(None, None)).is_False()
527+
expect(self.g.subscribe('sigmavirus24', 'github3.py')).is_True()
528+
self.mock_assertions()
529+
530+
def test_unfollow(self):
531+
self.request.return_value = generate_response('', 204)
532+
self.args = ('delete',
533+
'https://api.github.com/user/following/'
534+
'sigmavirus24')
535+
self.conf = {}
536+
537+
with expect.githuberror():
538+
self.g.unfollow('foo')
539+
540+
self.login()
541+
expect(self.g.unfollow(None)).is_False()
542+
expect(self.g.unfollow('sigmavirus24')).is_True()
543+
self.mock_assertions()
544+
545+
def test_unstar(self):
546+
self.request.return_value = generate_response('', 204)
547+
self.args = ('delete',
548+
'https://api.github.com/user/starred/'
549+
'sigmavirus24/github3.py')
550+
self.conf = {}
551+
552+
with expect.githuberror():
553+
self.g.unstar('foo', 'bar')
554+
555+
self.login()
556+
expect(self.g.unstar(None, None)).is_False()
557+
expect(self.g.unstar('sigmavirus24', 'github3.py')).is_True()
558+
self.mock_assertions()
559+
560+
def test_unsubscribe(self):
561+
self.request.return_value = generate_response('', 204)
562+
self.args = ('delete',
563+
'https://api.github.com/user/subscriptions/'
564+
'sigmavirus24/github3.py')
565+
self.conf = {}
566+
567+
with expect.githuberror():
568+
self.g.unsubscribe('foo', 'bar')
569+
570+
self.login()
571+
expect(self.g.unsubscribe(None, None)).is_False()
572+
expect(self.g.unsubscribe('sigmavirus24', 'github3.py')).is_True()
573+
self.mock_assertions()
574+
575+
def test_update_user(self):
576+
self.login()
577+
args = ('Ian Cordasco', 'example@mail.com', 'www.blog.com', 'company',
578+
'loc', True, 'bio')
579+
580+
with nested(patch.object(github3.github.GitHub, 'user'),
581+
patch.object(github3.users.User, 'update')) as (user, upd):
582+
user.return_value = github3.users.User(load('user'), self.g)
583+
upd.return_value = True
584+
expect(self.g.update_user(*args)).is_True()
585+
expect(user.called).is_True()
586+
expect(upd.called).is_True()
587+
upd.assert_called_with(*args)
588+
589+
def test_user(self):
590+
self.request.return_value = generate_response('user')
591+
self.args = ('get', 'https://api.github.com/users/sigmavir 341A us24')
592+
593+
expect(self.g.user('sigmavirus24')).isinstance(github3.users.User)
594+
self.mock_assertions()
595+
596+
self.args = ('get', 'https://api.github.com/user')
597+
self.login()
598+
expect(self.g.user()).isinstance(github3.users.User)
599+
self.mock_assertions()
600+
601+
# no test_zen

0 commit comments

Comments
 (0)
0