diff --git a/breadtube_bot/bot.py b/breadtube_bot/bot.py index ca0cc28..0f963d5 100644 --- a/breadtube_bot/bot.py +++ b/breadtube_bot/bot.py @@ -365,6 +365,7 @@ class Bot: break if selected_category is None: selected_category = category_ranges[-1][2] + self.logger.debug('Creating new channel: %s for id %s', subscription.name, subscription.channel_id) sub_channel = self.discord_manager.create_text_channel( self.guild_id, { 'name': subscription.name, diff --git a/breadtube_bot/youtube_subscription.py b/breadtube_bot/youtube_subscription.py index c51df43..c83ecae 100644 --- a/breadtube_bot/youtube_subscription.py +++ b/breadtube_bot/youtube_subscription.py @@ -68,7 +68,7 @@ class SubscriptionInfo: @staticmethod def discord_compatible_name(text: str) -> SubscriptionName: assert text, 'Channel name cannot be empty' - return SubscriptionName(unidecode(text[:100])) + return SubscriptionName(unidecode(text[:100].replace(' ', '-'))) @staticmethod def from_dict(info: dict) -> SubscriptionInfo: diff --git a/tests/data/subscriptions.csv b/tests/data/subscriptions.csv new file mode 100644 index 0000000..3e9a12b --- /dev/null +++ b/tests/data/subscriptions.csv @@ -0,0 +1,3 @@ +version https://git-lfs.github.com/spec/v1 +oid sha256:43048fa740f1583dcd33c27ffa38b15a03aa53177aa6c1ce23b2adebb04546af +size 91 diff --git a/tests/test_youtube_subscription.py b/tests/test_youtube_subscription.py new file mode 100644 index 0000000..5add24a --- /dev/null +++ b/tests/test_youtube_subscription.py @@ -0,0 +1,10 @@ +import logging +from pathlib import Path + +from breadtube_bot.youtube_subscription import SubscriptionHelper + + +def test_helper(): + subscriptions = SubscriptionHelper.read_text(Path('tests/data/subscriptions.csv').read_bytes()) + assert subscriptions['UUlDye0T4xTEq46sA5evq9fA'].name == 'a-gauche' + assert subscriptions['UUFrDDP81MX_QfOHrRZOgD4g'].name == 'l214'