Commit 6e32bf78 authored by Raphaël Hertzog's avatar Raphaël Hertzog

Fix capitalization of docstrings

parent 054857b2
......@@ -966,7 +966,7 @@ class UpdateVersionInformationTest(TestCase):
self.repo1.shorthand)
def test_task_cleanup_only(self):
"""test the task when it only has to process a cleanup"""
"""Test the task when it only has to process a cleanup."""
self.update.execute()
package_name = self.package.source_package_name
self.package.delete()
......
......@@ -941,7 +941,7 @@ class ProcessItemsTests(TestCase):
self.assertEqual(count, 1)
def test_items_cleanup_processed_list(self):
"""drops keys not associated to any object from the processed list"""
"""Drops keys not associated to any object from the processed list."""
unused_item = self.setup_item_to_cleanup()
# Check the removal from the processed list through return value of
......@@ -951,7 +951,7 @@ class ProcessItemsTests(TestCase):
self.assertTrue(self.task.item_needs_processing(unused_item))
def test_items_cleanup_processed_list_does_mark_data_modified(self):
"""when items are cleaned up, data is modified"""
"""When items are cleaned up, data is modified."""
self.setup_item_to_cleanup()
with mock.patch.object(self.task, 'data_mark_modified') as mocked:
......@@ -959,7 +959,7 @@ class ProcessItemsTests(TestCase):
mocked.assert_called_once_with()
def test_items_cleanup_processed_list_does_not_mark_data_modified(self):
"""nothing to cleanup, no data modified"""
"""Nothing to cleanup, no data modified."""
self.patch_items_all()
with mock.patch.object(self.task, 'data_mark_modified') as mocked:
self.task.items_cleanup_processed_list()
......@@ -1014,7 +1014,7 @@ class ProcessModelTests(TestCase):
self.assertIs(queryset, mock.sentinel.extended_queryset)
def test_items_extend_queryset(self):
"""default items_extend_queryset() just forwards the queryset"""
"""Default items_extend_queryset() just forwards the queryset."""
queryset = mock.sentinel.queryset
self.assertEqual(self.task.items_extend_queryset(queryset),
queryset)
......@@ -1046,7 +1046,7 @@ class ProcessModelTests(TestCase):
self.assertIs(data['get_absolute_url'], mock.sentinel.url)
def test_items_to_process_after_save_reload(self):
"""ensure we don't reprocess an item already seen"""
"""Ensure we don't reprocess an item already seen."""
srcpkgname = SourcePackageName.objects.create(name='dummy')
self.task.item_mark_processed(srcpkgname)
self.task.save_data()
......
......@@ -576,7 +576,7 @@ class ProcessMailTests(TestCase):
@mock.patch('distro_tracker.mail.management.commands.tracker_process_mail.'
'MailQueue')
def test_process_mail_command(self, mock_queue):
"""command is a simple wrapper around MailQueue.process_loop()"""
"""Command is a simple wrapper around MailQueue.process_loop()"""
call_command('tracker_process_mail')
mock_queue.assert_called_with()
mock_queue.return_value.process_loop.assert_called_with()
......@@ -221,7 +221,7 @@ class MailProcessorTest(TestCase, HelperMixin):
None)
def test_load_mail_from_file(self):
"""loads the mail to process from a file"""
"""Loads the mail to process from a file."""
mail_path = os.path.join(settings.DISTRO_TRACKER_DATA_PATH, 'a-mail')
self.create_mail(mail_path, subject='load_mail')
......@@ -230,7 +230,7 @@ class MailProcessorTest(TestCase, HelperMixin):
self.assertEqual(self.processor.message['Subject'], 'load_mail')
def test_init_with_filename(self):
"""can create object with filename"""
"""Can create object with filename."""
mail_path = os.path.join(settings.DISTRO_TRACKER_DATA_PATH, 'a-mail')
self.create_mail(mail_path, subject='load_mail')
......
......@@ -106,9 +106,7 @@ ignore =
# First line should be in imperative mood
D401,
# First line should not be the function's "signature"
D402,
# First word of the first line should be properly capitalized
D403
D402
enable-extensions = G
application-import-names = distro_tracker,django_email_accounts
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment