Skip to content

Merging the development into the main - Final PR - #173

Merged
BisoLab merged 1072 commits into
mainfrom
Development
May 3, 2026
Merged

BisoLab merged 1072 commits into
mainfrom
Development

Conversation

@BisoLab

@BisoLab BisoLab commented May 3, 2026

Copy link
Copy Markdown
Contributor

No description provided.

BisoLab and others added 30 commits April 28, 2026 02:36
… , copy icon in non owned playlists work fine
-ui tweaks in search module
-fixed comments wiring in feed
BisoLab and others added 26 commits May 3, 2026 02:11
PR - Fixed 95% of the work and only one feature left
Introduce constructor injection and test hooks to make upload and playlist logic testable. UploadTrackRemoteDataSource now accepts a Dio and a multipart-file factory (used instead of direct MultipartFile.fromFile). UploadTrackUseCase accepts fileFactory and fileExists overrides (with sensible defaults) and uses the injected checker instead of calling existsSync directly. UploadFormNotifier methods now accept an optional WidgetRef and allow overriding usecases/data sources/update/delete functions so tests can run without a sealed WidgetRef or real network. Also add a suite of playlist and upload unit tests covering datasource behavior, models, providers, and usecases.
Inject dependencies for upload and playlist tests
@BisoLab
BisoLab requested a review from a team as a code owner May 3, 2026 06:50
@BisoLab
BisoLab merged commit 2734482 into main May 3, 2026
1 of 2 checks 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.

10 participants