Implement stopwatch feature

[Themanwhosmellslikesugar]

No description provided.

[warewarewapain]

What do you think about using self.assertEqual here?

[warewarewapain]

Suggested change

	# we can't use async writing to a file because it won't execute in time
	# We can't use async writing to a file because it won't execute in time

[warewarewapain]

Suggested change

	@override_settings(HANDLERS_STOPWATCH=True)
	def test_setup_sets_event_loop_policy_when_stopwatch_enabled(self):
	"""Test _setup sets event loop policy when stopwatch is enabled."""
	bot = object.__new__(Bot)
	@override_settings(HANDLERS_STOPWATCH=True, TOKEN='secret')
	def test_setup_sets_event_loop_policy_when_stopwatch_enabled(self):
	"""Test _setup sets event loop policy when stopwatch is enabled."""
	bot = get_bot()

[warewarewapain]

Suggested change

	@override_settings(HANDLERS_STOPWATCH=False)
	def test_setup_does_not_set_event_loop_policy_when_disabled(self):
	"""Test _setup skips setting the event loop policy when disabled."""
	bot = object.__new__(Bot)
	with (
	@override_settings(HANDLERS_STOPWATCH=False, TOKEN='secret')
	def test_setup_does_not_set_event_loop_policy_when_disabled(self):
	"""Test _setup skips setting the event loop policy when disabled."""
	bot = get_bot()
	with (

[warewarewapain]

Suggested change

	with patch('hammett.stopwatch.collect_handler_stats') as mock_collect_handler_stats:
	mock_collect_handler_stats.side_effect = lambda handler: handler
	get_bot()
	with patch('hammett.stopwatch.collect_handler_stats') as mock_collect_handler_stats:
	mock_collect_handler_stats.side_effect = lambda handler: handler
	get_bot()

[Themanwhosmellslikesugar]

This does not affect the readability of the code and there is no point in arguing over this empty line. I suggest leaving it as it is

[warewarewapain]

What do you think about this? I believe we can still combine the two tests, and it won’t look complicated.

    def test_wraps_handler_preserves_assigned_attributes(self):
        """Test preserving wrapper assignments from the wrapped handler."""

        class TestScreen:
            """The class implements a test screen for testing purposes."""

            def handler(self):
                """Represent a stub handler for the testing purposes."""
                return 'original'

        def wrapper_handler():
            """Represent a wrapper handler for the testing purposes."""
            return 'wrapper'

        screen = TestScreen()
        handler = screen.handler
        wrapped_handler = wraps_handler(handler)(wrapper_handler)

        self.assertIs(wrapped_handler, wrapper_handler)
        self.assertEqual(wrapped_handler.__name__, handler.__name__)
        self.assertEqual(wrapped_handler.__qualname__, handler.__qualname__)
        self.assertEqual(wrapped_handler.__module__, handler.__module__)
        self.assertEqual(wrapped_handler.__doc__, handler.__doc__)
        self.assertIs(wrapped_handler.__wrapped__, handler)
        self.assertIs(wrapped_handler.__self__, screen)