Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Commit 3c1e833

Browse files
committed
Fix argument order, services should come before data.
Also use cancellation tokens
1 parent c08b776 commit 3c1e833

File tree

3 files changed

+12
-7
lines changed

3 files changed

+12
-7
lines changed

src/GitHub.Api/Application/ApplicationManagerBase.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ public void RestartRepository()
128128
{
129129
if (Environment.RepositoryPath != null)
130130
{
131-
repositoryManager = Unity.RepositoryManager.CreateInstance(Platform, TaskManager, GitClient, ProcessManager, Environment.RepositoryPath, Environment.FileSystem);
131+
repositoryManager = Unity.RepositoryManager.CreateInstance(Platform, TaskManager, GitClient, ProcessManager, Environment.FileSystem, Environment.RepositoryPath);
132132
repositoryManager.Initialize();
133133
Environment.Repository.Initialize(repositoryManager);
134134
repositoryManager.Start();

src/GitHub.Api/Git/RepositoryManager.cs

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,7 @@ class RepositoryManager : IRepositoryManager
100100
private readonly IProcessManager processManager;
101101
private readonly IRepositoryPathConfiguration repositoryPaths;
102102
private readonly IFileSystem fileSystem;
103+
private readonly CancellationToken token;
103104
private readonly IRepositoryWatcher watcher;
104105

105106
private bool isBusy;
@@ -116,11 +117,13 @@ class RepositoryManager : IRepositoryManager
116117
public RepositoryManager(IGitConfig gitConfig,
117118
IRepositoryWatcher repositoryWatcher, IGitClient gitClient,
118119
IProcessManager processManager,
119-
IRepositoryPathConfiguration repositoryPaths,
120-
IFileSystem fileSystem)
120+
IFileSystem fileSystem,
121+
CancellationToken token,
122+
IRepositoryPathConfiguration repositoryPaths)
121123
{
122124
this.repositoryPaths = repositoryPaths;
123125
this.fileSystem = fileSystem;
126+
this.token = token;
124127
this.gitClient = gitClient;
125128
this.processManager = processManager;
126129
this.watcher = repositoryWatcher;
@@ -129,7 +132,8 @@ public RepositoryManager(IGitConfig gitConfig,
129132
SetupWatcher();
130133
}
131134

132-
public static RepositoryManager CreateInstance(IPlatform platform, ITaskManager taskManager, IGitClient gitClient, IProcessManager processManager, NPath repositoryRoot, IFileSystem fileSystem)
135+
public static RepositoryManager CreateInstance(IPlatform platform, ITaskManager taskManager, IGitClient gitClient,
136+
IProcessManager processManager, IFileSystem fileSystem, NPath repositoryRoot)
133137
{
134138
var repositoryPathConfiguration = new RepositoryPathConfiguration(repositoryRoot);
135139
string filePath = repositoryPathConfiguration.DotGitConfig;
@@ -138,7 +142,8 @@ public static RepositoryManager CreateInstance(IPlatform platform, ITaskManager
138142
var repositoryWatcher = new RepositoryWatcher(platform, repositoryPathConfiguration, taskManager.Token);
139143

140144
return new RepositoryManager(gitConfig, repositoryWatcher,
141-
gitClient, processManager, repositoryPathConfiguration, fileSystem);
145+
gitClient, processManager, fileSystem,
146+
taskManager.Token, repositoryPathConfiguration);
142147
}
143148

144149
public void Initialize()
@@ -389,7 +394,7 @@ public void UpdateLocks()
389394

390395
private ITask HookupHandlers(ITask task, bool isExclusive, bool filesystemChangesExpected)
391396
{
392-
return new ActionTask(CancellationToken.None, () => {
397+
return new ActionTask(token, () => {
393398
if (isExclusive)
394399
{
395400
Logger.Trace("Starting Operation - Setting Busy Flag");

src/tests/IntegrationTests/BaseGitEnvironmentTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ protected IEnvironment Initialize(NPath repoPath, NPath environmentPath = null,
1414
{
1515
InitializePlatform(repoPath, environmentPath, enableEnvironmentTrace);
1616

17-
var repositoryManager = GitHub.Unity.RepositoryManager.CreateInstance(Platform, TaskManager, GitClient, ProcessManager, repoPath, Environment.FileSystem);
17+
var repositoryManager = GitHub.Unity.RepositoryManager.CreateInstance(Platform, TaskManager, GitClient, ProcessManager, Environment.FileSystem, repoPath);
1818
onRepositoryManagerCreated?.Invoke(repositoryManager);
1919

2020
RepositoryManager = repositoryManager;

0 commit comments

Comments
 (0)