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

Commit 70a8594

Browse files
committed
Fix tests
1 parent 5d5d555 commit 70a8594

File tree

4 files changed

+12
-32
lines changed

4 files changed

+12
-32
lines changed

src/tests/IntegrationTests/BaseGitEnvironmentTest.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
using System.Linq;
2+
using System.Threading;
23
using GitHub.Unity;
34
using System.Threading.Tasks;
45

@@ -10,8 +11,8 @@ protected async Task<IEnvironment> Initialize(NPath repoPath, NPath environmentP
1011
bool enableEnvironmentTrace = false)
1112
{
1213
TaskManager = new TaskManager();
13-
var sc = new ThreadSynchronizationContext(TaskManager.Token);
14-
TaskManager.UIScheduler = new SynchronizationContextTaskScheduler(sc);
14+
SyncContext = new ThreadSynchronizationContext(TaskManager.Token);
15+
TaskManager.UIScheduler = new SynchronizationContextTaskScheduler(SyncContext);
1516

1617
Environment = new IntegrationTestEnvironment(repoPath, SolutionDirectory, environmentPath, enableEnvironmentTrace);
1718

@@ -25,7 +26,6 @@ protected async Task<IEnvironment> Initialize(NPath repoPath, NPath environmentP
2526

2627
Platform.Initialize(ProcessManager, TaskManager);
2728

28-
2929
GitClient = new GitClient(Environment, ProcessManager, Platform.CredentialManager, TaskManager);
3030

3131
var usageTracker = new NullUsageTracker();
@@ -67,12 +67,13 @@ public override void OnTearDown()
6767
public IRepositoryManager RepositoryManager { get; private set; }
6868

6969
protected IPlatform Platform { get; private set; }
70-
70+
protected IApplicationManager ApplicationManager { get; set; }
7171
protected IProcessManager ProcessManager { get; private set; }
7272
protected ITaskManager TaskManager { get; private set; }
7373

7474
protected IProcessEnvironment GitEnvironment { get; private set; }
7575
protected IGitClient GitClient { get; set; }
76+
protected SynchronizationContext SyncContext { get; set; }
7677

7778
protected NPath DotGitConfig { get; private set; }
7879

src/tests/IntegrationTests/Git/IntegrationTestEnvironment.cs

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ public IntegrationTestEnvironment(NPath repoPath, NPath solutionDirectory, NPath
2828

2929
var installPath = solutionDirectory.Parent.Parent.Parent.Combine("GitHub.Api");
3030

31-
Initialize(installPath, solutionDirectory, repoPath.Combine("Assets"));
32-
Initialize();
31+
Initialize(UnityVersion, installPath, solutionDirectory, repoPath.Combine("Assets"));
32+
InitializeRepository();
3333

3434
this.enableTrace = enableTrace;
3535

@@ -40,14 +40,14 @@ public IntegrationTestEnvironment(NPath repoPath, NPath solutionDirectory, NPath
4040
}
4141
}
4242

43-
public void Initialize(NPath extensionInstallPath, NPath unityPath, NPath assetsPath)
43+
public void Initialize(string unityVersion, NPath extensionInstallPath, NPath unityPath, NPath assetsPath)
4444
{
45-
defaultEnvironment.Initialize(extensionInstallPath, unityPath, assetsPath);
45+
defaultEnvironment.Initialize(unityVersion, extensionInstallPath, unityPath, assetsPath);
4646
}
4747

48-
public void Initialize()
48+
public void InitializeRepository(NPath expectedPath = null)
4949
{
50-
defaultEnvironment.Initialize();
50+
defaultEnvironment.InitializeRepository(expectedPath);
5151
}
5252

5353
public string ExpandEnvironmentVariables(string name)
@@ -82,6 +82,7 @@ public string GetSpecialFolder(Environment.SpecialFolder folder)
8282

8383
public NPath Path => Environment.GetEnvironmentVariable("PATH").ToNPath();
8484
public string NewLine => Environment.NewLine;
85+
public string UnityVersion => "5.6";
8586

8687
public NPath GitExecutablePath
8788
{

src/tests/IntegrationTests/Git/RepositoryLocatorTests.cs

Lines changed: 0 additions & 21 deletions
This file was deleted.

src/tests/IntegrationTests/IntegrationTests.csproj

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@
7979
<Compile Include="BaseIntegrationTest.cs" />
8080
<Compile Include="Git\GitSetupTests.cs" />
8181
<Compile Include="Git\IntegrationTestEnvironment.cs" />
82-
<Compile Include="Git\RepositoryLocatorTests.cs" />
8382
<Compile Include="ProcessManagerExtensions.cs" />
8483
<Compile Include="Process\ProcessManagerIntegrationTests.cs" />
8584
<Compile Include="SetupFixture.cs" />

0 commit comments

Comments
 (0)