Repository: opsdroid/opsdroid (language: python)

Issue:
Package cleanup and updates
The goal of this PR is to lay the ground work for updating to new versions of Python. Make the CI all pass and generally try and declutter a little.


Discussion / hints:
## [Codecov](https://app.codecov.io/gh/opsdroid/opsdroid/pull/2010?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid) Report
All modified lines are covered by tests :white_check_mark:
> Comparison is base [(`41246da`)](https://app.codecov.io/gh/opsdroid/opsdroid/commit/41246da2f6f379a889dadd1d3b4e139b65d3c9fb?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid) 99.16% compared to head [(`af4bad3`)](https://app.codecov.io/gh/opsdroid/opsdroid/pull/2010?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid) 98.99%.

<details><summary>Additional details and impacted files</summary>


```diff
@@            Coverage Diff             @@
##           master    #2010      +/-   ##
==========================================
- Coverage   99.16%   98.99%   -0.17%     
==========================================
  Files          83       83              
  Lines        5395     5393       -2     
==========================================
- Hits         5350     5339      -11     
- Misses         45       54       +9     
```


| [Files](https://app.codecov.io/gh/opsdroid/opsdroid/pull/2010?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid) | Coverage Δ | |
|---|---|---|
| [opsdroid/\_\_init\_\_.py](https://app.codecov.io/gh/opsdroid/opsdroid/pull/2010?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid#diff-b3BzZHJvaWQvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | |

... and [1 file with indirect coverage changes](https://app.codecov.io/gh/opsdroid/opsdroid/pull/2010/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid)


</details>

[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/opsdroid/opsdroid/pull/2010?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid).   
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opsdroid).

Looks good to me. 👍 
It would be good to have another opinion if we should remove the video property test or not. Or, maybe, rework it.

Fix patch:
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 3927be121..eb5fda874 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-20.04, macos-latest, windows-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
         python-version: ["3.8", "3.9", "3.10"]
     steps:
       - uses: actions/checkout@v2
@@ -21,13 +21,9 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install libolm
-        if: runner.os != 'Windows'
+        if: runner.os == 'macos'
         run: |
-          if [ $RUNNER_OS = Linux ]; then
-              sudo apt install libolm3 libolm-dev -y
-          else
-              brew install libolm
-          fi
+          brew install libolm
       - name: Install ffmpeg on Ubuntu
         if: runner.os != 'Windows'
         run: |
diff --git a/setup.cfg b/setup.cfg
index b53c959af..9b414e371 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -88,6 +88,7 @@ connector_matrix =
 connector_matrix_e2e =
   bleach>=3.1.5
   matrix-nio[e2e]
+  python-olm
 connector_mattermost =
   mattermostdriver>=7.0.1
 connector_slack =


Test patch:
diff --git a/tests/test_events.py b/tests/test_events.py
index db0e65f25..29a74a5db 100644
--- a/tests/test_events.py
+++ b/tests/test_events.py
@@ -430,7 +430,6 @@ async def test_video(self):
         self.assertEqual(await event.get_file_bytes(), self.mkv_bytes)
         self.assertEqual(await event.get_mimetype(), "video/x-matroska")
         self.assertEqual(await event.get_bin(), self.vid_bin)
-        self.assertEqual(await event.get_properties(), self.props)
 
     async def test_explicit_mime_type(self):
         opsdroid = amock.CoroutineMock()