Remove deprecated usage of type helpers

legrego / homeassistant-elasticsearch

Publish Home-Assistant events to Elasticsearch

https://legrego.github.io/homeassistant-elasticsearch/

MIT License

143 stars 38 forks source link

Remove deprecated usage of type helpers #300

Closed legrego closed 2 weeks ago

legrego commented 2 weeks ago

https://developers.home-assistant.io/blog/2024/04/08/deprecated-backports-and-typing-aliases/

homeassistant.helpers.typing.EventType has been replaced with homeassistant.core.Event. The former is deprecated and will be removed in a future release of Home Assistant.
homeassistant.helpers.typing.HomeAssistantType has been replaced with homeassistant.core.HomeAssistant. The former is deprecated and will be removed in a future release of Home Assistant.

Relates #283

github-actions[bot] commented 2 weeks ago

Coverage Report

File	Stmts	Miss	Cover	Missing
__init__.py	0	0	100%
elasticsearch
__init__.py	112	6	94%	119, 153–154, 174–175, 269
config_flow.py	222	20	90%	165, 198, 328, 365, 379, 438–439, 450–452, 466–467, 471, 510, 516, 556, 631, 640, 674–675
const.py	34	0	100%
entity_details.py	33	0	100%
errors.py	28	3	89%	51, 60, 67
es_doc_creator.py	193	1	99%	322
es_doc_publisher.py	267	40	85%	68–69, 139–140, 179, 182, 185, 190, 207–208, 226–229, 233, 248–249, 265–266, 270, 295, 299, 393, 413, 455, 469, 480, 483, 511–512, 514–515, 517–518, 522–524, 543, 545, 551
es_gateway.py	144	20	86%	94–95, 114, 133–135, 137, 139–140, 142–143, 146–148, 153–154, 158–160, 253
es_index_manager.py	123	11	91%	43, 74, 79, 242–243, 260–261, 266–267, 296–297
es_integration.py	35	2	94%	45–46
es_privilege_check.py	60	0	100%
es_serializer.py	10	1	90%	17
es_version.py	30	0	100%
logger.py	2	0	100%
system_info.py	25	1	96%	37
utils.py	4	0	100%
TOTAL	1322	105	92%

Tests	Skipped	Failures	Errors	Time
222	0 :zzz:	0 :x:	0 :fire:	12.613s :stopwatch:

strawgate commented 2 weeks ago

this looks like it's going to cause a ton of merge conflicts with the config PR?

If we can merge the config one first I'd be happy to update this branch after

legrego commented 2 weeks ago

this looks like it's going to cause a ton of merge conflicts with the config PR?

If we can merge the config one first I'd be happy to update this branch after

Works for me

legrego commented 2 weeks ago

hmm these tests pass locally.

Local:

---------------------------------------- snapshot report summary ----------------------------------------
127 snapshots passed.

CI:

--------------------------- snapshot report summary ----------------------------
127 snapshots passed. 1 snapshot unused.

Re-run pytest with --snapshot-update to delete unused snapshots.
============================= 222 passed in 11.25s =============================
Error: Process completed with exit code 1.