Skip to content

Commit e0c0160

Browse files
jviaujviau
authored
Rename project.assets.json to test.assets.json (#10060) (#10074)
* Rename project.assets.json to test.assets.json * Copy files to output * update comment
1 parent 2662628 commit e0c0160

File tree

5 files changed

+50
-104
lines changed

5 files changed

+50
-104
lines changed

‎test/WebJobs.Script.Tests/Description/DotNet/PackageManagerTests.cs‎

Lines changed: 39 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,15 @@
11
// Copyright (c) .NET Foundation. All rights reserved.
22
// Licensed under the MIT License. See License.txt in the project root for license information.
33

4-
using System;
54
using System.IO;
65
using Microsoft.Azure.WebJobs.Script.Config;
76
using Microsoft.Azure.WebJobs.Script.Description;
8-
using WebJobs.Script.Tests;
97
using Xunit;
108

119
namespace Microsoft.Azure.WebJobs.Script.Tests
1210
{
1311
public class PackageManagerTests
1412
{
15-
private static readonly ScriptSettingsManager SettingsManager = ScriptSettingsManager.Instance;
16-
1713
[Theory]
1814
[InlineData(@"ProjectWithLockMatch", false)]
1915
[InlineData(@"FunctionWithNoProject", false)]
@@ -23,20 +19,51 @@ public class PackageManagerTests
2319
public void RequirePackageRestore_ReturnsExpectedResult(string projectPath, bool shouldRequireRestore)
2420
{
2521
projectPath = Path.Combine(Directory.GetCurrentDirectory(), "Description", "DotNet", "TestFiles", "PackageReferences", projectPath);
26-
bool result = PackageManager.RequiresPackageRestore(projectPath);
22+
try
23+
{
24+
CopyLockFile(projectPath);
25+
bool result = PackageManager.RequiresPackageRestore(projectPath);
26+
27+
Assert.True(Directory.Exists(projectPath));
2728

28-
Assert.True(Directory.Exists(projectPath));
29+
string message = $"Project in '{projectPath}' did not return expected result.";
30+
31+
// Using .True or .False (instead of .Equal) to trace additional information.
32+
if (shouldRequireRestore)
33+
{
34+
Assert.True(result, message);
35+
}
36+
else
37+
{
38+
Assert.False(result, message);
39+
}
40+
}
41+
finally
42+
{
43+
DeleteLockFile(projectPath);
44+
}
45+
}
2946

30-
string message = $"Project in '{projectPath}' did not return expected result.";
47+
private static void CopyLockFile(string projectPath)
48+
{
49+
// We save the lock file as something other than 'project.assets.json' so it is not flagged by component governance.
50+
// Only renaming to project.assets.json for the duration of the test.
51+
string lockFile = Path.Combine(projectPath, DotNetConstants.ProjectLockFileName);
52+
string sourceFile = Path.Combine(projectPath, "test.assets.json");
3153

32-
// Using .True or .False (instead of .Equal) to trace additional information.
33-
if (shouldRequireRestore)
54+
if (File.Exists(sourceFile))
3455
{
35-
Assert.True(result, message);
56+
File.Copy(sourceFile, lockFile, true);
3657
}
37-
else
58+
}
59+
60+
private static void DeleteLockFile(string projectPath)
61+
{
62+
// Clean up the 'project.assets.json' file so component governance does not flag it.
63+
string lockFile = Path.Combine(projectPath, DotNetConstants.ProjectLockFileName);
64+
if (File.Exists(lockFile))
3865
{
39-
Assert.False(result, message);
66+
File.Delete(lockFile);
4067
}
4168
}
4269
}

‎test/WebJobs.Script.Tests/Description/DotNet/TestFiles/PackageReferences/ProjectWithLockMatch/project.assets.json‎ renamed to ‎test/WebJobs.Script.Tests/Description/DotNet/TestFiles/PackageReferences/ProjectWithLockMatch/test.assets.json‎

File renamed without changes.

‎test/WebJobs.Script.Tests/Description/DotNet/TestFiles/PackageReferences/ProjectWithMismatchedLock/MismatchedPackageVersions/project.assets.json‎ renamed to ‎test/WebJobs.Script.Tests/Description/DotNet/TestFiles/PackageReferences/ProjectWithMismatchedLock/MismatchedPackageVersions/test.assets.json‎

File renamed without changes.

‎test/WebJobs.Script.Tests/Description/DotNet/TestFiles/PackageReferences/ProjectWithMismatchedLock/MismatchedProjectDependencies/project.assets.json‎ renamed to ‎test/WebJobs.Script.Tests/Description/DotNet/TestFiles/PackageReferences/ProjectWithMismatchedLock/MismatchedProjectDependencies/test.assets.json‎

File renamed without changes.

‎test/WebJobs.Script.Tests/WebJobs.Script.Tests.csproj‎

Lines changed: 11 additions & 92 deletions
Original file line numberDiff line numberDiff line change
@@ -28,24 +28,9 @@
2828
</ItemGroup>
2929

3030
<ItemGroup>
31-
<None Include="..\WebJobs.Script.Tests.Integration\TestScripts\**\*.*">
32-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
33-
<Link>TestScripts\%(RecursiveDir)%(Filename)%(Extension)</Link>
34-
</None>
35-
</ItemGroup>
36-
37-
<ItemGroup>
38-
<None Include="Description\DotNet\TestFiles\PackageReferences\ProjectWithLockMatch\function.proj">
39-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
40-
</None>
41-
<None Include="Description\DotNet\TestFiles\PackageReferences\ProjectWithMismatchedLock\MismatchedPackageVersions\function.proj">
42-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
43-
</None>
44-
<None Include="Description\DotNet\TestFiles\PackageReferences\ProjectWithMismatchedLock\MismatchedProjectDependencies\function.proj">
45-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
46-
</None>
47-
<None Include="Description\DotNet\TestFiles\PackageReferences\ProjectWithoutLock\function.proj">
31+
<None Include="..\WebJobs.Script.Tests.Integration\TestScripts\**">
4832
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
33+
<LinkBase>TestScripts</LinkBase>
4934
</None>
5035
</ItemGroup>
5136

@@ -86,84 +71,18 @@
8671
</ItemGroup>
8772

8873
<ItemGroup>
89-
<None Update="Description\DotNet\TestFiles\DepsFiles\function.deps.json">
90-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
91-
</None>
92-
<None Update="Description\DotNet\TestFiles\DepsFiles\function2.deps.json">
93-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
94-
</None>
95-
<None Update="Description\DotNet\TestFiles\DepsFiles\RidNativeDeps\function.deps.json">
96-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
97-
</None>
98-
<None Update="Description\DotNet\TestFiles\PackageReferences\FunctionWithNoProject\function.json">
99-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
100-
</None>
101-
<None Update="Description\DotNet\TestFiles\PackageReferences\ProjectWithLockMatch\project.assets.json">
102-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
103-
</None>
104-
<None Update="Description\DotNet\TestFiles\PackageReferences\ProjectWithMismatchedLock\MismatchedPackageVersions\project.lock.json">
105-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
106-
</None>
107-
<None Update="Description\DotNet\TestFiles\PackageReferences\ProjectWithMismatchedLock\MismatchedProjectDependencies\project.assets.json">
108-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
109-
</None>
110-
<None Update="Managment\Payloads\ExpectedConcurrencyPayload.json">
111-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
112-
</None>
113-
<None Update="Managment\Payloads\ExpectedHostJsonPayload.json">
114-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
115-
</None>
116-
<None Update="Managment\Payloads\ExpectedHttpExtensionsPayload.json">
117-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
118-
</None>
119-
<None Update="Microsoft.Azure.WebJobs.Script.WebHost.deps.json">
120-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
121-
</None>
122-
<None Update="TestFixture\HostOptionsProviderTests\TestBasicBindings.json">
123-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
124-
</None>
125-
<None Update="TestFixture\HostOptionsProviderTests\TestBasicConcurrency.json">
126-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
127-
</None>
128-
<None Update="TestFixture\HostOptionsProviderTests\TestDefaultConcurrency.json">
129-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
130-
</None>
131-
<None Update="TestFixture\HostOptionsProviderTests\TestIrregularNamingBindings.json">
132-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
133-
</None>
134-
<None Update="TestFixture\HostOptionsProviderTests\TestMultipleBindings.json">
135-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
136-
</None>
137-
<None Update="TestFixture\HostOptionsProviderTests\TestWihtoutExtensionsAttributeBindings.json">
138-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
139-
</None>
140-
<None Update="TestFixture\HostOptionsProviderTests\TestWihtoutIOptionsFormatter.json">
141-
<CopyToOutputDirectory>Always</CopyToOutputDirectory>
142-
</None>
143-
<None Update="Workers\Rpc\Resources\functions.png">
144-
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
145-
</None>
146-
<None Update="xunit.runner.json">
147-
<CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
148-
</None>
149-
</ItemGroup>
150-
151-
<ItemGroup>
152-
<None Remove="Resources\FileProvisioning\PowerShell\PSGalleryEmptyFeed.xml" />
153-
<None Remove="Resources\FileProvisioning\PowerShell\PSGallerySampleFeed.xml" />
154-
<None Remove="Resources\FileProvisioning\PowerShell\requirements_PSGalleryOffline.psd1" />
155-
<None Remove="Resources\FileProvisioning\PowerShell\requirements_PSGalleryOnline.psd1" />
156-
<None Remove="Resources\FileProvisioning\PowerShell\profile.ps1" />
74+
<None Include="Description\DotNet\TestFiles\**\*.proj" />
75+
<None Update="Description\DotNet\TestFiles\**" CopyToOutputDirectory="PreserveNewest" />
76+
<None Update="Microsoft.Azure.WebJobs.Script.WebHost.deps.json" CopyToOutputDirectory="PreserveNewest" />
77+
<None Update="TestFixture\HostOptionsProviderTests\*.json" CopyToOutputDirectory="PreserveNewest" />
78+
<None Update="Workers\Rpc\Resources\**" CopyToOutputDirectory="PreserveNewest" />
79+
<None Update="xunit.runner.json" CopyToOutputDirectory="PreserveNewest" />
80+
<None Remove="Resources\FileProvisioning\PowerShell\*" />
15781
</ItemGroup>
15882

15983
<ItemGroup>
160-
<EmbeddedResource Include="Resources\FileProvisioning\PowerShell\PSGallerySampleFeed.xml" />
161-
<EmbeddedResource Include="Resources\FileProvisioning\PowerShell\PSGalleryEmptyFeed.xml" />
162-
<EmbeddedResource Include="Resources\FileProvisioning\PowerShell\requirements_PSGalleryOffline.psd1" />
163-
<EmbeddedResource Include="Resources\FileProvisioning\PowerShell\requirements_PSGalleryOnline.psd1" />
164-
<EmbeddedResource Include="Resources\FileProvisioning\PowerShell\profile.ps1">
165-
<CopyToOutputDirectory>Never</CopyToOutputDirectory>
166-
</EmbeddedResource>
84+
<EmbeddedResource Include="Resources\FileProvisioning\PowerShell\*" />
85+
<EmbeddedResource Update="Resources\FileProvisioning\PowerShell\profile.ps1" CopyToOutputDirectory="Never" />
16786
</ItemGroup>
16887

16988
<Import Project="..\..\build\GrpcTestFix.targets" />

0 commit comments

Comments
 (0)