Resolve TODOs

src/FileReceiver.Bl.Impl/Factories/CallbackQueryHandlerFactory.cs

@@ -27,6 +27,8 @@ public ICallbackQueryHandler CreateCallbackHandler(CallbackQuery callbackQuery)
                     => _serviceProvider.GetService<EditProfileCallbackQueryHandler>(),
                 { Data: { } data } when data.StartsWith("fr-session")
                     => _serviceProvider.GetService<FileReceivingSessionCallbackQueryHandler>(),
+                { Data: { } data } when data.StartsWith("menu-")
+                    => _serviceProvider.GetService<BotMenuCallbackQueryHandler>(),
                 _ => _serviceProvider.GetService<DefaultCallbackQueryHandler>(),
             };
         }

src/FileReceiver.Bl.Impl/FileReceiverBlInstaller.cs

@@ -74,6 +74,7 @@ private static void AddCallbackQueryHandlers(this IServiceCollection services)
         {
             services.AddTransient<EditProfileCallbackQueryHandler, EditProfileCallbackQueryHandler>();
             services.AddTransient<FileReceivingSessionCallbackQueryHandler, FileReceivingSessionCallbackQueryHandler>();
+            services.AddTransient<BotMenuCallbackQueryHandler, BotMenuCallbackQueryHandler>();
             services.AddTransient<DefaultCallbackQueryHandler, DefaultCallbackQueryHandler>();
         }
 

src/FileReceiver.Bl.Impl/Handlers/CallbackQuery/BotMenuCallbackQueryHandler.cs

@@ -0,0 +1,41 @@
+using System.Threading.Tasks;
+
+using FileReceiver.Bl.Abstract.Factories;
+using FileReceiver.Bl.Abstract.Handlers;
+
+using Telegram.Bot.Types;
+
+namespace FileReceiver.Bl.Impl.Handlers.CallbackQuery
+{
+    public class BotMenuCallbackQueryHandler : ICallbackQueryHandler
+    {
+        private readonly ICommandHandlerFactory _commandHandlerFactory;
+
+        public BotMenuCallbackQueryHandler(ICommandHandlerFactory commandHandlerFactory)
+        {
+            _commandHandlerFactory = commandHandlerFactory;
+        }
+
+        public async Task HandleCallback(Telegram.Bot.Types.CallbackQuery callbackQuery)
+        {
+            var update = new Update()
+            {
+                CallbackQuery = callbackQuery
+            };
+
+            switch (callbackQuery)
+            {
+                case { Data: "menu-/profile", From: { } from }:
+                    await _commandHandlerFactory
+                        .CreateCommandHandler("/profile")
+                        .HandleCommandAsync(update);
+                    break;
+                case { Data: "menu-/send_file", From: { } from }:
+                    await _commandHandlerFactory
+                        .CreateCommandHandler("/send_file")
+                        .HandleCommandAsync(update);
+                    break;
+            }
+        }
+    }
+}

src/FileReceiver.Bl.Impl/Handlers/TelegramUpdate/EditProfileUpdateHandler.cs

@@ -57,7 +57,6 @@ public async Task HandleUpdateAsync(Update update)
                 TransactionData = transactionData,
             };
 
