Add unit tests

.github/workflows/status_embed.yaml

@@ -3,7 +3,7 @@ name: Status Embed
 on:
   workflow_run:
     workflows:
-      - Lint
+      - Validation
     types:
       - completed
 
@@ -15,7 +15,7 @@ jobs:
     steps:
       # A workflow_run event does not contain all the information
       # we need for a PR embed. That's why we upload an artifact
-      # with that information in the Lint workflow.
+      # with that information in the Validation workflow.
       - name: Get Pull Request Information
         id: pr_info
         if: github.event.workflow_run.event == 'pull_request'

.github/workflows/lint.yaml → .github/workflows/validation.yaml

@@ -1,4 +1,4 @@
-name: Lint
+name: Validation
 
 on:
   push:
@@ -20,8 +20,8 @@ env:
   PRE_COMMIT_HOME: ${{ github.workspace }}/.cache/pre-commit-cache
 
 jobs:
-  lint:
-    name: Lint
+  validation:
+    name: Validation
     runs-on: ubuntu-latest
 
     steps:
@@ -80,6 +80,17 @@ jobs:
         [flake8] %(code)s: %(text)s' \
         --extend-exclude '.cache'"
 
+      # Run all of the unit tests and generate coverage report with
+      # coverage.py
+      - name: Run tests and generate coverage report
+        run: pytest --cov --disable-warnings -q
+
+      # Publish test coverage report to coveralls.io, print a
+      # "job" link in the output of the GitHub Action
+      - name: Publish coverage report to coveralls.io
+        env:
+            GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: coveralls --service=github
 
       # Prepare the Pull Request Payload artifact. If this fails, we
       # we fail silently using the `continue-on-error` option. It's

README.md

@@ -1,8 +1,9 @@
 # Neutron Bot
 
-[![mit](https://img.shields.io/badge/Licensed%20under-GPL-red.svg?style=flat-square)](./LICENSE)
 ![Python package](https://github.com/Codin-Nerds/Neutron-Bot/workflows/Python%20package/badge.svg)
+[![Coverage Status](https://coveralls.io/repos/github/Codin-Nerds/Neutron-Bot/badge.svg?branch=main)](https://coveralls.io/github/Codin-Nerds/Neutron-Bot?branch=main)
 [![made-with-python](https://img.shields.io/badge/Made%20with-Python%203.9-ffe900.svg?longCache=true&style=flat-square&colorB=00a1ff&logo=python&logoColor=88889e)](https://www.python.org/)
+[![license](https://img.shields.io/badge/Licensed%20under-GPL-red.svg?style=flat-square)](./LICENSE)
 [![Discord](https://img.shields.io/static/v1?label=The%20Codin'%20Nerds&logo=discord&message=%3E600%20members&color=%237289DA&logoColor=white)](https://discord.gg/Dhz9pM7)
 
 ## About the bot

bot/__main__.py

@@ -18,4 +18,4 @@
 
 
 if __name__ == "__main__":
-    bot.run(config.TOKEN)
+    bot.run(config.TOKEN)  # pragma: no cover

bot/cogs/core/error_handler.py

@@ -134,7 +134,7 @@ async def handle_json_decode_error(self, ctx: Context, exception: JSONDecodeErro
             The error occurred on *`line {exception.lineno} column {exception.colno} (char {exception.pos})`*
             """
         )
-        if exception.lines:
+        if hasattr(exception, "lines") and len(exception.lines) >= exception.lineno:
             msg += textwrap.dedent(
                 f"""
                 ```

bot/cogs/moderation/slowmode.py

@@ -25,7 +25,7 @@ async def slow_mode(self, ctx: Context, duration: Duration) -> None:
         await ctx.channel.edit(slowmode_delay=duration)
 
         if duration:
-            log_msg = f"ser {ctx.author} applied slowmode to #{ctx.channel} for {stringify_duration(duration)}"
+            log_msg = f"User {ctx.author} applied slowmode to #{ctx.channel} for {stringify_duration(duration)}"
             msg = f"⏱️ Applied slowmode for this channel, time delay: {stringify_duration(duration)}."
         else:
             log_msg = f"User {ctx.author} removed slowmode from #{ctx.channel}"

bot/core/bot.py

@@ -25,6 +25,12 @@ def __init__(self, *args, **kwargs) -> None:
         self.initial_call = True
         self._ignored_logs = defaultdict(set)
 
+        # These attributes will be set later, once the bot starts up
+        # they can't be set here, because they need asyncio's event loop
+        # however we still want to define them here to provide type hints
+        self.http_session: t.Optional[aiohttp.ClientSession] = None
+        self.db_engine: t.Optional[AsyncEngine] = None
+
     async def load_extensions(self) -> None:
         """Load all listed cogs."""
         for extension in autoload.EXTENSIONS: