Skip to content

Commit

Permalink
test: suppress CA1307 and add StringComparison overload calls
Browse files Browse the repository at this point in the history
  • Loading branch information
oliverbooth committed Aug 27, 2023
1 parent 43d2e8f commit 2fb91d5
Showing 1 changed file with 44 additions and 0 deletions.
44 changes: 44 additions & 0 deletions X10D.Tests/src/Text/StringTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,9 @@ public void CountSubstring_ShouldHonor_StringComparison()
Assert.Multiple(() =>
{
Assert.That("Hello World".CountSubstring('E'), Is.Zero);
#pragma warning disable CA1307
Assert.That("Hello World".CountSubstring("E"), Is.Zero);
#pragma warning restore CA1307
Assert.That("Hello World".CountSubstring("E", StringComparison.OrdinalIgnoreCase), Is.EqualTo(1));
});
}
Expand All @@ -175,7 +177,9 @@ public void CountSubstring_ShouldReturn0_GivenNoInstanceChar()
Assert.Multiple(() =>
{
Assert.That("Hello World".CountSubstring('z'), Is.Zero);
#pragma warning disable CA1307
Assert.That("Hello World".CountSubstring("z"), Is.Zero);
#pragma warning restore CA1307
Assert.That("Hello World".CountSubstring("z", StringComparison.OrdinalIgnoreCase), Is.Zero);
});
}
Expand All @@ -186,7 +190,9 @@ public void CountSubstring_ShouldReturn1_GivenSingleInstanceChar()
Assert.Multiple(() =>
{
Assert.That("Hello World".CountSubstring('e'), Is.EqualTo(1));
#pragma warning disable CA1307
Assert.That("Hello World".CountSubstring("e"), Is.EqualTo(1));
#pragma warning restore CA1307
Assert.That("Hello World".CountSubstring("e", StringComparison.OrdinalIgnoreCase), Is.EqualTo(1));
});
}
Expand All @@ -197,7 +203,9 @@ public void CountSubstring_ShouldReturn0_GivenEmptyHaystack()
Assert.Multiple(() =>
{
Assert.That(string.Empty.CountSubstring('\0'), Is.Zero);
#pragma warning disable CA1307
Assert.That(string.Empty.CountSubstring(string.Empty), Is.Zero);
#pragma warning restore CA1307
Assert.That(string.Empty.CountSubstring(string.Empty, StringComparison.OrdinalIgnoreCase), Is.Zero);
});
}
Expand All @@ -217,7 +225,10 @@ public void EnsureEndsWith_ShouldPrependChar_GivenEndsWithReturnFalse()
const string value = "Hello Worl";
const char substring = 'd';

#pragma warning disable CA1307
Assert.That(value.EnsureEndsWith(substring), Is.EqualTo("Hello World"));
#pragma warning restore CA1307
Assert.That(value.EnsureEndsWith(substring, StringComparison.Ordinal), Is.EqualTo("Hello World"));
}

[Test]
Expand All @@ -226,7 +237,10 @@ public void EnsureEndsWith_ShouldReturnChar_GivenEndsWithReturnTrue()
const string value = "A";
const char substring = 'A';

#pragma warning disable CA1307
Assert.That(value.EnsureEndsWith(substring), Is.EqualTo(value));
#pragma warning restore CA1307
Assert.That(value.EnsureEndsWith(substring, StringComparison.Ordinal), Is.EqualTo(value));
}

[Test]
Expand All @@ -235,7 +249,10 @@ public void EnsureStartsWith_ShouldPrependChar_GivenEndsWithReturnFalse()
const string value = "B";
const char substring = 'A';

#pragma warning disable CA1307
Assert.That(value.EnsureStartsWith(substring), Is.EqualTo("AB"));
#pragma warning restore CA1307
Assert.That(value.EnsureStartsWith(substring, StringComparison.Ordinal), Is.EqualTo("AB"));
}

