Added logging in test

Author: Zeegaan

src/Umbraco.Tests.Integration/Umbraco.Web.BackOffice/Controllers/ContentControllerTests.cs

@@ -444,8 +444,12 @@ public async Task PostSave_Validates_Domains_Exist()
             body = body.TrimStart(AngularJsonMediaTypeFormatter.XsrfPrefix);
             ContentItemDisplay display = JsonConvert.DeserializeObject<ContentItemDisplay>(body);
 
-            Thread.CurrentThread.CurrentCulture = new CultureInfo("en-US");
+            TestContext.Progress.Write($"======CURRENT THREAD CULTURE IN TEST======{Thread.CurrentThread.CurrentCulture.Name}");
             ILocalizedTextService localizedTextService = GetRequiredService<ILocalizedTextService>();
+            foreach (var culture in localizedTextService.GetSupportedCultures())
+            {
+                TestContext.Progress.Write($"======SUPPORTED CULTURES IN TEXT SERVICE======{culture.Name}");
+            }
             var expectedMessage = localizedTextService.Localize("speechBubbles", "publishWithNoDomains", new []{"en-US"});
 
             Assert.Multiple(() =>
@@ -507,6 +511,7 @@ public async Task PostSave_Validates_All_Cultures_Has_Domains()
             body = body.TrimStart(AngularJsonMediaTypeFormatter.XsrfPrefix);
             ContentItemDisplay display = JsonConvert.DeserializeObject<ContentItemDisplay>(body);
 
+
             ILocalizedTextService localizedTextService = GetRequiredService<ILocalizedTextService>();
             var expectedMessage = localizedTextService.Localize("speechBubbles", "publishWithMissingDomain", new []{"en-US"});
 

src/Umbraco.Tests.UnitTests/Umbraco.Web.BackOffice/Controllers/ContentControllerTests.cs

@@ -261,8 +261,7 @@ private ContentController CreateContentController(IDomainService domainService)
                 Mock.Of<ISqlContext>(),
                 Mock.Of<IJsonSerializer>(),
                 Mock.Of<IScopeProvider>(),
-                Mock.Of<IAuthorizationService>(),
-                Mock.Of<IUserDataService>()
+                Mock.Of<IAuthorizationService>()
             );
 
             return controller;

src/Umbraco.Web.BackOffice/Controllers/ContentController.cs

@@ -65,7 +65,6 @@ public class ContentController : ContentControllerBase
         private readonly Lazy<IDictionary<string, ILanguage>> _allLangs;
         private readonly ILogger<ContentController> _logger;
         private readonly IScopeProvider _scopeProvider;
-        private readonly IUserDataService _userDataService;
 
         public object Domains { get; private set; }
 
@@ -91,8 +90,7 @@ public ContentController(
             ISqlContext sqlContext,
             IJsonSerializer serializer,
             IScopeProvider scopeProvider,
-            IAuthorizationService authorizationService,
-            IUserDataService userDataService)
+            IAuthorizationService authorizationService)
             : base(cultureDictionary, loggerFactory, shortStringHelper, eventMessages, localizedTextService, serializer)
         {
             _propertyEditors = propertyEditors;
@@ -114,7 +112,6 @@ public ContentController(
             _logger = loggerFactory.CreateLogger<ContentController>();
             _scopeProvider = scopeProvider;
             _allLangs = new Lazy<IDictionary<string, ILanguage>>(() => _localizationService.GetAllLanguages().ToDictionary(x => x.IsoCode, x => x, StringComparer.InvariantCultureIgnoreCase));
-            _userDataService = userDataService;
         }
 
         /// <summary>
@@ -1473,17 +1470,14 @@ internal void AddDomainWarnings(IContent persistedContent, string[] culturesPubl
             {
                 assignedDomains.UnionWith(_domainService.GetAssignedDomains(ancestorID, true));
             }
-            foreach (var data in _userDataService.GetUserData())
-            {
-                _logger.LogWarning($"{data.Name} : {data.Data}");
-            }
+
             // No domains at all, add a warning, to add domains.
             if (assignedDomains.Count == 0)
             {
                 globalNotifications.AddWarningNotification(
                     _localizedTextService.Localize("auditTrails", "publish"),
                     _localizedTextService.Localize("speechBubbles", "publishWithNoDomains"));
-                _logger.LogWarning("NOT REGISTRED DOMAIN FOR: {Cultures}");
+
                 _logger.LogWarning("The root node {RootNodeName} was published with multiple cultures, but no domains are configured, this will cause routing and caching issues, please register domains for: {Cultures}",
                     persistedContent.Name, string.Join(", ", publishedCultures));
                 return;