From 6d4746eb291bffe3978663d66228a4fc58b59793 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotr=20Kie=C5=82kowicz?= Date: Wed, 26 Feb 2025 16:53:55 +0100 Subject: [PATCH 1/4] Bump test packages --- Directory.Packages.props | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Directory.Packages.props b/Directory.Packages.props index 005c5e99363..af722d60487 100644 --- a/Directory.Packages.props +++ b/Directory.Packages.props @@ -109,7 +109,7 @@ - + @@ -121,7 +121,7 @@ - + From 2963d4696c8a02fa75222cd4c4c07c4d5c3ea1ab Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotr=20Kie=C5=82kowicz?= Date: Wed, 26 Feb 2025 16:54:02 +0100 Subject: [PATCH 2/4] xUnit2031 --- test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs | 2 +- test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs | 2 +- test/OpenTelemetry.Tests/Resources/ResourceBuilderTests.cs | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs b/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs index bdf93ae7598..069f5651031 100644 --- a/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs +++ b/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs @@ -188,7 +188,7 @@ public void MetricInstrumentationScopeIsExportedCorrectly() Assert.NotNull(metric.MeterTags); - Assert.Single(metric.MeterTags.Where(kvp => kvp.Key == meterTags[0].Key && kvp.Value == meterTags[0].Value)); + Assert.Single(metric.MeterTags, kvp => kvp.Key == meterTags[0].Key && kvp.Value == meterTags[0].Value); } [Fact] diff --git a/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs b/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs index 26506edf945..0fc735124af 100644 --- a/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs +++ b/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs @@ -137,7 +137,7 @@ public void AddViewWithExceptionInUserCallbackNoDefault() { var counter1 = meter1.CreateCounter("counter1"); counter1.Add(1); - Assert.Single(inMemoryEventListener.Events.Where((e) => e.EventId == 41)); + Assert.Single(inMemoryEventListener.Events, e => e.EventId == 41); } meterProvider.ForceFlush(MaxTimeToAllowForFlush); diff --git a/test/OpenTelemetry.Tests/Resources/ResourceBuilderTests.cs b/test/OpenTelemetry.Tests/Resources/ResourceBuilderTests.cs index 15ecdeedc88..3fec4a65c71 100644 --- a/test/OpenTelemetry.Tests/Resources/ResourceBuilderTests.cs +++ b/test/OpenTelemetry.Tests/Resources/ResourceBuilderTests.cs @@ -13,7 +13,7 @@ public void ServiceResource_ServiceName() var resource = ResourceBuilder.CreateEmpty().AddService("my-service").Build(); Assert.Equal(2, resource.Attributes.Count()); Assert.Contains(new KeyValuePair(ResourceSemanticConventions.AttributeServiceName, "my-service"), resource.Attributes); - Assert.Single(resource.Attributes.Where(kvp => kvp.Key == ResourceSemanticConventions.AttributeServiceName)); + Assert.Single(resource.Attributes, kvp => kvp.Key == ResourceSemanticConventions.AttributeServiceName); Assert.True(Guid.TryParse((string)resource.Attributes.Single(kvp => kvp.Key == ResourceSemanticConventions.AttributeServiceInstance).Value, out _)); } From 9b85d591cad1c8aca56d6862ef44250e06d1644b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotr=20Kie=C5=82kowicz?= Date: Wed, 26 Feb 2025 17:16:19 +0100 Subject: [PATCH 3/4] xUnit1042 --- .../Metrics/MetricTestData.cs | 92 ++++++++++--------- 1 file changed, 47 insertions(+), 45 deletions(-) diff --git a/test/OpenTelemetry.Tests/Metrics/MetricTestData.cs b/test/OpenTelemetry.Tests/Metrics/MetricTestData.cs index 76084371ce1..3ab818e7ebb 100644 --- a/test/OpenTelemetry.Tests/Metrics/MetricTestData.cs +++ b/test/OpenTelemetry.Tests/Metrics/MetricTestData.cs @@ -1,59 +1,61 @@ // Copyright The OpenTelemetry Authors // SPDX-License-Identifier: Apache-2.0 +using Xunit; + namespace OpenTelemetry.Metrics.Tests; public class MetricTestData { - public static IEnumerable InvalidInstrumentNames - => new List - { - new object[] { " " }, - new object[] { "-first-char-not-alphabetic" }, - new object[] { "1first-char-not-alphabetic" }, - new object[] { "invalid+separator" }, - new object[] { new string('m', 256) }, - new object[] { "a\xb5" }, // `\xb5` is the Micro character - }; + public static TheoryData InvalidInstrumentNames + => + [ + " ", + "-first-char-not-alphabetic", + "1first-char-not-alphabetic", + "invalid+separator", + new('m', 256), + "a\xb5", // `\xb5` is the Micro character + ]; - public static IEnumerable ValidInstrumentNames - => new List - { - new object[] { "m" }, - new object[] { "first-char-alphabetic" }, - new object[] { "my-2-instrument" }, - new object[] { "my.metric" }, - new object[] { "my_metric2" }, - new object[] { new string('m', 255) }, - new object[] { "CaSe-InSeNsItIvE" }, - new object[] { "my_metric/environment/database" }, - }; + public static TheoryData ValidInstrumentNames + => + [ + "m", + "first-char-alphabetic", + "my-2-instrument", + "my.metric", + "my_metric2", + new('m', 255), + "CaSe-InSeNsItIvE", + "my_metric/environment/database", + ]; - public static IEnumerable InvalidHistogramBoundaries - => new List - { - new object[] { new double[] { 0, 0 } }, - new object[] { new double[] { 1, 0 } }, - new object[] { new double[] { 0, 1, 1, 2 } }, - new object[] { new double[] { 0, 1, 2, -1 } }, - }; + public static TheoryData InvalidHistogramBoundaries + => + [ + [0.0, 0.0], + [1.0, 0.0], + [0.0, 1.0, 1.0, 2.0], + [0.0, 1.0, 2.0, -1.0], + ]; - public static IEnumerable ValidHistogramMinMax - => new List - { - new object[] { new double[] { -10, 0, 1, 9, 10, 11, 19 }, new HistogramConfiguration(), -10, 19 }, - new object[] { new double[] { double.NegativeInfinity }, new HistogramConfiguration(), double.NegativeInfinity, double.NegativeInfinity }, - new object[] { new double[] { double.NegativeInfinity, 0, double.PositiveInfinity }, new HistogramConfiguration(), double.NegativeInfinity, double.PositiveInfinity }, - new object[] { new double[] { 1 }, new HistogramConfiguration(), 1, 1 }, - new object[] { new double[] { 5, 100, 4, 101, -2, 97 }, new ExplicitBucketHistogramConfiguration() { Boundaries = [10.0, 20.0] }, -2, 101 }, - new object[] { new double[] { 5, 100, 4, 101, -2, 97 }, new Base2ExponentialBucketHistogramConfiguration(), 4, 101 }, - }; + public static TheoryData ValidHistogramMinMax => + new() + { + { [-10.0, 0.0, 1.0, 9.0, 10.0, 11.0, 19.0], new HistogramConfiguration(), -10.0, 19.0 }, + { [double.NegativeInfinity], new HistogramConfiguration(), double.NegativeInfinity, double.NegativeInfinity }, + { [double.NegativeInfinity, 0.0, double.PositiveInfinity], new HistogramConfiguration(), double.NegativeInfinity, double.PositiveInfinity }, + { [1.0], new HistogramConfiguration(), 1.0, 1.0 }, + { [5.0, 100.0, 4.0, 101.0, -2.0, 97.0], new ExplicitBucketHistogramConfiguration { Boundaries = [10.0, 20.0] }, -2.0, 101.0 }, + { [5.0, 100.0, 4.0, 101.0, -2.0, 97.0], new Base2ExponentialBucketHistogramConfiguration(), 4.0, 101.0 }, + }; - public static IEnumerable InvalidHistogramMinMax - => new List + public static TheoryData InvalidHistogramMinMax + => new() { - new object[] { new double[] { 1 }, new HistogramConfiguration() { RecordMinMax = false } }, - new object[] { new double[] { 1 }, new ExplicitBucketHistogramConfiguration() { Boundaries = [10.0, 20.0], RecordMinMax = false } }, - new object[] { new double[] { 1 }, new Base2ExponentialBucketHistogramConfiguration() { RecordMinMax = false } }, + { [1.0], new HistogramConfiguration { RecordMinMax = false } }, + { [1.0], new ExplicitBucketHistogramConfiguration { Boundaries = [10.0, 20.0], RecordMinMax = false } }, + { [1.0], new Base2ExponentialBucketHistogramConfiguration { RecordMinMax = false } }, }; } From ff27ad396ddfdfe33757e0ea72153cad0c781dcc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotr=20Kie=C5=82kowicz?= Date: Wed, 26 Feb 2025 17:18:54 +0100 Subject: [PATCH 4/4] xUnit2024 --- test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs b/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs index 069f5651031..a2366647e1f 100644 --- a/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs +++ b/test/OpenTelemetry.Tests/Metrics/MetricApiTests.cs @@ -584,7 +584,7 @@ public void MeterSourcesWildcardSupportMatchTest(bool hasView) meterProvider.ForceFlush(MaxTimeToAllowForFlush); - Assert.True(exportedItems.Count == 5); // "SomeCompany.SomeProduct.SomeComponent" will not be subscribed. + Assert.Equal(5, exportedItems.Count); // "SomeCompany.SomeProduct.SomeComponent" will not be subscribed. if (hasView) { @@ -629,7 +629,7 @@ public void MeterSourcesWildcardSupportNegativeTestNoMeterAdded(bool hasView) meter2.CreateObservableGauge("myGauge2", () => measurement); meterProvider.ForceFlush(MaxTimeToAllowForFlush); - Assert.True(exportedItems.Count == 0); + Assert.Empty(exportedItems); } [Theory]