chore: use require.Len instead of assert.Len

testcontainers / testcontainers-go

Testcontainers for Go is a Go package that makes it simple to create and clean up container-based dependencies for automated integration/smoke tests. The clean, easy-to-use API enables developers to programmatically define containers that should be run as part of a test and clean up those resources when the test is done.

https://golang.testcontainers.org

MIT License

3.68k stars 504 forks source link

chore: use require.Len instead of assert.Len #2854

Closed mmorel-35 closed 3 weeks ago

mmorel-35 commented 3 weeks ago

What does this PR do?

This uses require.Len instead of assert.Len calls when element is reused for tests.

Why is it important?

When assert.Len is used, this is generally to check the length of a list that is generally used in at least one other test.

Related issues

Relates #2808

netlify[bot] commented 3 weeks ago

Deploy Preview for testcontainers-go ready!

Name	Link
Latest commit	8ccc448987651164ea5e1284e0d48ad194bee29d
Latest deploy log	https://app.netlify.com/sites/testcontainers-go/deploys/67207ce720375000087728c9
Deploy Preview	https://deploy-preview-2854--testcontainers-go.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.