open-source-mirrors
/
recyclarr
mirror of https://github.com/recyclarr/recyclarr
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

refactor: Address internal sealed class warnings

Browse Source 
- Made lots of classes internal & sealed as needed
- Removed Cli.TestLibrary project. TestLibrary projects are only useful
  if they're going to be reusable in other tests. The Cli project is at
  the bottom of the dependency chain and doesn't need to be reusable.
pull/433/head
Robert Dailey 2 months ago
parent e9a667e6ee
commit 173be37918
196 changed files with 297 additions and 526 deletions
Show Stats Download Patch File Download Diff File

3
.editorconfig
Unescape View File

@ -42,6 +42,3 @@ dotnet_diagnostic.CA2007.severity = none
# Implement standard exception constructors
dotnet_diagnostic.CA1032.severity = none
# Consider making public types internal
dotnet_diagnostic.CA1515.severity = none

2
Directory.Build.props
Unescape View File

@ -32,6 +32,8 @@
<!-- For only NON-TEST projects -->
<ItemGroup Condition="!$(MSBuildProjectName.EndsWith('Tests'))">
<InternalsVisibleTo Include="$(MSBuildProjectName).Tests" />
<!-- For NSubstitute -->
<InternalsVisibleTo Include="DynamicProxyGenAssembly2" />
</ItemGroup>
<!-- ReferenceTrimmer - run build with /p:EnableReferenceTrimmer=true to enable -->

7
Recyclarr.sln
Unescape View File

@ -30,8 +30,6 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Recyclarr.Core.TestLibrary"
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Recyclarr.Cli.Tests", "tests\Recyclarr.Cli.Tests\Recyclarr.Cli.Tests.csproj", "{59886A2A-338F-4864-9735-1633433B1F8F}"
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Recyclarr.Cli.TestLibrary", "tests\Recyclarr.Cli.TestLibrary\Recyclarr.Cli.TestLibrary.csproj", "{7B730BDB-1519-4847-BA23-998AB3750E31}"
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Recyclarr.Core", "src\Recyclarr.Core\Recyclarr.Core.csproj", "{14F923B3-13EB-46D2-8AD0-D0811B46952A}"
EndProject
Global
@ -60,10 +58,6 @@ Global
{59886A2A-338F-4864-9735-1633433B1F8F}.Debug|Any CPU.Build.0 = Debug|Any CPU
{59886A2A-338F-4864-9735-1633433B1F8F}.Release|Any CPU.ActiveCfg = Release|Any CPU
{59886A2A-338F-4864-9735-1633433B1F8F}.Release|Any CPU.Build.0 = Release|Any CPU
{7B730BDB-1519-4847-BA23-998AB3750E31}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{7B730BDB-1519-4847-BA23-998AB3750E31}.Debug|Any CPU.Build.0 = Debug|Any CPU
{7B730BDB-1519-4847-BA23-998AB3750E31}.Release|Any CPU.ActiveCfg = Release|Any CPU
{7B730BDB-1519-4847-BA23-998AB3750E31}.Release|Any CPU.Build.0 = Release|Any CPU
{14F923B3-13EB-46D2-8AD0-D0811B46952A}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{14F923B3-13EB-46D2-8AD0-D0811B46952A}.Debug|Any CPU.Build.0 = Debug|Any CPU
{14F923B3-13EB-46D2-8AD0-D0811B46952A}.Release|Any CPU.ActiveCfg = Release|Any CPU
@ -80,6 +74,5 @@ Global
{A4EC7E0D-C591-4874-B9AC-EB12A96F3E83} = {18E17C53-F600-40AE-82C1-3CD1E547C307}
{DE198BA1-2906-43BB-9CDB-977B9218A670} = {18E17C53-F600-40AE-82C1-3CD1E547C307}
{59886A2A-338F-4864-9735-1633433B1F8F} = {18E17C53-F600-40AE-82C1-3CD1E547C307}
{7B730BDB-1519-4847-BA23-998AB3750E31} = {18E17C53-F600-40AE-82C1-3CD1E547C307}
EndGlobalSection
EndGlobal

6
src/Recyclarr.Cli/CompositionRoot.cs
Unescape View File

