-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix unlisted tracks in track upload challenge #2382
Conversation
@@ -135,6 +150,17 @@ def test_track_upload_challenge(app): | |||
assert user_challenge.current_step_count == 1 | |||
assert not user_challenge.is_complete | |||
|
|||
# Ensure unlisted track is not counted | |||
session.add(unlisted_track) | |||
bus.dispatch(ChallengeEvent.track_upload, 30000000, 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be 30000001?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't affect the test but for consistency yep, will change
@@ -58,6 +58,7 @@ def _get_num_track_uploads_by_user( | |||
Track.blocknumber >= block_number, | |||
Track.is_current == True, | |||
Track.is_delete == False, | |||
Track.is_unlisted == False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we do a stem check too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great idea
* Fix unlisted tracks in track upload challenge * PR revisions * Fix lint
Description
Stop counting unlisted tracks towards track upload completion.
Tests
How will this change be monitored?