-            // TODO: rewrite with pattern matching
             switch (editingAction)
             {
                 case ProfileEditingAction.UpdateFirstName:

src/FileReceiver.Bl.Impl/Handlers/TelegramUpdate/FileReceivingSessionCreatingUpdateHandler.cs

@@ -111,10 +111,9 @@ private async Task SetFileSizeConstraintAsync(FileReceivingSessionDataDto data)
 
                 await _receivingSessionService.SetFileSizeConstraintAsync(data.SessionId, maxFileSize);
                 await _botMessagesService.SendTextMessageAsync(data.UserId,
-                    // TODO: Add regex pattern examples
                     "Now send me file name constraints written with regex pattern. " +
                     "You can check your pattern [here](https://regex101.com/). " +
-                    "To send several patterns separate them with a coma");
+                    "If you don't want to use any file name constraints send `-1`.");
             }
             catch (Exception ex)
             {

src/FileReceiver.Bl.Impl/Services/BotMessagesService.cs

@@ -1,6 +1,7 @@
 using System.Threading.Tasks;
 
 using FileReceiver.Bl.Abstract.Services;
+using FileReceiver.Common.Constants;
 
 using Microsoft.Extensions.Logging;
 
@@ -25,14 +26,17 @@ public BotMessagesService(
 
         public async Task SendMenuAsync(long userOrChatId)
         {
-            // TODO: Create a real bot menu with buttons which will invoke command handlers and replace plain text with resources file
-            var sentMsg = await _botClient.SendTextMessageAsync(userOrChatId, "This is a bot menu...");
+            var sentMsg = await _botClient.SendTextMessageAsync(
+                userOrChatId,
+                "Menu:",
+                ParseMode.Markdown,
+                replyMarkup: Keyboards.BotMenu);
+
             if (sentMsg is null) _logger.LogWarning("Message wasn't sent to {userId}", userOrChatId);
         }
 
         public async Task SendErrorAsync(long userOrChatId, string errorMessage)
         {
-            // TODO: Update error sending, maybe add an image or smth like that and replace plain text with resources file
             var sentMsg = await _botClient.SendTextMessageAsync(userOrChatId,
                 $"Sorry, an error happen. Error description: {errorMessage}");
             if (sentMsg is null) _logger.LogWarning("Message wasn't sent to {userId}", userOrChatId);

src/FileReceiver.Bl.Impl/Services/FileReceivingService.cs

@@ -59,7 +59,6 @@ public async Task FinishReceivingTransaction(long userId)
 
         public async Task<bool> SaveDocument(long userId, Guid sessionId, Document document)
         {
-            // TODO: Add constraints check
             var transaction = await _transactionService.Get(userId, TransactionType.FileSending);
             var memStream = new MemoryStream();
             var fileInfo = await _botClient.GetFileAsync(document.FileId);

src/FileReceiver.Bl.Impl/Services/FileReceivingSessionService.cs

@@ -4,6 +4,7 @@
 using AutoMapper;
 
 using FileReceiver.Bl.Abstract.Services;
+using FileReceiver.Common.Constants;
 using FileReceiver.Common.Enums;
 using FileReceiver.Common.Exceptions;
 using FileReceiver.Common.Models;
@@ -47,7 +48,7 @@ public async Task CreateFileReceivingSessionAsync(long userId)
                 UserId = userId,
                 Constrains = new ConstraintsModel(),
                 SessionState = FileReceivingSessionState.FileSizeConstraintSet,
-                MaxFiles = 50, // TODO: should be moved to a constants file
+                MaxFiles = DefaultValues.FilesAtSession,
                 CreateData = DateTimeOffset.UtcNow,
             };
 

src/FileReceiver.Common/Constants/DefaultValues.cs

@@ -0,0 +1,7 @@
+namespace FileReceiver.Common.Constants
+{
+    public static class DefaultValues
+    {
+        public const int FilesAtSession = 50;
+    }
+}

src/FileReceiver.Common/Constants/Keyboards.cs

@@ -20,5 +20,16 @@ public static class Keyboards
             },
         });
 
+        public static readonly InlineKeyboardMarkup BotMenu = new(new[]
+        {
+            new[]
+            {
+                InlineKeyboardButton.WithCallbackData("Profile", "menu-/profile"),
+            },
+            new[]
+            {
+                InlineKeyboardButton.WithCallbackData("Send File", "menu-/send_file"),
+            },
+        });
     }
 }

tests/FileReceiver.Tests.Bl/ServicesTests/BotMessagesServiceTests.cs

@@ -2,6 +2,7 @@
 
 using FileReceiver.Bl.Abstract.Services;
 using FileReceiver.Bl.Impl.Services;
+using FileReceiver.Common.Constants;
 
 using Microsoft.Extensions.Logging;
 
@@ -36,7 +37,11 @@ public async Task SendMenuAsync_ShouldReturnNothing_WhenUserIdValid(long userId)
             await _sut.SendMenuAsync(userId);
 
             // Assert
-            await _botClient.Received(1).SendTextMessageAsync(userId, "This is a bot menu...");
+            await _botClient.Received(1).SendTextMessageAsync(
+                userId,
+                "Menu:",
+                ParseMode.Markdown,
+                replyMarkup: Keyboards.BotMenu);
         }
 
         [Theory]