Remove unused platform parameter in item.destination

This commit is contained in:
Šarūnas Nejus 2024-09-15 03:15:14 +01:00
parent 06dd2738bb
commit d3186859ac
No known key found for this signature in database
GPG key ID: DD28F6704DBE3435
2 changed files with 8 additions and 6 deletions

View file

@ -1077,7 +1077,6 @@ class Item(LibModel):
self,
relative_to_libdir=False,
basedir=None,
platform=None,
path_formats=None,
) -> bytes:
"""Return the path in the library directory designated for the item
@ -1087,7 +1086,6 @@ class Item(LibModel):
library's base directory for the destination.
"""
db = self._check_db()
platform = platform or sys.platform
basedir = basedir or db.directory
path_formats = path_formats or db.path_formats
@ -1117,7 +1115,7 @@ class Item(LibModel):
subpath = self.evaluate_template(subpath_tmpl, True)
# Prepare path for output: normalize Unicode characters.
if platform == "darwin":
if sys.platform == "darwin":
subpath = unicodedata.normalize("NFD", subpath)
else:
subpath = unicodedata.normalize("NFC", subpath)

View file

@ -23,6 +23,7 @@ import sys
import time
import unicodedata
import unittest
from unittest.mock import patch
import pytest
from mediafile import MediaFile, UnreadableFileError
@ -411,13 +412,15 @@ class DestinationTest(BeetsTestCase):
def test_unicode_normalized_nfd_on_mac(self):
instr = unicodedata.normalize("NFC", "caf\xe9")
self.lib.path_formats = [("default", instr)]
dest = self.i.destination(platform="darwin", relative_to_libdir=True)
with patch("sys.platform", "darwin"):
dest = self.i.destination(relative_to_libdir=True)
assert as_string(dest) == unicodedata.normalize("NFD", instr)
def test_unicode_normalized_nfc_on_linux(self):
instr = unicodedata.normalize("NFD", "caf\xe9")
self.lib.path_formats = [("default", instr)]
dest = self.i.destination(platform="linux", relative_to_libdir=True)
with patch("sys.platform", "linux"):
dest = self.i.destination(relative_to_libdir=True)
assert as_string(dest) == unicodedata.normalize("NFC", instr)
def test_non_mbcs_characters_on_windows(self):
@ -436,7 +439,8 @@ class DestinationTest(BeetsTestCase):
def test_unicode_extension_in_fragment(self):
self.lib.path_formats = [("default", "foo")]
self.i.path = util.bytestring_path("bar.caf\xe9")
dest = self.i.destination(platform="linux", relative_to_libdir=True)
with patch("sys.platform", "linux"):
dest = self.i.destination(relative_to_libdir=True)
assert as_string(dest) == "foo.caf\xe9"
def test_asciify_and_replace(self):