Merge branch 'master' into joshferge/feat/assign-prs-to-user-v2

Author: JoshFerge

src/sentry/integrations/slack/unfurl/explore.py

@@ -157,8 +157,10 @@ def _unfurl_explore(
             _logger.warning("Failed to generate chart for explore unfurl")
             continue
 
+        # Only one chart/y-axis is supported at a time in Explore
+        title = f"{defaults['title']} - {y_axes[0]}"
         unfurls[link.url] = SlackDiscoverMessageBuilder(
-            title=defaults["title"],
+            title=title,
             chart_url=url,
         ).build()
 

tests/sentry/integrations/slack/test_unfurl.py

@@ -1522,7 +1522,9 @@ def test_unfurl_explore(
 
         assert (
             unfurls[url]
-            == SlackDiscoverMessageBuilder(title="Explore Traces", chart_url="chart-url").build()
+            == SlackDiscoverMessageBuilder(
+                title="Explore Traces - avg(span.duration)", chart_url="chart-url"
+            ).build()
         )
         assert len(mock_generate_chart.mock_calls) == 1
         assert mock_generate_chart.call_args[0][0] == ChartType.SLACK_EXPLORE_LINE
@@ -1695,7 +1697,9 @@ def test_unfurl_explore_end_to_end(
         assert len(unfurls) == 1
         assert (
             unfurls[url]
-            == SlackDiscoverMessageBuilder(title="Explore Traces", chart_url="chart-url").build()
+            == SlackDiscoverMessageBuilder(
+                title="Explore Traces - avg(span.duration)", chart_url="chart-url"
+            ).build()
         )
 
     @patch(
@@ -1806,7 +1810,9 @@ def test_unfurl_explore_logs(
 
         assert (
             unfurls[url]
-            == SlackDiscoverMessageBuilder(title="Explore Logs", chart_url="chart-url").build()
+            == SlackDiscoverMessageBuilder(
+                title="Explore Logs - sum(payload_size)", chart_url="chart-url"
+            ).build()
         )
         assert len(mock_generate_chart.mock_calls) == 1
         chart_data = mock_generate_chart.call_args[0][1]