Skip to content

Commit a2c6926

Browse files
CharliePooleCharliePoole
committed
Fix errors in dotnet tool package
1 parent fc73722 commit a2c6926

File tree

3 files changed

+49
-27
lines changed

3 files changed

+49
-27
lines changed

‎build.cake‎

Lines changed: 26 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -262,18 +262,12 @@ BuildSettings.Packages.AddRange(new PackageDefinition[] {
262262
source: BuildSettings.NuGetDirectory + "runners/nunit.console-runner-with-extensions.nuspec",
263263
checks: new PackageCheck[] { HasFile("LICENSE.txt") }),
264264

265-
NUnitConsoleRunnerNetCorePackage = new NuGetPackage(
265+
NUnitConsoleRunnerNetCorePackage = new DotNetToolPackage(
266266
id: "NUnit.ConsoleRunner.NetCore",
267267
source: BuildSettings.NuGetDirectory + "runners/nunit.console-runner.netcore.nuspec",
268-
checks: new PackageCheck[] {
269-
HasFiles("LICENSE.txt", "NOTICES.txt"),
270-
HasDirectory("tools/net6.0").WithFiles("nunit3-console.exe", "nunit3-console.dll", "nunit-console.nuget.addins").AndFiles(ENGINE_FILES)
271-
},
272-
symbols: new PackageCheck[] {
273-
HasDirectory("tools/net6.0").WithFile("nunit3-console.pdb").AndFiles(ENGINE_PDB_FILES)
274-
},
268+
checks: new PackageCheck[] { HasFiles("nunit.exe") },
275269
testRunner: new ConsoleRunnerSelfTester(BuildSettings.NuGetTestDirectory
276-
+ $"NUnit.ConsoleRunner.NetCore.{BuildSettings.PackageVersion}/tools/net6.0/nunit3-console.exe"),
270+
+ $"NUnit.ConsoleRunner.NetCore.{BuildSettings.PackageVersion}/nunit.exe"),
277271
tests: NetCoreRunnerTests),
278272

279273
NUnitConsoleRunnerChocolateyPackage = new ChocolateyPackage(
@@ -388,6 +382,29 @@ public class ConsoleRunnerSelfTester : TestRunner, IPackageTestRunner
388382
}
389383
}
390384

385+
//////////////////////////////////////////////////////////////////////
386+
// DOTNET TOOL PACKAGE
387+
//////////////////////////////////////////////////////////////////////
388+
389+
// TODO: Temporary custom package class to be moved into the recipe
390+
391+
public class DotNetToolPackage : NuGetPackage
392+
{
393+
public DotNetToolPackage(string id, string source, string basePath = null,
394+
IPackageTestRunner testRunner = null, TestRunnerSource testRunnerSource = null,
395+
PackageCheck[] checks = null, PackageCheck[] symbols = null, IEnumerable<PackageTest> tests = null)
396+
: base(id, source, basePath: basePath, testRunner: testRunner, testRunnerSource: testRunnerSource,
397+
checks: checks, symbols: symbols, tests: tests) { }
398+
399+
public override void InstallPackage()
400+
{
401+
var arguments = $"tool install {PackageId} --version {BuildSettings.PackageVersion} " +
402+
$"--add-source \"{BuildSettings.PackageDirectory}\" --tool-path \"{PackageTestDirectory}\"";
403+
Console.WriteLine($"Executing dotnet {arguments}");
404+
_context.StartProcess("dotnet", arguments);
405+
}
406+
}
407+
391408
//////////////////////////////////////////////////////////////////////
392409
// ADDITIONAL TARGETS USED FOR RECOVERY AND DEBUGGING
393410
//////////////////////////////////////////////////////////////////////

‎nuget/runners/nunit.console-runner.netcore.nuspec‎

Lines changed: 21 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -21,29 +21,32 @@
2121
<releaseNotes>https://docs.nunit.org/articles/nunit/release-notes/console-and-engine.html</releaseNotes>
2222
<language>en-US</language>
2323
<tags>nunit test testing tdd runner</tags>
24-
<copyright>Copyright (c) 2021 Charlie Poole, Rob Prouse</copyright>
24+
<copyright>Copyright (c) 2021-2024 Charlie Poole, Rob Prouse</copyright>
25+
<packageTypes>
26+
<packageType name="DotnetTool" />
27+
</packageTypes>
2528
</metadata>
2629
<files>
2730
<file src="../../LICENSE.txt" />
2831
<file src="../../NOTICES.txt" />
2932

