Non compliant package didn't fail validation (#6436)

Author: xavierdecoster

src/NuGetGallery/Security/RequirePackageMetadataCompliancePolicy.cs

@@ -116,7 +116,8 @@ private bool IsPackageMetadataCompliant(Package package, State state, out IList<
 
             // Author validation
             if (!package.FlattenedAuthors
-                .Split(new[] { ',', ' ' }, StringSplitOptions.RemoveEmptyEntries)
+                .Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries)
+                .Select(s => s.Trim())
                 .Contains(state.RequiredCoOwnerUsername, StringComparer.InvariantCultureIgnoreCase))
             {
                 complianceFailures.Add(string.Format(CultureInfo.CurrentCulture, Strings.SecurityPolicy_RequiredAuthorMissing, state.RequiredCoOwnerUsername));

tests/NuGetGallery.Facts/Security/RequirePackageMetadataCompliancePolicyFacts.cs

@@ -335,7 +335,7 @@ public static IReadOnlyCollection<Package> CreateNonCompliantPackages()
 
                 // Ensure authors is non-compliant.
                 var nonCompliantPackage4 = CreateCompliantPackage(version, newPackageRegistration);
-                nonCompliantPackage4.FlattenedAuthors = "NuGet";
+                nonCompliantPackage4.FlattenedAuthors = "Microsoft Communications Platform";
                 nonCompliantPackages.Add(nonCompliantPackage4);
 
                 return nonCompliantPackages;