Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Convert AbstractClassTests
  • Loading branch information
MattKotsenas committed Jun 6, 2024
commit 85d89f557ad1f64b8e2ff8c973c46a5afc5a10d9

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

53 changes: 23 additions & 30 deletions Source/Moq.Analyzers.Test/AbstractClassTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,10 @@ private void TestBadWithGeneric()
""");
}

// [Fact]
public Task ShouldPassOnGenericTypesWithNoArgs()
[Fact]
public async Task ShouldPassOnGenericTypesWithNoArgs()
{
return Verify(VerifyCSharpDiagnostic(
await VerifyCSharpDiagnostic(
"""
namespace Moq.Analyzers.Test.Data.AbstractClass.GenericNoArgs;

Expand All @@ -76,14 +76,13 @@ private void TestForBaseGenericNoArgs()
var mock2 = new Mock<AbstractGenericClassDefaultCtor<object>>(MockBehavior.Default);
}
}
"""
));
""");
}

// [Fact]
public Task ShouldFailOnMismatchArgs()
[Fact]
public async Task ShouldFailOnMismatchArgs()
{
return Verify(VerifyCSharpDiagnostic(
await VerifyCSharpDiagnostic(
"""
namespace Moq.Analyzers.Test.Data.AbstractClass.MismatchArgs;

Expand All @@ -110,23 +109,22 @@ internal class MyUnitTests
private void TestBad()
{
// The class has a ctor that takes an Int32 but passes a String
var mock = new Mock<AbstractClassWithCtor>("42");
var mock = new Mock<AbstractClassWithCtor>{|Moq1002:("42")|};

// The class has a ctor with two arguments [Int32, String], but they are passed in reverse order
var mock1 = new Mock<AbstractClassWithCtor>("42", 42);
var mock1 = new Mock<AbstractClassWithCtor>{|Moq1002:("42", 42)|};

// The class has a ctor but does not take any arguments
var mock2 = new Mock<AbstractClassDefaultCtor>(42);
var mock2 = new Mock<AbstractClassDefaultCtor>{|Moq1002:(42)|};
}
}
"""
));
""");
}

// [Fact]
public Task ShouldPassWithNoArgs()
[Fact]
public async Task ShouldPassWithNoArgs()
{
return Verify(VerifyCSharpDiagnostic(
await VerifyCSharpDiagnostic(
"""
namespace Moq.Analyzers.Test.Data.AbstractClass.NoArgs;

Expand All @@ -146,14 +144,13 @@ private void TestForBaseNoArgs()
mock.As<AbstractClassDefaultCtor>();
}
}
"""
));
""");
}

// [Fact(Skip = "I think this _should_ fail, but currently passes. Tracked by #55.")]
public Task ShouldFailWithArgsNonePassed()
[Fact(Skip = "I think this _should_ fail, but currently passes. Tracked by #55.")]
public async Task ShouldFailWithArgsNonePassed()
{
return Verify(VerifyCSharpDiagnostic(
await VerifyCSharpDiagnostic(
"""
namespace Moq.Analyzers.Test.Data.AbstractClass.WithArgsNonePassed;

Expand All @@ -175,18 +172,15 @@ private void TestForBaseWithArgsNonePassed()
{
var mock = new Mock<AbstractClassWithCtor>();
mock.As<AbstractClassWithCtor>();

var mock2 = new Mock<AbstractClassDefaultCtor>(MockBehavior.Default);
}
}
"""
));
""");
}

// [Fact]
public Task ShouldPassWithArgsPassed()
[Fact]
public async Task ShouldPassWithArgsPassed()
{
return Verify(VerifyCSharpDiagnostic(
await VerifyCSharpDiagnostic(
"""
namespace Moq.Analyzers.Test.DataAbstractClass.WithArgsPassed;

Expand Down Expand Up @@ -226,7 +220,6 @@ private void TestForBaseWithArgsPassed()
var mock6 = new Mock<AbstractGenericClassWithCtor<object>>(MockBehavior.Default, 42);
}
}
"""
));
""");
}
}