diff --git a/tests/components/facebook/test_notify.py b/tests/components/facebook/test_notify.py index bbaa1f12516..77ae544646d 100644 --- a/tests/components/facebook/test_notify.py +++ b/tests/components/facebook/test_notify.py @@ -10,13 +10,15 @@ from homeassistant.core import HomeAssistant @pytest.fixture -def facebook(): +def facebook() -> fb.FacebookNotificationService: """Fixture for facebook.""" access_token = "page-access-token" return fb.FacebookNotificationService(access_token) -async def test_send_simple_message(hass: HomeAssistant, facebook) -> None: +async def test_send_simple_message( + hass: HomeAssistant, facebook: fb.FacebookNotificationService +) -> None: """Test sending a simple message with success.""" with requests_mock.Mocker() as mock: mock.register_uri(requests_mock.POST, fb.BASE_URL, status_code=HTTPStatus.OK) @@ -40,7 +42,9 @@ async def test_send_simple_message(hass: HomeAssistant, facebook) -> None: assert mock.last_request.qs == expected_params -async def test_send_multiple_message(hass: HomeAssistant, facebook) -> None: +async def test_send_multiple_message( + hass: HomeAssistant, facebook: fb.FacebookNotificationService +) -> None: """Test sending a message to multiple targets.""" with requests_mock.Mocker() as mock: mock.register_uri(requests_mock.POST, fb.BASE_URL, status_code=HTTPStatus.OK) @@ -66,7 +70,9 @@ async def test_send_multiple_message(hass: HomeAssistant, facebook) -> None: assert request.qs == expected_params -async def test_send_message_attachment(hass: HomeAssistant, facebook) -> None: +async def test_send_message_attachment( + hass: HomeAssistant, facebook: fb.FacebookNotificationService +) -> None: """Test sending a message with a remote attachment.""" with requests_mock.Mocker() as mock: mock.register_uri(requests_mock.POST, fb.BASE_URL, status_code=HTTPStatus.OK) @@ -95,32 +101,36 @@ async def test_send_message_attachment(hass: HomeAssistant, facebook) -> None: expected_params = {"access_token": ["page-access-token"]} assert mock.last_request.qs == expected_params - async def test_send_targetless_message(hass, facebook): - """Test sending a message without a target.""" - with requests_mock.Mocker() as mock: - mock.register_uri( - requests_mock.POST, fb.BASE_URL, status_code=HTTPStatus.OK - ) - facebook.send_message(message="going nowhere") - assert not mock.called +async def test_send_targetless_message( + hass: HomeAssistant, facebook: fb.FacebookNotificationService +) -> None: + """Test sending a message without a target.""" + with requests_mock.Mocker() as mock: + mock.register_uri(requests_mock.POST, fb.BASE_URL, status_code=HTTPStatus.OK) - async def test_send_message_with_400(hass, facebook): - """Test sending a message with a 400 from Facebook.""" - with requests_mock.Mocker() as mock: - mock.register_uri( - requests_mock.POST, - fb.BASE_URL, - status_code=HTTPStatus.BAD_REQUEST, - json={ - "error": { - "message": "Invalid OAuth access token.", - "type": "OAuthException", - "code": 190, - "fbtrace_id": "G4Da2pFp2Dp", - } - }, - ) - facebook.send_message(message="nope!", target=["+15555551234"]) - assert mock.called - assert mock.call_count == 1 + facebook.send_message(message="going nowhere") + assert not mock.called + + +async def test_send_message_with_400( + hass: HomeAssistant, facebook: fb.FacebookNotificationService +) -> None: + """Test sending a message with a 400 from Facebook.""" + with requests_mock.Mocker() as mock: + mock.register_uri( + requests_mock.POST, + fb.BASE_URL, + status_code=HTTPStatus.BAD_REQUEST, + json={ + "error": { + "message": "Invalid OAuth access token.", + "type": "OAuthException", + "code": 190, + "fbtrace_id": "G4Da2pFp2Dp", + } + }, + ) + facebook.send_message(message="nope!", target=["+15555551234"]) + assert mock.called + assert mock.call_count == 1