Fix testcases

Author: agile.zhou

test/ApiSiteTests/TestApiConfigController.cs

@@ -1,24 +1,25 @@
-using System.Collections.Generic;
-using System.Threading.Tasks;
 using AgileConfig.Server.Apisite.Controllers;
 using AgileConfig.Server.Apisite.Controllers.api.Models;
 using AgileConfig.Server.Apisite.Metrics;
 using AgileConfig.Server.Apisite.Models;
 using AgileConfig.Server.Common.EventBus;
 using AgileConfig.Server.Data.Entity;
 using AgileConfig.Server.IService;
+using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Caching.Memory;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
 using Moq;
+using System.Collections.Generic;
+using System.Threading.Tasks;
 
 namespace ApiSiteTests;
 
 [TestClass]
 public class TestApiConfigController
 {
     [TestMethod]
-    public async Task TestGet()
+    public async Task GetAppConfig_WithValidApp_ReturnsConfigs()
     {
         App newApp()
         {
@@ -55,8 +56,6 @@ List<Config> newConfigs()
         }
 
         var configService = new Mock<IConfigService>();
-        //configService.Setup(s => s.GetPublishedConfigsAsync("001"))
-        //    .ReturnsAsync(newConfigs);
         configService.Setup(s => s.GetPublishedConfigsByAppIdWithInheritance(It.IsAny<string>(), It.IsAny<string>()))
             .ReturnsAsync(newConfigs);
 
@@ -65,20 +64,32 @@ List<Config> newConfigs()
         var eventBus = new Mock<ITinyEventBus>();
         var meterService = new Mock<IMeterService>();
 
+        var httpContext = new DefaultHttpContext();
+        httpContext.Request.Headers["Authorization"] = "Basic MDAxOjE=";
+
         var ctrl = new AgileConfig.Server.Apisite.Controllers.api.ConfigController(
             configService.Object,
             appService.Object,
             memoryCache,
             meterService.Object,
             new ConfigController(configService.Object, appService.Object, userSErvice.Object, eventBus.Object)
         );
+        ctrl.ControllerContext = new ControllerContext
+        {
+            HttpContext = httpContext
+        };
+
         var act = await ctrl.GetAppConfig("001", new EnvString { Value = "DEV" });
 
         Assert.IsNotNull(act);
         Assert.IsNotNull(act.Value);
         Assert.IsInstanceOfType(act.Value, typeof(List<ApiConfigVM>));
         Assert.AreEqual(2, act.Value.Count);
+    }
 
+    [TestMethod]
+    public async Task GetAppConfig_WithDisabledApp_ReturnsNotFound()
+    {
         App newApp1()
         {
             return new App
@@ -87,17 +98,29 @@ App newApp1()
             };
         }
 
-        appService = new Mock<IAppService>();
+        var appService = new Mock<IAppService>();
         appService.Setup(s => s.GetAsync(It.IsAny<string>())).ReturnsAsync(newApp1);
+        var configService = new Mock<IConfigService>();
+        IMemoryCache memoryCache = null;
+        var userSErvice = new Mock<IUserService>();
+        var eventBus = new Mock<ITinyEventBus>();
+        var meterService = new Mock<IMeterService>();
+        var httpContext = new DefaultHttpContext();
+        httpContext.Request.Headers["Authorization"] = "Basic MDAxOjE=";
 
-        ctrl = new AgileConfig.Server.Apisite.Controllers.api.ConfigController(
+        var ctrl = new AgileConfig.Server.Apisite.Controllers.api.ConfigController(
             configService.Object,
             appService.Object,
             memoryCache,
             meterService.Object,
             new ConfigController(configService.Object, appService.Object, userSErvice.Object, eventBus.Object)
         );
-        act = await ctrl.GetAppConfig("001", new EnvString { Value = "DEV" });
+        ctrl.ControllerContext = new ControllerContext
+        {
+            HttpContext = httpContext
+        };
+
+        var act = await ctrl.GetAppConfig("001", new EnvString { Value = "DEV" });
 
         Assert.IsNotNull(act);
         Assert.IsNull(act.Value);