Removed update authorisation on Project and Nft update endpoints

Author: folathecoder

MarketPlaceApi.Tests/Controllers/NftLikeControllerTest.cs

@@ -53,44 +53,44 @@ public async Task GetNftLikeById_ReturnsOkResult()
     okResult.Value.Should().Be(nftLike);
   }
 
-  
+
 
   [Fact]
   public async Task PostNftLike_ReturnsCreatedAtActionResult()
   {
-      // Arrange
-      var nftLikeDto = new NftLikeCreateDto
+    // Arrange
+    var nftLikeDto = new NftLikeCreateDto
+    {
+      nftId = "1"
+    };
+    var nftLike = _mapper.Map<NftLike>(nftLikeDto);
+
+    // Mock the _nftService.GetAsync method to return a valid nft object
+    A.CallTo(() => _mockNftService.GetAsync(nftLike.nftId)).Returns(Task.FromResult(A.Fake<Nft>()));
+
+    // Mock the _nftLikeService.GetNftLikeByUserIdAndNftId to return null
+    A.CallTo(() => _mockNftLikeService.GetNftLikeByUserIdAndNftId("1", "1")).Returns(Task.FromResult((NftLike)null));
+
+    // Set up the HttpContext with the required header
+    var httpContext = new DefaultHttpContext();
+    httpContext.Request.Headers["userId"] = "1";
+
+    var controller = new NftLikeController(_mockNftLikeService, _mockNftService, _mapper)
+    {
+      ControllerContext = new ControllerContext
       {
-          nftId = "1"
-      };
-      var nftLike = _mapper.Map<NftLike>(nftLikeDto);
-
-      // Mock the _nftService.GetAsync method to return a valid nft object
-      A.CallTo(() => _mockNftService.GetAsync(nftLike.nftId)).Returns(Task.FromResult(A.Fake<Nft>()));
-
-      // Mock the _nftLikeService.GetNftLikeByUserIdAndNftId to return null
-      A.CallTo(() => _mockNftLikeService.GetNftLikeByUserIdAndNftId("1", "1")).Returns(Task.FromResult((NftLike)null));
-
-      // Set up the HttpContext with the required header
-      var httpContext = new DefaultHttpContext();
-      httpContext.Request.Headers["userId"] = "1";
-
-      var controller = new NftLikeController(_mockNftLikeService, _mockNftService, _mapper)
-      {
-          ControllerContext = new ControllerContext
-          {
-              HttpContext = httpContext
-          }
-      };
+        HttpContext = httpContext
+      }
+    };
 
     // Act
-      var result = await controller.Post(nftLikeDto);
+    var result = await controller.Post(nftLikeDto);
 
-      // Assert
-      result.Should().NotBeNull();
-      result.Should().BeOfType<CreatedAtActionResult>();
-      var createdAtResult = result.Should().BeOfType<CreatedAtActionResult>().Subject;
-      createdAtResult.StatusCode.Should().Be(201);
+    // Assert
+    result.Should().NotBeNull();
+    result.Should().BeOfType<CreatedAtActionResult>();
+    var createdAtResult = result.Should().BeOfType<CreatedAtActionResult>().Subject;
+    createdAtResult.StatusCode.Should().Be(201);
 
   }
 
@@ -140,7 +140,7 @@ public async Task GetNftLikesByUserId_ReturnsOkResult()
     A.CallTo(() => _mockNftLikeService.GetNftLikesByUserId("1")).Returns(Task.FromResult(nftLikes));
 
     var controller = new NftLikeController(_mockNftLikeService, _mockNftService, _mapper);
-    
+
 
     //Act
     var result = await controller.GetNftLikesByUserId("1");
@@ -172,6 +172,6 @@ public async Task GetNftLikesByUserAndNftId_ReturnsOkResult()
     var okResult = result.Should().BeOfType<OkObjectResult>().Subject;
     okResult.StatusCode.Should().Be(200);
     okResult.Value.Should().Be(nftLike);
-    
+
   }
 }

MarketPlaceApi.Tests/Controllers/PortfolioControllerTest.cs

@@ -133,7 +133,7 @@ public async Task GetPortfolioByUserId_ReturnsOkResult()
 
     A.CallTo(() => _mockPortfolioService.GetPortfolioByUserId("1")).Returns(Task.FromResult(portfolio));
     var controller = new PortfolioController(_mockPortfolioService, _mockProjectService, _mapper);
-    
+
     //Act
     var result = await controller.GetPortfolioByUserId("1");
 
