From de2c7806ca52e9c5470705f5ebb4955f8ba3942c Mon Sep 17 00:00:00 2001 From: David Medina Date: Tue, 7 Feb 2012 22:51:46 +0100 Subject: Fix litle bugs --- pygithub3/requests/users/emails.py | 8 +++++--- pygithub3/requests/users/user.py | 9 ++++++++- 2 files changed, 13 insertions(+), 4 deletions(-) (limited to 'pygithub3/requests/users') diff --git a/pygithub3/requests/users/emails.py b/pygithub3/requests/users/emails.py index 7622fcc..410bd9a 100644 --- a/pygithub3/requests/users/emails.py +++ b/pygithub3/requests/users/emails.py @@ -3,12 +3,12 @@ import re -from . import Request +from . import Request, ValidationError class List(Request): - uri = 'users/emails' + uri = 'user/emails' class Add(Request): @@ -16,9 +16,11 @@ class Add(Request): uri = 'user/emails' def clean_body(self): - def is_email(email): return re.match(r'.*', email) # TODO: email regex ;) + if not self.body: + raise ValidationError("'%s' request needs emails" + % (self.__class__.__name__)) return filter(is_email, self.body) diff --git a/pygithub3/requests/users/user.py b/pygithub3/requests/users/user.py index 78ab6cd..740b3dd 100644 --- a/pygithub3/requests/users/user.py +++ b/pygithub3/requests/users/user.py @@ -22,4 +22,11 @@ class Update(Request): resource = User uri = 'user' body_schema = ( - 'name', 'email', 'blog', 'company', 'location','hireable', 'bio') + 'name', 'email', 'blog', 'company', 'location', 'hireable', 'bio') + + def clean_body(self): + if not self.body: + raise ValidationError("'%s' request needs data. You can use " + "'%s' keys" % (self.__class__.__name__, + self.body_schema)) + return self.body -- cgit v1.2.3-59-g8ed1b