30-
<file src="net6.0/nunit3-console.exe" target="tools/net6.0/" />
31-
<file src="net6.0/nunit3-console.pdb" target="tools/net6.0/" />
32-
<file src="net6.0/nunit3-console.dll" target="tools/net6.0/" />
33-
<file src="net6.0/nunit3-console.dll.config" target="tools/net6.0" />
34-
<file src="net6.0/nunit3-console.deps.json" target="tools/net6.0" />
35-
<file src="net6.0/nunit3-console.runtimeconfig.json" target="tools/net6.0" />
36-
<file src="net6.0/nunit.engine.core.dll" target="tools/net6.0" />
37-
<file src="net6.0/nunit.engine.core.pdb" target="tools/net6.0" />
38-
<file src="net6.0/nunit.engine.dll" target="tools/net6.0" />
39-
<file src="net6.0/nunit.engine.pdb" target="tools/net6.0" />
40-
<file src="net6.0/nunit.engine.api.dll" target="tools/net6.0" />
41-
<file src="net6.0/nunit.engine.api.pdb" target="tools/net6.0" />
42-
<file src="net6.0/nunit.engine.api.xml" target="tools/net6.0" />
43-
<file src="net6.0/testcentric.engine.metadata.dll" target="tools/net6.0" />
44-
<file src="net6.0/Microsoft.Extensions.DependencyModel.dll" target="tools/net6.0" />
45-
<file src="../../nuget/runners/nunit.console.nuget.addins" target="tools/net6.0"/>
46-
<file src="../../nuget/runners/DotnetToolSettings.xml" target="tools/net6.0"/>
33+
<file src="net6.0/nunit3-console.exe" target="tools/net6.0/any" />
34+
<file src="net6.0/nunit3-console.pdb" target="tools/net6.0/any" />
35+
<file src="net6.0/nunit3-console.dll" target="tools/net6.0/any" />
36+
<file src="net6.0/nunit3-console.dll.config" target="tools/net6.0/any" />
37+
<file src="net6.0/nunit3-console.deps.json" target="tools/net6.0/any" />
38+
<file src="net6.0/nunit3-console.runtimeconfig.json" target="tools/net6.0/any" />
39+
<file src="net6.0/nunit.engine.core.dll" target="tools/net6.0/any" />
40+
<file src="net6.0/nunit.engine.core.pdb" target="tools/net6.0/any" />
41+
<file src="net6.0/nunit.engine.dll" target="tools/net6.0/any" />
42+
<file src="net6.0/nunit.engine.pdb" target="tools/net6.0/any" />
43+
<file src="net6.0/nunit.engine.api.dll" target="tools/net6.0/any" />
44+
<file src="net6.0/nunit.engine.api.pdb" target="tools/net6.0/any" />
45+
<file src="net6.0/nunit.engine.api.xml" target="tools/net6.0/any" />
46+
<file src="net6.0/testcentric.engine.metadata.dll" target="tools/net6.0/any" />
47+
<file src="net6.0/Microsoft.Extensions.DependencyModel.dll" target="tools/net6.0/any" />
48+
<file src="../../nuget/runners/nunit.console.nuget.addins" target="tools/net6.0/any"/>
49+
<file src="../../nuget/runners/DotnetToolSettings.xml" target="tools/net6.0/any"/>
4750

4851
<file src="../../nunit_256.png" target="images"/>
4952
</files>

‎src/NUnitConsole/nunit3-console/nunit3-console.csproj‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
<AssemblyName>nunit3-console</AssemblyName>
77
<TargetFrameworks>net462;net6.0;net8.0</TargetFrameworks>
88
<RollForward>Major</RollForward>
9+
<DebugType>portable</DebugType>
10+
<DebugSymbols>true</DebugSymbols>
911
</PropertyGroup>
1012

1113
<PropertyGroup>

0 commit comments

Comments
 (0)