Merge pull request #333 from ITU-BDSA2024-GROUP10/see-the-following-and-follower-count-on-about-me

See the following and follower count on about me

Author: RasmusLarsen02

Chirp/src/Chirp.Core/AuthorService.cs

@@ -6,6 +6,7 @@ namespace Chirp.Core;
 public interface IAuthorService
 {
     public List<AuthorDTO> GetFollows(string username);
+    public List<AuthorDTO> GetFollowers(string username);
     public bool Follow(string currentUser, string userToFollow);
     public bool Unfollow(string currentUser, string userToUnFollow);
     public bool MakeFollowersUnfollow(string username);
@@ -16,6 +17,11 @@ public List<AuthorDTO> GetFollows(string username)
     {
         return db.GetAuthorFollows(username).Result;
     }
+    
+    public List<AuthorDTO> GetFollowers(string username)
+    {
+        return db.GetAuthorFollowers(username).Result;
+    }
 
     public bool MakeFollowersUnfollow(string username)
     {

Chirp/src/Chirp.Core/IAuthorRepository.cs

@@ -7,6 +7,7 @@ public interface IAuthorRepository
     public Task<AuthorDTO?> GetAuthorByName(string name);
     public Task<bool> AddAuthor(AuthorDTO author);
     public Task<List<AuthorDTO>> GetAuthorFollows(string username);
+    public Task<List<AuthorDTO>> GetAuthorFollowers(string username);
     public Task<bool> Follow (string currentUser, string userToFollow);
     public Task<bool> UnFollow (string currentUser, string userToUnFollow);
     public Task<bool> MakeFollowersUnfollow(string user);

Chirp/src/Chirp.Infrastructure/AuthorRepository.cs

@@ -36,6 +36,17 @@ public async Task<List<AuthorDTO>> GetAuthorFollows(string username)
         return GetFollowing(username).Result.Select(a => new AuthorDTO(a.UserName!, a.Email!)).ToList();
     }
 
+    public async Task<List<AuthorDTO>> GetAuthorFollowers(string username)
+    {
+        if (!await UserExists(username)) throw new UserDoesNotExist();
+        var followers = await context.Authors
+            .Where(a => a.NormalizedUserName == username.ToUpper())
+            .Select(a => a.Followers)
+            .FirstOrDefaultAsync() ?? new List<Author>();
+        
+        return followers.Select(a => new AuthorDTO(a.UserName!, a.Email!)).ToList();
+    }
+
     public async Task<bool> Follow(string currentUser, string userToFollow)
     {
         if (currentUser == userToFollow) throw new ArgumentException("You cannot follow yourself");

Chirp/src/Chirp.Web/Pages/AboutMe.cshtml

@@ -17,6 +17,12 @@
 <div>
     <h2>Name: @Model.Author!.UserName</h2>
     <h3>Email: @Model.Author.Email</h3>
+    <div style="display: flex;">
+        <h4>Following: @Model.AmountYouFollow</h4>
+        <div style="margin-right: 25px"></div>
+        <h4>Followers: @Model.AmountOfFollowers</h4>
+    </div>
+    
     @if (Model.Cheeps.Count == 1)
     {
         <h4>You have Cheep'd: @Model.Cheeps.Count time</h4>
@@ -40,4 +46,3 @@
 
     @await Component.InvokeAsync("CheepList", new {cheeps = Model.Cheeps, targetPage = "/AboutMe"})
 </div>
-

Chirp/src/Chirp.Web/Pages/AboutMe.cshtml.cs

@@ -8,34 +8,59 @@
 using Microsoft.AspNetCore.Identity;
 using Microsoft.EntityFrameworkCore.Metadata.Internal;
 using Microsoft.AspNetCore.Authentication;
+using NUnit.Framework;
 
 namespace Chirp.Web.Pages;
 
 public class AboutMe(IAuthorService authorService, ICheepService cheepService, SignInManager<Author> signInManager, UserManager<Author> userManager) : PageModel
 {
     public Author? Author { get; set; }
     public List<CheepDTO> Cheeps { get; set; } = [];
+    public int AmountYouFollow { get; set; } = 0;
+    public int AmountOfFollowers { get; set; } = 0;
     public async void OnGet()
     {
         Author = await userManager.FindByNameAsync(User.Identity!.Name!);
-        Cheeps = (cheepService.GetCheepsFromAuthor(Author!.UserName!)).ToList();
+        if (Author?.UserName == null)
+        {
+            Redirect("/NotFound");
+            return;
+        }
+        
+        SetUserInfo(Author.UserName);
     }
 
     public async Task<ActionResult> OnPostConfirmDelete()
     {
-        await SignOutAndDeleteUser();
+        try
+        {
+            await SignOutAndDeleteUser();
+        }
+        catch
+        {
+            return Redirect("/NotFound");
+        }
+        
         return Redirect("/");
     }
 
     public async Task SignOutAndDeleteUser()
     {
         var authorName = User.Identity!.Name!;
-        await signInManager.SignOutAsync();
+        
         Author = await userManager.FindByNameAsync(authorName);
-        if (Author != null)
+        if (Author?.UserName != null)
         {
-            authorService.MakeFollowersUnfollow(Author.UserName!);
+            await signInManager.SignOutAsync();
+            authorService.MakeFollowersUnfollow(Author.UserName);
             await userManager.DeleteAsync(Author);
-        } else throw new Exception("Author could not be found");
+        } else throw new Exception("Author data is not valid");
+    }
+
+    private void SetUserInfo(string authorUsername)
+    {
+        Cheeps = cheepService.GetCheepsFromAuthor(authorUsername);
+        AmountYouFollow = authorService.GetFollows(authorUsername).Count;
+        AmountOfFollowers = authorService.GetFollowers(authorUsername).Count;
     }
 }

Chirp/test/PlaywrightTests/EndToEndTests.cs

@@ -68,4 +68,51 @@ public async Task ChirpTextRedirectsToMainPage()
         await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = "Public Timeline" })).ToBeVisibleAsync();
         Assert.That(Page.Url, Is.EqualTo($"{RazorBaseUrl}/?page=1"));
     }
