Skip to content

Commit 6c6c553

Browse files
zivkanzivkan
committed
Fix tests
1 parent 754df9c commit 6c6c553

12 files changed

+8639
-8043
lines changed

‎test/NuGet.Core.Tests/NuGet.PackageManagement.Test/NotThreadSafeResourceCollection.cs

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
// Copyright (c) .NET Foundation. All rights reserved.
2+
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
3+
4+
using Xunit;
5+
6+
namespace NuGet.PackageManagement.Test
7+
{
8+
[CollectionDefinition(nameof(NotThreadSafeResourceCollection), DisableParallelization = true)]
9+
public class NotThreadSafeResourceCollection
10+
{
11+
// This class has no code, and is never created. Its purpose is simply
12+
// to be the place to apply [CollectionDefinition]
13+
}
14+
}

‎test/NuGet.Core.Tests/NuGet.PackageManagement.Test/NuGet.PackageManagement.Test.csproj

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
<ItemGroup>
1010
<ProjectReference Include="..\..\TestUtilities\Test.Utility\Test.Utility.csproj" />
1111
<ProjectReference Include="..\..\..\src\NuGet.Core\NuGet.PackageManagement\NuGet.PackageManagement.csproj" />
12-
<ProjectReference Include="..\NuGet.Configuration.Test\NuGet.Configuration.Test.csproj" />
1312
</ItemGroup>
1413

1514
<ItemGroup Condition=" '$(TargetFramework)' == '$(NETFXTargetFramework)' ">

‎test/NuGet.Core.Tests/NuGet.PackageManagement.Test/NuGetPackageManagerTests.cs

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

‎test/NuGet.Core.Tests/NuGet.PackageManagement.Test/NuGetPackageManagerTests/BatchedEventTests.cs

Lines changed: 757 additions & 0 deletions
Large diffs are not rendered by default.

‎test/NuGet.Core.Tests/NuGet.PackageManagement.Test/NuGetPackageManagerTests/ConstructorTests.cs

Lines changed: 156 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,156 @@
1+
// Copyright (c) .NET Foundation. All rights reserved.
2+
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
3+
4+
using System;
5+
using Moq;
6+
using NuGet.Commands;
7+
using NuGet.Configuration;
8+
using NuGet.Protocol.Core.Types;
9+
using Xunit;
10+
11+
namespace NuGet.PackageManagement.Test.NuGetPackageManagerTests
12+
{
13+
public class ConstructorTests
14+
{
15+
[Fact]
16+
public void Constructor_WithNullISourceRepositoryProvider_Throws()
17+
{
18+
var solutionManager = new Mock<ISolutionManager>();
19+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
20+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
21+
null,
22+
new Mock<ISettings>().Object,
23+
solutionManager.Object,
24+
null,
25+
new Mock<IRestoreProgressReporter>().Object
26+
));
27+
}
28+
29+
[Fact]
30+
public void Constructor_WithNullISettings_Throws()
31+
{
32+
var solutionManager = new Mock<ISolutionManager>();
33+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
34+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
35+
new Mock<ISourceRepositoryProvider>().Object,
36+
null,
37+
solutionManager.Object,
38+
new Mock<IDeleteOnRestartManager>().Object,
39+
new Mock<IRestoreProgressReporter>().Object
40+
));
41+
}
42+
43+
[Fact]
44+
public void Constructor_WithNullISolutionManager_Throws()
45+
{
46+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
47+
new Mock<ISourceRepositoryProvider>().Object,
48+
new Mock<ISettings>().Object,
49+
null,
50+
new Mock<IDeleteOnRestartManager>().Object,
51+
new Mock<IRestoreProgressReporter>().Object
52+
));
53+
}
54+
55+
[Fact]
56+
public void Constructor_WithNullIDeleteOnRestartManager_Throws()
57+
{
58+
var solutionManager = new Mock<ISolutionManager>();
59+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
60+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
61+
new Mock<ISourceRepositoryProvider>().Object,
62+
new Mock<ISettings>().Object,
63+
solutionManager.Object,
64+
null,
65+
new Mock<IRestoreProgressReporter>().Object
66+
));
67+
}
68+
69+
[Fact]
70+
public void Constructor_WithNullProgressReporter_Throws()
71+
{
72+
var solutionManager = new Mock<ISolutionManager>();
73+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
74+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
75+
new Mock<ISourceRepositoryProvider>().Object,
76+
new Mock<ISettings>().Object,
77+
solutionManager.Object,
78+
new Mock<IDeleteOnRestartManager>().Object,
79+
reporter: null
80+
));
81+
}
82+
83+
[Fact]
84+
public void Constructor_With6Arguments_WithNullProgressReporter_DoesNotThrows()
85+
{
86+
var solutionManager = new Mock<ISolutionManager>();
87+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
88+
_ = new NuGetPackageManager(
89+
new Mock<ISourceRepositoryProvider>().Object,
90+
new Mock<ISettings>().Object,
91+
solutionManager.Object,
92+
new Mock<IDeleteOnRestartManager>().Object,
93+
reporter: null,
94+
excludeVersion: true
95+
);
96+
}
97+
98+
[Fact]
99+
public void Constructor_With6Arguments_WithNullISourceRepositoryProvider_Throws()
100+
{
101+
var solutionManager = new Mock<ISolutionManager>();
102+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
103+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
104+
null,
105+
new Mock<ISettings>().Object,
106+
solutionManager.Object,
107+
null,
108+
new Mock<IRestoreProgressReporter>().Object,
109+
excludeVersion: true
110+
));
111+
}
112+
113+
[Fact]
114+
public void Constructor_With6Arguments_WithNullISettings_Throws()
115+
{
116+
var solutionManager = new Mock<ISolutionManager>();
117+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
118+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
119+
new Mock<ISourceRepositoryProvider>().Object,
120+
null,
121+
solutionManager.Object,
122+
new Mock<IDeleteOnRestartManager>().Object,
123+
new Mock<IRestoreProgressReporter>().Object,
124+
excludeVersion: true
125+
));
126+
}
127+
128+
[Fact]
129+
public void Constructor_With6Arguments_WithNullISolutionManager_Throws()
130+
{
131+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
132+
new Mock<ISourceRepositoryProvider>().Object,
133+
new Mock<ISettings>().Object,
134+
null,
135+
new Mock<IDeleteOnRestartManager>().Object,
136+
new Mock<IRestoreProgressReporter>().Object,
137+
excludeVersion: true
138+
));
139+
}
140+
141+
[Fact]
142+
public void Constructor_With6Arguments_WithNullIDeleteOnRestartManager_Throws()
143+
{
144+
var solutionManager = new Mock<ISolutionManager>();
145+
solutionManager.Setup(e => e.SolutionDirectory).Returns(@"C:\");
146+
Assert.Throws<ArgumentNullException>(() => new NuGetPackageManager(
147+
new Mock<ISourceRepositoryProvider>().Object,
148+
new Mock<ISettings>().Object,
149+
solutionManager.Object,
150+
null,
151+
new Mock<IRestoreProgressReporter>().Object,
152+
excludeVersion: true
153+
));
154+
}
155+
}
156+
}

0 commit comments

Comments
 (0)