Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit fb62c08

Browse files
committed
The clone service is not mocked by default
Some services are not mocked by default, so getting the default service provider was returning a real instance, and setting a Returns on it does not make NSubstitute happy.
1 parent b625370 commit fb62c08

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/UnitTests/GitHub.App/Services/RepositoryCreationServiceTests.cs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,9 @@ public class TheCreateRepositoryMethod : TestBaseClass
1616
[Fact]
1717
public void CreatesRepositoryOnlineViaApiAndThenClonesIt()
1818
{
19-
var provider = Substitutes.ServiceProvider;
19+
var cloneService = Substitutes.RepositoryCloneService;
20+
var provider = Substitutes.GetServiceProvider(cloneService);
21+
2022
var newRepository = new NewRepository("octokit.net");
2123
var repository = new TestRepository("octokit.net", "https://github.com/octokit/octokit.net");
2224
var account = Substitute.For<IAccount>();
@@ -25,7 +27,6 @@ public void CreatesRepositoryOnlineViaApiAndThenClonesIt()
2527
var apiClient = Substitute.For<IApiClient>();
2628
apiClient.CreateRepository(newRepository, "octokit", false)
2729
.Returns(Observable.Return(repository));
28-
var cloneService = provider.GetRepositoryCloneService();
2930
cloneService.CloneRepository("https://github.com/octokit/octokit.net", "octokit.net", @"c:\dev")
3031
.Returns(Observable.Return(Unit.Default));
3132
var creator = provider.GetRepositoryCreationService();

0 commit comments

Comments
 (0)