diff --git a/superset/reports/notifications/slack.py b/superset/reports/notifications/slack.py index b70cab4435..9ecf8de6d5 100644 --- a/superset/reports/notifications/slack.py +++ b/superset/reports/notifications/slack.py @@ -184,7 +184,7 @@ Error: %(text)s # files_upload returns SlackResponse as we run it in sync mode. if files: for file in files: - client.files_upload( + client.files_upload_v2( channels=channel, file=file, initial_comment=body, diff --git a/tests/integration_tests/reports/commands_tests.py b/tests/integration_tests/reports/commands_tests.py index 30d1834852..e57912759a 100644 --- a/tests/integration_tests/reports/commands_tests.py +++ b/tests/integration_tests/reports/commands_tests.py @@ -1098,7 +1098,7 @@ def test_email_dashboard_report_schedule_force_screenshot( @pytest.mark.usefixtures( "load_birth_names_dashboard_with_slices", "create_report_slack_chart" ) -@patch("superset.utils.slack.WebClient.files_upload") +@patch("superset.utils.slack.WebClient.files_upload_v2") @patch("superset.utils.screenshots.ChartScreenshot.get_screenshot") def test_slack_chart_report_schedule( screenshot_mock, @@ -1186,7 +1186,7 @@ def test_slack_chart_report_schedule_with_errors( @pytest.mark.usefixtures( "load_birth_names_dashboard_with_slices", "create_report_slack_chart_with_csv" ) -@patch("superset.utils.slack.WebClient.files_upload") +@patch("superset.utils.slack.WebClient.files_upload_v2") @patch("superset.utils.csv.urllib.request.urlopen") @patch("superset.utils.csv.urllib.request.OpenerDirector.open") @patch("superset.utils.csv.get_chart_csv_data") @@ -1353,7 +1353,7 @@ def test_report_schedule_success_grace(create_alert_slack_chart_success): @pytest.mark.usefixtures("create_alert_slack_chart_grace") -@patch("superset.utils.slack.WebClient.files_upload") +@patch("superset.utils.slack.WebClient.files_upload_v2") @patch("superset.utils.screenshots.ChartScreenshot.get_screenshot") def test_report_schedule_success_grace_end( screenshot_mock, file_upload_mock, create_alert_slack_chart_grace