Skip to content

Commit

Permalink
GitHubSync update - release-6.1 (#703)
Browse files Browse the repository at this point in the history
* GitHubSync update - release-6.1

* Consolidate Azure.Storage.Blobs to 12.21.2

* Consolidate NServiceBus.ClaimCheck to 1.0.1

* Bump Particular.Approvals to 2.0.0

* Bump Particular.Packaging to 4.2.0

* Bump Microsoft.NET.Test.Sdk to 17.12.0

* Clean up the declaration of the reference for GitHubActionsTestLogger

* Remove unnecessary dependency.

* Add NServiceBus reference

* Update testing packages

* Update Fody

* Add NUnit.Analyzers

* Disable automatic version range for package

---------

Co-authored-by: internalautomation[bot] <85681268+internalautomation[bot]@users.noreply.github.com>
Co-authored-by: Tamara Rivera <tamita.rivera@gmail.com>
Co-authored-by: Brandon Ording <bording@gmail.com>
  • Loading branch information
3 people authored Dec 2, 2024
1 parent 583bd86 commit 424f909
Show file tree
Hide file tree
Showing 16 changed files with 136 additions and 54 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,11 @@
</ItemGroup>

<ItemGroup>
<PackageReference Include="Azure.Storage.Blobs" Version="12.21.2" />
<PackageReference Include="GitHubActionsTestLogger" Version="2.4.1">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.0" />
<PackageReference Include="NServiceBus.AcceptanceTests.Sources" Version="9.2.2" GeneratePathProperty="true" />
<PackageReference Include="NServiceBus.ClaimCheck" Version="1.0.1" />
<PackageReference Include="NUnit" Version="4.1.0" />
<PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
<PackageReference Include="NServiceBus.AcceptanceTests.Sources" Version="9.2.3" GeneratePathProperty="true" />
<PackageReference Include="NUnit" Version="4.2.2" />
<PackageReference Include="NUnit.Analyzers" Version="4.4.0" />
<PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
</ItemGroup>

Expand Down
4 changes: 1 addition & 3 deletions src/AcceptanceTests/When_using_claimcheck_with_client.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,9 @@
using System.Threading.Tasks;
using AcceptanceTesting;
using NServiceBus;
using NServiceBus.ClaimCheck;
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework;
using NUnit.Framework.Legacy;

public class When_using_claimcheck_with_client : NServiceBusAcceptanceTest
{
Expand All @@ -27,7 +25,7 @@ public async Task Should_work()
.Done(c => c.MessageReceived)
.Run();

CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,9 @@
using System.Threading.Tasks;
using AcceptanceTesting;
using NServiceBus;
using NServiceBus.ClaimCheck;
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework;
using NUnit.Framework.Legacy;

public class When_using_claimcheck_with_connection_string : NServiceBusAcceptanceTest
{
Expand All @@ -27,7 +25,7 @@ public async Task Should_work()
.Done(c => c.MessageReceived)
.Run();

CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,11 @@
using System;
using System.Threading.Tasks;
using AcceptanceTesting;
using NUnit.Framework;
using Azure.Storage.Blobs;
using Microsoft.Extensions.DependencyInjection;
using NServiceBus.ClaimCheck;
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework.Legacy;
using NUnit.Framework;

public class When_using_claimcheck_with_custom_provider : NServiceBusAcceptanceTest
{
Expand All @@ -28,7 +26,7 @@ public async Task Should_work()
.Run();

Assert.That(context.ProviderWasCalled);
CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
4 changes: 1 addition & 3 deletions src/AcceptanceTests/When_using_claimcheck_with_expiry.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,9 @@
using System.Threading.Tasks;
using AcceptanceTesting;
using NServiceBus;
using NServiceBus.ClaimCheck;
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework;
using NUnit.Framework.Legacy;

public class When_using_claimcheck_with_expiry : NServiceBusAcceptanceTest
{
Expand All @@ -27,7 +25,7 @@ public async Task Should_work()
.Done(c => c.MessageReceived)
.Run();

CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
16 changes: 10 additions & 6 deletions src/DataBus/NServiceBus.DataBus.AzureBlobStorage.csproj
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>net8.0</TargetFramework>
Expand All @@ -8,12 +8,16 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Azure.Storage.Blobs" Version="[12.19.1, 13.0.0)" />
<PackageReference Include="Fody" Version="6.8.1" PrivateAssets="All" />
<PackageReference Include="Microsoft.IO.RecyclableMemoryStream" Version="3.0.1" />
<PackageReference Include="NServiceBus.ClaimCheck" Version="[1.0.1,2.0.0)" />
<PackageReference Include="Azure.Storage.Blobs" Version="12.21.2" />
<PackageReference Include="Microsoft.IO.RecyclableMemoryStream" Version="3.0.1" AutomaticVersionRange="false" />
<PackageReference Include="NServiceBus" Version="9.2.3" />
<PackageReference Include="NServiceBus.ClaimCheck" Version="1.0.1" />
</ItemGroup>

<ItemGroup>
<PackageReference Include="Fody" Version="6.9.1" PrivateAssets="All" />
<PackageReference Include="Obsolete.Fody" Version="5.3.0" PrivateAssets="All" />
<PackageReference Include="Particular.Packaging" Version="4.1.0" PrivateAssets="All" />
<PackageReference Include="Particular.Packaging" Version="4.2.0" PrivateAssets="All" />
</ItemGroup>

<ItemGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,12 +9,11 @@
</ItemGroup>

<ItemGroup>
<PackageReference Include="Azure.Storage.Blobs" Version="12.21.2" />
<PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.0" />
<PackageReference Include="NServiceBus.AcceptanceTests.Sources" Version="9.2.2" GeneratePathProperty="true" />
<PackageReference Include="NServiceBus.ClaimCheck" Version="1.0.1" />
<PackageReference Include="NUnit" Version="4.1.0" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
<PackageReference Include="NServiceBus.AcceptanceTests.Sources" Version="9.2.3" GeneratePathProperty="true" />
<PackageReference Include="NUnit" Version="4.2.2" />
<PackageReference Include="NUnit.Analyzers" Version="4.4.0" />
<PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
</ItemGroup>

Expand Down
3 changes: 1 addition & 2 deletions src/DataBusAcceptanceTests/When_using_databus_with_client.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework;
using NUnit.Framework.Legacy;

#pragma warning disable CS0618 // Type or member is obsolete
public class When_using_databus_with_client : NServiceBusAcceptanceTest
Expand All @@ -27,7 +26,7 @@ public async Task Should_work()
.Done(c => c.MessageReceived)
.Run();

CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework;
using NUnit.Framework.Legacy;

#pragma warning disable CS0618 // Type or member is obsolete
public class When_using_databus_with_connection_string : NServiceBusAcceptanceTest
Expand All @@ -24,7 +23,7 @@ public async Task Should_work()
.Done(c => c.MessageReceived)
.Run();

CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,11 @@
using System;
using System.Threading.Tasks;
using AcceptanceTesting;
using NUnit.Framework;
using Azure.Storage.Blobs;
using Microsoft.Extensions.DependencyInjection;
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework.Legacy;
using NUnit.Framework;

#pragma warning disable CS0618 // Type or member is obsolete
public class When_using_databus_with_custom_provider : NServiceBusAcceptanceTest
Expand All @@ -28,7 +27,7 @@ public async Task Should_work()
.Run();

Assert.That(context.ProviderWasCalled);
CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
3 changes: 1 addition & 2 deletions src/DataBusAcceptanceTests/When_using_databus_with_expiry.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
using NServiceBus.AcceptanceTests;
using NServiceBus.AcceptanceTests.EndpointTemplates;
using NUnit.Framework;
using NUnit.Framework.Legacy;

#pragma warning disable CS0618 // Type or member is obsolete
public class When_using_databus_with_expiry : NServiceBusAcceptanceTest
Expand All @@ -24,7 +23,7 @@ public async Task Should_work()
.Done(c => c.MessageReceived)
.Run();

CollectionAssert.AreEqual(payloadToSend, context.PayloadReceived);
Assert.That(context.PayloadReceived, Is.EqualTo(payloadToSend).AsCollection);
}

public class Context : ScenarioContext
Expand Down
4 changes: 2 additions & 2 deletions src/Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@
<EnableNETAnalyzers>true</EnableNETAnalyzers>
<AnalysisLevel Condition="'$(AnalysisLevel)' == ''">5.0</AnalysisLevel>
<EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
<!-- NuGetAuditMode set to 'all' for tool projects in Directory.Build.targets, other project types default to 'direct' -->
<NuGetAuditLevel>low</NuGetAuditLevel>
<NuGetAuditMode Condition="'$(NuGetAuditMode)' == ''">all</NuGetAuditMode>
<!-- To lock the version of Particular.Analyzers, for example, in a release branch, set this property in Custom.Build.props -->
<ParticularAnalyzersVersion Condition="'$(ParticularAnalyzersVersion)' == ''">2.1.2</ParticularAnalyzersVersion>
<ParticularAnalyzersVersion Condition="'$(ParticularAnalyzersVersion)' == ''">2.1.3</ParticularAnalyzersVersion>
<NServiceBusKey>0024000004800000940000000602000000240000525341310004000001000100dde965e6172e019ac82c2639ffe494dd2e7dd16347c34762a05732b492e110f2e4e2e1b5ef2d85c848ccfb671ee20a47c8d1376276708dc30a90ff1121b647ba3b7259a6bc383b2034938ef0e275b58b920375ac605076178123693c6c4f1331661a62eba28c249386855637780e3ff5f23a6d854700eaa6803ef48907513b92</NServiceBusKey>
<NServiceBusTestsKey>00240000048000009400000006020000002400005253413100040000010001007f16e21368ff041183fab592d9e8ed37e7be355e93323147a1d29983d6e591b04282e4da0c9e18bd901e112c0033925eb7d7872c2f1706655891c5c9d57297994f707d16ee9a8f40d978f064ee1ffc73c0db3f4712691b23bf596f75130f4ec978cf78757ec034625a5f27e6bb50c618931ea49f6f628fd74271c32959efb1c5</NServiceBusTestsKey>
</PropertyGroup>
Expand Down
4 changes: 1 addition & 3 deletions src/Directory.Build.targets
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
<Project>

<PropertyGroup>
<NuGetAuditMode Condition="'$(PackAsTool)' == 'true'">all</NuGetAuditMode>
</PropertyGroup>
<Import Project="msbuild\AutomaticVersionRanges.targets" Condition="Exists('msbuild\AutomaticVersionRanges.targets')" />

</Project>
12 changes: 5 additions & 7 deletions src/Tests/NServiceBus.DataBus.AzureBlobStorage.Tests.csproj
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>net8.0</TargetFramework>
Expand All @@ -11,14 +11,12 @@
</ItemGroup>

<ItemGroup>
<PackageReference Include="Azure.Storage.Blobs" Version="12.21.2" />
<PackageReference Include="GitHubActionsTestLogger" Version="2.4.1" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.0" />
<PackageReference Include="NServiceBus" Version="9.2.2" />
<PackageReference Include="NServiceBus.ClaimCheck" Version="1.0.1" />
<PackageReference Include="NUnit" Version="4.1.0" />
<PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
<PackageReference Include="NUnit" Version="4.2.2" />
<PackageReference Include="NUnit.Analyzers" Version="4.4.0" />
<PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
<PackageReference Include="Particular.Approvals" Version="1.0.0" />
<PackageReference Include="Particular.Approvals" Version="2.0.0" />
<PackageReference Include="PublicApiGenerator" Version="11.1.0" />
</ItemGroup>

Expand Down
42 changes: 42 additions & 0 deletions src/msbuild/AutomaticVersionRanges.targets
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
<Project>

<PropertyGroup>
<AutomaticVersionRangesEnabled Condition="'$(AutomaticVersionRangesEnabled)' == '' And '$(Configuration)' == 'Debug'">false</AutomaticVersionRangesEnabled>
<AutomaticVersionRangesEnabled Condition="'$(AutomaticVersionRangesEnabled)' == '' And '$(IsPackable)' == 'false'">false</AutomaticVersionRangesEnabled>
<AutomaticVersionRangesEnabled Condition="'$(AutomaticVersionRangesEnabled)' == '' And '$(ManagePackageVersionsCentrally)' == 'true'">false</AutomaticVersionRangesEnabled>
<AutomaticVersionRangesEnabled Condition="'$(AutomaticVersionRangesEnabled)' == ''">true</AutomaticVersionRangesEnabled>
</PropertyGroup>

<UsingTask TaskName="ConvertToVersionRange" TaskFactory="RoslynCodeTaskFactory" AssemblyFile="$(MSBuildToolsPath)\Microsoft.Build.Tasks.Core.dll">
<Task>
<Code Source="$(MSBuildThisFileDirectory)ConvertToVersionRange.cs" />
</Task>
</UsingTask>

<Target Name="ConvertProjectReferenceVersionsToVersionRanges" AfterTargets="_GetProjectReferenceVersions" Condition="'$(AutomaticVersionRangesEnabled)' == 'true'">
<PropertyGroup>
<NumberOfProjectReferences>@(_ProjectReferencesWithVersions->Count())</NumberOfProjectReferences>
</PropertyGroup>
<ConvertToVersionRange Condition="$(NumberOfProjectReferences) &gt; 0" References="@(_ProjectReferencesWithVersions)" VersionProperty="ProjectVersion">
<Output TaskParameter="ReferencesWithVersionRanges" ItemName="_ProjectReferencesWithVersionRanges" />
</ConvertToVersionRange>
<ItemGroup Condition="$(NumberOfProjectReferences) &gt; 0">
<_ProjectReferencesWithVersions Remove="@(_ProjectReferencesWithVersions)" />
<_ProjectReferencesWithVersions Include="@(_ProjectReferencesWithVersionRanges)" />
</ItemGroup>
</Target>

<Target Name="ConvertPackageReferenceVersionsToVersionRanges" BeforeTargets="CollectPackageReferences" Condition="'$(AutomaticVersionRangesEnabled)' == 'true'">
<PropertyGroup>
<NumberOfPackageReferences>@(PackageReference->Count())</NumberOfPackageReferences>
</PropertyGroup>
<ConvertToVersionRange Condition="$(NumberOfPackageReferences) &gt; 0" References="@(PackageReference)" VersionProperty="Version">
<Output TaskParameter="ReferencesWithVersionRanges" ItemName="_PackageReferencesWithVersionRanges" />
</ConvertToVersionRange>
<ItemGroup Condition="$(NumberOfPackageReferences) &gt; 0">
<PackageReference Remove="@(PackageReference)" />
<PackageReference Include="@(_PackageReferencesWithVersionRanges)" />
</ItemGroup>
</Target>

</Project>
57 changes: 57 additions & 0 deletions src/msbuild/ConvertToVersionRange.cs
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
using System;
using System.Text.RegularExpressions;
using Microsoft.Build.Framework;
using Microsoft.Build.Utilities;

public class ConvertToVersionRange : Task
{
[Required]
public ITaskItem[] References { get; set; } = [];

[Required]
public string VersionProperty { get; set; } = string.Empty;

[Output]
public ITaskItem[] ReferencesWithVersionRanges { get; private set; } = [];

public override bool Execute()
{
var success = true;

foreach (var reference in References)
{
var automaticVersionRange = reference.GetMetadata("AutomaticVersionRange");

if (automaticVersionRange.Equals("false", StringComparison.OrdinalIgnoreCase))
{
continue;
}

var privateAssets = reference.GetMetadata("PrivateAssets");

if (privateAssets.Equals("All", StringComparison.OrdinalIgnoreCase))
{
continue;
}

var version = reference.GetMetadata(VersionProperty);
var match = Regex.Match(version, @"^\d+");

if (match.Value.Equals(string.Empty, StringComparison.Ordinal))
{
Log.LogError("Reference '{0}' with version '{1}' is not valid for automatic version range conversion. Fix the version or exclude the reference from conversion by setting 'AutomaticVersionRange=\"false\"' on the reference.", reference.ItemSpec, version);
success = false;
continue;
}

var nextMajor = Convert.ToInt32(match.Value) + 1;

var versionRange = $"[{version}, {nextMajor}.0.0)";
reference.SetMetadata(VersionProperty, versionRange);
}

ReferencesWithVersionRanges = References;

return success;
}
}

0 comments on commit 424f909

Please sign in to comment.