[PAY-3814] Endless Listen Streak Challenge

[dharit-tan]

Description

• First challenge will have step_count = 7 and amount = 7, will only be marked complete once 7-day listen streak is achieved.
• For subsequent days of the streak we add 1 new row per day with step_count = 1 and amount = 1.
• Specifier is now date-dependent to allow multiple user_challenge listen streak rows per user (formerly only 1 per user).
• Unfortunately need to add db query in generate_specifier to determine whether to use the existing user_challenge row, or create a new one. But this is the only challenge that will have this query.
• There will still only ever be 1 ChallengeListenStreak row per user - another db query to determine whether one exists or not in on_after_challenge_creation.
• Allow querying of user_challenge rows without specifiers for tests

How Has This Been Tested?

Integration tests pass

[changeset-bot]

⚠️ No Changeset found

Latest commit: 7511756

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[rickyrombo]

I don't think generate_specifier needs to go to the DB actually?

There's several possible outcomes:

• This is day 1 of the 7 days, and we need to create a new challenge
  • Use today's date, should_create_challenge is True
• This is day 2-7, and we need to update the existing challenge
  • Can use anything for the specifier, as should_create_challenge is False and instead we will update the existing challenge in update_user_challenges
• This is day 8+, and we need to create a new challenge for the endless streak
  • Use today's date, should_create_challenge is True

Is there someplace where we need the existing challenge's specifier?

EDIT: yes, it's in challenge manager before calling update challenge on the updater

[isaacsolo]

looking gooooood

[raymondjacobson]

lgtm looks like you've got a test failure though