[Backend.Tests] Add sealed internal to Mock classes (#3049)

sillsdev · Jun 5, 2024 · c6f2191 · c6f2191
1 parent a05e09b
commit c6f2191
Show file tree

Hide file tree

Showing 17 changed files with 21 additions and 21 deletions.
diff --git a/Backend.Tests/Controllers/LiftControllerTests.cs b/Backend.Tests/Controllers/LiftControllerTests.cs
@@ -684,7 +684,7 @@ public void TestRoundtrip(RoundTripObj roundTripObj)
             }
         }
 
-        private class MockLogger : ILogger<LiftController>
+        private sealed class MockLogger : ILogger<LiftController>
         {
             public IDisposable BeginScope<TState>(TState state)
             {

diff --git a/Backend.Tests/Mocks/BannerRepositoryMock.cs b/Backend.Tests/Mocks/BannerRepositoryMock.cs
@@ -6,7 +6,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class BannerRepositoryMock : IBannerRepository
+    sealed internal class BannerRepositoryMock : IBannerRepository
     {
         private Dictionary<BannerType, Banner> _banners;
 

diff --git a/Backend.Tests/Mocks/EmailServiceMock.cs b/Backend.Tests/Mocks/EmailServiceMock.cs
@@ -4,7 +4,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    internal class EmailServiceMock : IEmailService
+    sealed internal class EmailServiceMock : IEmailService
     {
         public Task<bool> SendEmail(MimeMessage msg)
         {

diff --git a/Backend.Tests/Mocks/HubContextMock.cs b/Backend.Tests/Mocks/HubContextMock.cs
@@ -9,14 +9,14 @@ namespace Backend.Tests.Mocks
     /// <summary>
     /// A *very* sparse, mostly unimplemented Mock of SignalR HubContext.
     /// </summary>
-    public class HubContextMock : IHubContext<CombineHub>
+    sealed internal class HubContextMock : IHubContext<CombineHub>
     {
         public IHubClients Clients => new HubClientsMock();
 
         public IGroupManager Groups => throw new System.NotImplementedException();
     }
 
-    public class HubClientsMock : IHubClients
+    sealed internal class HubClientsMock : IHubClients
     {
         public IClientProxy AllExcept(IReadOnlyList<string> excludedConnectionIds)
         {
@@ -61,7 +61,7 @@ public IClientProxy Users(IReadOnlyList<string> userIds)
         public IClientProxy All => new ClientProxyMock();
     }
 
-    public class ClientProxyMock : IClientProxy
+    sealed internal class ClientProxyMock : IClientProxy
     {
         // Disable this warning as this mock simply needs to return an empty Task, not await anything.
 #pragma warning disable 1998

diff --git a/Backend.Tests/Mocks/MergeBlacklistRepositoryMock.cs b/Backend.Tests/Mocks/MergeBlacklistRepositoryMock.cs
@@ -8,7 +8,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class MergeBlacklistRepositoryMock : IMergeBlacklistRepository
+    sealed internal class MergeBlacklistRepositoryMock : IMergeBlacklistRepository
     {
         private readonly List<MergeWordSet> _mergeBlacklist;
 

diff --git a/Backend.Tests/Mocks/MergeGraylistRepositoryMock.cs b/Backend.Tests/Mocks/MergeGraylistRepositoryMock.cs
@@ -8,7 +8,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class MergeGraylistRepositoryMock : IMergeGraylistRepository
+    sealed internal class MergeGraylistRepositoryMock : IMergeGraylistRepository
     {
         private readonly List<MergeWordSet> _mergeGraylist;
 

diff --git a/Backend.Tests/Mocks/PasswordResetContextMock.cs b/Backend.Tests/Mocks/PasswordResetContextMock.cs
@@ -6,7 +6,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class PasswordResetContextMock : IPasswordResetContext
+    sealed internal class PasswordResetContextMock : IPasswordResetContext
     {
         private List<PasswordReset> _resets;
 

diff --git a/Backend.Tests/Mocks/PasswordResetServiceMock.cs b/Backend.Tests/Mocks/PasswordResetServiceMock.cs
@@ -4,7 +4,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    internal class PasswordResetServiceMock : IPasswordResetService
+    sealed internal class PasswordResetServiceMock : IPasswordResetService
     {
         public Task<PasswordReset> CreatePasswordReset(string email)
         {

diff --git a/Backend.Tests/Mocks/PermissionServiceMock.cs b/Backend.Tests/Mocks/PermissionServiceMock.cs
@@ -7,7 +7,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    internal class PermissionServiceMock : IPermissionService
+    sealed internal class PermissionServiceMock : IPermissionService
     {
         private readonly IUserRepository _userRepo;
         private const string NoHttpContextAvailable = "NO_HTTP_CONTEXT_AVAILABLE";
@@ -157,7 +157,7 @@ public string GetUserId(HttpContext? request)
     }
 
     [Serializable]
-    public class UserAuthenticationException : Exception
+    internal class UserAuthenticationException : Exception
     {
         public UserAuthenticationException() { }
 

diff --git a/Backend.Tests/Mocks/ProjectRepositoryMock.cs b/Backend.Tests/Mocks/ProjectRepositoryMock.cs
@@ -9,7 +9,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class ProjectRepositoryMock : IProjectRepository
+    sealed internal class ProjectRepositoryMock : IProjectRepository
     {
         private readonly List<Project> _projects;
 

diff --git a/Backend.Tests/Mocks/SemanticDomainRepositoryMock.cs b/Backend.Tests/Mocks/SemanticDomainRepositoryMock.cs
@@ -5,7 +5,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class SemanticDomainRepositoryMock : ISemanticDomainRepository
+    sealed internal class SemanticDomainRepositoryMock : ISemanticDomainRepository
     {
         private object? _responseObj;
 

diff --git a/Backend.Tests/Mocks/SpeakerRepositoryMock.cs b/Backend.Tests/Mocks/SpeakerRepositoryMock.cs
@@ -8,7 +8,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class SpeakerRepositoryMock : ISpeakerRepository
+    sealed internal class SpeakerRepositoryMock : ISpeakerRepository
     {
         private readonly List<Speaker> _speakers;
 

diff --git a/Backend.Tests/Mocks/StatisticsServiceMock.cs b/Backend.Tests/Mocks/StatisticsServiceMock.cs
@@ -6,7 +6,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    internal class StatisticsServiceMock : IStatisticsService
+    sealed internal class StatisticsServiceMock : IStatisticsService
     {
         public Task<List<SemanticDomainCount>> GetSemanticDomainCounts(string projectId, string lang)
         {

diff --git a/Backend.Tests/Mocks/UserEditRepositoryMock.cs b/Backend.Tests/Mocks/UserEditRepositoryMock.cs
@@ -7,7 +7,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class UserEditRepositoryMock : IUserEditRepository
+    sealed internal class UserEditRepositoryMock : IUserEditRepository
     {
         private readonly List<UserEdit> _userEdits;
 

diff --git a/Backend.Tests/Mocks/UserRepositoryMock.cs b/Backend.Tests/Mocks/UserRepositoryMock.cs
@@ -9,7 +9,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class UserRepositoryMock : IUserRepository
+    sealed internal class UserRepositoryMock : IUserRepository
     {
         private readonly List<User> _users;
 
@@ -103,7 +103,7 @@ public Task<ResultOfUpdate> ChangePassword(string userId, string password)
     }
 
     [Serializable]
-    public class UserCreationException : Exception
+    internal class UserCreationException : Exception
     {
         public UserCreationException() { }
 

diff --git a/Backend.Tests/Mocks/UserRoleRepositoryMock.cs b/Backend.Tests/Mocks/UserRoleRepositoryMock.cs
@@ -8,7 +8,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class UserRoleRepositoryMock : IUserRoleRepository
+    sealed internal class UserRoleRepositoryMock : IUserRoleRepository
     {
         private readonly List<UserRole> _userRoles;
 

diff --git a/Backend.Tests/Mocks/WordRepositoryMock.cs b/Backend.Tests/Mocks/WordRepositoryMock.cs
@@ -7,7 +7,7 @@
 
 namespace Backend.Tests.Mocks
 {
-    public class WordRepositoryMock : IWordRepository
+    sealed internal class WordRepositoryMock : IWordRepository
     {
         private readonly List<Word> _words;
         private readonly List<Word> _frontier;