added help links for analyzers (#62)

Meir017 · web-flow · commit 686a33bb1cd4 · 2018-07-07T21:54:06.000+03:00
* added some help links * more help links * completed help links * fixed #74 * completed
diff --git a/src/FluentAssertions.Analyzers.Tests/Tips/StringTests.cs b/src/FluentAssertions.Analyzers.Tests/Tips/StringTests.cs
@@ -85,8 +85,6 @@ public class StringTests
 
         [AssertionDataTestMethod]
         [AssertionDiagnostic("string.IsNullOrEmpty(actual).Should().BeTrue({0});")]
-        [AssertionDiagnostic("string.IsNullOrEmpty(actual).Should().BeTrue({0}).And.ToString();")]
-        [AssertionDiagnostic("string.IsNullOrEmpty(actual.ToString()).Should().BeTrue({0});")]
         [AssertionDiagnostic("string.IsNullOrEmpty(actual.ToString()).Should().BeTrue({0}).And.ToString();")]
         [Implemented]
         public void StringShouldBeNullOrEmpty_TestAnalyzer(string assertion) => VerifyCSharpDiagnostic<StringShouldBeNullOrEmptyAnalyzer>(assertion);
@@ -95,12 +93,6 @@ public class StringTests
         [AssertionCodeFix(
             oldAssertion: "string.IsNullOrEmpty(actual).Should().BeTrue({0});",
             newAssertion: "actual.Should().BeNullOrEmpty({0});")]
-        [AssertionCodeFix(
-            oldAssertion: "string.IsNullOrEmpty(actual).Should().BeTrue({0}).And.ToString();",
-            newAssertion: "actual.Should().BeNullOrEmpty({0}).And.ToString();")]
-        [AssertionCodeFix(
-            oldAssertion: "string.IsNullOrEmpty(actual.ToString()).Should().BeTrue({0});",
-            newAssertion: "actual.ToString().Should().BeNullOrEmpty({0});")]
         [AssertionCodeFix(
             oldAssertion: "string.IsNullOrEmpty(actual.ToString()).Should().BeTrue({0}).And.ToString();",
             newAssertion: "actual.ToString().Should().BeNullOrEmpty({0}).And.ToString();")]
diff --git a/src/FluentAssertions.Analyzers/Constants.cs b/src/FluentAssertions.Analyzers/Constants.cs
@@ -6,6 +6,7 @@ public static class DiagnosticProperties
         {
             public const string Title = nameof(Title);
             public const string VisitorName = nameof(VisitorName);
+            public const string HelpLink = nameof(HelpLink);
         }
 
         public static class Tips
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldBeInAscendingOrder.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldBeInAscendingOrder.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class OrderByShouldEqualSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class OrderByShouldEqualSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public OrderByShouldEqualSyntaxVisitor() : base(MemberValidator.MathodContainingLambda("OrderBy"), MemberValidator.Should, MemberValidator.ArgumentIsVariable("Equal"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldBeInDescendingOrder.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldBeInDescendingOrder.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class OrderByDescendingShouldEqualSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class OrderByDescendingShouldEqualSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public OrderByDescendingShouldEqualSyntaxVisitor() : base(MemberValidator.MathodContainingLambda("OrderByDescending"), MemberValidator.Should, MemberValidator.ArgumentIsVariable("Equal"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldContainItem.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldContainItem.cs
@@ -26,7 +26,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class ContainsShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class ContainsShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public ContainsShouldBeTrueSyntaxVisitor() : base(new MemberValidator("Contains"), MemberValidator.Should, new MemberValidator("BeTrue"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldEqualOtherCollectionByComparer.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldEqualOtherCollectionByComparer.cs
@@ -27,7 +27,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class SelectShouldEqualOtherCollectionSelectSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class SelectShouldEqualOtherCollectionSelectSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public SelectShouldEqualOtherCollectionSelectSyntaxVisitor()
                 : base(MemberValidator.MathodContainingLambda("Select"), MemberValidator.Should, new MemberValidator("Equal", MathodContainingArgumentInvokingLambda))
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountGreaterOrEqualTo.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountGreaterOrEqualTo.cs
@@ -14,7 +14,7 @@ public class CollectionShouldHaveCountGreaterOrEqualToAnalyzer : CollectionAnaly
         public const string DiagnosticId = Constants.Tips.Collections.CollectionShouldHaveCountGreaterOrEqualTo;
         public const string Category = Constants.Tips.Category;
 
-        public const string Message = "Use .Should()### instead.";
+        public const string Message = "Use .Should().HaveCountGreaterOrEqualTo() instead.";
 
         protected override DiagnosticDescriptor Rule => new DiagnosticDescriptor(DiagnosticId, Title, Message, Category, DiagnosticSeverity.Info, true);
         protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class CountShouldBeGreaterOrEqualToSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class CountShouldBeGreaterOrEqualToSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public CountShouldBeGreaterOrEqualToSyntaxVisitor() : base(new MemberValidator("Count"), MemberValidator.Should, new MemberValidator("BeGreaterOrEqualTo"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountGreaterThan.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountGreaterThan.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class CountShouldBeGreaterThanSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class CountShouldBeGreaterThanSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public CountShouldBeGreaterThanSyntaxVisitor() : base(new MemberValidator("Count"), MemberValidator.Should, new MemberValidator("BeGreaterThan"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountLessOrEqualTo.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountLessOrEqualTo.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
                 yield return new CountShouldBeLessOrEqualToSyntaxVisitor();
             }
         }
-        private class CountShouldBeLessOrEqualToSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class CountShouldBeLessOrEqualToSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public CountShouldBeLessOrEqualToSyntaxVisitor() : base(new MemberValidator("Count"), MemberValidator.Should, new MemberValidator("BeLessOrEqualTo"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountLessThan.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveCountLessThan.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class CountShouldBeLessThanSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class CountShouldBeLessThanSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public CountShouldBeLessThanSyntaxVisitor() : base(new MemberValidator("Count"), MemberValidator.Should, new MemberValidator("BeLessThan"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveSameCount.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldHaveSameCount.cs
@@ -26,7 +26,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class ShouldHaveCountOtherCollectionCountSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class ShouldHaveCountOtherCollectionCountSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public ShouldHaveCountOtherCollectionCountSyntaxVisitor() : base(MemberValidator.Should, new MemberValidator("HaveCount", HasArgumentInvokingCountMethod))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldIntersectWith.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldIntersectWith.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class IntersectShouldNotBeEmptySyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class IntersectShouldNotBeEmptySyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public IntersectShouldNotBeEmptySyntaxVisitor() : base(MemberValidator.HasArguments("Intersect"), MemberValidator.Should, new MemberValidator("NotBeEmpty"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotBeEmpty.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotBeEmpty.cs
@@ -27,7 +27,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class AnyShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class AnyShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public AnyShouldBeTrueSyntaxVisitor() : base(MemberValidator.MathodNotContainingLambda("Any"), MemberValidator.Should, new MemberValidator("BeTrue"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotContainItem.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotContainItem.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class ContainsShouldBeFalseSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class ContainsShouldBeFalseSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public ContainsShouldBeFalseSyntaxVisitor() : base(new MemberValidator("Contains"), MemberValidator.Should, new MemberValidator("BeFalse"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotContainNulls.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotContainNulls.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class SelectShouldNotContainNullsSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class SelectShouldNotContainNullsSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public SelectShouldNotContainNullsSyntaxVisitor() : base(MemberValidator.MathodContainingLambda("Select"), MemberValidator.Should, new MemberValidator("NotContainNulls"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotContainProperty.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotContainProperty.cs
@@ -41,9 +41,9 @@ public class WhereShouldBeEmptySyntaxVisitor : FluentAssertionsCSharpSyntaxVisit
             }
         }
 
-        public class ShouldOnlyContainNotSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class ShouldOnlyContainSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
-            public ShouldOnlyContainNotSyntaxVisitor() : base(MemberValidator.Should, MemberValidator.MathodContainingLambda("OnlyContain"))
+            public ShouldOnlyContainSyntaxVisitor() : base(MemberValidator.Should, MemberValidator.MathodContainingLambda("OnlyContain"))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotHaveCount.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotHaveCount.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class CountShouldNotBeSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class CountShouldNotBeSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public CountShouldNotBeSyntaxVisitor() : base(MemberValidator.HasNoArguments("Count"), MemberValidator.Should, MemberValidator.ArgumentIsIdentifierOrLiteral("NotBe"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotHaveSameCount.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotHaveSameCount.cs
@@ -26,7 +26,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class CountShouldNotBeOtherCollectionCountSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class CountShouldNotBeOtherCollectionCountSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public CountShouldNotBeOtherCollectionCountSyntaxVisitor() : base(MemberValidator.HasNoArguments("Count"), MemberValidator.Should, new MemberValidator("NotBe", HasArgumentInvokingCountMethod))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotIntersectWith.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldNotIntersectWith.cs
@@ -24,7 +24,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
                 yield return new IntersectShouldBeEmptySyntaxVisitor();
             }
         }
-        private class IntersectShouldBeEmptySyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class IntersectShouldBeEmptySyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public IntersectShouldBeEmptySyntaxVisitor() : base(MemberValidator.HasArguments("Intersect"), MemberValidator.Should, new MemberValidator("BeEmpty"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldOnlyHaveUniqueItems.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldOnlyHaveUniqueItems.cs
@@ -27,7 +27,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class ShouldHaveSameCountThisCollectionDistinctSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class ShouldHaveSameCountThisCollectionDistinctSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public ShouldHaveSameCountThisCollectionDistinctSyntaxVisitor() : base(MemberValidator.Should, new MemberValidator("HaveSameCount", ArgumentInvokesDistinctMethod))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldOnlyHaveUniqueItemsByComparer.cs b/src/FluentAssertions.Analyzers/Tips/Collections/CollectionShouldOnlyHaveUniqueItemsByComparer.cs
@@ -26,7 +26,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-        private class SelectShouldOnlyHaveUniqueItemsSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class SelectShouldOnlyHaveUniqueItemsSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
             public SelectShouldOnlyHaveUniqueItemsSyntaxVisitor() : base(MemberValidator.MathodContainingLambda("Select"), MemberValidator.Should, new MemberValidator("OnlyHaveUniqueItems"))
             {
diff --git a/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldContainKey.cs b/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldContainKey.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new ContainsKeyShouldBeTrue();
+                yield return new ContainsKeyShouldBeTrueSyntaxVisitor();
             }
         }
 
-		private class ContainsKeyShouldBeTrue : FluentAssertionsCSharpSyntaxVisitor
+		public class ContainsKeyShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
 		{
-			public ContainsKeyShouldBeTrue() : base(new MemberValidator("ContainsKey"), MemberValidator.Should, new MemberValidator("BeTrue"))
+			public ContainsKeyShouldBeTrueSyntaxVisitor() : base(new MemberValidator("ContainsKey"), MemberValidator.Should, new MemberValidator("BeTrue"))
 			{
 			}
 		}
diff --git a/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldContainValue.cs b/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldContainValue.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new ContainsValueShouldBeTrue();
+                yield return new ContainsValueShouldBeTrueSyntaxVisitor();
             }
         }
 
-        private class ContainsValueShouldBeTrue : FluentAssertionsCSharpSyntaxVisitor
+        public class ContainsValueShouldBeTrueSyntaxVisitor: FluentAssertionsCSharpSyntaxVisitor
         {
-            public ContainsValueShouldBeTrue() : base(new MemberValidator("ContainsValue"), MemberValidator.Should, new MemberValidator("BeTrue"))
+            public ContainsValueShouldBeTrueSyntaxVisitor() : base(new MemberValidator("ContainsValue"), MemberValidator.Should, new MemberValidator("BeTrue"))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldNotContainKey.cs b/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldNotContainKey.cs
@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
             }
         }
 
-		private class ContainsKeyShouldBeFalseSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+		public class ContainsKeyShouldBeFalseSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
 		{
 			public ContainsKeyShouldBeFalseSyntaxVisitor() : base(new MemberValidator("ContainsKey"), MemberValidator.Should, new MemberValidator("BeFalse"))
 			{
diff --git a/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldNotContainValue.cs b/src/FluentAssertions.Analyzers/Tips/Dictionaries/DictionaryShouldNotContainValue.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new ContainsValueShouldBeFalse();
+                yield return new ContainsValueShouldBeFalseSyntaxVisitor();
             }
         }
 
-        private class ContainsValueShouldBeFalse : FluentAssertionsCSharpSyntaxVisitor
+        public class ContainsValueShouldBeFalseSyntaxVisitor: FluentAssertionsCSharpSyntaxVisitor
         {
-            public ContainsValueShouldBeFalse() : base(new MemberValidator("ContainsValue"), MemberValidator.Should, new MemberValidator("BeFalse"))
+            public ContainsValueShouldBeFalseSyntaxVisitor() : base(new MemberValidator("ContainsValue"), MemberValidator.Should, new MemberValidator("BeFalse"))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Numerics/NumericShouldBeNegative.cs b/src/FluentAssertions.Analyzers/Tips/Numerics/NumericShouldBeNegative.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new NumericShouldBeNegativeSyntaxVisitor();
+                yield return new NumericShouldBeBeLessThan0SyntaxVisitor();
             }
         }
 
-        public class NumericShouldBeNegativeSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class NumericShouldBeBeLessThan0SyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
-            public NumericShouldBeNegativeSyntaxVisitor() : base(MemberValidator.Should, MemberValidator.ArgumentIsLiteral("BeLessThan", 0))
+            public NumericShouldBeBeLessThan0SyntaxVisitor() : base(MemberValidator.Should, MemberValidator.ArgumentIsLiteral("BeLessThan", 0))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Numerics/NumericShouldBePositive.cs b/src/FluentAssertions.Analyzers/Tips/Numerics/NumericShouldBePositive.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new NumericShouldBePositiveSyntaxVisitor();
+                yield return new NumericShouldBeBeGreaterThan0SyntaxVisitor();
             }
         }
 
-        public class NumericShouldBePositiveSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class NumericShouldBeBeGreaterThan0SyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
-            public NumericShouldBePositiveSyntaxVisitor() : base(MemberValidator.Should, MemberValidator.ArgumentIsLiteral("BeGreaterThan", 0))
+            public NumericShouldBeBeGreaterThan0SyntaxVisitor() : base(MemberValidator.Should, MemberValidator.ArgumentIsLiteral("BeGreaterThan", 0))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Strings/StringShouldBeNullOrEmpty.cs b/src/FluentAssertions.Analyzers/Tips/Strings/StringShouldBeNullOrEmpty.cs
@@ -22,13 +22,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new StringShouldBeNullOrEmptySyntaxVisitor();
+                yield return new StringIsNullOrEmptyShouldBeTrueSyntaxVisitor();
             }
         }
 
-        public class StringShouldBeNullOrEmptySyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class StringIsNullOrEmptyShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
-            public StringShouldBeNullOrEmptySyntaxVisitor() : base(new MemberValidator("IsNullOrEmpty"), MemberValidator.Should, new MemberValidator("BeTrue"))
+            public StringIsNullOrEmptyShouldBeTrueSyntaxVisitor() : base(new MemberValidator("IsNullOrEmpty"), MemberValidator.Should, new MemberValidator("BeTrue"))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Strings/StringShouldEndWith.cs b/src/FluentAssertions.Analyzers/Tips/Strings/StringShouldEndWith.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new StringShouldEndWithSyntaxVisitor();
+                yield return new EndWithShouldBeTrueSyntaxVisitor();
             }
         }
 
-        public class StringShouldEndWithSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class EndWithShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
-            public StringShouldEndWithSyntaxVisitor() : base(new MemberValidator("EndsWith"), MemberValidator.Should, new MemberValidator("BeTrue"))
+            public EndWithShouldBeTrueSyntaxVisitor() : base(new MemberValidator("EndsWith"), MemberValidator.Should, new MemberValidator("BeTrue"))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Tips/Strings/StringShouldHaveLength.cs b/src/FluentAssertions.Analyzers/Tips/Strings/StringShouldHaveLength.cs
@@ -21,13 +21,13 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors
         {
             get
             {
-                yield return new StringShouldHaveLengthSyntaxVisitor();
+                yield return new LengthShouldBeSyntaxVisitor();
             }
         }
 
-        public class StringShouldHaveLengthSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
+        public class LengthShouldBeSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor
         {
-            public StringShouldHaveLengthSyntaxVisitor() : base(new MemberValidator("Length"), MemberValidator.Should, new MemberValidator("Be"))
+            public LengthShouldBeSyntaxVisitor() : base(new MemberValidator("Length"), MemberValidator.Should, new MemberValidator("Be"))
             {
             }
         }
diff --git a/src/FluentAssertions.Analyzers/Utilities/FluentAssertionsAnalyzer.cs b/src/FluentAssertions.Analyzers/Utilities/FluentAssertionsAnalyzer.cs
diff --git a/src/FluentAssertions.Analyzers/Utilities/FluentAssertionsCSharpSyntaxVisitor.cs b/src/FluentAssertions.Analyzers/Utilities/FluentAssertionsCSharpSyntaxVisitor.cs
diff --git a/src/FluentAssertions.Analyzers/Utilities/HelpLinks.cs b/src/FluentAssertions.Analyzers/Utilities/HelpLinks.cs

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@ public static class DiagnosticProperties`
`6`	`6`	`{`
`7`	`7`	`public const string Title = nameof(Title);`
`8`	`8`	`public const string VisitorName = nameof(VisitorName);`
	`9`	`+ public const string HelpLink = nameof(HelpLink);`
`9`	`10`	`}`
`10`	`11`
`11`	`12`	`public static class Tips`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors`
`25`	`25`	`}`
`26`	`26`	`}`
`27`	`27`
`28`		`- private class OrderByShouldEqualSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
	`28`	`+ public class OrderByShouldEqualSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
`29`	`29`	`{`
`30`	`30`	`public OrderByShouldEqualSyntaxVisitor() : base(MemberValidator.MathodContainingLambda("OrderBy"), MemberValidator.Should, MemberValidator.ArgumentIsVariable("Equal"))`
`31`	`31`	`{`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors`
`26`	`26`	`}`
`27`	`27`	`}`
`28`	`28`
`29`		`- private class ContainsShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
	`29`	`+ public class ContainsShouldBeTrueSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
`30`	`30`	`{`
`31`	`31`	`public ContainsShouldBeTrueSyntaxVisitor() : base(new MemberValidator("Contains"), MemberValidator.Should, new MemberValidator("BeTrue"))`
`32`	`32`	`{`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`
`30`		`- private class SelectShouldEqualOtherCollectionSelectSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
	`30`	`+ public class SelectShouldEqualOtherCollectionSelectSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
`31`	`31`	`{`
`32`	`32`	`public SelectShouldEqualOtherCollectionSelectSyntaxVisitor()`
`33`	`33`	`: base(MemberValidator.MathodContainingLambda("Select"), MemberValidator.Should, new MemberValidator("Equal", MathodContainingArgumentInvokingLambda))`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ public class CollectionShouldHaveCountGreaterOrEqualToAnalyzer : CollectionAnaly`
`14`	`14`	`public const string DiagnosticId = Constants.Tips.Collections.CollectionShouldHaveCountGreaterOrEqualTo;`
`15`	`15`	`public const string Category = Constants.Tips.Category;`
`16`	`16`
`17`		`- public const string Message = "Use .Should()### instead.";`
	`17`	`+ public const string Message = "Use .Should().HaveCountGreaterOrEqualTo() instead.";`
`18`	`18`
`19`	`19`	`protected override DiagnosticDescriptor Rule => new DiagnosticDescriptor(DiagnosticId, Title, Message, Category, DiagnosticSeverity.Info, true);`
`20`	`20`	`protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors`
`@@ -25,7 +25,7 @@ protected override IEnumerable<FluentAssertionsCSharpSyntaxVisitor> Visitors`
`25`	`25`	`}`
`26`	`26`	`}`
`27`	`27`
`28`		`- private class CountShouldBeGreaterOrEqualToSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
	`28`	`+ public class CountShouldBeGreaterOrEqualToSyntaxVisitor : FluentAssertionsCSharpSyntaxVisitor`
`29`	`29`	`{`
`30`	`30`	`public CountShouldBeGreaterOrEqualToSyntaxVisitor() : base(new MemberValidator("Count"), MemberValidator.Should, new MemberValidator("BeGreaterOrEqualTo"))`
`31`	`31`	`{`