+
+    [Test]
+    public async Task ForgetMe()
+    {
+        #region Arrange
+
+        var testAuthor1 = new TestAuthorBuilder(RazorFactory.GetUserManager())
+            .WithUsernameAndEmail("mr. test1")
+            .GetTestAuthor();
+            
+        var testAuthor2 = new TestAuthorBuilder(RazorFactory.GetUserManager())
+            .WithUsernameAndEmail("mr. test2")
+            .GetTestAuthor();
+            
+        var testAuthor3 = new TestAuthorBuilder(RazorFactory.GetUserManager())
+            .WithUsernameAndEmail("mr. test3")
+            .GetTestAuthor();
+        
+        testAuthor1.AddFollow(testAuthor2);
+        testAuthor3.AddFollow(testAuthor1);
+        
+        //because of the following relation, the other testauthors gets created when we create testAuthor1
+        testAuthor1.Create(); 
+
+        #endregion
+        
+        await RazorPageUtils.Login(testAuthor1);
+        await Page.GetByRole(AriaRole.Link, new() { Name = "About Me" }).ClickAsync();
+        await Page.GetByRole(AriaRole.Button, new() { Name = "Forget Me" }).ClickAsync();
+        await Page.GetByRole(AriaRole.Button, new() { Name = "Yes, Delete" }).ClickAsync();
+        
+        var author1 = await RazorFactory.GetDbContext().Authors.FindAsync(testAuthor1.UserName);
+        var author2Followers = RazorFactory.GetDbContext().Authors
+            .Where(a => a.NormalizedUserName == testAuthor2.UserName!.ToUpper())
+            .SelectMany(a => a.Followers);
+        
+        var author3Following = RazorFactory.GetDbContext().Authors
+            .Where(a => a.NormalizedUserName == testAuthor3.UserName!.ToUpper())
+            .SelectMany(a => a.Following);
+        
+        await Expect(Page.Locator("body")).ToContainTextAsync("login");
+        Assert.That(Page.Url, Is.EqualTo($"{RazorBaseUrl}/?page=1"));
+        Assert.That(author1, Is.Null);
+        Assert.That(author2Followers, Is.Empty);
+        Assert.That(author3Following, Is.Empty);
+        
+    }
 }

Chirp/test/PlaywrightTests/UITests/AboutMeTests.cs

