Skip to content

Fix enhance integration test: wrong param name and import#4

Merged
daveronan merged 1 commit into
masterfrom
mix-enhance-update
Apr 21, 2026
Merged

Fix enhance integration test: wrong param name and import#4
daveronan merged 1 commit into
masterfrom
mix-enhance-update

Conversation

@daveronan
Copy link
Copy Markdown
Collaborator

@daveronan daveronan commented Apr 21, 2026

Summary

  • Fix retrieve_enhanced_track call in integration test: max_attempts -> timeout to match the current method signature
  • Fix import: MusicalStyle -> EnhanceMusicalStyle (correct enum for the enhance endpoint)

Test plan

  • 143 unit tests passing
  • Run enhance integration test manually to verify (@pytest.mark.skip by default)

Note

Low Risk
Low risk: changes are confined to a skipped integration test and only adjust argument names/imports to match current client APIs.

Overview
Fixes the enhance integration test to match current SDK interfaces by switching the style enum from MusicalStyle to EnhanceMusicalStyle and updating the polling call to use timeout instead of the old max_attempts parameter.

Reviewed by Cursor Bugbot for commit 3b2fb9d. Bugbot is set up for automated code reviews on this repo. Configure here.

- max_attempts -> timeout to match retrieve_enhanced_track signature
- MusicalStyle -> EnhanceMusicalStyle (correct enum for enhance)

Made-with: Cursor
@daveronan daveronan merged commit 9b96fb5 into master Apr 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant