Fix pylint issues

moshthepitt · Feb 12, 2019 · 512050a · 512050a
1 parent 6d53033
commit 512050a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/small_small_hr/emails.py b/small_small_hr/emails.py
@@ -8,7 +8,7 @@
 from django.utils.translation import ugettext as _
 
 
-def send_email(  # pylint: disable=too-many-locals
+def send_email(  # pylint: disable=too-many-arguments, too-many-locals
         name: str, email: str, subject: str, message: str, obj: object = None,
         cc_list: list = None, template: str = 'generic'):
     """

diff --git a/tests/test_forms.py b/tests/test_forms.py
@@ -470,6 +470,8 @@ def test_leave_oversubscribe_off(self, mock):
         """
         Test leave oversubscribe when SSHR_ALLOW_OVERSUBSCRIBE is False
         """
+        mock.return_value = None
+
         user = mommy.make('auth.User', first_name='Bob', last_name='Ndoe')
         staffprofile = mommy.make('small_small_hr.StaffProfile', user=user)
         staffprofile.leave_days = 21