@@ -143,7 +143,7 @@ public async Task GetPortfolioByUserId_ReturnsOkResult()
     var okResult = result.Should().BeOfType<OkObjectResult>().Subject;
     okResult.StatusCode.Should().Be(200);
     okResult.Value.Should().Be(portfolio);
-   
+
   }
 
   [Fact]
@@ -153,8 +153,8 @@ public async Task GetPortfolioByProjectId_ReturnsOkResult()
     var portfolio = A.Fake<Portfolio>();
     A.CallTo(() => _mockPortfolioService.GetPortfolioByProjectId("1")).Returns(Task.FromResult(portfolio));
     var controller = new PortfolioController(_mockPortfolioService, _mockProjectService, _mapper);
-    
-    
+
+
     //Act
     var result = await controller.GetPortfolioByProjectId("1");
 

MarketPlaceApi.Tests/Controllers/ProjectControllerTest.cs

@@ -48,7 +48,7 @@ public async Task GetProjectById_ReturnsOkResult()
     A.CallTo(() => _mockProjectService.GetAsync("1")).Returns(Task.FromResult(project));
     var controller = new ProjectController(_mockProjectService, _mockProjectUpdateService, _mockProjectDetailService, _mockCategoryService, _mapper);
 
-    
+
     //Act
     var result = await controller.Get("1");
 
@@ -172,7 +172,7 @@ public async Task SearchProjects_ReturnsOkResult()
 
     //Act
     var result = await controller.SearchProjects("1", true);
-    
+
     //Assert
     result.Should().NotBeNull();
     result.Should().BeOfType<OkObjectResult>();
@@ -193,7 +193,7 @@ public async Task GetProjectByWalletAddress_ReturnsOkResult()
 
     //Act
     var result = await controller.GetProjectByWalletAddress("1");
-    
+
     //Assert
     result.Should().NotBeNull();
     result.Should().BeOfType<OkObjectResult>();

MarketPlaceApi.Tests/Controllers/UserControllerTest.cs

@@ -3,7 +3,7 @@
 namespace MARKETPLACEAPI.Tests.Controllers;
 
 
-public class UserControllerTest 
+public class UserControllerTest
 {
   private readonly IUserService _mockUserService;
   private readonly IPortfolioService _mockPortfolioService;
@@ -27,12 +27,12 @@ public UserControllerTest()
   }
 
   [Fact]
-  public async Task GetUsers ()
+  public async Task GetUsers()
   {
     // Arrange
     var userController = new UserController(_mockUserService, _mockPortfolioService, _mockProjectService, _mockUserNftService, _mockProjectLikeService, _mockNftLikeService, _mapper);
     var users = A.Fake<IList<User>>();
-    
+
     A.CallTo(() => _mockUserService.GetAsync()).Returns(Task.FromResult(users));
 
     // Act
@@ -70,7 +70,7 @@ public async Task GetUserById_ReturnsOkResult()
     A.CallTo(() => _mockProjectService.GetProjectsByUserId("1")).Returns(Task.FromResult(listedProjects));
 
 
-    
+
 
     // Assert
     result.Should().NotBeNull();
@@ -85,7 +85,7 @@ public async Task Update_ReturnsNoContentResult()
     // Arrange
     var userController = new UserController(_mockUserService, _mockPortfolioService, _mockProjectService, _mockUserNftService, _mockProjectLikeService, _mockNftLikeService, _mapper);
     userController.ControllerContext.HttpContext = new DefaultHttpContext();
-    userController.ControllerContext.HttpContext.Request.Headers["userId"] = "1";    var userDto = A.Fake<UserUpdateDto>();
+    userController.ControllerContext.HttpContext.Request.Headers["userId"] = "1"; var userDto = A.Fake<UserUpdateDto>();
 
     var user = A.Fake<User>();
     A.CallTo(() => _mockUserService.GetAsync("1")).Returns(Task.FromResult(user));
@@ -145,8 +145,8 @@ public async Task GetByWalletAddress_ReturnsOkResult()
   public async Task GetMe_ReturnsOkResult()
   {
     // Arrange
-    var userController = new UserController(_mockUserService, _mockPortfolioService, 
-    _mockProjectService, _mockUserNftService, _mockProjectLikeService, 
+    var userController = new UserController(_mockUserService, _mockPortfolioService,
+    _mockProjectService, _mockUserNftService, _mockProjectLikeService,
     _mockNftLikeService, _mapper);
     userController.ControllerContext.HttpContext = new DefaultHttpContext();
     userController.ControllerContext.HttpContext.Request.Headers["userId"] = "1";