automatic-ripping-machine/test
TJSeit 5e9dfd7035
[BUGFIX] - Fix track filename mismatch issue #1530 (#1594)
* Fix off-by-one track filename bug when moving files after rip (bug #1530)

* updating file name matching logic

* reverting uneeded change

* using authoritative file naming

Authoritative Filename Propagation:

The filename variable is defined as the authoritative source of truth for the destination path (filepathname).
Before executing the HandBrake command, track.filename and track.orig_filename are updated with the authoritative filename.
Database Commit:

The updated values of track.filename are committed to the database before proceeding with the HandBrake command.
Error Handling:

If HandBrake fails, the authoritative name is already saved in the database, ensuring consistency for debugging and recovery.

* Update handbrake.py

* minimized changes

* Remove filename assignment in handbrake.py

Remove assignment of track.filename and track.orig_filename.

* Initial plan

* Fix filename path duplication bug in handbrake_mkv function

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Fix filename path duplication in handbrake_main_feature function

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Initial plan

* Fix MakeMKV track numbering to be 1-indexed for consistency with HandBrake

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Fix logging message - remove incorrect -1 from no_of_titles

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Initial plan

* Implement fuzzy file matching for MKV transcoding filename mismatches

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Update test file to be standalone and fix flake8 issues

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Initial plan

* Fix PEP8 compliance issues in test_ripper_makemkv_track_numbering.py

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Initial plan

* Clean up PR: remove track numbering changes, keep file matching fix

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Bump version from 2.20.4 to 2.20.5

* Initial plan

* Refactor handbrake.py to reduce code duplication

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Further cleanup: remove redundant error logging in handbrake_mkv

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Initial plan

* Fix SonarQube code duplication issues in test file

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Initial plan

* Fix useless self-assignment in handbrake.py line 134

Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>

* Resolving SonarQube issues and bumping version

* Removing unnecessary try catch

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TJSeit <7307974+TJSeit@users.noreply.github.com>
Co-authored-by: 1337-server <sndspamfilter@gmail.com>
2025-11-04 21:12:56 +10:30
..
unittest [BUGFIX] - Fix track filename mismatch issue #1530 (#1594) 2025-11-04 21:12:56 +10:30
__init__.py added qa to devtools for test and resolved qa issues 2023-06-19 22:04:03 +09:30