@ -22,7 +22,7 @@ using Spectre.Console.Cli;
namespace Recyclarr.Cli;
public static class CompositionRoot
internal static class CompositionRoot
{
public static void Setup(ContainerBuilder builder)
{
@ -52,7 +52,7 @@ public static class CompositionRoot
builder.RegisterType<FlurlHttpExceptionHandler>();
// Sync
builder.RegisterType<SyncProcessor>().As<ISyncProcessor>();
builder.RegisterType<SyncProcessor>();
// Configuration
builder.RegisterType<ConfigCreationProcessor>().As<IConfigCreationProcessor>();
@ -70,7 +70,7 @@ public static class CompositionRoot
private static void RegisterMigrations(ContainerBuilder builder)
{
builder.RegisterType<MigrationExecutor>().As<IMigrationExecutor>();
builder.RegisterType<MigrationExecutor>();
// Migration Steps
builder

2
src/Recyclarr.Cli/Console/CliSetup.cs
Unescape View File

@ -5,7 +5,7 @@ using Spectre.Console.Cli;
namespace Recyclarr.Cli.Console;
public static class CliSetup
internal static class CliSetup
{
private static void AddCommands(IConfigurator cli)
{

2
src/Recyclarr.Cli/Console/CommandException.cs
Unescape View File

@ -1,3 +1,3 @@
namespace Recyclarr.Cli.Console;
public class CommandException(string? message) : Exception(message);
internal class CommandException(string? message) : Exception(message);

2
src/Recyclarr.Cli/Console/Commands/BaseCommandSettings.cs
Unescape View File

@ -3,7 +3,7 @@ using Spectre.Console.Cli;
namespace Recyclarr.Cli.Console.Commands;
public class BaseCommandSettings : CommandSettings
internal class BaseCommandSettings : CommandSettings
{
public CancellationToken CancellationToken { get; set; }

5
src/Recyclarr.Cli/Console/Commands/ConfigCreateCommand.cs
Unescape View File

@ -10,20 +10,19 @@ namespace Recyclarr.Cli.Console.Commands;
[UsedImplicitly]
[Description("Create a starter configuration file.")]
public class ConfigCreateCommand(
internal class ConfigCreateCommand(
ILogger log,
IConfigCreationProcessor processor,
IMultiRepoUpdater repoUpdater
) : AsyncCommand<ConfigCreateCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
[SuppressMessage(
"Performance",
"CA1819:Properties should not return arrays",
Justification = "Spectre.Console requires it"
)]
public class CliSettings : BaseCommandSettings, ICreateConfigSettings
internal class CliSettings : BaseCommandSettings, ICreateConfigSettings
{
[CommandOption("-p|--path")]
[Description("Path to where the configuration file should be created.")]

4
src/Recyclarr.Cli/Console/Commands/ConfigListLocalCommand.cs
Unescape View File

@ -9,13 +9,13 @@ namespace Recyclarr.Cli.Console.Commands;
[UsedImplicitly]
[Description("List local configuration files.")]
public class ConfigListLocalCommand(
internal class ConfigListLocalCommand(
ConfigListLocalProcessor processor,
IMultiRepoUpdater repoUpdater
) : AsyncCommand<ConfigListLocalCommand.CliSettings>
{
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
public class CliSettings : BaseCommandSettings;
internal class CliSettings : BaseCommandSettings;
public override async Task<int> ExecuteAsync(CommandContext context, CliSettings settings)
{

10
src/Recyclarr.Cli/Console/Commands/ConfigListTemplatesCommand.cs
Unescape View File

@ -1,5 +1,6 @@
using System.ComponentModel;
using System.Diagnostics.CodeAnalysis;
using Recyclarr.Cli.Console.Settings;
using Recyclarr.Cli.Processors;
using Recyclarr.Cli.Processors.Config;
using Recyclarr.Repo;
@ -9,13 +10,13 @@ namespace Recyclarr.Cli.Console.Commands;
[UsedImplicitly]
[Description("List local configuration files.")]
public class ConfigListTemplatesCommand(
internal class ConfigListTemplatesCommand(
ConfigListTemplateProcessor processor,
IMultiRepoUpdater repoUpdater
) : AsyncCommand<ConfigListTemplatesCommand.CliSettings>
{
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
public class CliSettings : BaseCommandSettings, IConfigListTemplatesSettings
internal class CliSettings : BaseCommandSettings, IConfigListTemplatesSettings
{
[CommandOption("-i|--includes")]
[Description(
@ -32,8 +33,3 @@ public class ConfigListTemplatesCommand(
return (int)ExitStatus.Succeeded;
}
}
public interface IConfigListTemplatesSettings
{
bool Includes { get; }
}

5
src/Recyclarr.Cli/Console/Commands/DeleteCustomFormatsCommand.cs
Unescape View File

@ -9,17 +9,16 @@ namespace Recyclarr.Cli.Console.Commands;
[Description("Delete things from services like Radarr and Sonarr")]
[UsedImplicitly]
public class DeleteCustomFormatsCommand(IDeleteCustomFormatsProcessor processor)
internal class DeleteCustomFormatsCommand(IDeleteCustomFormatsProcessor processor)
: AsyncCommand<DeleteCustomFormatsCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
[SuppressMessage(
"Performance",
"CA1819:Properties should not return arrays",
Justification = "Spectre.Console requires it"
)]
public class CliSettings : BaseCommandSettings, IDeleteCustomFormatSettings
internal class CliSettings : BaseCommandSettings, IDeleteCustomFormatSettings
{
[CommandArgument(0, "<instance_name>")]
[Description("The name of the instance to delete CFs from.")]

8
src/Recyclarr.Cli/Console/Commands/ListCustomFormatsCommand.cs
Unescape View File

@ -14,12 +14,14 @@ namespace Recyclarr.Cli.Console.Commands;
[UsedImplicitly]
[Description("List custom formats in the guide for a particular service.")]
public class ListCustomFormatsCommand(CustomFormatDataLister lister, IMultiRepoUpdater repoUpdater)
: AsyncCommand<ListCustomFormatsCommand.CliSettings>
internal class ListCustomFormatsCommand(
CustomFormatDataLister lister,
IMultiRepoUpdater repoUpdater
) : AsyncCommand<ListCustomFormatsCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
public class CliSettings : BaseCommandSettings, IListCustomFormatSettings
internal class CliSettings : BaseCommandSettings, IListCustomFormatSettings
{
[CommandArgument(0, "<service_type>")]
[EnumDescription<SupportedServices>("The service type to obtain information about.")]

4
src/Recyclarr.Cli/Console/Commands/ListMediaNamingCommand.cs
Unescape View File

@ -11,12 +11,12 @@ namespace Recyclarr.Cli.Console.Commands;
[UsedImplicitly]
[Description("List media naming formats in the guide for a particular service.")]
public class ListMediaNamingCommand(MediaNamingDataLister lister, IMultiRepoUpdater repoUpdater)
internal class ListMediaNamingCommand(MediaNamingDataLister lister, IMultiRepoUpdater repoUpdater)
: AsyncCommand<ListMediaNamingCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
public class CliSettings : BaseCommandSettings
internal class CliSettings : BaseCommandSettings
{
[CommandArgument(0, "<service_type>")]
[EnumDescription<SupportedServices>("The service type to obtain information about.")]

4
src/Recyclarr.Cli/Console/Commands/ListQualitiesCommand.cs
Unescape View File

@ -12,12 +12,12 @@ namespace Recyclarr.Cli.Console.Commands;
#pragma warning disable CS8765
[UsedImplicitly]
[Description("List quality definitions in the guide for a particular service.")]
public class ListQualitiesCommand(QualitySizeDataLister lister, IMultiRepoUpdater repoUpdater)
internal class ListQualitiesCommand(QualitySizeDataLister lister, IMultiRepoUpdater repoUpdater)
: AsyncCommand<ListQualitiesCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
public class CliSettings : BaseCommandSettings
internal class CliSettings : BaseCommandSettings
{
[CommandArgument(0, "<service_type>")]
[EnumDescription<SupportedServices>("The service type to obtain information about.")]

4
src/Recyclarr.Cli/Console/Commands/MigrateCommand.cs
Unescape View File

@ -13,12 +13,12 @@ namespace Recyclarr.Cli.Console.Commands;
[UsedImplicitly]
[Description("Perform migration steps that may be needed between versions")]
public class MigrateCommand(IAnsiConsole console, IMigrationExecutor migration)
internal class MigrateCommand(IAnsiConsole console, MigrationExecutor migration)
: Command<MigrateCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
public class CliSettings : BaseCommandSettings;
internal class CliSettings : BaseCommandSettings;
public override int Execute(CommandContext context, CliSettings settings)
{

9
src/Recyclarr.Cli/Console/Commands/SyncCommand.cs
Unescape View File

@ -12,20 +12,19 @@ namespace Recyclarr.Cli.Console.Commands;
[Description("Sync the guide to services")]
[UsedImplicitly]
public class SyncCommand(
IMigrationExecutor migration,
internal class SyncCommand(
MigrationExecutor migration,
IMultiRepoUpdater repoUpdater,
ISyncProcessor syncProcessor
SyncProcessor syncProcessor
) : AsyncCommand<SyncCommand.CliSettings>
{
[UsedImplicitly]
[SuppressMessage("Design", "CA1034:Nested types should not be visible")]
[SuppressMessage(
"Performance",
"CA1819:Properties should not return arrays",
Justification = "Spectre.Console requires it"
)]
public class CliSettings : BaseCommandSettings, ISyncSettings
internal class CliSettings : BaseCommandSettings, ISyncSettings
{
[CommandArgument(0, "[service]")]
[EnumDescription<SupportedServices>(

2
src/Recyclarr.Cli/Console/Helpers/EnumDescriptionAttribute.cs
Unescape View File

@ -6,7 +6,7 @@ using System.Text;
namespace Recyclarr.Cli.Console.Helpers;
[AttributeUsage(AttributeTargets.Property)]
public sealed class EnumDescriptionAttribute<TEnum> : DescriptionAttribute
internal sealed class EnumDescriptionAttribute<TEnum> : DescriptionAttribute
where TEnum : Enum
{
public override string Description { get; }

6
src/Recyclarr.Cli/Console/Settings/IConfigListTemplatesSettings.cs
Unescape View File

@ -0,0 +1,6 @@
namespace Recyclarr.Cli.Console.Settings;
internal interface IConfigListTemplatesSettings
{
bool Includes { get; }
}

2
src/Recyclarr.Cli/Console/Settings/ICreateConfigSettings.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Console.Settings;
public interface ICreateConfigSettings
internal interface ICreateConfigSettings
{
public string? Path { get; }
public IReadOnlyCollection<string> Templates { get; }

2
src/Recyclarr.Cli/Console/Settings/IDeleteCustomFormatSettings.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Console.Settings;
public interface IDeleteCustomFormatSettings
internal interface IDeleteCustomFormatSettings
{
public string InstanceName { get; }
public IReadOnlyCollection<string> CustomFormatNames { get; }

2
src/Recyclarr.Cli/Console/Settings/IListCustomFormatSettings.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.TrashGuide;
namespace Recyclarr.Cli.Console.Settings;
public interface IListCustomFormatSettings
internal interface IListCustomFormatSettings
{
SupportedServices Service { get; }
bool ScoreSets { get; }

2
src/Recyclarr.Cli/Console/Settings/ISyncSettings.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.TrashGuide;
namespace Recyclarr.Cli.Console.Settings;
public interface ISyncSettings
internal interface ISyncSettings
{
SupportedServices? Service { get; }
IReadOnlyCollection<string> Configs { get; }

2
src/Recyclarr.Cli/Console/Setup/AppDataDirSetupTask.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.Platform;
namespace Recyclarr.Cli.Console.Setup;
public class AppDataDirSetupTask(IAppDataSetup appDataSetup) : IGlobalSetupTask
internal class AppDataDirSetupTask(IAppDataSetup appDataSetup) : IGlobalSetupTask
{
public void OnStart(BaseCommandSettings cmd)
{

2
src/Recyclarr.Cli/Console/Setup/CompositeGlobalSetupTask.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.Cli.Console.Commands;
namespace Recyclarr.Cli.Console.Setup;
[UsedImplicitly]
public class CompositeGlobalSetupTask(IOrderedEnumerable<Lazy<IGlobalSetupTask>> tasks)
internal class CompositeGlobalSetupTask(IOrderedEnumerable<Lazy<IGlobalSetupTask>> tasks)
: IGlobalSetupTask
{
public void OnStart(BaseCommandSettings cmd)

2
src/Recyclarr.Cli/Console/Setup/IGlobalSetupTask.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.Cli.Console.Commands;
namespace Recyclarr.Cli.Console.Setup;
public interface IGlobalSetupTask
internal interface IGlobalSetupTask
{
void OnStart(BaseCommandSettings cmd);
void OnFinish();

2
src/Recyclarr.Cli/Console/Setup/JanitorCleanupTask.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.Settings;
namespace Recyclarr.Cli.Console.Setup;
public class JanitorCleanupTask(
internal class JanitorCleanupTask(
LogJanitor janitor,
ILogger log,
ISettings<LogJanitorSettings> settings

2
src/Recyclarr.Cli/Console/Setup/LoggerSetupTask.cs
Unescape View File

@ -6,7 +6,7 @@ using Serilog.Events;
namespace Recyclarr.Cli.Console.Setup;
public class LoggerSetupTask(
internal class LoggerSetupTask(
LoggingLevelSwitch loggingLevelSwitch,
LoggerFactory loggerFactory,
IEnumerable<ILogConfigurator> logConfigurators

2
src/Recyclarr.Cli/Console/Setup/ProgramInformationDisplayTask.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.Platform;
namespace Recyclarr.Cli.Console.Setup;
public class ProgramInformationDisplayTask(ILogger log, IAppPaths paths) : IGlobalSetupTask
internal class ProgramInformationDisplayTask(ILogger log, IAppPaths paths) : IGlobalSetupTask
{
public void OnStart(BaseCommandSettings cmd)
{

2
src/Recyclarr.Cli/Logging/LogJanitor.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.Platform;
namespace Recyclarr.Cli.Logging;
public class LogJanitor(IAppPaths paths)
internal class LogJanitor(IAppPaths paths)
{
public void DeleteOldestLogFiles(int numberOfNewestToKeep)
{

2
src/Recyclarr.Cli/Logging/LoggerFactory.cs
Unescape View File

@ -6,7 +6,7 @@ using Serilog.Templates.Themes;
namespace Recyclarr.Cli.Logging;
public class LoggerFactory(IEnvironment env, LoggingLevelSwitch levelSwitch)
internal class LoggerFactory(IEnvironment env, LoggingLevelSwitch levelSwitch)
{
public ILogger Logger { get; private set; } =
LogSetup

7
src/Recyclarr.Cli/Migration/IMigrationExecutor.cs
Unescape View File

@ -1,7 +0,0 @@
namespace Recyclarr.Cli.Migration;
public interface IMigrationExecutor
{
void PerformAllMigrationSteps(bool withDiagnostics);
void CheckNeededMigrations();
}

2
src/Recyclarr.Cli/Migration/MigrationException.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Migration;
public class MigrationException(
internal class MigrationException(
Exception originalException,
string operationDescription,
IReadOnlyCollection<string> remediation

4
src/Recyclarr.Cli/Migration/MigrationExecutor.cs
Unescape View File

@ -3,10 +3,10 @@ using Spectre.Console;
namespace Recyclarr.Cli.Migration;
public class MigrationExecutor(
internal class MigrationExecutor(
IOrderedEnumerable<IMigrationStep> migrationSteps,
IAnsiConsole console
) : IMigrationExecutor
)
{
public void PerformAllMigrationSteps(bool withDiagnostics)
{

2
src/Recyclarr.Cli/Migration/RequiredMigrationException.cs
Unescape View File

@ -1,3 +1,3 @@
namespace Recyclarr.Cli.Migration;
public class RequiredMigrationException() : Exception("Some REQUIRED migrations did not pass");
internal class RequiredMigrationException() : Exception("Some REQUIRED migrations did not pass");

2
src/Recyclarr.Cli/Migration/Steps/DeleteRepoDirMigrationStep.cs
Unescape View File

@ -6,7 +6,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Migration.Steps;
[UsedImplicitly]
public class DeleteRepoDirMigrationStep(IAppPaths paths) : IMigrationStep
internal class DeleteRepoDirMigrationStep(IAppPaths paths) : IMigrationStep
{
public string Description => "Delete old repo directory";
public IReadOnlyCollection<string> Remediation =>

2
src/Recyclarr.Cli/Migration/Steps/IMigrationStep.cs
Unescape View File

@ -2,7 +2,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Migration.Steps;
public interface IMigrationStep
internal interface IMigrationStep
{
/// <summary>
/// A description printed to the user so that they understand the purpose of this migration step, and

2
src/Recyclarr.Cli/Migration/Steps/MoveOsxAppDataDotnet8.cs
Unescape View File

@ -5,7 +5,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Migration.Steps;
[UsedImplicitly]
public class MoveOsxAppDataDotnet8(
internal class MoveOsxAppDataDotnet8(
IAppPaths paths,
IEnvironment env,
IRuntimeInformation runtimeInfo,

6
src/Recyclarr.Cli/Pipelines/CustomFormat/Cache/CustomFormatCache.cs
Unescape View File

@ -5,7 +5,7 @@ using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Pipelines.CustomFormat.Cache;
public record TrashIdMapping(string TrashId, string CustomFormatName, int CustomFormatId);
internal record TrashIdMapping(string TrashId, string CustomFormatName, int CustomFormatId);
[CacheObjectName("custom-format-cache")]
[SuppressMessage("Design", "CA1002:Do not expose generic lists", Justification = "POCO")]
@ -14,12 +14,12 @@ public record TrashIdMapping(string TrashId, string CustomFormatName, int Custom
"CA2227:Collection properties should be read only",
Justification = "POCO"
)]
public record CustomFormatCacheObject() : CacheObject(1)
internal record CustomFormatCacheObject() : CacheObject(1)
{
public List<TrashIdMapping> TrashIdMappings { get; set; } = [];
}
public class CustomFormatCache(CustomFormatCacheObject cacheObject) : BaseCache(cacheObject)
internal class CustomFormatCache(CustomFormatCacheObject cacheObject) : BaseCache(cacheObject)
{
public IReadOnlyList<TrashIdMapping> TrashIdMappings => cacheObject.TrashIdMappings;

2
src/Recyclarr.Cli/Pipelines/CustomFormat/Cache/CustomFormatCachePersister.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.Config.Models;
namespace Recyclarr.Cli.Pipelines.CustomFormat.Cache;
public class CustomFormatCachePersister(
internal class CustomFormatCachePersister(
ILogger log,
ICacheStoragePath storagePath,
IServiceConfiguration config

2
src/Recyclarr.Cli/Pipelines/CustomFormat/CustomFormatDataLister.cs
Unescape View File

@ -5,7 +5,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Pipelines.CustomFormat;
public class CustomFormatDataLister(IAnsiConsole console, ICustomFormatGuideService guide)
internal class CustomFormatDataLister(IAnsiConsole console, ICustomFormatGuideService guide)
{
public void List(IListCustomFormatSettings settings)
{

2
src/Recyclarr.Cli/Pipelines/CustomFormat/Models/ConflictingCustomFormat.cs
Unescape View File

@ -2,4 +2,4 @@ using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Pipelines.CustomFormat.Models;
public record ConflictingCustomFormat(CustomFormatData GuideCf, int ConflictingId);
internal record ConflictingCustomFormat(CustomFormatData GuideCf, int ConflictingId);

2
src/Recyclarr.Cli/Pipelines/CustomFormat/Models/CustomFormatTransactionData.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Pipelines.CustomFormat.Models;
public record CustomFormatTransactionData
internal record CustomFormatTransactionData
{
public Collection<TrashIdMapping> DeletedCustomFormats { get; } = [];
public Collection<CustomFormatData> NewCustomFormats { get; } = [];

10
src/Recyclarr.Cli/Pipelines/CustomFormat/Models/ProcessedConfigData.cs
Unescape View File

@ -1,10 +0,0 @@
using Recyclarr.Config.Models;
using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Pipelines.CustomFormat.Models;
public class ProcessedConfigData
{
public ICollection<CustomFormatData> CustomFormats { get; init; } = [];
public ICollection<AssignScoresToConfig> QualityProfiles { get; init; } = [];
}

2
src/Recyclarr.Cli/Pipelines/CustomFormat/Models/ProcessedCustomFormatCache.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Pipelines.CustomFormat.Models;
public class ProcessedCustomFormatCache : IPipelineCache
internal class ProcessedCustomFormatCache : IPipelineCache
{
private readonly List<CustomFormatData> _customFormats = [];

2
src/Recyclarr.Cli/Pipelines/IPipelineCache.cs
Unescape View File

@ -3,7 +3,7 @@ namespace Recyclarr.Cli.Pipelines;
/// <summary>
/// Defines a mechanism for state sharing between pipelines.
/// </summary>
public interface IPipelineCache
internal interface IPipelineCache
{
void Clear();
}

6
src/Recyclarr.Cli/Pipelines/ISyncPipeline.cs
Unescape View File

@ -2,9 +2,9 @@ using Recyclarr.Cli.Console.Settings;
namespace Recyclarr.Cli.Pipelines;
// This interface is valuable because it allows having a collection of generic sync pipelines without needing to be
// aware of the generic parameters.
public interface ISyncPipeline
// This interface is valuable because it allows having a collection of generic sync pipelines
// without needing to be aware of the generic parameters.
internal interface ISyncPipeline
{
public Task Execute(ISyncSettings settings, CancellationToken ct);
}

2
src/Recyclarr.Cli/Pipelines/MediaNaming/MediaNamingDataLister.cs
Unescape View File

@ -6,7 +6,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Pipelines.MediaNaming;
public class MediaNamingDataLister(IAnsiConsole console, IMediaNamingGuideService guide)
internal class MediaNamingDataLister(IAnsiConsole console, IMediaNamingGuideService guide)
{
public void ListNaming(SupportedServices serviceType)
{

2
src/Recyclarr.Cli/Pipelines/MediaNaming/PipelinePhases/Config/IServiceBasedMediaNamingConfigPhase.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.TrashGuide.MediaNaming;
namespace Recyclarr.Cli.Pipelines.MediaNaming.PipelinePhases.Config;
public interface IServiceBasedMediaNamingConfigPhase
internal interface IServiceBasedMediaNamingConfigPhase
{
Task<MediaNamingDto> ProcessNaming(IMediaNamingGuideService guide, NamingFormatLookup lookup);
}

2
src/Recyclarr.Cli/Pipelines/MediaNaming/PipelinePhases/Config/NamingFormatLookup.cs
Unescape View File

@ -2,7 +2,7 @@ using System.Diagnostics.CodeAnalysis;
namespace Recyclarr.Cli.Pipelines.MediaNaming.PipelinePhases.Config;
public class NamingFormatLookup
internal class NamingFormatLookup
{
private readonly List<InvalidNamingConfig> _errors = [];
public IReadOnlyCollection<InvalidNamingConfig> Errors => _errors;

2
src/Recyclarr.Cli/Pipelines/MediaNaming/PipelinePhases/Config/RadarrMediaNamingConfigPhase.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.TrashGuide.MediaNaming;
namespace Recyclarr.Cli.Pipelines.MediaNaming.PipelinePhases.Config;
public class RadarrMediaNamingConfigPhase(RadarrConfiguration config)
internal class RadarrMediaNamingConfigPhase(RadarrConfiguration config)
: IServiceBasedMediaNamingConfigPhase
{
public Task<MediaNamingDto> ProcessNaming(

2
src/Recyclarr.Cli/Pipelines/MediaNaming/PipelinePhases/Config/SonarrMediaNamingConfigPhase.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.TrashGuide.MediaNaming;
namespace Recyclarr.Cli.Pipelines.MediaNaming.PipelinePhases.Config;
public class SonarrMediaNamingConfigPhase(SonarrConfiguration config)
internal class SonarrMediaNamingConfigPhase(SonarrConfiguration config)
: IServiceBasedMediaNamingConfigPhase
{
public Task<MediaNamingDto> ProcessNaming(

4
src/Recyclarr.Cli/Pipelines/MediaNaming/PipelinePhases/MediaNamingConfigPhase.cs
Unescape View File

@ -7,9 +7,9 @@ using Recyclarr.TrashGuide.MediaNaming;
namespace Recyclarr.Cli.Pipelines.MediaNaming.PipelinePhases;
public record InvalidNamingConfig(string Type, string ConfigValue);
internal record InvalidNamingConfig(string Type, string ConfigValue);
public record ProcessedNamingConfig
internal record ProcessedNamingConfig
{
public required MediaNamingDto Dto { get; init; }
public IReadOnlyCollection<InvalidNamingConfig> InvalidNaming { get; init; } = [];

2
src/Recyclarr.Cli/Pipelines/PipelineAutofacModule.cs
Unescape View File

@ -19,7 +19,7 @@ using Recyclarr.TrashGuide.QualitySize;
namespace Recyclarr.Cli.Pipelines;
public class PipelineAutofacModule : Module
internal class PipelineAutofacModule : Module
{
protected override void Load(ContainerBuilder builder)
{

9
src/Recyclarr.Cli/Pipelines/QualityProfile/Models/ProcessedQualityProfileData.cs
Unescape View File

@ -3,9 +3,14 @@ using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Pipelines.QualityProfile.Models;
public record ProcessedQualityProfileScore(string TrashId, string CfName, int FormatId, int Score);
internal record ProcessedQualityProfileScore(
string TrashId,
string CfName,
int FormatId,
int Score
);
public record ProcessedQualityProfileData
internal record ProcessedQualityProfileData
{
public required QualityProfileConfig Profile { get; init; }
public bool ShouldCreate { get; init; } = true;

2
src/Recyclarr.Cli/Pipelines/QualityProfile/Models/QualityProfileServiceData.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.ServarrApi.QualityProfile;
namespace Recyclarr.Cli.Pipelines.QualityProfile.Models;
public record QualityProfileServiceData(
internal record QualityProfileServiceData(
IReadOnlyList<QualityProfileDto> Profiles,
QualityProfileDto Schema
);

4
src/Recyclarr.Cli/Pipelines/QualityProfile/Models/QualityProfileTransactionData.cs
Unescape View File

@ -3,13 +3,13 @@ using FluentValidation.Results;
namespace Recyclarr.Cli.Pipelines.QualityProfile.Models;
public record InvalidProfileData(
internal record InvalidProfileData(
UpdatedQualityProfile Profile,
IReadOnlyCollection<ValidationFailure> Errors
);
[SuppressMessage("Usage", "CA2227:Collection properties should be read only")]
public record QualityProfileTransactionData
internal record QualityProfileTransactionData
{
public ICollection<string> NonExistentProfiles { get; init; } = [];
public ICollection<InvalidProfileData> InvalidProfiles { get; init; } = [];

2
src/Recyclarr.Cli/Pipelines/QualityProfile/Models/QualityProfileUpdateReason.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Pipelines.QualityProfile.Models;
public enum QualityProfileUpdateReason
internal enum QualityProfileUpdateReason
{
New,
Changed,

2
src/Recyclarr.Cli/Pipelines/QualityProfile/QualityItemOrganizer.cs
Unescape View File

@ -5,7 +5,7 @@ using Recyclarr.ServarrApi.QualityProfile;
namespace Recyclarr.Cli.Pipelines.QualityProfile;
public class QualityItemOrganizer
internal class QualityItemOrganizer
{
private readonly List<string> _invalidItemNames = [];

2
src/Recyclarr.Cli/Pipelines/QualityProfile/QualityProfileExtensions.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.ServarrApi.QualityProfile;
namespace Recyclarr.Cli.Pipelines.QualityProfile;
public static class QualityProfileExtensions
internal static class QualityProfileExtensions
{
private static IEnumerable<ProfileItemDto> FlattenItems(IEnumerable<ProfileItemDto> items)
{

4
src/Recyclarr.Cli/Pipelines/QualityProfile/QualityProfileStatCalculator.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.ServarrApi.QualityProfile;
namespace Recyclarr.Cli.Pipelines.QualityProfile;
public record ProfileWithStats
internal record ProfileWithStats
{
public required UpdatedQualityProfile Profile { get; set; }
public bool ProfileChanged { get; set; }
@ -14,7 +14,7 @@ public record ProfileWithStats
public bool HasChanges => ProfileChanged || ScoresChanged || QualitiesChanged;
}
public class QualityProfileStatCalculator(ILogger log)
internal class QualityProfileStatCalculator(ILogger log)
{
public ProfileWithStats Calculate(UpdatedQualityProfile profile)
{

4
src/Recyclarr.Cli/Pipelines/QualityProfile/UpdatedFormatScore.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.ServarrApi.QualityProfile;
namespace Recyclarr.Cli.Pipelines.QualityProfile;
public enum FormatScoreUpdateReason
internal enum FormatScoreUpdateReason
{
/// <summary>
/// A score who's value did not change.
@ -29,7 +29,7 @@ public enum FormatScoreUpdateReason
New,
}
public record UpdatedFormatScore(
internal record UpdatedFormatScore(
ProfileFormatItemDto Dto,
int NewScore,
FormatScoreUpdateReason Reason

4
src/Recyclarr.Cli/Pipelines/QualityProfile/UpdatedQualityProfile.cs
Unescape View File

@ -3,14 +3,14 @@ using Recyclarr.ServarrApi.QualityProfile;
namespace Recyclarr.Cli.Pipelines.QualityProfile;
public record UpdatedQualities
internal record UpdatedQualities
{
public ICollection<string> InvalidQualityNames { get; init; } = [];
public IReadOnlyCollection<ProfileItemDto> Items { get; init; } = [];
public int NumWantedItems { get; init; }
}
public record UpdatedQualityProfile
internal record UpdatedQualityProfile
{
public required QualityProfileDto ProfileDto { get; init; }
public required ProcessedQualityProfileData ProfileConfig { get; init; }

2
src/Recyclarr.Cli/Pipelines/QualityProfile/UpdatedQualityProfileValidator.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.Common.Extensions;
namespace Recyclarr.Cli.Pipelines.QualityProfile;
public class UpdatedQualityProfileValidator : AbstractValidator<UpdatedQualityProfile>
internal class UpdatedQualityProfileValidator : AbstractValidator<UpdatedQualityProfile>
{
public UpdatedQualityProfileValidator()
{

2
src/Recyclarr.Cli/Pipelines/QualitySize/PipelinePhases/Limits/IQualityItemLimitFactory.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.TrashGuide.QualitySize;
namespace Recyclarr.Cli.Pipelines.QualitySize.PipelinePhases.Limits;
public interface IQualityItemLimitFactory
internal interface IQualityItemLimitFactory
{
Task<QualityItemLimits> Create(SupportedServices serviceType, CancellationToken ct);
}

2
src/Recyclarr.Cli/Pipelines/QualitySize/PipelinePhases/Limits/QualityItemLimitFactory.cs
Unescape View File

@ -4,7 +4,7 @@ using Recyclarr.TrashGuide.QualitySize;
namespace Recyclarr.Cli.Pipelines.QualitySize.PipelinePhases.Limits;
public class QualityItemLimitFactory(
internal class QualityItemLimitFactory(
IIndex<SupportedServices, IQualityItemLimitFetcher> limitFactory
) : IQualityItemLimitFactory
{

2
src/Recyclarr.Cli/Pipelines/QualitySize/PipelinePhases/Limits/RadarrQualityItemLimitFetcher.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.TrashGuide.QualitySize;
namespace Recyclarr.Cli.Pipelines.QualitySize.PipelinePhases.Limits;
public class RadarrQualityItemLimitFetcher(IRadarrCapabilityFetcher capabilityFetcher)
internal class RadarrQualityItemLimitFetcher(IRadarrCapabilityFetcher capabilityFetcher)
: IQualityItemLimitFetcher
{
private QualityItemLimits? _cachedLimits;

2
src/Recyclarr.Cli/Pipelines/QualitySize/PipelinePhases/Limits/SonarrQualityItemLimitFetcher.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.TrashGuide.QualitySize;
namespace Recyclarr.Cli.Pipelines.QualitySize.PipelinePhases.Limits;
public class SonarrQualityItemLimitFetcher(ISonarrCapabilityFetcher capabilityFetcher)
internal class SonarrQualityItemLimitFetcher(ISonarrCapabilityFetcher capabilityFetcher)
: IQualityItemLimitFetcher
{
private QualityItemLimits? _cachedLimits;

2
src/Recyclarr.Cli/Pipelines/QualitySize/QualitySizeDataLister.cs
Unescape View File

@ -4,7 +4,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Pipelines.QualitySize;
public class QualitySizeDataLister(IAnsiConsole console, IQualitySizeGuideService guide)
internal class QualitySizeDataLister(IAnsiConsole console, IQualitySizeGuideService guide)
{
public void ListQualities(SupportedServices serviceType)
{

2
src/Recyclarr.Cli/Processors/Config/ConfigCreationProcessor.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.Cli.Console.Settings;
namespace Recyclarr.Cli.Processors.Config;
public class ConfigCreationProcessor(IOrderedEnumerable<IConfigCreator> creators)
internal class ConfigCreationProcessor(IOrderedEnumerable<IConfigCreator> creators)
: IConfigCreationProcessor
{
public void Process(ICreateConfigSettings settings)

2
src/Recyclarr.Cli/Processors/Config/ConfigListLocalProcessor.cs
Unescape View File

@ -9,7 +9,7 @@ using Spectre.Console.Rendering;
namespace Recyclarr.Cli.Processors.Config;
public class ConfigListLocalProcessor(
internal class ConfigListLocalProcessor(
IAnsiConsole console,
ConfigurationRegistry configRegistry,
IAppPaths paths

4
src/Recyclarr.Cli/Processors/Config/ConfigListTemplateProcessor.cs
Unescape View File

@ -1,11 +1,11 @@
using System.Globalization;
using Recyclarr.Cli.Console.Commands;
using Recyclarr.Cli.Console.Settings;
using Recyclarr.TrashGuide;
using Spectre.Console;
namespace Recyclarr.Cli.Processors.Config;
public class ConfigListTemplateProcessor(
internal class ConfigListTemplateProcessor(
IAnsiConsole console,
IConfigTemplateGuideService guideService
)

2
src/Recyclarr.Cli/Processors/Config/FileExistsException.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Processors.Config;
public class FileExistsException(string attemptedPath)
internal class FileExistsException(string attemptedPath)
: Exception($"File already exists: {attemptedPath}")
{
public string AttemptedPath { get; } = attemptedPath;

2
src/Recyclarr.Cli/Processors/Config/IConfigCreationProcessor.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.Cli.Console.Settings;
namespace Recyclarr.Cli.Processors.Config;
public interface IConfigCreationProcessor
internal interface IConfigCreationProcessor
{
void Process(ICreateConfigSettings settings);
}

2
src/Recyclarr.Cli/Processors/Config/IConfigCreator.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.Cli.Console.Settings;
namespace Recyclarr.Cli.Processors.Config;
public interface IConfigCreator
internal interface IConfigCreator
{
bool CanHandle(ICreateConfigSettings settings);
void Create(ICreateConfigSettings settings);

2
src/Recyclarr.Cli/Processors/Config/LocalConfigCreator.cs
Unescape View File

@ -6,7 +6,7 @@ using Recyclarr.Platform;
namespace Recyclarr.Cli.Processors.Config;
public class LocalConfigCreator(
internal class LocalConfigCreator(
ILogger log,
IAppPaths paths,
IFileSystem fs,

2
src/Recyclarr.Cli/Processors/Config/TemplateConfigCreator.cs
Unescape View File

@ -6,7 +6,7 @@ using Recyclarr.TrashGuide;
namespace Recyclarr.Cli.Processors.Config;
public class TemplateConfigCreator(
internal class TemplateConfigCreator(
ILogger log,
IConfigTemplateGuideService templates,
IAppPaths paths

2
src/Recyclarr.Cli/Processors/Delete/DeleteCustomFormatsProcessor.cs
Unescape View File

@ -19,7 +19,7 @@ internal class CustomFormatConfigurationScope(ILifetimeScope scope) : Configurat
scope.Resolve<ICustomFormatApiService>();
}
public class DeleteCustomFormatsProcessor(
internal class DeleteCustomFormatsProcessor(
ILogger log,
IAnsiConsole console,
ConfigurationRegistry configRegistry,

2
src/Recyclarr.Cli/Processors/Delete/IDeleteCustomFormatsProcessor.cs
Unescape View File

@ -2,7 +2,7 @@ using Recyclarr.Cli.Console.Settings;
namespace Recyclarr.Cli.Processors.Delete;
public interface IDeleteCustomFormatsProcessor
internal interface IDeleteCustomFormatsProcessor
{
Task Process(IDeleteCustomFormatSettings settings, CancellationToken ct);
}

2
src/Recyclarr.Cli/Processors/ErrorHandling/ConsoleExceptionHandler.cs
Unescape View File

@ -10,7 +10,7 @@ using YamlDotNet.Core;
namespace Recyclarr.Cli.Processors.ErrorHandling;
public class ConsoleExceptionHandler(ILogger log)
internal class ConsoleExceptionHandler(ILogger log)
{
public async Task<bool> HandleException(Exception sourceException)
{

2
src/Recyclarr.Cli/Processors/ErrorHandling/ErrorResponseParser.cs
Unescape View File

@ -5,7 +5,7 @@ using Recyclarr.Json;
namespace Recyclarr.Cli.Processors.ErrorHandling;
public sealed class ErrorResponseParser(ILogger log, string responseBody)
internal sealed class ErrorResponseParser(ILogger log, string responseBody)
{
private readonly Func<Stream> _streamFactory = () =>
new MemoryStream(Encoding.UTF8.GetBytes(responseBody));

2
src/Recyclarr.Cli/Processors/ErrorHandling/FlurlHttpExceptionHandler.cs
Unescape View File

@ -3,7 +3,7 @@ using Recyclarr.Common.Extensions;
namespace Recyclarr.Cli.Processors.ErrorHandling;
public class FlurlHttpExceptionHandler(ILogger log)
internal class FlurlHttpExceptionHandler(ILogger log)
{
[SuppressMessage("Design", "CA1031:Do not catch general exception types")]
public async Task ProcessServiceErrorMessages(IServiceErrorMessageExtractor extractor)

2
src/Recyclarr.Cli/Processors/ErrorHandling/IServiceErrorMessageExtractor.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Processors.ErrorHandling;
public interface IServiceErrorMessageExtractor
internal interface IServiceErrorMessageExtractor
{
Task<string> GetErrorMessage();
int? HttpStatusCode { get; }

2
src/Recyclarr.Cli/Processors/ErrorHandling/ServiceErrorMessageExtractor.cs
Unescape View File

@ -2,7 +2,7 @@ using Flurl.Http;
namespace Recyclarr.Cli.Processors.ErrorHandling;
public class ServiceErrorMessageExtractor(FlurlHttpException e) : IServiceErrorMessageExtractor
internal class ServiceErrorMessageExtractor(FlurlHttpException e) : IServiceErrorMessageExtractor
{
public HttpRequestError? HttpError
{

2
src/Recyclarr.Cli/Processors/ExitStatus.cs
Unescape View File

@ -1,6 +1,6 @@
namespace Recyclarr.Cli.Processors;
public enum ExitStatus
internal enum ExitStatus
{
Succeeded = 0,
Failed,

2
src/Recyclarr.Cli/Processors/FatalException.cs
Unescape View File

@ -1,4 +1,4 @@
namespace Recyclarr.Cli.Processors;
public class FatalException(string? message, Exception? innerException = null)
internal class FatalException(string? message, Exception? innerException = null)
: Exception(message, innerException);

9
src/Recyclarr.Cli/Processors/IServiceProcessor.cs
Unescape View File

@ -1,9 +0,0 @@
using Recyclarr.Cli.Console.Settings;
using Recyclarr.Config.Models;
namespace Recyclarr.Cli.Processors;
public interface IServiceProcessor
{
Task Process(ISyncSettings settings, IServiceConfiguration config);
}

8
src/Recyclarr.Cli/Processors/Sync/ISyncProcessor.cs
Unescape View File

@ -1,8 +0,0 @@
using Recyclarr.Cli.Console.Settings;
namespace Recyclarr.Cli.Processors.Sync;
public interface ISyncProcessor
{
Task<ExitStatus> Process(ISyncSettings settings, CancellationToken ct);
}

6
src/Recyclarr.Cli/Processors/Sync/SyncProcessor.cs
Unescape View File

@ -12,19 +12,19 @@ using Spectre.Console;
namespace Recyclarr.Cli.Processors.Sync;
[UsedImplicitly]
public class SyncBasedConfigurationScope(ILifetimeScope scope) : ConfigurationScope(scope)
internal class SyncBasedConfigurationScope(ILifetimeScope scope) : ConfigurationScope(scope)
{
public ISyncPipeline Pipelines { get; } = scope.Resolve<ISyncPipeline>();
}
[SuppressMessage("Design", "CA1031:Do not catch general exception types")]
public class SyncProcessor(
internal class SyncProcessor(
IAnsiConsole console,
ConfigurationRegistry configRegistry,
ConfigurationScopeFactory configScopeFactory,
ConsoleExceptionHandler exceptionHandler,
NotificationService notify
) : ISyncProcessor
)
{
public async Task<ExitStatus> Process(ISyncSettings settings, CancellationToken ct)
{

1
src/Recyclarr.Core/GlobalUsings.cs
Unescape View File

@ -1,3 +1,2 @@
global using JetBrains.Annotations;
global using Serilog;
global using SuperLinq;

4
tests/Directory.Build.props
Unescape View File

@ -6,7 +6,7 @@
<DefaultItemExcludes>$(DefaultItemExcludes);TestResults\*\**</DefaultItemExcludes>
</PropertyGroup>
<ItemGroup>
<ItemGroup Condition="$(MSBuildProjectName.EndsWith('.Tests'))">
<!-- Packages that apply to all tests -->
<PackageReference Include="AgileObjects.ReadableExpressions" />
<PackageReference Include="AutofacContrib.NSubstitute" />
@ -30,7 +30,7 @@
<PackageReference Include="TestableIO.System.IO.Abstractions.TestingHelpers" />
</ItemGroup>
<ItemGroup>
<ItemGroup Condition="$(MSBuildProjectName.EndsWith('.Tests'))">
<Using Include="AutoFixture.NUnit4" />
<Using Include="FluentAssertions" />
<Using Include="NSubstitute" />

7
tests/Recyclarr.Cli.TestLibrary/Recyclarr.Cli.TestLibrary.csproj
Unescape View File

@ -1,7 +0,0 @@
<Project Sdk="Microsoft.NET.Sdk">
<ItemGroup>
<ProjectReference Include="..\..\src\Recyclarr.Cli\Recyclarr.Cli.csproj" />
<ProjectReference Include="..\..\src\Recyclarr.Core\Recyclarr.Core.csproj" />
<ProjectReference Include="..\Recyclarr.Core.TestLibrary\Recyclarr.Core.TestLibrary.csproj" />
</ItemGroup>
</Project>

5
tests/Recyclarr.Cli.Tests/Cache/CustomFormatCacheTest.cs
Unescape View File

@ -1,12 +1,11 @@
using Recyclarr.Cli.Pipelines.CustomFormat.Cache;
using Recyclarr.Cli.Pipelines.CustomFormat.Models;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Core.TestLibrary;
namespace Recyclarr.Cli.Tests.Cache;
[TestFixture]
public class CustomFormatCacheTest
internal sealed class CustomFormatCacheTest
{
[Test]
public void New_updated_and_changed_are_added()

3
tests/Recyclarr.Cli.Tests/IntegrationTests/AutoMapperConfigurationTest.cs
Unescape View File

@ -1,9 +1,8 @@
using AutoMapper;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
internal sealed class AutoMapperConfigurationTest : CliIntegrationFixture
{
[Test]

3
tests/Recyclarr.Cli.Tests/IntegrationTests/BaseCommandSetupIntegrationTest.cs
Unescape View File

@ -1,11 +1,10 @@
using System.IO.Abstractions;
using Recyclarr.Cli.Console.Setup;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Settings;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
internal sealed class BaseCommandSetupIntegrationTest : CliIntegrationFixture
{
[Test]

2
tests/Recyclarr.Cli.Tests/IntegrationTests/CliCommandIntegrationTest.cs
Unescape View File

@ -1,6 +1,6 @@
using System.IO.Abstractions;
using Recyclarr.Cli.Console;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Repo;
using Recyclarr.TestLibrary;
using Spectre.Console.Cli;

3
tests/Recyclarr.Cli.Tests/IntegrationTests/CompositionRootTest.cs
Unescape View File

@ -10,8 +10,7 @@ using Spectre.Console;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
public class CompositionRootTest
internal sealed class CompositionRootTest
{
[SuppressMessage(
"Microsoft.Performance",

3
tests/Recyclarr.Cli.Tests/IntegrationTests/ConfigCreationProcessorIntegrationTest.cs
Unescape View File

@ -2,12 +2,11 @@ using System.IO.Abstractions;
using Recyclarr.Cli.Console.Commands;
using Recyclarr.Cli.Console.Settings;
using Recyclarr.Cli.Processors.Config;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Repo;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
internal sealed class ConfigCreationProcessorIntegrationTest : CliIntegrationFixture
{
[Test]

3
tests/Recyclarr.Cli.Tests/IntegrationTests/CustomFormatTransactionPhaseTest.cs
Unescape View File

@ -2,14 +2,13 @@ using Recyclarr.Cli.Pipelines.CustomFormat;
using Recyclarr.Cli.Pipelines.CustomFormat.Cache;
using Recyclarr.Cli.Pipelines.CustomFormat.Models;
using Recyclarr.Cli.Pipelines.CustomFormat.PipelinePhases;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Config;
using Recyclarr.Core.TestLibrary;
using Recyclarr.TrashGuide.CustomFormat;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
internal sealed class CustomFormatTransactionPhaseTest : CliIntegrationFixture
{
[Test]

3
tests/Recyclarr.Cli.Tests/IntegrationTests/ServiceCompatibilityIntegrationTest.cs
Unescape View File

@ -1,10 +1,9 @@
using System.IO.Abstractions;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Settings;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
internal sealed class ServiceCompatibilityIntegrationTest : CliIntegrationFixture
{
[Test]

3
tests/Recyclarr.Cli.Tests/IntegrationTests/TemplateConfigCreatorIntegrationTest.cs
Unescape View File

@ -1,12 +1,11 @@
using System.IO.Abstractions;
using Recyclarr.Cli.Console.Settings;
using Recyclarr.Cli.Processors.Config;
using Recyclarr.Cli.TestLibrary;
using Recyclarr.Cli.Tests.Reusable;
using Recyclarr.Repo;
namespace Recyclarr.Cli.Tests.IntegrationTests;
[TestFixture]
internal sealed class TemplateConfigCreatorIntegrationTest : CliIntegrationFixture
{
[Test]

Some files were not shown because too many files have changed in this diff Show More

Write Preview
Loading…
Cancel
Save