MAS-129 Add tests

lambda/MAS.Tests/Infrastructure/TestBase.cs

@@ -2,11 +2,12 @@
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.TestHost;
 using Microsoft.Extensions.Configuration;
+using System;
 using System.Net.Http;
 
 namespace MAS.Tests.Infrastructure
 {
-    public class TestBase
+    public class TestBase : IDisposable
     {
         protected readonly TestServer _server;
         protected readonly HttpClient _client;
@@ -30,5 +31,10 @@ public TestBase()
             _server = new TestServer(builder);
             _client = _server.CreateClient(); 
         }
+
+        public virtual void Dispose()
+        {
+            // Do "global" teardown here; Called after every test method.
+        }
     }
 }

lambda/MAS.Tests/UnitTests/DailyEmailTests.CanCreateEmailWithTwoItemsDifferentEvidenceType.approved.txt

@@ -0,0 +1 @@
+<div class='evidenceType'><strong>Some evidence type</strong><div class='item'>Some Title<br>Some source<br>Some speciality<br>Some short summary<br><a href='https://www.medicinesresources.nhs.uk/abc'>SPS Comment</a></div></div><div class='evidenceType'><strong>Some evidence type 2</strong><div class='item'>Some Title<br>Some source<br>Some speciality 2<br>Some short summary<br><a href='https://www.medicinesresources.nhs.uk/abc'>SPS Comment</a></div></div>

lambda/MAS.Tests/UnitTests/DailyEmailTests.CanCreateEmailWithTwoItemsSharingEvidenceType.approved.txt

@@ -0,0 +1 @@
+<div class='evidenceType'><strong>Some evidence type</strong><div class='item'>Some Title<br>Some source<br>Some speciality<br>Some short summary<br><a href='https://www.medicinesresources.nhs.uk/abc'>SPS Comment</a></div><div class='item'>Some Title<br>Some source<br>Some speciality<br>Some short summary<br><a href='https://www.medicinesresources.nhs.uk/abc'>SPS Comment</a></div></div>

lambda/MAS.Tests/UnitTests/DailyEmailTests.CanCreateSingleItemEmail.approved.txt

@@ -0,0 +1 @@
+<div class='evidenceType'><strong>Some evidence type</strong><div class='item'>Some Title<br>Some source<br>Some speciality<br>Some short summary<br><a href='https://www.medicinesresources.nhs.uk/abc'>SPS Comment</a></div></div>

lambda/MAS.Tests/UnitTests/DailyEmailTests.ItemsWithManySpecialitiesRenderCorrectly.approved.txt

@@ -0,0 +1 @@
+<div class='evidenceType'><strong>Some evidence type</strong><div class='item'>Some Title<br>Some source<br>Some speciality | Another speciality<br>Some short summary<br><a href='https://www.medicinesresources.nhs.uk/abc'>SPS Comment</a></div></div>

lambda/MAS.Tests/UnitTests/DailyEmailTests.cs

@@ -0,0 +1,142 @@
+using MailChimp.Net.Core;
+using MailChimp.Net.Interfaces;
+using MailChimp.Net.Models;
+using MAS.Models;
+using MAS.Services;
+using MAS.Tests.Infrastructure;
+using Microsoft.Extensions.Logging;
+using Moq;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
+using Xunit;
+using Source = MAS.Models.Source;
+
+namespace MAS.Tests.UnitTests
+{
+    public class DailyEmailTests : TestBase
+    {
+
+        Mock<ILogger<MailService>> MockLogger;
+        Mock<IMailChimpManager> MockMailChimpManager;
+        MailService MailService;
+
+        public DailyEmailTests()
+        {
+            //Arrange
+            MockLogger = new Mock<ILogger<MailService>>();
+
+            MockMailChimpManager = new Mock<IMailChimpManager>();
+            MockMailChimpManager.Setup(x => x.Campaigns.AddAsync(It.IsAny<Campaign>())).ReturnsAsync(new Campaign() { Id = "1234" });
+            MockMailChimpManager.Setup(x => x.Content.AddOrUpdateAsync(It.IsAny<string>(), It.IsAny<ContentRequest>()));
+            MockMailChimpManager.Setup(x => x.Campaigns.SendAsync(It.IsAny<string>()));
+
+            MailService = new MailService(MockMailChimpManager.Object, MockLogger.Object);
+        }
+
+        public override void Dispose()
+        {
+
+        }
+
+        Item exampleItem = new Item()
+        {
+            Id = "123",
+            Title = "Some Title",
+            Slug = "abc",
+            ShortSummary = "Some short summary",
+            ResourceLinks = "",
+            Comment = "",
+            Speciality = new List<Speciality>
+                {
+                    new Speciality()
+                    {
+                        Key = "1a",
+                        Title = "Some speciality",
+
+                    }
+                },
+            EvidenceType = new EvidenceType()
+            {
+                Key = "1b",
+                Title = "Some evidence type"
+            },
+            Source = new Source()
+            {
+                Id = "1c",
+                Title = "Some source"
+            }
+        };
+        Item exampleItem2 = new Item()
+        {
+            Id = "123",
+            Title = "Some Title",
+            Slug = "abc",
+            ShortSummary = "Some short summary",
+            ResourceLinks = "",
+            Comment = "",
+            Speciality = new List<Speciality>
+            {
+                    new Speciality()
+                    {
+                        Key = "1a",
+                        Title = "Some speciality 2",
+
+                    }
+            },
+            EvidenceType = new EvidenceType()
+            {
+                Key = "1c",
+                Title = "Some evidence type 2"
+            },
+            Source = new Source()
+            {
+                Id = "1c",
+                Title = "Some source"
+            }
+        };
+
+        [Fact]
+        public void CanCreateSingleItemEmail()
+        {
+            var items = new List<Item> { exampleItem };
+            var actualHtml = this.MailService.CreateDailyEmailBody(items);
+
+            actualHtml.ShouldMatchApproved();
+
+        }
+
+        [Fact]
+        public void CanCreateEmailWithTwoItemsSharingEvidenceType()
+        {
+            var items = new List<Item> { exampleItem, exampleItem };
+            var actualHtml = this.MailService.CreateDailyEmailBody(items);
+
+            actualHtml.ShouldMatchApproved();
+
+        }
+
+        [Fact]
+        public void CanCreateEmailWithTwoItemsDifferentEvidenceType()
+        {
+            var items = new List<Item> { exampleItem, exampleItem2 };
+            var actualHtml = this.MailService.CreateDailyEmailBody(items);
+
+            actualHtml.ShouldMatchApproved();
+
+        }
+
+        [Fact]
+        public void ItemsWithManySpecialitiesRenderCorrectly()
+        {
+            exampleItem.Speciality.Add(new Speciality() { Key = "abcd", Title = "Another speciality" });
+            var items = new List<Item> { exampleItem };
+            var actualHtml = this.MailService.CreateDailyEmailBody(items);
+
+            actualHtml.ShouldMatchApproved();
+
+        }
+
+    }
+}

lambda/MAS/Services/ContentService.cs

@@ -30,8 +30,8 @@ public async Task<IEnumerable<Item>> GetItemsAsync()
                 try
                 {
                     var jsonStr = await client.DownloadStringTaskAsync(new Uri(AppSettings.CMSConfig.URI));
-                    var json = JsonConvert.DeserializeObject<Item[]>(jsonStr);
-                    return json;
+                    var item = JsonConvert.DeserializeObject<Item[]>(jsonStr);
+                    return item;
                 }
                 catch(Exception e)
                 {