refactor: simplify PlayerServiceTests and make environment configurable

Remove unnecessary IDisposable pattern from PlayerServiceTests since
there's no cleanup needed after removing environment variable dependency.
Add optional environmentName parameter to PlayerMocks.InitServiceMocks()
to enable testing environment-specific behavior.

Changes:
- Remove IDisposable implementation from PlayerServiceTests
- Add environmentName parameter to InitServiceMocks (defaults to Development)

Author: nanotaboada

test/Dotnet.Samples.AspNetCore.WebApi.Tests/Unit/PlayerServiceTests.cs

@@ -7,10 +7,8 @@
 
 namespace Dotnet.Samples.AspNetCore.WebApi.Tests.Unit;
 
-public class PlayerServiceTests : IDisposable
+public class PlayerServiceTests
 {
-    private bool _disposed;
-
     /* -------------------------------------------------------------------------
      * Create
      * ---------------------------------------------------------------------- */
@@ -331,18 +329,4 @@ private static async Task<long> ExecutionTimeAsync(Func<Task> awaitable)
 
         return stopwatch.ElapsedMilliseconds;
     }
-
-    protected virtual void Dispose(bool disposing)
-    {
-        if (!_disposed)
-        {
-            _disposed = true;
-        }
-    }
-
-    public void Dispose()
-    {
-        Dispose(true);
-        GC.SuppressFinalize(this);
-    }
 }

test/Dotnet.Samples.AspNetCore.WebApi.Tests/Utilities/PlayerMocks.cs

@@ -49,14 +49,14 @@ public static (
             Mock<IMemoryCache> memoryCache,
             Mock<IMapper> mapper,
             Mock<IHostEnvironment> environment
-        ) InitServiceMocks(object? cacheValue = null)
+        ) InitServiceMocks(object? cacheValue = null, string environmentName = "Development")
         {
             var repository = new Mock<IPlayerRepository>();
             var logger = new Mock<ILogger<PlayerService>>();
             var memoryCache = SetupMemoryCacheMock(cacheValue ?? It.IsAny<object>());
             var mapper = new Mock<IMapper>();
             var environment = new Mock<IHostEnvironment>();
-            environment.Setup(env => env.EnvironmentName).Returns("Development");
+            environment.Setup(env => env.EnvironmentName).Returns(environmentName);
             return (repository, logger, memoryCache, mapper, environment);
         }