Skip to content

Created tests for startup logging#950

Merged
kosayoda merged 4 commits into
python-discord:masterfrom
ks129:logging-tests
Jun 22, 2020
Merged

Created tests for startup logging#950
kosayoda merged 4 commits into
python-discord:masterfrom
ks129:logging-tests

Conversation

@ks129
Copy link
Copy Markdown
Contributor

@ks129 ks129 commented May 17, 2020

The title says all. Closes #583

@ks129 ks129 requested a review from a team as a code owner May 17, 2020 13:23
@ks129 ks129 requested review from dementati and jb3 and removed request for a team May 17, 2020 13:23
Comment thread tests/bot/cogs/test_logging.py Outdated
Comment thread tests/bot/cogs/test_logging.py Outdated
@MarkKoz MarkKoz added a: tests Related to tests (e.g. unit tests) p: 3 - low Low Priority s: waiting for author Waiting for author to address a review or respond to a comment labels May 21, 2020
- Remove embed attributes checks
- Replace `self.dev_log.assert_awaited_once_with` with `self.dev_log.assert_awaited_once`.
@MarkKoz MarkKoz added status: needs review and removed s: waiting for author Waiting for author to address a review or respond to a comment labels May 29, 2020
@ghost ghost added the needs 1 review label Jun 13, 2020
@ghost ghost removed the needs 1 approval label Jun 22, 2020
@kosayoda kosayoda merged commit 03b6131 into python-discord:master Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: tests Related to tests (e.g. unit tests) p: 3 - low Low Priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Write unit tests for bot/cogs/logging.py

4 participants