[Test]
Expand All @@ -244,7 +261,10 @@ public void EnsureStartsWith_ShouldReturnChar_GivenEndsWithReturnTrue()
const string value = "A";
const char substring = 'A';

#pragma warning disable CA1307
Assert.That(value.EnsureStartsWith(substring), Is.EqualTo(value));
#pragma warning restore CA1307
Assert.That(value.EnsureStartsWith(substring, StringComparison.Ordinal), Is.EqualTo(value));
}

[Test]
Expand All @@ -253,31 +273,43 @@ public void EnsureEndsWith_ShouldAppendSubstring_GivenEndsWithReturnFalse()
const string value = "Hello ";
const string substring = "World";

#pragma warning disable CA1307
Assert.That(value.EnsureEndsWith(substring), Is.EqualTo("Hello World"));
#pragma warning restore CA1307
Assert.That(value.EnsureEndsWith(substring, StringComparison.Ordinal), Is.EqualTo("Hello World"));
}

[Test]
public void EnsureEndsWith_ShouldReturnString_GivenEndsWithReturnTrue()
{
const string substring = "World";

#pragma warning disable CA1307
Assert.That(substring.EnsureEndsWith(substring), Is.EqualTo(substring));
#pragma warning restore CA1307
Assert.That(substring.EnsureEndsWith(substring, StringComparison.Ordinal), Is.EqualTo(substring));
}

[Test]
public void EnsureEndsWith_ShouldReturnString_GivenEmptySourceString()
{
const string substring = "World";

#pragma warning disable CA1307
Assert.That(string.Empty.EnsureEndsWith(substring), Is.EqualTo(substring));
#pragma warning restore CA1307
Assert.That(string.Empty.EnsureEndsWith(substring, StringComparison.Ordinal), Is.EqualTo(substring));
}

[Test]
public void EnsureEndsWith_ShouldReturnString_GivenEmptySubString()
{
const string substring = "World";

#pragma warning disable CA1307
Assert.That(substring.EnsureEndsWith(string.Empty), Is.EqualTo(substring));
#pragma warning restore CA1307
Assert.That(substring.EnsureEndsWith(string.Empty, StringComparison.Ordinal), Is.EqualTo(substring));
}

[Test]
Expand All @@ -286,31 +318,43 @@ public void EnsureStartsWith_ShouldAppendSubstring_GivenEndsWithReturnFalse()
const string value = "World";
const string substring = "Hello ";

#pragma warning disable CA1307
Assert.That(value.EnsureStartsWith(substring), Is.EqualTo("Hello World"));
#pragma warning restore CA1307
Assert.That(value.EnsureStartsWith(substring, StringComparison.Ordinal), Is.EqualTo("Hello World"));
}

[Test]
public void EnsureStartsWith_ShouldReturnString_GivenEndsWithReturnTrue()
{
const string substring = "World";

#pragma warning disable CA1307
Assert.That(substring.EnsureStartsWith(substring), Is.EqualTo(substring));
#pragma warning restore CA1307
Assert.That(substring.EnsureStartsWith(substring, StringComparison.Ordinal), Is.EqualTo(substring));
}

[Test]
public void EnsureStartsWith_ShouldReturnString_GivenEmptySourceString()
{
const string substring = "World";

#pragma warning disable CA1307
Assert.That(string.Empty.EnsureStartsWith(substring), Is.EqualTo(substring));
#pragma warning restore CA1307
Assert.That(string.Empty.EnsureStartsWith(substring, StringComparison.Ordinal), Is.EqualTo(substring));
}

[Test]
public void EnsureStartsWith_ShouldReturnString_GivenEmptySubString()
{
const string substring = "World";

#pragma warning disable CA1307
Assert.That(substring.EnsureStartsWith(string.Empty), Is.EqualTo(substring));
#pragma warning restore CA1307
Assert.That(substring.EnsureStartsWith(string.Empty, StringComparison.Ordinal), Is.EqualTo(substring));
}

[Test]
Expand Down

0 comments on commit 2fb91d5

Please sign in to comment.