suite: fix SetupSubTest and TearDownSubTest execution order #1471

linusbarth · 2023-09-15T10:34:46Z

Summary

The order of execution of SetupSubTest and TearDownSubTest methods was changed so that the testing context inside these methods is correct.

Changes

The SetupSubTest was moved to after the SetT that set the child testing context
The TearDownSubTest was moved to before the deferred SetT that reset the testing context (execution wise)
The TearDownSubTest and SetT were deferred in one block. This was fine beforehand because a panic in the TearDownSubTest would have had no influence on the "suite.SetT(oldT)". Now since a panic in the TearDownSubTest would lead to omitting the "suite.SetT(oldT)" this defer was split into two separate defers.

Motivation

There were two problems with the order of execution in the Suite.Run() method:

One could not access the correct testing context ("s.T()") inside the SetupSubTest and TearDownSubTest methods. If the testing context was used for e.g. assertions of mocks in the TearDownSubTest, the results would not be "attached" to the correct test in the test output.
The behavior was different to the order of execution for "root" tests regarding the SetupTest and TearDownTest methods (see lines 167-201). This could confuse users of the library.

Related issues

Closes #1465

dolmen · 2023-10-10T12:43:40Z

suite/suite.go

+			if tearDownSubTest, ok := suite.s.(TearDownSubTest); ok {
+				tearDownSubTest.TearDownSubTest()
+			}
+		}()


Move the if outside of the defer:

if tearDownSubTest, ok := suite.s.(TearDownSubTest); ok { defer tearDownSubTest.TearDownSubTest() }

Done. Also I took the liberty to extract the defer-function in line 102. Please note if I should revert that. I think it should not produce noticeable change.

dolmen

We also need test that enforces the order. That test should fail before your fix and succeed after.

linusbarth · 2023-10-10T15:52:09Z

We also need test that enforces the order. That test should fail before your fix and succeed after.

done

dolmen · 2023-10-15T22:28:23Z

@linusbarth Please drop the last commit (merge commit). Use git rebase instead.

dolmen · 2023-10-15T22:42:37Z

suite/suite_test.go

