[PR #259] [MERGED] Add test for percent search bug #350

New issue

Closed

opened 2026-02-27 20:24:22 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 20:24:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/259
Author: @marioortizmanero
Created: 9/25/2021
Status: ✅ Merged
Merged: 9/30/2021
Merged by: @ramsayleung

Base: master ← Head: percent-search

📝 Commits (6)

c877029 Add test for percent search bug
d80ab80 Fix current_user_saved_{tracks,albums}
fc56139 Update changelog
ef3a000 Fix examples and tests
d0add84 Merge pull request #260 from ramsayleung/is-playable-tests
731355a Merge branch 'master' into percent-search

📊 Changes

6 files changed (+48 additions, -20 deletions)

View changed files

📝 CHANGELOG.md (+1 -0)
📝 examples/pagination_async.rs (+2 -2)
📝 examples/pagination_manual.rs (+1 -1)
📝 examples/pagination_sync.rs (+1 -1)
📝 src/clients/oauth.rs (+18 -4)
📝 tests/test_with_oauth.rs (+25 -12)

📄 Description

Description

This closes #141, which has been fixed magically :P

Dependencies

None

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

I've added a new test that's equivalent to the one that was failing.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ramsayleung/rspotify/pull/259 **Author:** [@marioortizmanero](https://github.com/marioortizmanero) **Created:** 9/25/2021 **Status:** ✅ Merged **Merged:** 9/30/2021 **Merged by:** [@ramsayleung](https://github.com/ramsayleung) **Base:** `master` ← **Head:** `percent-search` --- ### 📝 Commits (6) - [`c877029`](https://github.com/ramsayleung/rspotify/commit/c87702935c5ab3cfb8f00fd71bd12f8a346a0533) Add test for percent search bug - [`d80ab80`](https://github.com/ramsayleung/rspotify/commit/d80ab80d797344c8aa722f704525ad68dcfc4576) Fix `current_user_saved_{tracks,albums}` - [`fc56139`](https://github.com/ramsayleung/rspotify/commit/fc561395a9e543e4fb1468d229c798994b6a9ae9) Update changelog - [`ef3a000`](https://github.com/ramsayleung/rspotify/commit/ef3a000ce02679df0d1f56c1daf3596915332cee) Fix examples and tests - [`d0add84`](https://github.com/ramsayleung/rspotify/commit/d0add840cdc3b0ee6c82e42aaecc082c8169895a) Merge pull request #260 from ramsayleung/is-playable-tests - [`731355a`](https://github.com/ramsayleung/rspotify/commit/731355ae045e514d29a18b59dcaef282fa825b2f) Merge branch 'master' into percent-search ### 📊 Changes **6 files changed** (+48 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -0) 📝 `examples/pagination_async.rs` (+2 -2) 📝 `examples/pagination_manual.rs` (+1 -1) 📝 `examples/pagination_sync.rs` (+1 -1) 📝 `src/clients/oauth.rs` (+18 -4) 📝 `tests/test_with_oauth.rs` (+25 -12) </details> ### 📄 Description ## Description This closes #141, which has been fixed magically :P ## Dependencies None ## Type of change Please delete options that are not relevant. - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update ## How Has This Been Tested? I've added a new test that's equivalent to the one that was failing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>