|
| 1 | +""" |
| 2 | +Tests for MusicBrainz integration in views. |
| 3 | +
|
| 4 | +These tests verify the application behavior, not the external API. |
| 5 | +""" |
| 6 | + |
| 7 | +from unittest.mock import MagicMock, patch |
| 8 | + |
| 9 | +import requests |
| 10 | +from django.urls import reverse |
| 11 | + |
| 12 | +from core.services.musicbrainz import MusicBrainzResult |
| 13 | + |
| 14 | + |
| 15 | +class TestMusicBrainzSearchView: |
| 16 | + """Tests for the musicbrainz_search_htmx view.""" |
| 17 | + |
| 18 | + def test_returns_empty_for_short_query(self, logged_in_client): |
| 19 | + """Returns empty results for queries shorter than minimum length.""" |
| 20 | + response = logged_in_client.get(reverse("musicbrainz_search_htmx"), {"q": "a"}) |
| 21 | + |
| 22 | + assert response.status_code == 200 |
| 23 | + assert "partials/musicbrainz/musicbrainz_suggestions.html" in [t.name for t in response.templates] |
| 24 | + assert response.context["results"] == [] |
| 25 | + |
| 26 | + def test_returns_empty_for_empty_query(self, logged_in_client): |
| 27 | + """Returns empty results for empty query.""" |
| 28 | + response = logged_in_client.get(reverse("musicbrainz_search_htmx"), {"q": ""}) |
| 29 | + |
| 30 | + assert response.status_code == 200 |
| 31 | + assert response.context["results"] == [] |
| 32 | + |
| 33 | + @patch("core.views.get_musicbrainz_client") |
| 34 | + def test_returns_search_results(self, mock_get_client, logged_in_client): |
| 35 | + """Returns search results from MusicBrainz client.""" |
| 36 | + mock_client = MagicMock() |
| 37 | + mock_client.search_releases.return_value = [ |
| 38 | + MusicBrainzResult( |
| 39 | + mbid="test-mbid-123", |
| 40 | + title="Abbey Road", |
| 41 | + artists=["The Beatles"], |
| 42 | + year=1969, |
| 43 | + country="GB", |
| 44 | + label="Apple Records", |
| 45 | + ) |
| 46 | + ] |
| 47 | + mock_get_client.return_value = mock_client |
| 48 | + |
| 49 | + response = logged_in_client.get(reverse("musicbrainz_search_htmx"), {"q": "abbey road"}) |
| 50 | + |
| 51 | + assert response.status_code == 200 |
| 52 | + assert len(response.context["results"]) == 1 |
| 53 | + assert response.context["results"][0].title == "Abbey Road" |
| 54 | + mock_client.search_releases.assert_called_once() |
| 55 | + |
| 56 | + @patch("core.views.get_musicbrainz_client") |
| 57 | + def test_handles_api_error_gracefully(self, mock_get_client, logged_in_client): |
| 58 | + """Handles API errors gracefully and shows error message.""" |
| 59 | + mock_client = MagicMock() |
| 60 | + mock_client.search_releases.side_effect = requests.RequestException("API Error") |
| 61 | + mock_get_client.return_value = mock_client |
| 62 | + |
| 63 | + response = logged_in_client.get(reverse("musicbrainz_search_htmx"), {"q": "test query"}) |
| 64 | + |
| 65 | + assert response.status_code == 200 |
| 66 | + assert "error" in response.context |
| 67 | + assert response.context["error"] == "Search failed" |
| 68 | + |
| 69 | + def test_preserves_media_id_in_context(self, logged_in_client): |
| 70 | + """Preserves media_id in context for editing existing media.""" |
| 71 | + response = logged_in_client.get(reverse("musicbrainz_search_htmx"), {"q": "", "media_id": "42"}) |
| 72 | + |
| 73 | + assert response.status_code == 200 |
| 74 | + assert response.context["media_id"] == "42" |
| 75 | + |
| 76 | + @patch("core.views.get_musicbrainz_client") |
| 77 | + def test_preserves_query_in_context(self, mock_get_client, logged_in_client): |
| 78 | + """Preserves search query in context.""" |
| 79 | + mock_client = MagicMock() |
| 80 | + mock_client.search_releases.return_value = [] |
| 81 | + mock_get_client.return_value = mock_client |
| 82 | + |
| 83 | + response = logged_in_client.get(reverse("musicbrainz_search_htmx"), {"q": "test"}) |
| 84 | + |
| 85 | + assert response.status_code == 200 |
| 86 | + assert response.context["query"] == "test" |
0 commit comments