Skip to content

Commit b18eaef

Browse files
committed
TD-3931-Controller refactor -DataService reference removed from the controller.
1 parent f6ac84d commit b18eaef

File tree

2 files changed

+7
-9
lines changed

2 files changed

+7
-9
lines changed

DigitalLearningSolutions.Web.Tests/Controllers/TrackingSystem/Delegates/DelegateApprovalsControllerTests.cs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
namespace DigitalLearningSolutions.Web.Tests.Controllers.TrackingSystem.Delegates
22
{
3-
using DigitalLearningSolutions.Data.DataServices.UserDataService;
43
using DigitalLearningSolutions.Web.Controllers.TrackingSystem.Delegates;
54
using DigitalLearningSolutions.Web.Services;
65
using DigitalLearningSolutions.Web.Tests.ControllerHelpers;
@@ -12,14 +11,14 @@ public class DelegateApprovalsControllerTests
1211
{
1312
private DelegateApprovalsController delegateApprovalsController = null!;
1413
private IDelegateApprovalsService delegateApprovalsService = null!;
15-
private IUserDataService userDataService = null!;
14+
private IUserService userService = null!;
1615

1716
[SetUp]
1817
public void Setup()
1918
{
20-
userDataService = A.Fake<IUserDataService>();
19+
userService = A.Fake<IUserService>();
2120
delegateApprovalsService = A.Fake<IDelegateApprovalsService>();
22-
delegateApprovalsController = new DelegateApprovalsController(delegateApprovalsService, userDataService)
21+
delegateApprovalsController = new DelegateApprovalsController(delegateApprovalsService, userService)
2322
.WithDefaultContext()
2423
.WithMockUser(true);
2524
}

DigitalLearningSolutions.Web/Controllers/TrackingSystem/Delegates/DelegateApprovalsController.cs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
namespace DigitalLearningSolutions.Web.Controllers.TrackingSystem.Delegates
22
{
33
using System.Linq;
4-
using DigitalLearningSolutions.Data.DataServices.UserDataService;
54
using DigitalLearningSolutions.Data.Enums;
65
using DigitalLearningSolutions.Web.Attributes;
76
using DigitalLearningSolutions.Web.Helpers;
@@ -20,15 +19,15 @@
2019
public class DelegateApprovalsController : Controller
2120
{
2221
private readonly IDelegateApprovalsService delegateApprovalsService;
23-
private readonly IUserDataService userDataService;
22+
private readonly IUserService userService;
2423

2524
public DelegateApprovalsController(
2625
IDelegateApprovalsService delegateApprovalsService,
27-
IUserDataService userDataService
26+
IUserService userService
2827
)
2928
{
3029
this.delegateApprovalsService = delegateApprovalsService;
31-
this.userDataService = userDataService;
30+
this.userService = userService;
3231
}
3332

3433
public IActionResult Index()
@@ -65,7 +64,7 @@ public IActionResult ApproveDelegatesForCentre()
6564
[Route("/TrackingSystem/Delegates/Reject")]
6665
public IActionResult DelegateRejectionPage(int delegateId)
6766
{
68-
var delegateEntity = userDataService.GetDelegateById(delegateId);
67+
var delegateEntity = userService.GetDelegateById(delegateId);
6968
var model = new RejectDelegateViewModel(delegateEntity);
7069
return View(model);
7170
}

0 commit comments

Comments
 (0)