@@ -259,10 +259,14 @@ func (suite *SuiteTester) TestSubtest() {

 func (suite *SuiteTester) TearDownSubTest() {
 	suite.TearDownSubTestRunCount++
+	// We should get the *testing.T for the test that is to be torn down


Move this comment as 3rd argument of Contains below.

dolmen · 2023-10-15T22:42:47Z

suite/suite_test.go

 }

 func (suite *SuiteTester) SetupSubTest() {
 	suite.SetupSubTestRunCount++
+	// We should get the *testing.T for the test that is to be set up


Move this comment as 3rd argument of Contains below.

There were two problems with the order of execution in the Suite.Run() method: - One could not access the correct testing context ("s.T()") inside the SetupSubTest and TearDownSubTest methods. If the testing context was used for e.g. assertions of mocks in the TearDownSubTest, the results would not be "attached" to the correct test in the test output. - The behavior was different to the order of execution for "root" tests (see lines 167-201) regarding the SetupTest and TearDownTest methods. This could confuse users of the library. Also the logic to be deferred was joined together. This was fine beforehand because a panic in the TearDownSubTest would have had no influence on the "suite.SetT(oldT)". Now since a panic in the TearDownSubTest would lead to omitting the "suite.SetT(oldT)" this defer was split into two separate defers.

tomhutch

Nice, I'd be really happy to see this fix get in.

tomhutch · 2023-10-16T17:05:56Z

suite/suite_test.go

@@ -259,10 +259,12 @@ func (suite *SuiteTester) TestSubtest() {

 func (suite *SuiteTester) TearDownSubTest() {
 	suite.TearDownSubTestRunCount++
+	suite.Contains(suite.T().Name(), "subtest", "We should get the *testing.T for the test that is to be torn down")


Updating the test suite attributes here and performing assertions alongside the others in the main test body will be more consistent with the current testing setup:

testify/suite/suite_test.go

Lines 154 to 172 in ac5cd69

SetupSuiteRunCount int

TearDownSuiteRunCount int

SetupTestRunCount int

TearDownTestRunCount int

TestOneRunCount int

TestTwoRunCount int

TestSubtestRunCount int

NonTestMethodRunCount int

SetupSubTestRunCount int

TearDownSubTestRunCount int

SuiteNameBefore []string

TestNameBefore []string

SuiteNameAfter []string

TestNameAfter []string

TimeBefore []time.Time

TimeAfter []time.Time

See this for reference:
https://github.com/stretchr/testify/pull/1393/files#diff-ca4ffdb7050ad570420ac5682fefad8481384709c2884051cd57071283ce9ca4

Thanks for your review! You're right, that is more consistent. I mostly just copied your referenced testing code now into my branch.

suite/suite.go

This fix adds panic handling for subtests which will achieve: - subtests will fail for the correct test context when panicking - the test execution is not stopped; the next subtest will be executed

This MR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [github.com/stretchr/testify](https://github.com/stretchr/testify) | require | minor | `v1.8.0` -> `v1.9.0` | --- ### Release Notes <details> <summary>stretchr/testify (github.com/stretchr/testify)</summary> ### [`v1.9.0`](https://github.com/stretchr/testify/releases/tag/v1.9.0) [Compare Source](stretchr/testify@v1.8.4...v1.9.0) #### What's Changed - Fix Go modules version by [@SuperQ](https://github.com/SuperQ) in stretchr/testify#1394 - Document that require is not safe to call in created goroutines by [@programmer04](https://github.com/programmer04) in stretchr/testify#1392 - Remove myself from MAINTAINERS.md by [@mvdkleijn](https://github.com/mvdkleijn) in stretchr/testify#1367 - Correct spelling/grammar by [@echarrod](https://github.com/echarrod) in stretchr/testify#1389 - docs: Update URLs in README by [@davidjb](https://github.com/davidjb) in stretchr/testify#1349 - Update mockery link to Github Pages in README by [@LandonTClipp](https://github.com/LandonTClipp) in stretchr/testify#1346 - docs: Fix typos in tests and comments by [@alexandear](https://github.com/alexandear) in stretchr/testify#1410 - CI: tests from go1.17 by [@SuperQ](https://github.com/SuperQ) in stretchr/testify#1409 - Fix adding ? when no values passed by [@lesichkovm](https://github.com/lesichkovm) in stretchr/testify#1320 - codegen: use standard header for generated files by [@dolmen](https://github.com/dolmen) in stretchr/testify#1406 - mock: AssertExpectations log reason only on failure by [@hikyaru-suzuki](https://github.com/hikyaru-suzuki) in stretchr/testify#1360 - assert: fix flaky TestNeverTrue by [@dolmen](https://github.com/dolmen) in stretchr/testify#1417 - README: fix typos "set up" vs "setup" by [@ossan-dev](https://github.com/ossan-dev) in stretchr/testify#1428 - mock: move regexp compilation outside of `Called` by [@aud10slave](https://github.com/aud10slave) in stretchr/testify#631 - assert: refactor internal func getLen() by [@dolmen](https://github.com/dolmen) in stretchr/testify#1445 - mock: deprecate type AnythingOfTypeArgument ([#1434](stretchr/testify#1434)) by [@dolmen](https://github.com/dolmen) in stretchr/testify#1441 - Remove no longer needed assert.canConvert by [@alexandear](https://github.com/alexandear) in stretchr/testify#1470 - assert: ObjectsAreEqual: use time.Equal for time.Time types by [@tscales](https://github.com/tscales) in stretchr/testify#1464 - Bump actions/checkout from 3 to 4 by [@dependabot](https://github.com/dependabot) in stretchr/testify#1466 - Bump actions/setup-go from 3.2.0 to 4.1.0 by [@dependabot](https://github.com/dependabot) in stretchr/testify#1451 - fix: make EventuallyWithT concurrency safe by [@czeslavo](https://github.com/czeslavo) in stretchr/testify#1395 - assert: fix httpCode and HTTPBody occur panic when http.Handler read Body by [@hidu](https://github.com/hidu) in stretchr/testify#1484 - assert.EqualExportedValues: fix handling of arrays by [@zrbecker](https://github.com/zrbecker) in stretchr/testify#1473 - .github: use latest Go versions by [@kevinburkesegment](https://github.com/kevinburkesegment) in stretchr/testify#1489 - assert: Deprecate EqualExportedValues by [@HaraldNordgren](https://github.com/HaraldNordgren) in stretchr/testify#1488 - suite: refactor test assertions by [@alexandear](https://github.com/alexandear) in stretchr/testify#1474 - suite: fix SetupSubTest and TearDownSubTest execution order by [@linusbarth](https://github.com/linusbarth) in stretchr/testify#1471 - docs: Fix deprecation comments for http package by [@alexandear](https://github.com/alexandear) in stretchr/testify#1335 - Add map support doc comments to Subset and NotSubset by [@jedevc](https://github.com/jedevc) in stretchr/testify#1306 - TestErrorIs/TestNotErrorIs: check error message contents by [@craig65535](https://github.com/craig65535) in stretchr/testify#1435 - suite: fix subtest names (fix [#1501](stretchr/testify#1501)) by [@dolmen](https://github.com/dolmen) in stretchr/testify#1504 - assert: improve unsafe.Pointer tests by [@dolmen](https://github.com/dolmen) in stretchr/testify#1505 - assert: simplify isNil implementation by [@dolmen](https://github.com/dolmen) in stretchr/testify#1506 - assert.InEpsilonSlice: fix expected/actual order and other improvements by [@dolmen](https://github.com/dolmen) in stretchr/testify#1483 - Fix dependency cycle with objx [#1292](stretchr/testify#1292) by [@dolmen](https://github.com/dolmen) in stretchr/testify#1453 - mock: refactor TestIsArgsEqual by [@dolmen](https://github.com/dolmen) in stretchr/testify#1444 - mock: optimize argument matching checks by [@dolmen](https://github.com/dolmen) in stretchr/testify#1416 - assert: fix TestEventuallyTimeout by [@dolmen](https://github.com/dolmen) in stretchr/testify#1412 - CI: add go 1.21 in GitHub Actions by [@dolmen](https://github.com/dolmen) in stretchr/testify#1450 - suite: fix recoverAndFailOnPanic to report test failure at the right location by [@dolmen](https://github.com/dolmen) in stretchr/testify#1502 - Update maintainers by [@brackendawson](https://github.com/brackendawson) in stretchr/testify#1533 - assert: Fix EqualValues to handle overflow/underflow by [@arjunmahishi](https://github.com/arjunmahishi) in stretchr/testify#1531 - assert: better formatting for Len() error by [@kevinburkesegment](https://github.com/kevinburkesegment) in stretchr/testify#1485 - Ensure AssertExpectations does not fail in skipped tests by [@ianrose14](https://github.com/ianrose14) in stretchr/testify#1331 - suite: fix deadlock in suite.Require()/Assert() by [@arjunmahishi](https://github.com/arjunmahishi) in stretchr/testify#1535 - Revert "assert: ObjectsAreEqual: use time.Equal for time.Time type" by [@brackendawson](https://github.com/brackendawson) in stretchr/testify#1537 - \[chore] Add issue templates by [@arjunmahishi](https://github.com/arjunmahishi) in stretchr/testify#1538 - Update the build status badge by [@brackendawson](https://github.com/brackendawson) in stretchr/testify#1540 - Update Github workflows setup-go to V5 by [@hendrywiranto](https://github.com/hendrywiranto) in stretchr/testify#1545 - Support Pointer to Struct in EqualExportedValues by [@Lucaber](https://github.com/Lucaber) in stretchr/testify#1517 - README: drop link to gorc by [@guettli](https://github.com/guettli) in stretchr/testify#1248 - http_assertions: honour the msgAndArgs provided with each assertion by [@arjunmahishi](https://github.com/arjunmahishi) in stretchr/testify#1548 - fix typos in comments and tests by [@ccoVeille](https://github.com/ccoVeille) in stretchr/testify#1247 - Include the auto-release notes in releases by [@brackendawson](https://github.com/brackendawson) in stretchr/testify#1550 - Add `NotImplements` and variants by [@hslatman](https://github.com/hslatman) in stretchr/testify#1385 - Add support to compare uintptr by [@bogdandrutu](https://github.com/bogdandrutu) in stretchr/testify#1339 - build(deps): bump github.com/stretchr/objx from 0.5.1 to 0.5.2 by [@dependabot](https://github.com/dependabot) in stretchr/testify#1552 #### New Contributors - [@SuperQ](https://github.com/SuperQ) made their first contribution in stretchr/testify#1394 - [@programmer04](https://github.com/programmer04) made their first contribution in stretchr/testify#1392 - [@echarrod](https://github.com/echarrod) made their first contribution in stretchr/testify#1389 - [@davidjb](https://github.com/davidjb) made their first contribution in stretchr/testify#1349 - [@LandonTClipp](https://github.com/LandonTClipp) made their first contribution in stretchr/testify#1346 - [@alexandear](https://github.com/alexandear) made their first contribution in stretchr/testify#1410 - [@lesichkovm](https://github.com/lesichkovm) made their first contribution in stretchr/testify#1320 - [@dolmen](https://github.com/dolmen) made their first contribution in stretchr/testify#1406 - [@hikyaru-suzuki](https://github.com/hikyaru-suzuki) made their first contribution in stretchr/testify#1360 - [@ossan-dev](https://github.com/ossan-dev) made their first contribution in stretchr/testify#1428 - [@aud10slave](https://github.com/aud10slave) made their first contribution in stretchr/testify#631 - [@tscales](https://github.com/tscales) made their first contribution in stretchr/testify#1464 - [@czeslavo](https://github.com/czeslavo) made their first contribution in stretchr/testify#1395 - [@hidu](https://github.com/hidu) made their first contribution in stretchr/testify#1484 - [@zrbecker](https://github.com/zrbecker) made their first contribution in stretchr/testify#1473 - [@kevinburkesegment](https://github.com/kevinburkesegment) made their first contribution in stretchr/testify#1489 - [@linusbarth](https://github.com/linusbarth) made their first contribution in stretchr/testify#1471 - [@jedevc](https://github.com/jedevc) made their first contribution in stretchr/testify#1306 - [@craig65535](https://github.com/craig65535) made their first contribution in stretchr/testify#1435 - [@arjunmahishi](https://github.com/arjunmahishi) made their first contribution in stretchr/testify#1531 - [@ianrose14](https://github.com/ianrose14) made their first contribution in stretchr/testify#1331 - [@hendrywiranto](https://github.com/hendrywiranto) made their first contribution in stretchr/testify#1545 - [@Lucaber](https://github.com/Lucaber) made their first contribution in stretchr/testify#1517 - [@guettli](https://github.com/guettli) made their first contribution in stretchr/testify#1248 - [@ccoVeille](https://github.com/ccoVeille) made their first contribution in stretchr/testify#1247 - [@hslatman](https://github.com/hslatman) made their first contribution in stretchr/testify#1385 - [@bogdandrutu](https://github.com/bogdandrutu) made their first contribution in stretchr/testify#1339 **Full Changelog**: stretchr/testify@v1.8.4...v1.9.0 ### [`v1.8.4`](https://github.com/stretchr/testify/releases/tag/v1.8.4) [Compare Source](stretchr/testify@v1.8.3...v1.8.4) #### What's Changed - Create GitHub release when new release tag is pushed by [@aldas](https://github.com/aldas) in stretchr/testify#1354 #### New Contributors - [@aldas](https://github.com/aldas) made their first contribution in stretchr/testify#1354 **Full Changelog**: stretchr/testify@v1.8.3...v1.8.4 ### [`v1.8.3`](https://github.com/stretchr/testify/releases/tag/v1.8.3) [Compare Source](stretchr/testify@v1.8.2...v1.8.3) #### What's Changed - Compare public elements of struct by [@mchlp](https://github.com/mchlp) in stretchr/testify#1309 - assert: fix error message formatting for NotContains by [@wwade](https://github.com/wwade) in stretchr/testify#1362 - allow testing for functional options by [@nbaztec](https://github.com/nbaztec) in stretchr/testify#1023 - add EventuallyWithT assertion by [@tobikris](https://github.com/tobikris) in stretchr/testify#1264 - EqualExportedValues: Handle nested pointer, slice and map fields by [@HaraldNordgren](https://github.com/HaraldNordgren) in stretchr/testify#1379 #### New Contributors - [@mchlp](https://github.com/mchlp) made their first contribution in stretchr/testify#1309 - [@wwade](https://github.com/wwade) made their first contribution in stretchr/testify#1362 - [@nbaztec](https://github.com/nbaztec) made their first contribution in stretchr/testify#1023 - [@tobikris](https://github.com/tobikris) made their first contribution in stretchr/testify#1264 **Full Changelog**: stretchr/testify@v1.8.2...v1.8.3 ### [`v1.8.2`](https://github.com/stretchr/testify/releases/tag/v1.8.2) [Compare Source](stretchr/testify@v1.8.1...v1.8.2) #### What's Changed - Add opportunity to trigger setup/teardown for subtest by [@qerdcv](https://github.com/qerdcv) in stretchr/testify#1246 - fix: fix bug for check unsafe.Pointer isNil by [@sunpe](https://github.com/sunpe) in stretchr/testify#1319 - Fix Call.Unset() panic (issue [#1236](stretchr/testify#1236)) by [@lisitsky](https://github.com/lisitsky) in stretchr/testify#1250 - Fix `CallerInfo()` source file paths by [@bozaro](https://github.com/bozaro) in stretchr/testify#1288 - assert: Fix Subset/NotSubset when map is missing keys from the subset by [@danielwhite](https://github.com/danielwhite) in stretchr/testify#1261 #### New Contributors - [@qerdcv](https://github.com/qerdcv) made their first contribution in stretchr/testify#1246 - [@sunpe](https://github.com/sunpe) made their first contribution in stretchr/testify#1319 - [@lisitsky](https://github.com/lisitsky) made their first contribution in stretchr/testify#1250 - [@bozaro](https://github.com/bozaro) made their first contribution in stretchr/testify#1288 - [@danielwhite](https://github.com/danielwhite) made their first contribution in stretchr/testify#1261 **Full Changelog**: stretchr/testify@v1.8.1...v1.8.2 ### [`v1.8.1`](https://github.com/stretchr/testify/releases/tag/v1.8.1) [Compare Source](stretchr/testify@v1.8.0...v1.8.1) #### What's Changed - Bump github.com/stretchr/objx from 0.4.0 to 0.5.0 by [@dependabot](https://github.com/dependabot) in stretchr/testify#1283 **Full Changelog**: stretchr/testify@v1.8.0...v1.8.1 </details> --- - [ ] If you want to rebase/retry this MR, check this box ---    See merge request alpine/infra/build-server-status!12

* [configgrpc] Add `ToServerContext` (#9624) Adds a new function, `ToServerContext` which does what `ToServer` does, but takes a `context.Context`. After the next release we'll deprecate `ToServerContext` and rename it to `ToServer`. Related to https://github.com/open-telemetry/opentelemetry-collector/issues/9490 --------- Co-authored-by: Dmitrii Anoshin <[email protected]> * [chore] Fix prepare-release.yml (#9675) **Description:** Fixes https://github.com/open-telemetry/opentelemetry-collector/actions/runs/8139621131/workflow ``` The workflow is not valid. .github/workflows/prepare-release.yml (Line: 54, Col: 9): Job 'prepare-release' depends on unknown job 'validate-version'. ``` #9409 introduced some errors because the workflow is not tested on PRs, #9668 partly fixed them and I mistakenly believed Github would notify me of further errors in the workflow, the same way it did with this one, but it did not. * [chore] Prepare release v1.3.0/v0.96.0 (#9680) The following commands were run to prepare this release: - make chlog-update VERSION=v1.3.0/v0.96.0 - make prepare-release PREVIOUS_VERSION=1.2.0 RELEASE_CANDIDATE=1.3.0 MODSET=stable - make prepare-release PREVIOUS_VERSION=0.95.0 RELEASE_CANDIDATE=0.96.0 MODSET=beta * [chore] Update release schedule (#9678) **Description:** Updates release schedule. The March 18th release overlaps with KubeCon EU, so I am shifting everything by one week starting with that release. This means the next release cycle will have three weeks. * [chore] Add reminder to update release schedule to release issue template (#9679) **Description:** Add reminder to update the release schedule * [chore] Update prepare release examples (#9677) **Description:** After #8975 we decided to not do any more explicit release candidates so we can update the examples. Note that due to #9676 this is not tested until we explicitly run the workflow on the next release, but the change is small enough to seem safe to merge. * [cmd/mdatagen] Pull new changes from contrib (#9683) To completely migrate mdatagen from contrib to core, we need to pull latest changes: - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31500 - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31503 - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31520 - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31525 - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31530 - https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/31532 --------- Co-authored-by: Alex Boten <[email protected]> * [chore] fix type in confighttp readme (#9690) This doc incorrectly uses the otlp exporter as an example of using confighttp. That exporter doesn't use confighttp Signed-off-by: Alex Boten <[email protected]> * Update module github.com/stretchr/testify to v1.9.0 (#9695) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/stretchr/testify](https://github.com/stretchr/testify) | `v1.8.4` -> `v1.9.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fstretchr%2ftestify/v1.9.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fstretchr%2ftestify/v1.9.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fstretchr%2ftestify/v1.8.4/v1.9.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fstretchr%2ftestify/v1.8.4/v1.9.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>stretchr/testify (github.com/stretchr/testify)</summary> ### [`v1.9.0`](https://github.com/stretchr/testify/releases/tag/v1.9.0) [Compare Source](https://github.com/stretchr/testify/compare/v1.8.4...v1.9.0) #### What's Changed - Fix Go modules version by [@SuperQ](https://github.com/SuperQ) in [https://github.com/stretchr/testify/pull/1394](https://github.com/stretchr/testify/pull/1394) - Document that require is not safe to call in created goroutines by [@programmer04](https://github.com/programmer04) in [https://github.com/stretchr/testify/pull/1392](https://github.com/stretchr/testify/pull/1392) - Remove myself from MAINTAINERS.md by [@mvdkleijn](https://github.com/mvdkleijn) in [https://github.com/stretchr/testify/pull/1367](https://github.com/stretchr/testify/pull/1367) - Correct spelling/grammar by [@echarrod](https://github.com/echarrod) in [https://github.com/stretchr/testify/pull/1389](https://github.com/stretchr/testify/pull/1389) - docs: Update URLs in README by [@davidjb](https://github.com/davidjb) in [https://github.com/stretchr/testify/pull/1349](https://github.com/stretchr/testify/pull/1349) - Update mockery link to Github Pages in README by [@LandonTClipp](https://github.com/LandonTClipp) in [https://github.com/stretchr/testify/pull/1346](https://github.com/stretchr/testify/pull/1346) - docs: Fix typos in tests and comments by [@alexandear](https://github.com/alexandear) in [https://github.com/stretchr/testify/pull/1410](https://github.com/stretchr/testify/pull/1410) - CI: tests from go1.17 by [@SuperQ](https://github.com/SuperQ) in [https://github.com/stretchr/testify/pull/1409](https://github.com/stretchr/testify/pull/1409) - Fix adding ? when no values passed by [@lesichkovm](https://github.com/lesichkovm) in [https://github.com/stretchr/testify/pull/1320](https://github.com/stretchr/testify/pull/1320) - codegen: use standard header for generated files by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1406](https://github.com/stretchr/testify/pull/1406) - mock: AssertExpectations log reason only on failure by [@hikyaru-suzuki](https://github.com/hikyaru-suzuki) in [https://github.com/stretchr/testify/pull/1360](https://github.com/stretchr/testify/pull/1360) - assert: fix flaky TestNeverTrue by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1417](https://github.com/stretchr/testify/pull/1417) - README: fix typos "set up" vs "setup" by [@ossan-dev](https://github.com/ossan-dev) in [https://github.com/stretchr/testify/pull/1428](https://github.com/stretchr/testify/pull/1428) - mock: move regexp compilation outside of `Called` by [@aud10slave](https://github.com/aud10slave) in [https://github.com/stretchr/testify/pull/631](https://github.com/stretchr/testify/pull/631) - assert: refactor internal func getLen() by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1445](https://github.com/stretchr/testify/pull/1445) - mock: deprecate type AnythingOfTypeArgument ([#1434](https://github.com/stretchr/testify/issues/1434)) by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1441](https://github.com/stretchr/testify/pull/1441) - Remove no longer needed assert.canConvert by [@alexandear](https://github.com/alexandear) in [https://github.com/stretchr/testify/pull/1470](https://github.com/stretchr/testify/pull/1470) - assert: ObjectsAreEqual: use time.Equal for time.Time types by [@tscales](https://github.com/tscales) in [https://github.com/stretchr/testify/pull/1464](https://github.com/stretchr/testify/pull/1464) - Bump actions/checkout from 3 to 4 by [@dependabot](https://github.com/dependabot) in [https://github.com/stretchr/testify/pull/1466](https://github.com/stretchr/testify/pull/1466) - Bump actions/setup-go from 3.2.0 to 4.1.0 by [@dependabot](https://github.com/dependabot) in [https://github.com/stretchr/testify/pull/1451](https://github.com/stretchr/testify/pull/1451) - fix: make EventuallyWithT concurrency safe by [@czeslavo](https://github.com/czeslavo) in [https://github.com/stretchr/testify/pull/1395](https://github.com/stretchr/testify/pull/1395) - assert: fix httpCode and HTTPBody occur panic when http.Handler read Body by [@hidu](https://github.com/hidu) in [https://github.com/stretchr/testify/pull/1484](https://github.com/stretchr/testify/pull/1484) - assert.EqualExportedValues: fix handling of arrays by [@zrbecker](https://github.com/zrbecker) in [https://github.com/stretchr/testify/pull/1473](https://github.com/stretchr/testify/pull/1473) - .github: use latest Go versions by [@kevinburkesegment](https://github.com/kevinburkesegment) in [https://github.com/stretchr/testify/pull/1489](https://github.com/stretchr/testify/pull/1489) - assert: Deprecate EqualExportedValues by [@HaraldNordgren](https://github.com/HaraldNordgren) in [https://github.com/stretchr/testify/pull/1488](https://github.com/stretchr/testify/pull/1488) - suite: refactor test assertions by [@alexandear](https://github.com/alexandear) in [https://github.com/stretchr/testify/pull/1474](https://github.com/stretchr/testify/pull/1474) - suite: fix SetupSubTest and TearDownSubTest execution order by [@linusbarth](https://github.com/linusbarth) in [https://github.com/stretchr/testify/pull/1471](https://github.com/stretchr/testify/pull/1471) - docs: Fix deprecation comments for http package by [@alexandear](https://github.com/alexandear) in [https://github.com/stretchr/testify/pull/1335](https://github.com/stretchr/testify/pull/1335) - Add map support doc comments to Subset and NotSubset by [@jedevc](https://github.com/jedevc) in [https://github.com/stretchr/testify/pull/1306](https://github.com/stretchr/testify/pull/1306) - TestErrorIs/TestNotErrorIs: check error message contents by [@craig65535](https://github.com/craig65535) in [https://github.com/stretchr/testify/pull/1435](https://github.com/stretchr/testify/pull/1435) - suite: fix subtest names (fix [#1501](https://github.com/stretchr/testify/issues/1501)) by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1504](https://github.com/stretchr/testify/pull/1504) - assert: improve unsafe.Pointer tests by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1505](https://github.com/stretchr/testify/pull/1505) - assert: simplify isNil implementation by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1506](https://github.com/stretchr/testify/pull/1506) - assert.InEpsilonSlice: fix expected/actual order and other improvements by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1483](https://github.com/stretchr/testify/pull/1483) - Fix dependency cycle with objx [#1292](https://github.com/stretchr/testify/issues/1292) by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1453](https://github.com/stretchr/testify/pull/1453) - mock: refactor TestIsArgsEqual by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1444](https://github.com/stretchr/testify/pull/1444) - mock: optimize argument matching checks by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1416](https://github.com/stretchr/testify/pull/1416) - assert: fix TestEventuallyTimeout by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1412](https://github.com/stretchr/testify/pull/1412) - CI: add go 1.21 in GitHub Actions by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1450](https://github.com/stretchr/testify/pull/1450) - suite: fix recoverAndFailOnPanic to report test failure at the right location by [@dolmen](https://github.com/dolmen) in [https://github.com/stretchr/testify/pull/1502](https://github.com/stretchr/testify/pull/1502) - Update maintainers by [@brackendawson](https://github.com/brackendawson) in [https://github.com/stretchr/testify/pull/1533](https://github.com/stretchr/testify/pull/1533) - assert: Fix EqualValues to handle overflow/underflow by [@arjunmahishi](https://github.com/arjunmahishi) in [https://github.com/stretchr/testify/pull/1531](https://github.com/stretchr/testify/pull/1531) - assert: better formatting for Len() error by [@kevinburkesegment](https://github.com/kevinburkesegment) in [https://github.com/stretchr/testify/pull/1485](https://github.com/stretchr/testify/pull/1485) - Ensure AssertExpectations does not fail in skipped tests by [@ianrose14](https://github.com/ianrose14) in [https://github.com/stretchr/testify/pull/1331](https://github.com/stretchr/testify/pull/1331) - suite: fix deadlock in suite.Require()/Assert() by [@arjunmahishi](https://github.com/arjunmahishi) in [https://github.com/stretchr/testify/pull/1535](https://github.com/stretchr/testify/pull/1535) - Revert "assert: ObjectsAreEqual: use time.Equal for time.Time type" by [@brackendawson](https://github.com/brackendawson) in [https://github.com/stretchr/testify/pull/1537](https://github.com/stretchr/testify/pull/1537) - \[chore] Add issue templates by [@arjunmahishi](https://github.com/arjunmahishi) in [https://github.com/stretchr/testify/pull/1538](https://github.com/stretchr/testify/pull/1538) - Update the build status badge by [@brackendawson](https://github.com/brackendawson) in [https://github.com/stretchr/testify/pull/1540](https://github.com/stretchr/testify/pull/1540) - Update Github workflows setup-go to V5 by [@hendrywiranto](https://github.com/hendrywiranto) in [https://github.com/stretchr/testify/pull/1545](https://github.com/stretchr/testify/pull/1545) - Support Pointer to Struct in EqualExportedValues by [@Lucaber](https://github.com/Lucaber) in [https://github.com/stretchr/testify/pull/1517](https://github.com/stretchr/testify/pull/1517) - README: drop link to gorc by [@guettli](https://github.com/guettli) in [https://github.com/stretchr/testify/pull/1248](https://github.com/stretchr/testify/pull/1248) - http_assertions: honour the msgAndArgs provided with each assertion by [@arjunmahishi](https://github.com/arjunmahishi) in [https://github.com/stretchr/testify/pull/1548](https://github.com/stretchr/testify/pull/1548) - fix typos in comments and tests by [@ccoVeille](https://github.com/ccoVeille) in [https://github.com/stretchr/testify/pull/1247](https://github.com/stretchr/testify/pull/1247) - Include the auto-release notes in releases by [@brackendawson](https://github.com/brackendawson) in [https://github.com/stretchr/testify/pull/1550](https://github.com/stretchr/testify/pull/1550) - Add `NotImplements` and variants by [@hslatman](https://github.com/hslatman) in [https://github.com/stretchr/testify/pull/1385](https://github.com/stretchr/testify/pull/1385) - Add support to compare uintptr by [@bogdandrutu](https://github.com/bogdandrutu) in [https://github.com/stretchr/testify/pull/1339](https://github.com/stretchr/testify/pull/1339) - build(deps): bump github.com/stretchr/objx from 0.5.1 to 0.5.2 by [@dependabot](https://github.com/dependabot) in [https://github.com/stretchr/testify/pull/1552](https://github.com/stretchr/testify/pull/1552) #### New Contributors - [@SuperQ](https://github.com/SuperQ) made their first contribution in [https://github.com/stretchr/testify/pull/1394](https://github.com/stretchr/testify/pull/1394) - [@programmer04](https://github.com/programmer04) made their first contribution in [https://github.com/stretchr/testify/pull/1392](https://github.com/stretchr/testify/pull/1392) - [@echarrod](https://github.com/echarrod) made their first contribution in [https://github.com/stretchr/testify/pull/1389](https://github.com/stretchr/testify/pull/1389) - [@davidjb](https://github.com/davidjb) made their first contribution in [https://github.com/stretchr/testify/pull/1349](https://github.com/stretchr/testify/pull/1349) - [@LandonTClipp](https://github.com/LandonTClipp) made their first contribution in [https://github.com/stretchr/testify/pull/1346](https://github.com/stretchr/testify/pull/1346) - [@alexandear](https://github.com/alexandear) made their first contribution in [https://github.com/stretchr/testify/pull/1410](https://github.com/stretchr/testify/pull/1410) - [@lesichkovm](https://github.com/lesichkovm) made their first contribution in [https://github.com/stretchr/testify/pull/1320](https://github.com/stretchr/testify/pull/1320) - [@dolmen](https://github.com/dolmen) made their first contribution in [https://github.com/stretchr/testify/pull/1406](https://github.com/stretchr/testify/pull/1406) - [@hikyaru-suzuki](https://github.com/hikyaru-suzuki) made their first contribution in [https://github.com/stretchr/testify/pull/1360](https://github.com/stretchr/testify/pull/1360) - [@ossan-dev](https://github.com/ossan-dev) made their first contribution in [https://github.com/stretchr/testify/pull/1428](https://github.com/stretchr/testify/pull/1428) - [@aud10slave](https://github.com/aud10slave) made their first contribution in [https://github.com/stretchr/testify/pull/631](https://github.com/stretchr/testify/pull/631) - [@tscales](https://github.com/tscales) made their first contribution in [https://github.com/stretchr/testify/pull/1464](https://github.com/stretchr/testify/pull/1464) - [@czeslavo](https://github.com/czeslavo) made their first contribution in [https://github.com/stretchr/testify/pull/1395](https://github.com/stretchr/testify/pull/1395) - [@hidu](https://github.com/hidu) made their first contribution in [https://github.com/stretchr/testify/pull/1484](https://github.com/stretchr/testify/pull/1484) - [@zrbecker](https://github.com/zrbecker) made their first contribution in [https://github.com/stretchr/testify/pull/1473](https://github.com/stretchr/testify/pull/1473) - [@kevinburkesegment](https://github.com/kevinburkesegment) made their first contribution in [https://github.com/stretchr/testify/pull/1489](https://github.com/stretchr/testify/pull/1489) - [@linusbarth](https://github.com/linusbarth) made their first contribution in [https://github.com/stretchr/testify/pull/1471](https://github.com/stretchr/testify/pull/1471) - [@jedevc](https://github.com/jedevc) made their first contribution in [https://github.com/stretchr/testify/pull/1306](https://github.com/stretchr/testify/pull/1306) - [@craig65535](https://github.com/craig65535) made their first contribution in [https://github.com/stretchr/testify/pull/1435](https://github.com/stretchr/testify/pull/1435) - [@arjunmahishi](https://github.com/arjunmahishi) made their first contribution in [https://github.com/stretchr/testify/pull/1531](https://github.com/stretchr/testify/pull/1531) - [@ianrose14](https://github.com/ianrose14) made their first contribution in [https://github.com/stretchr/testify/pull/1331](https://github.com/stretchr/testify/pull/1331) - [@hendrywiranto](https://github.com/hendrywiranto) made their first contribution in [https://github.com/stretchr/testify/pull/1545](https://github.com/stretchr/testify/pull/1545) - [@Lucaber](https://github.com/Lucaber) made their first contribution in [https://github.com/stretchr/testify/pull/1517](https://github.com/stretchr/testify/pull/1517) - [@guettli](https://github.com/guettli) made their first contribution in [https://github.com/stretchr/testify/pull/1248](https://github.com/stretchr/testify/pull/1248) - [@ccoVeille](https://github.com/ccoVeille) made their first contribution in [https://github.com/stretchr/testify/pull/1247](https://github.com/stretchr/testify/pull/1247) - [@hslatman](https://github.com/hslatman) made their first contribution in [https://github.com/stretchr/testify/pull/1385](https://github.com/stretchr/testify/pull/1385) - [@bogdandrutu](https://github.com/bogdandrutu) made their first contribution in [https://github.com/stretchr/testify/pull/1339](https://github.com/stretchr/testify/pull/1339) **Full Changelog**: https://github.com/stretchr/testify/compare/v1.8.4...v1.9.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * Update module go.opentelemetry.io/build-tools/semconvgen to v0.13.0 (#9701) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [go.opentelemetry.io/build-tools/semconvgen](https://github.com/open-telemetry/opentelemetry-go-build-tools) | `v0.12.0` -> `v0.13.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fbuild-tools%2fsemconvgen/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/go.opentelemetry.io%2fbuild-tools%2fsemconvgen/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/go.opentelemetry.io%2fbuild-tools%2fsemconvgen/v0.12.0/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fbuild-tools%2fsemconvgen/v0.12.0/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>open-telemetry/opentelemetry-go-build-tools (go.opentelemetry.io/build-tools/semconvgen)</summary> ### [`v0.13.0`](https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/HEAD/CHANGELOG.md#v0130) [Compare Source](https://github.com/open-telemetry/opentelemetry-go-build-tools/compare/v0.12.0...v0.13.0) ##### 🛑 Breaking changes 🛑 - `all`: bump minimal Go version to 1.20 ([#474](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/474)) ##### 💡 Enhancements 💡 - `multimod`: ignore excluded-modules when using sync to update dependencies ([#442](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/442)) This allows users of the sync command to sync all modules in a monorepo, including those listed in the excluded-modules. This is useful for repositories where some modules may not yet be ready for releasing (therefore listed under excluded-modules) but their dependencies still need to be managed via multimod. - `crosslink`: Add `--skip` flag to ignore specified go modules ([#480](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/480)) - `multimod`: add support for `--commit-hash` to allow users to overwrite the tag in a versions.yaml file ([#422](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/422)) This feature allows users to specify a tag (i.e. main) when they want to update modules. - `chloggen`: support a custom changelog summary template ([#501](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/501)) The changelog summary template can be customized by configuring a custom template with the `summary_template` configuration setting. The default template provides a starting point for a custom template: https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/v0.13.0/chloggen/internal/chlog/summary.tmpl ##### 🧰 Bug fixes 🧰 - `crosslink`: Fix Windows produces backslashes instead of slashes ([#458](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/458)) This fixes the issue of Windows produces backslashes instead of slashes when crosslinking dependencies in go.mod files on Windows. - `dbotconf`: Fix Windows produces backslashes instead of slashes ([#264](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/264)) This fixes the issue of Windows produces backslashes instead of slashes when generating Dependabot configuration files on Windows. - `multimod`: Fix tagging on Windows ([#464](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/464)) This fixes the issue of `multimod tag` failing on Windows. - `multimod`: Fix to log 'Using versioning file' and 'Successfully deleted module tags' to stderr instead of stdout ([#507](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/507)) - `chloggen`: change generated files permissions from 0755 to 0644 ([#457](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/457)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * Update module golang.org/x/tools to v0.19.0 (#9708) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | golang.org/x/tools | `v0.18.0` -> `v0.19.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/golang.org%2fx%2ftools/v0.19.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/golang.org%2fx%2ftools/v0.19.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/golang.org%2fx%2ftools/v0.18.0/v0.19.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/golang.org%2fx%2ftools/v0.18.0/v0.19.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * [cmd/mdatagen] Update the scope name generation method (#9693) Don't use hardcoded "go.opentelemetry.io/collector" prefix. Provide a way to specify the `scope_name` in metadata.yaml. If not provided, try to use the go package name. Updates https://github.com/open-telemetry/opentelemetry-collector/issues/9494 * Update module go.opentelemetry.io/collector/exporter/otlphttpexporter to v0.96.0 (#9703) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [go.opentelemetry.io/collector/exporter/otlphttpexporter](https://github.com/open-telemetry/opentelemetry-collector) | `v0.95.0` -> `v0.96.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlphttpexporter/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlphttpexporter/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlphttpexporter/v0.95.0/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlphttpexporter/v0.95.0/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>open-telemetry/opentelemetry-collector (go.opentelemetry.io/collector/exporter/otlphttpexporter)</summary> ### [`v0.96.0`](https://github.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v130v0960) [Compare Source](https://github.com/open-telemetry/opentelemetry-collector/compare/v0.95.0...v0.96.0) ##### 🛑 Breaking changes 🛑 - `configgrpc`: Remove deprecated `GRPCClientSettings`, `GRPCServerSettings`, and `ServerConfig.ToListenerContext`. ([#9616](https://github.com/open-telemetry/opentelemetry-collector/issues/9616)) - `confighttp`: Remove deprecated `HTTPClientSettings`, `NewDefaultHTTPClientSettings`, and `CORSSettings`. ([#9625](https://github.com/open-telemetry/opentelemetry-collector/issues/9625)) - `confignet`: Removes deprecated `NetAddr` and `TCPAddr` ([#9614](https://github.com/open-telemetry/opentelemetry-collector/issues/9614)) ##### 💡 Enhancements 💡 - `configtls`: Add `include_system_ca_certs_pool` to configtls, allowing to load system certs and additional custom certs. ([#7774](https://github.com/open-telemetry/opentelemetry-collector/issues/7774)) - `otelcol`: Add `ConfigProviderSettings` to `CollectorSettings` ([#4759](https://github.com/open-telemetry/opentelemetry-collector/issues/4759)) This allows passing a custom list of `confmap.Provider`s to `otelcol.NewCommand`. - `pdata`: Update to OTLP v1.1.0 ([#9587](https://github.com/open-telemetry/opentelemetry-collector/issues/9587)) Introduces Span and SpanLink flags. - `confmap`: Update mapstructure to use a maintained fork, github.com/go-viper/mapstructure/v2. ([#9634](https://github.com/open-telemetry/opentelemetry-collector/issues/9634)) [https://github.com/mitchellh/mapstructure/issues/349](https://github.com/mitchellh/mapstructure/issues/349)/349 for context. ##### 🧰 Bug fixes 🧰 - `configretry`: Allow max_elapsed_time to be set to 0 for indefinite retries ([#9641](https://github.com/open-telemetry/opentelemetry-collector/issues/9641)) - `client`: Make `Metadata.Get` thread safe ([#9595](https://github.com/open-telemetry/opentelemetry-collector/issues/9595)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update module go.opentelemetry.io/collector/receiver/otlpreceiver to v0.96.0 (#9704) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [go.opentelemetry.io/collector/receiver/otlpreceiver](https://github.com/open-telemetry/opentelemetry-collector) | `v0.95.0` -> `v0.96.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2freceiver%2fotlpreceiver/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/go.opentelemetry.io%2fcollector%2freceiver%2fotlpreceiver/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/go.opentelemetry.io%2fcollector%2freceiver%2fotlpreceiver/v0.95.0/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2freceiver%2fotlpreceiver/v0.95.0/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>open-telemetry/opentelemetry-collector (go.opentelemetry.io/collector/receiver/otlpreceiver)</summary> ### [`v0.96.0`](https://github.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v130v0960) [Compare Source](https://github.com/open-telemetry/opentelemetry-collector/compare/v0.95.0...v0.96.0) ##### 🛑 Breaking changes 🛑 - `configgrpc`: Remove deprecated `GRPCClientSettings`, `GRPCServerSettings`, and `ServerConfig.ToListenerContext`. ([#9616](https://github.com/open-telemetry/opentelemetry-collector/issues/9616)) - `confighttp`: Remove deprecated `HTTPClientSettings`, `NewDefaultHTTPClientSettings`, and `CORSSettings`. ([#9625](https://github.com/open-telemetry/opentelemetry-collector/issues/9625)) - `confignet`: Removes deprecated `NetAddr` and `TCPAddr` ([#9614](https://github.com/open-telemetry/opentelemetry-collector/issues/9614)) ##### 💡 Enhancements 💡 - `configtls`: Add `include_system_ca_certs_pool` to configtls, allowing to load system certs and additional custom certs. ([#7774](https://github.com/open-telemetry/opentelemetry-collector/issues/7774)) - `otelcol`: Add `ConfigProviderSettings` to `CollectorSettings` ([#4759](https://github.com/open-telemetry/opentelemetry-collector/issues/4759)) This allows passing a custom list of `confmap.Provider`s to `otelcol.NewCommand`. - `pdata`: Update to OTLP v1.1.0 ([#9587](https://github.com/open-telemetry/opentelemetry-collector/issues/9587)) Introduces Span and SpanLink flags. - `confmap`: Update mapstructure to use a maintained fork, github.com/go-viper/mapstructure/v2. ([#9634](https://github.com/open-telemetry/opentelemetry-collector/issues/9634)) [https://github.com/mitchellh/mapstructure/issues/349](https://github.com/mitchellh/mapstructure/issues/349)/349 for context. ##### 🧰 Bug fixes 🧰 - `configretry`: Allow max_elapsed_time to be set to 0 for indefinite retries ([#9641](https://github.com/open-telemetry/opentelemetry-collector/issues/9641)) - `client`: Make `Metadata.Get` thread safe ([#9595](https://github.com/open-telemetry/opentelemetry-collector/issues/9595)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * [chore] [cmd/mdatagen] Update status.go template (#9713) To produce the same output as mdatagen in contrib. It makes it easy to compare the diff for the mdatagen migration. * Update module golang.org/x/sys to v0.18.0 (#9706) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | golang.org/x/sys | `v0.17.0` -> `v0.18.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/golang.org%2fx%2fsys/v0.18.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/golang.org%2fx%2fsys/v0.18.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/golang.org%2fx%2fsys/v0.17.0/v0.18.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/golang.org%2fx%2fsys/v0.17.0/v0.18.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> Co-authored-by: Alex Boten <[email protected]> * Update module google.golang.org/protobuf to v1.33.0 (#9712) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [google.golang.org/protobuf](https://github.com/protocolbuffers/protobuf-go) | `v1.32.0` -> `v1.33.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/google.golang.org%2fprotobuf/v1.33.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/google.golang.org%2fprotobuf/v1.33.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/google.golang.org%2fprotobuf/v1.32.0/v1.33.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/google.golang.org%2fprotobuf/v1.32.0/v1.33.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * Update module go.opentelemetry.io/collector/exporter/otlpexporter to v0.96.0 (#9702) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [go.opentelemetry.io/collector/exporter/otlpexporter](https://github.com/open-telemetry/opentelemetry-collector) | `v0.95.0` -> `v0.96.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlpexporter/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlpexporter/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlpexporter/v0.95.0/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fcollector%2fexporter%2fotlpexporter/v0.95.0/v0.96.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>open-telemetry/opentelemetry-collector (go.opentelemetry.io/collector/exporter/otlpexporter)</summary> ### [`v0.96.0`](https://github.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v130v0960) [Compare Source](https://github.com/open-telemetry/opentelemetry-collector/compare/v0.95.0...v0.96.0) ##### 🛑 Breaking changes 🛑 - `configgrpc`: Remove deprecated `GRPCClientSettings`, `GRPCServerSettings`, and `ServerConfig.ToListenerContext`. ([#9616](https://github.com/open-telemetry/opentelemetry-collector/issues/9616)) - `confighttp`: Remove deprecated `HTTPClientSettings`, `NewDefaultHTTPClientSettings`, and `CORSSettings`. ([#9625](https://github.com/open-telemetry/opentelemetry-collector/issues/9625)) - `confignet`: Removes deprecated `NetAddr` and `TCPAddr` ([#9614](https://github.com/open-telemetry/opentelemetry-collector/issues/9614)) ##### 💡 Enhancements 💡 - `configtls`: Add `include_system_ca_certs_pool` to configtls, allowing to load system certs and additional custom certs. ([#7774](https://github.com/open-telemetry/opentelemetry-collector/issues/7774)) - `otelcol`: Add `ConfigProviderSettings` to `CollectorSettings` ([#4759](https://github.com/open-telemetry/opentelemetry-collector/issues/4759)) This allows passing a custom list of `confmap.Provider`s to `otelcol.NewCommand`. - `pdata`: Update to OTLP v1.1.0 ([#9587](https://github.com/open-telemetry/opentelemetry-collector/issues/9587)) Introduces Span and SpanLink flags. - `confmap`: Update mapstructure to use a maintained fork, github.com/go-viper/mapstructure/v2. ([#9634](https://github.com/open-telemetry/opentelemetry-collector/issues/9634)) [https://github.com/mitchellh/mapstructure/issues/349](https://github.com/mitchellh/mapstructure/issues/349)/349 for context. ##### 🧰 Bug fixes 🧰 - `configretry`: Allow max_elapsed_time to be set to 0 for indefinite retries ([#9641](https://github.com/open-telemetry/opentelemetry-collector/issues/9641)) - `client`: Make `Metadata.Get` thread safe ([#9595](https://github.com/open-telemetry/opentelemetry-collector/issues/9595)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Alex Boten <[email protected]> * Update github-actions deps (#9691) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/cache](https://github.com/actions/cache) | action | patch | `v4.0.0` -> `v4.0.1` | | [github/codeql-action](https://github.com/github/codeql-action) | action | patch | `v3.24.5` -> `v3.24.6` | --- ### Release Notes <details> <summary>actions/cache (actions/cache)</summary> ### [`v4.0.1`](https://github.com/actions/cache/releases/tag/v4.0.1) [Compare Source](https://github.com/actions/cache/compare/v4.0.0...v4.0.1) ##### What's Changed - Update README.md by [@yacaovsnc](https://github.com/yacaovsnc) in [https://github.com/actions/cache/pull/1304](https://github.com/actions/cache/pull/1304) - Update examples by [@yacaovsnc](https://github.com/yacaovsnc) in [https://github.com/actions/cache/pull/1305](https://github.com/actions/cache/pull/1305) - Update actions/cache publish flow by [@bethanyj28](https://github.com/bethanyj28) in [https://github.com/actions/cache/pull/1340](https://github.com/actions/cache/pull/1340) - Update [@actions/cache](https://github.com/actions/cache) by [@bethanyj28](https://github.com/bethanyj28) in [https://github.com/actions/cache/pull/1341](https://github.com/actions/cache/pull/1341) ##### New Contributors - [@yacaovsnc](https://github.com/yacaovsnc) made their first contribution in [https://github.com/actions/cache/pull/1304](https://github.com/actions/cache/pull/1304) **Full Changelog**: https://github.com/actions/cache/compare/v4...v4.0.1 </details> <details> <summary>github/codeql-action (github/codeql-action)</summary> ### [`v3.24.6`](https://github.com/github/codeql-action/compare/v3.24.5...v3.24.6) [Compare Source](https://github.com/github/codeql-action/compare/v3.24.5...v3.24.6) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update module golang.org/x/net to v0.22.0 (#9705) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | golang.org/x/net | `v0.21.0` -> `v0.22.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/golang.org%2fx%2fnet/v0.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/golang.org%2fx%2fnet/v0.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/golang.org%2fx%2fnet/v0.21.0/v0.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/golang.org%2fx%2fnet/v0.21.0/v0.22.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * Update module google.golang.org/grpc to v1.62.1 (#9711) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [google.golang.org/grpc](https://github.com/grpc/grpc-go) | `v1.62.0` -> `v1.62.1` | [![age](https://developer.mend.io/api/mc/badges/age/go/google.golang.org%2fgrpc/v1.62.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/google.golang.org%2fgrpc/v1.62.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/google.golang.org%2fgrpc/v1.62.0/v1.62.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/google.golang.org%2fgrpc/v1.62.0/v1.62.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>grpc/grpc-go (google.golang.org/grpc)</summary> ### [`v1.62.1`](https://github.com/grpc/grpc-go/releases/tag/v1.62.1): Release 1.62.1 [Compare Source](https://github.com/grpc/grpc-go/compare/v1.62.0...v1.62.1) ### Bug Fixes - xds: fix a bug that results in `no matching virtual host found` RPC errors due to a difference between the target and LDS resource names ([#6997](https://github.com/grpc/grpc-go/issues/6997)) - server: fixed stats handler data `InPayload.Length` for unary RPC calls ([#6766](https://github.com/grpc/grpc-go/issues/6766)) - Special Thanks: [@hueypark](https://github.com/hueypark) - grpc: the experimental `RecvBufferPool` `DialOption` and `ServerOption` are now active during unary RPCs with compression ([#6766](https://github.com/grpc/grpc-go/issues/6766)) - Special Thanks: [@hueypark](https://github.com/hueypark) - grpc: trim whitespaces in `accept-encoding` header before determining compressors - Special Thanks: [@sercand](https://github.com/sercand) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * Update module go.opentelemetry.io/build-tools/multimod to v0.13.0 (#9700) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [go.opentelemetry.io/build-tools/multimod](https://github.com/open-telemetry/opentelemetry-go-build-tools) | `v0.12.0` -> `v0.13.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fbuild-tools%2fmultimod/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/go.opentelemetry.io%2fbuild-tools%2fmultimod/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/go.opentelemetry.io%2fbuild-tools%2fmultimod/v0.12.0/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fbuild-tools%2fmultimod/v0.12.0/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>open-telemetry/opentelemetry-go-build-tools (go.opentelemetry.io/build-tools/multimod)</summary> ### [`v0.13.0`](https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/HEAD/CHANGELOG.md#v0130) [Compare Source](https://github.com/open-telemetry/opentelemetry-go-build-tools/compare/v0.12.0...v0.13.0) ##### 🛑 Breaking changes 🛑 - `all`: bump minimal Go version to 1.20 ([#474](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/474)) ##### 💡 Enhancements 💡 - `multimod`: ignore excluded-modules when using sync to update dependencies ([#442](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/442)) This allows users of the sync command to sync all modules in a monorepo, including those listed in the excluded-modules. This is useful for repositories where some modules may not yet be ready for releasing (therefore listed under excluded-modules) but their dependencies still need to be managed via multimod. - `crosslink`: Add `--skip` flag to ignore specified go modules ([#480](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/480)) - `multimod`: add support for `--commit-hash` to allow users to overwrite the tag in a versions.yaml file ([#422](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/422)) This feature allows users to specify a tag (i.e. main) when they want to update modules. - `chloggen`: support a custom changelog summary template ([#501](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/501)) The changelog summary template can be customized by configuring a custom template with the `summary_template` configuration setting. The default template provides a starting point for a custom template: https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/v0.13.0/chloggen/internal/chlog/summary.tmpl ##### 🧰 Bug fixes 🧰 - `crosslink`: Fix Windows produces backslashes instead of slashes ([#458](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/458)) This fixes the issue of Windows produces backslashes instead of slashes when crosslinking dependencies in go.mod files on Windows. - `dbotconf`: Fix Windows produces backslashes instead of slashes ([#264](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/264)) This fixes the issue of Windows produces backslashes instead of slashes when generating Dependabot configuration files on Windows. - `multimod`: Fix tagging on Windows ([#464](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/464)) This fixes the issue of `multimod tag` failing on Windows. - `multimod`: Fix to log 'Using versioning file' and 'Successfully deleted module tags' to stderr instead of stdout ([#507](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/507)) - `chloggen`: change generated files permissions from 0755 to 0644 ([#457](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/457)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <[email protected]> * Update module go.opentelemetry.io/build-tools/crosslink to v0.13.0 (#9699) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [go.opentelemetry.io/build-tools/crosslink](https://github.com/open-telemetry/opentelemetry-go-build-tools) | `v0.12.1-0.20240121161735-d70c842b1bf5` -> `v0.13.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/go.opentelemetry.io%2fbuild-tools%2fcrosslink/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/go.opentelemetry.io%2fbuild-tools%2fcrosslink/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/go.opentelemetry.io%2fbuild-tools%2fcrosslink/v0.12.1-0.20240121161735-d70c842b1bf5/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/go.opentelemetry.io%2fbuild-tools%2fcrosslink/v0.12.1-0.20240121161735-d70c842b1bf5/v0.13.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>open-telemetry/opentelemetry-go-build-tools (go.opentelemetry.io/build-tools/crosslink)</summary> ### [`v0.13.0`](https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/HEAD/CHANGELOG.md#v0130) [Compare Source](https://github.com/open-telemetry/opentelemetry-go-build-tools/compare/v0.12.0...v0.13.0) ##### 🛑 Breaking changes 🛑 - `all`: bump minimal Go version to 1.20 ([#474](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/474)) ##### 💡 Enhancements 💡 - `multimod`: ignore excluded-modules when using sync to update dependencies ([#442](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/442)) This allows users of the sync command to sync all modules in a monorepo, including those listed in the excluded-modules. This is useful for repositories where some modules may not yet be ready for releasing (therefore listed under excluded-modules) but their dependencies still need to be managed via multimod. - `crosslink`: Add `--skip` flag to ignore specified go modules ([#480](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/480)) - `multimod`: add support for `--commit-hash` to allow users to overwrite the tag in a versions.yaml file ([#422](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/422)) This feature allows users to specify a tag (i.e. main) when they want to update modules. - `chloggen`: support a custom changelog summary template ([#501](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/501)) The changelog summary template can be customized by configuring a custom template with the `summary_template` configuration setting. The default template provides a starting point for a custom template: https://github.com/open-telemetry/opentelemetry-go-build-tools/blob/v0.13.0/chloggen/internal/chlog/summary.tmpl ##### 🧰 Bug fixes 🧰 - `crosslink`: Fix Windows produces backslashes instead of slashes ([#458](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/458)) This fixes the issue of Windows produces backslashes instead of slashes when crosslinking dependencies in go.mod files on Windows. - `dbotconf`: Fix Windows produces backslashes instead of slashes ([#264](https://github.com/open-telemetry/opentelemetry-go-build-tools/issues/264)) This fixes the issue of Windows produces backslashes instead of slashes when generating …

Without this mocks are not verified as expected (every table-test case'). While this would work, one might expect behavior like ``` for _, tt := range testcases { t.Run(tt.name, func(t* testing.T) { init-controller-and-mocks // test-code // mocks are verified here, at the end of subtest } } ``` Unfortunately, that's not the behavior you get with `s.Run(subtest func())` as controller is not re-initialized. In order to addres this gap testify introduced `SetupSubTest/TearDownSubTest` in v1.8.2 See stretchr/testify#1246 for more details SubTest behavior was fixed in v1.9.0 - otherwise failed mock expectations made parent test to fail, but child still succeeded. See stretchr/testify#1471 for more details

dolmen added pkg-suite Change related to package testify/suite bug labels Oct 10, 2023

dolmen requested changes Oct 10, 2023

View reviewed changes

dolmen added the hacktoberfest-accepted Hacktoberfest label Oct 10, 2023

dolmen requested changes Oct 10, 2023

View reviewed changes

dolmen changed the title ~~fix: SetupSubTest and TearDownSubTest execution order~~ suite: fix SetupSubTest and TearDownSubTest execution order Oct 10, 2023

dolmen force-pushed the fix-setup-and-teardown-subtest-execution-order branch from 4f8dd94 to 7d0eaf9 Compare October 15, 2023 22:35

dolmen requested changes Oct 15, 2023

View reviewed changes

linusbarth force-pushed the fix-setup-and-teardown-subtest-execution-order branch 3 times, most recently from 33e63d9 to c46c72b Compare October 16, 2023 00:59

linusbarth added 5 commits October 16, 2023 03:04

improve: defer-style in Suite.Run()

83e44bc

test: call order of setup/teardown for subtests

4657972

test: testing.T correctness in subtests setup/teardown

aa01ed0

improve: move comment to msgAndArgs-param in test

ac5cd69

linusbarth force-pushed the fix-setup-and-teardown-subtest-execution-order branch from c46c72b to ac5cd69 Compare October 16, 2023 01:04

This was referenced Oct 16, 2023

suite: fix test failures #1421

Merged

Add flag -testify.shuffle for suites #1413

Open

Ensure that subtest name is set in SetupSubTest/TearDownSubTest #1393

Closed

thekidder approved these changes Oct 16, 2023

View reviewed changes

dolmen mentioned this pull request Oct 16, 2023

suite: recover and fail on subtest #1455

Closed

dolmen previously approved these changes Oct 16, 2023

View reviewed changes

tomhutch reviewed Oct 16, 2023

View reviewed changes

improve: tests for asserting test names in subtests

c8efd3c

linusbarth dismissed dolmen’s stale review via c8efd3c October 17, 2023 14:36

xwjdsh reviewed Oct 17, 2023

View reviewed changes

suite/suite.go Show resolved Hide resolved

fix: panic behavior for subtests

91478fe

This fix adds panic handling for subtests which will achieve: - subtests will fail for the correct test context when panicking - the test execution is not stopped; the next subtest will be executed

renovate bot mentioned this pull request Apr 3, 2024

Update all non-major dependencies bitrise-io/bitrise-workflow-editor#823

Merged

1 task

renovate bot mentioned this pull request Jul 22, 2024

chore(deps): update all moul/u#74

Open

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

suite: fix SetupSubTest and TearDownSubTest execution order #1471

suite: fix SetupSubTest and TearDownSubTest execution order #1471

linusbarth commented Sep 15, 2023

dolmen Oct 10, 2023 •

edited

Loading

linusbarth Oct 10, 2023

dolmen left a comment

linusbarth commented Oct 10, 2023

dolmen commented Oct 15, 2023

dolmen Oct 15, 2023

linusbarth Oct 16, 2023

dolmen Oct 15, 2023

linusbarth Oct 16, 2023

tomhutch left a comment

tomhutch Oct 16, 2023

linusbarth Oct 17, 2023 •

edited

Loading

	SetupSuiteRunCount int
	TearDownSuiteRunCount int
	SetupTestRunCount int
	TearDownTestRunCount int
	TestOneRunCount int
	TestTwoRunCount int
	TestSubtestRunCount int
	NonTestMethodRunCount int
	SetupSubTestRunCount int
	TearDownSubTestRunCount int

	SuiteNameBefore []string
	TestNameBefore []string

	SuiteNameAfter []string
	TestNameAfter []string

	TimeBefore []time.Time
	TimeAfter []time.Time

suite: fix SetupSubTest and TearDownSubTest execution order #1471

suite: fix SetupSubTest and TearDownSubTest execution order #1471

Conversation

linusbarth commented Sep 15, 2023

Summary

Changes

Motivation

Related issues

dolmen Oct 10, 2023 • edited Loading

Choose a reason for hiding this comment

linusbarth Oct 10, 2023

Choose a reason for hiding this comment

dolmen left a comment

Choose a reason for hiding this comment

linusbarth commented Oct 10, 2023

dolmen commented Oct 15, 2023

dolmen Oct 15, 2023

Choose a reason for hiding this comment

linusbarth Oct 16, 2023

Choose a reason for hiding this comment

dolmen Oct 15, 2023

Choose a reason for hiding this comment

linusbarth Oct 16, 2023

Choose a reason for hiding this comment

tomhutch left a comment

Choose a reason for hiding this comment

tomhutch Oct 16, 2023

Choose a reason for hiding this comment

linusbarth Oct 17, 2023 • edited Loading

Choose a reason for hiding this comment

dolmen Oct 10, 2023 •

edited

Loading

linusbarth Oct 17, 2023 •

edited

Loading