Skip to content

Commit d115715

Browse files
committed
Give share elements id's
1 parent 3f39099 commit d115715

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

LinkDotNet.Blog.UnitTests/Web/Shared/ShareBlogPostTests.cs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
using System.Linq;
2-
using AngleSharp.Html.Dom;
2+
using AngleSharp.Dom;
33
using Blazored.Toast.Services;
44
using Bunit;
55
using Bunit.TestDoubles;
@@ -21,7 +21,7 @@ public void ShouldCopyLinkToClipboard()
2121
Services.GetRequiredService<FakeNavigationManager>().NavigateTo("blogPost/1");
2222
var cut = RenderComponent<ShareBlogPost>();
2323

24-
cut.FindAll("a")[1].Click();
24+
cut.Find("#share-clipboard").Click();
2525

2626
var copyToClipboardInvocation = JSInterop.Invocations.SingleOrDefault(i => i.Identifier == "navigator.clipboard.writeText");
2727
copyToClipboardInvocation.Arguments[0].Should().Be("http://localhost/blogPost/1");
@@ -35,8 +35,7 @@ public void ShouldShareToLinkedIn()
3535

3636
var cut = RenderComponent<ShareBlogPost>();
3737

38-
var linkedInShare = (IHtmlAnchorElement)cut.FindAll("a")[0];
39-
linkedInShare.Href.Should()
40-
.Be("https://www.linkedin.com/shareArticle?mini=true&url=http://localhost/blogPost/1");
38+
var linkedInShare = cut.Find("#share-linkedin").Attributes.Single(s => s.Name == "href").Value;
39+
linkedInShare.Should().Be("https://www.linkedin.com/shareArticle?mini=true&url=http://localhost/blogPost/1");
4140
}
4241
}

LinkDotNet.Blog.Web/Shared/ShareBlogPost.razor

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
@inject IToastService toastService
44

55
<div class="flex-row">
6-
<a href="@LinkedInShare" target="_blank"><i class="fab fa-linkedin fa-lg"></i></a>
7-
<a href="javascript:void(0)" @onclick="CopyToClipboard"><i class="fas fa-link fa-lg"></i></a>
6+
<a id="share-linkedin" href="@LinkedInShare" target="_blank"><i class="fab fa-linkedin fa-lg"></i></a>
7+
<a id="share-clipboard" href="javascript:void(0)" @onclick="CopyToClipboard"><i class="fas fa-link fa-lg"></i></a>
88
</div>
99
@code {
1010
private string LinkedInShare => $"https://www.linkedin.com/shareArticle?mini=true&url={navigationManager.Uri}";

0 commit comments

Comments
 (0)