From b42ebe6f1bf71d83b02f68dcbf6bbe2c768e6f97 Mon Sep 17 00:00:00 2001
From: Conner Turnbull <133619638+cturnbull-bitwarden@users.noreply.github.com>
Date: Mon, 3 Jun 2024 09:58:53 -0400
Subject: [PATCH] Fix Broken Icon Unit Test (#4151)

* Updated test domain from ameritrade.com to tdameritrade.com to fix failing test

* Added a skip attribute
---
 test/Icons.Test/Services/IconFetchingServiceTests.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/Icons.Test/Services/IconFetchingServiceTests.cs b/test/Icons.Test/Services/IconFetchingServiceTests.cs
index cc23975ec5..9ab57add5f 100644
--- a/test/Icons.Test/Services/IconFetchingServiceTests.cs
+++ b/test/Icons.Test/Services/IconFetchingServiceTests.cs
@@ -5,13 +5,13 @@ namespace Bit.Icons.Test.Services;
 
 public class IconFetchingServiceTests : ServiceTestBase<IconFetchingService>
 {
-    [Theory]
+    [Theory(Skip = "Run ad-hoc")]
     [InlineData("www.twitter.com")] // https site
     [InlineData("www.google.com")] // https site
     [InlineData("neverssl.com")] // http site
     [InlineData("neopets.com")] // uses favicon.ico
     [InlineData("hopin.com")] // uses svg+xml format
-    [InlineData("ameritrade.com")] // redirects to tdameritrade.com
+    [InlineData("tdameritrade.com")]
     [InlineData("icloud.com")]
     public async Task GetIconAsync_Success(string domain)
     {