Skip to content

Commit 839bf7d

Browse files
committed
Fix total cache update, improve the test asserts
1 parent 8375919 commit 839bf7d

File tree

5 files changed

+11
-0
lines changed

5 files changed

+11
-0
lines changed

src/StaticWebAssetsSdk/Targets/Microsoft.NET.Sdk.StaticWebAssets.Compression.targets

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -242,6 +242,7 @@ Copyright (c) .NET Foundation. All rights reserved.
242242
<StaticWebAsset Include="@(_CompressionBuildStaticWebAsset)" />
243243
<StaticWebAssetEndpoint Include="@(_CompressionBuildStaticWebAssetEndpoint)" />
244244
<_CompressionCurrentProjectBuildAssets Include="@(StaticWebAsset)" />
245+
<FileWrites Include="@(_ResolveBuildCompressedStaticWebAssetsCachePath)" />
245246
</ItemGroup>
246247

247248
<ApplyCompressionNegotiation

src/StaticWebAssetsSdk/Targets/Microsoft.NET.Sdk.StaticWebAssets.JSModules.targets

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,7 @@ Copyright (c) .NET Foundation. All rights reserved.
165165
<StaticWebAsset Include="@(_JSModuleStaticWebAsset)" />
166166
<StaticWebAssetEndpoint Remove="@(_FilteredJSModuleStaticWebAssetEndpointWithProperty)" />
167167
<StaticWebAssetEndpoint Include="@(_FilteredJSModuleStaticWebAssetEndpointWithProperty)" />
168+
<FileWrites Include="$(_ResolveJsInitializerModuleStaticWebAssetsCachePath)" />
168169
</ItemGroup>
169170

170171
</Target>
@@ -487,6 +488,9 @@ Copyright (c) .NET Foundation. All rights reserved.
487488
<!-- Remove the items from their original groups since they've now become a StaticWebAsset -->
488489
<Content Remove="@(_JsFileModuleStaticWebAsset->'%(OriginalItemSpec)')" />
489490
<None Remove="@(_JsFileModuleStaticWebAsset->'%(OriginalItemSpec)')" />
491+
492+
<FileWrites Include="$(_ResolveJSModuleStaticWebAssetsRazorCachePath)" />
493+
<FileWrites Include="$(_ResolveJSModuleStaticWebAssetsCshtmlCachePath)" />
490494
</ItemGroup>
491495

492496
</Target>

src/StaticWebAssetsSdk/Targets/Microsoft.NET.Sdk.StaticWebAssets.targets

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -707,6 +707,8 @@ Copyright (c) .NET Foundation. All rights reserved.
707707
</StaticWebAssetDiscoveryPattern>
708708

709709
<Content Remove="@(StaticWebAsset)" />
710+
711+
<FileWrites Include="$(_ResolveProjectStaticWebAssetsCachePath)" />
710712
</ItemGroup>
711713

712714
</Target>

src/StaticWebAssetsSdk/Tasks/DefineStaticWebAssets.Cache.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,8 @@ private void TotalUpdate(byte[] propertiesHash, byte[] fingerprintPatternsHash,
165165
GlobalPropertiesHash = propertiesHash;
166166
FingerprintPatternsHash = fingerprintPatternsHash;
167167
PropertyOverridesHash = propertyOverridesHash;
168+
CachedAssets.Clear();
169+
CachedCopyCandidates.Clear();
168170
InputHashes = [.. inputsByHash.Keys];
169171
_inputByHash = inputsByHash;
170172
}

test/Microsoft.NET.Sdk.Razor.Tests/StaticWebAssets/DiscoverStaticWebAssetsTest.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -601,6 +601,8 @@ public void DefineStaticWebAssetsCache_Recomputes_All_WhenPropertiesChange(Updat
601601

602602
Assert.False(cache.IsUpToDate());
603603
Assert.Same(inputHashes, cache.OutOfDateInputs());
604+
Assert.Empty(cache.CachedAssets);
605+
Assert.Empty(cache.CachedCopyCandidates);
604606
}
605607

606608
[Fact]

0 commit comments

Comments
 (0)