Fix Home Assistant media viewer for new Frigate versions

[madsciencetist]

blakeblackshear/frigate#6320 changed the events API to move box, region, score and top_score into a "data" sub-struct

[madsciencetist]

@NickM-27 Do these old references in frigate need to be updated as well?
maintainer.py:31
ha_notifications.md:79
mqtt.md:25-92

[NickM-27]

No those are all correct, all of those are different to the event DB structure (which comes from the API)

[madsciencetist]

Even maintainer.py:31? Those are Events

[NickM-27]

Even maintainer.py:31? Those are Events

They are just labeled that way because mypy requires a label, they are actually just a dict and come from here https://github.com/blakeblackshear/frigate/blob/f2ff55dba2c8548fedca3f4a4a3da7d7a5feaaf8/frigate/object_processing.py#L264 so yes it is correct

[dermotduffy]

@madsciencetist Mind fixing up the tests and we can get this in? Thanks in advance!

[madsciencetist]

Fixed

[codecov]

Codecov Report

Merging #518 (a9b6e61) into dev (96f912b) will not change coverage.
Report is 12 commits behind head on dev.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               dev      #518   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines         1929      1936    +7     
=========================================
+ Hits          1929      1936    +7     

Files Changed	Coverage Δ
custom_components/frigate/media_source.py	100.00% <ø> (ø)
custom_components/frigate/camera.py	100.00% <100.00%> (ø)
custom_components/frigate/const.py	100.00% <100.00%> (ø)

[dermotduffy]

@madsciencetist Almost! Looks like formatting isn't quite right, run the precommit manually and commit/push the changes?

[madsciencetist]

Might I suggest that you not require workflow approval, so that CI failures are immediately evident!

[dermotduffy]

Might I suggest that you not require workflow approval, so that CI failures are immediately evident!

I'll counter-suggest installing the precommit in your local client with pre-commit install so it runs before your commits, that will catch these issues immediately in your local environment. With that said, I wouldn't object at all to changing the workflow approvals, but that's something only the repo owner (not me!) can do.

[dermotduffy]

Thanks @madsciencetist !