@@ -29,16 +29,33 @@ public async Task CannotSeeAboutMeLinkWhenNotRegistered()
     [Test]
     public async Task AboutMePageShowsUserInfo()
     {
-        var testAuthor = new TestAuthorBuilder(RazorFactory.GetUserManager())
-            .WithDefault()
-            .Create();
+        var testAuthor1 = new TestAuthorBuilder(RazorFactory.GetUserManager())
+            .WithUsernameAndEmail("mr. test1")
+            .GetTestAuthor();
+        var testAuthor2 = new TestAuthorBuilder(RazorFactory.GetUserManager())
+            .WithUsernameAndEmail("mr. test2")
+            .GetTestAuthor();
 
-        await RazorPageUtils.Login(testAuthor);
+        var testAuthorMain = new TestAuthorBuilder(RazorFactory.GetUserManager())
+            .WithUsernameAndEmail("mr. testMain")
+            .GetTestAuthor();
+        
+        testAuthorMain.AddFollow(testAuthor1);
+        testAuthorMain.AddFollow(testAuthor2);
+        testAuthor1.AddFollow(testAuthorMain);
+        
+        testAuthorMain.Create();
+        
+        await GenerateCheeps(testAuthorMain.author, 365);
+        
+        await RazorPageUtils.Login(testAuthorMain);
 
         await Page.GetByRole(AriaRole.Link, new() { Name = "About Me" }).ClickAsync();
-        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = $"Name: {testAuthor.UserName}" })).ToBeVisibleAsync();
-        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = $"Email: {testAuthor.Email}" })).ToBeVisibleAsync();
-        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = "You have Cheep'd: 0 times" })).ToBeVisibleAsync();
+        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = $"Name: {testAuthorMain.UserName}" })).ToBeVisibleAsync();
+        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = $"Email: {testAuthorMain.Email}" })).ToBeVisibleAsync();
+        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = "You have Cheep'd: 365 times" })).ToBeVisibleAsync();
+        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = "Following: 2" })).ToBeVisibleAsync();
+        await Expect(Page.GetByRole(AriaRole.Heading, new() { Name = "Followers: 1" })).ToBeVisibleAsync();
     }
 
     [Test]

Chirp/test/PlaywrightTests/Utils/TestAuthorBuilder.cs

@@ -6,8 +6,10 @@ namespace PlaywrightTests.Utils;
 
 public class TestAuthor
 {
-    public TestAuthor()
+    private TestAuthorBuilder _testAuthorBuilder { get; set; }
+    public TestAuthor(TestAuthorBuilder testAuthorBuilder)
     {
+        _testAuthorBuilder = testAuthorBuilder;
     }
 
     public Author author { get; set; } = new Author();
@@ -16,6 +18,18 @@ public TestAuthor()
     public string? UserName => author.UserName;
     public string? Email => author.Email;
     public List<Author> Follows => author.Following;
+    public List<Author> Followers => author.Followers;
+    
+    public void AddFollow(TestAuthor testAuthor)
+    {
+        testAuthor.Followers.Add(author);
+        author.Following.Add(testAuthor.author);
+    }
+
+    public void Create()
+    {
+        _testAuthorBuilder.Create();
+    }
 }
 
 public class TestAuthorBuilder
@@ -25,7 +39,7 @@ public class TestAuthorBuilder
 
     public TestAuthorBuilder(UserManager<Author> userManager)
     {
-        _testAuthor = new TestAuthor();
+        _testAuthor = new TestAuthor(this);
         _userManager = userManager;
         _testAuthor.Password = "Password123!";
     }
@@ -41,6 +55,11 @@ public TestAuthor Create()
         _userManager.CreateAsync(_testAuthor.author, _testAuthor.Password).Wait();
         return _testAuthor;
     }
+    
+    public TestAuthor GetTestAuthor()
+    {
+        return _testAuthor;
+    }
 
     public TestAuthorBuilder WithUsername(string username)
     {

Chirp/test/Utils/TestUtilities/InMemoryCostumeWebApplicationFactory.cs

@@ -52,6 +52,7 @@ protected override void ConfigureWebHost(IWebHostBuilder builder)
             {
                 var connection = container.GetRequiredService<DbConnection>();
                 options.UseSqlite(connection);
+                options.EnableSensitiveDataLogging();
             });
         });