mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-11-14 02:10:21 +00:00
rebase: Bump github.com/onsi/ginkgo/v2 from 2.10.0 to 2.11.0
Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.10.0 to 2.11.0. - [Release notes](https://github.com/onsi/ginkgo/releases) - [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md) - [Commits](https://github.com/onsi/ginkgo/compare/v2.10.0...v2.11.0) --- updated-dependencies: - dependency-name: github.com/onsi/ginkgo/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
This commit is contained in:
parent
029276b569
commit
e8f1bbe36b
2
go.mod
2
go.mod
@ -22,7 +22,7 @@ require (
|
|||||||
github.com/kubernetes-csi/csi-lib-utils v0.14.0
|
github.com/kubernetes-csi/csi-lib-utils v0.14.0
|
||||||
github.com/kubernetes-csi/external-snapshotter/client/v6 v6.2.0
|
github.com/kubernetes-csi/external-snapshotter/client/v6 v6.2.0
|
||||||
github.com/libopenstorage/secrets v0.0.0-20210908194121-a1d19aa9713a
|
github.com/libopenstorage/secrets v0.0.0-20210908194121-a1d19aa9713a
|
||||||
github.com/onsi/ginkgo/v2 v2.10.0
|
github.com/onsi/ginkgo/v2 v2.11.0
|
||||||
github.com/onsi/gomega v1.27.8
|
github.com/onsi/gomega v1.27.8
|
||||||
github.com/pkg/xattr v0.4.9
|
github.com/pkg/xattr v0.4.9
|
||||||
github.com/prometheus/client_golang v1.16.0
|
github.com/prometheus/client_golang v1.16.0
|
||||||
|
4
go.sum
4
go.sum
@ -915,8 +915,8 @@ github.com/onsi/ginkgo/v2 v2.7.0/go.mod h1:yjiuMwPokqY1XauOgju45q3sJt6VzQ/Fict1L
|
|||||||
github.com/onsi/ginkgo/v2 v2.8.1/go.mod h1:N1/NbDngAFcSLdyZ+/aYTYGSlq9qMCS/cNKGJjy+csc=
|
github.com/onsi/ginkgo/v2 v2.8.1/go.mod h1:N1/NbDngAFcSLdyZ+/aYTYGSlq9qMCS/cNKGJjy+csc=
|
||||||
github.com/onsi/ginkgo/v2 v2.9.0/go.mod h1:4xkjoL/tZv4SMWeww56BU5kAt19mVB47gTWxmrTcxyk=
|
github.com/onsi/ginkgo/v2 v2.9.0/go.mod h1:4xkjoL/tZv4SMWeww56BU5kAt19mVB47gTWxmrTcxyk=
|
||||||
github.com/onsi/ginkgo/v2 v2.9.1/go.mod h1:FEcmzVcCHl+4o9bQZVab+4dC9+j+91t2FHSzmGAPfuo=
|
github.com/onsi/ginkgo/v2 v2.9.1/go.mod h1:FEcmzVcCHl+4o9bQZVab+4dC9+j+91t2FHSzmGAPfuo=
|
||||||
github.com/onsi/ginkgo/v2 v2.10.0 h1:sfUl4qgLdvkChZrWCYndY2EAu9BRIw1YphNAzy1VNWs=
|
github.com/onsi/ginkgo/v2 v2.11.0 h1:WgqUCUt/lT6yXoQ8Wef0fsNn5cAuMK7+KT9UFRz2tcU=
|
||||||
github.com/onsi/ginkgo/v2 v2.10.0/go.mod h1:UDQOh5wbQUlMnkLfVaIUMtQ1Vus92oM+P2JX1aulgcE=
|
github.com/onsi/ginkgo/v2 v2.11.0/go.mod h1:ZhrRA5XmEE3x3rhlzamx/JJvujdZoJ2uvgI7kR0iZvM=
|
||||||
github.com/onsi/gomega v1.4.2/go.mod h1:ex+gbHU/CVuBBDIJjb2X0qEXbFg53c61hWP/1CpauHY=
|
github.com/onsi/gomega v1.4.2/go.mod h1:ex+gbHU/CVuBBDIJjb2X0qEXbFg53c61hWP/1CpauHY=
|
||||||
github.com/onsi/gomega v1.4.3/go.mod h1:ex+gbHU/CVuBBDIJjb2X0qEXbFg53c61hWP/1CpauHY=
|
github.com/onsi/gomega v1.4.3/go.mod h1:ex+gbHU/CVuBBDIJjb2X0qEXbFg53c61hWP/1CpauHY=
|
||||||
github.com/onsi/gomega v1.5.0/go.mod h1:ex+gbHU/CVuBBDIJjb2X0qEXbFg53c61hWP/1CpauHY=
|
github.com/onsi/gomega v1.5.0/go.mod h1:ex+gbHU/CVuBBDIJjb2X0qEXbFg53c61hWP/1CpauHY=
|
||||||
|
18
vendor/github.com/onsi/ginkgo/v2/CHANGELOG.md
generated
vendored
18
vendor/github.com/onsi/ginkgo/v2/CHANGELOG.md
generated
vendored
@ -1,3 +1,21 @@
|
|||||||
|
## 2.11.0
|
||||||
|
|
||||||
|
In prior versions of Ginkgo specs the CLI filter flags (e.g. `--focus`, `--label-filter`) would _override_ any programmatic focus. This behavior has proved surprising and confusing in at least the following ways:
|
||||||
|
|
||||||
|
- users cannot combine programmatic filters and CLI filters to more efficiently select subsets of tests
|
||||||
|
- CLI filters can override programmatic focus on CI systems resulting in an exit code of 0 despite the presence of (incorrectly!) committed focused specs.
|
||||||
|
|
||||||
|
Going forward Ginkgo will AND all programmatic and CLI filters. Moreover, the presence of any programmatic focused tests will always result in a non-zero exit code.
|
||||||
|
|
||||||
|
This change is technically a change in Ginkgo's external contract and may require some users to make changes to successfully adopt. Specifically: it's possible some users were intentionally using CLI filters to override programmatic focus. If this is you please open an issue so we can explore solutions to the underlying problem you are trying to solve.
|
||||||
|
|
||||||
|
### Fixes
|
||||||
|
- Programmatic focus is no longer overwrriten by CLI filters [d6bba86]
|
||||||
|
|
||||||
|
### Maintenance
|
||||||
|
- Bump github.com/onsi/gomega from 1.27.7 to 1.27.8 (#1218) [4a70a38]
|
||||||
|
- Bump golang.org/x/sys from 0.8.0 to 0.9.0 (#1219) [97eda4d]
|
||||||
|
|
||||||
## 2.10.0
|
## 2.10.0
|
||||||
|
|
||||||
### Features
|
### Features
|
||||||
|
39
vendor/github.com/onsi/ginkgo/v2/internal/focus.go
generated
vendored
39
vendor/github.com/onsi/ginkgo/v2/internal/focus.go
generated
vendored
@ -8,12 +8,12 @@ import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
/*
|
/*
|
||||||
If a container marked as focus has a descendant that is also marked as focus, Ginkgo's policy is to
|
If a container marked as focus has a descendant that is also marked as focus, Ginkgo's policy is to
|
||||||
unmark the container's focus. This gives developers a more intuitive experience when debugging specs.
|
unmark the container's focus. This gives developers a more intuitive experience when debugging specs.
|
||||||
It is common to focus a container to just run a subset of specs, then identify the specific specs within the container to focus -
|
It is common to focus a container to just run a subset of specs, then identify the specific specs within the container to focus -
|
||||||
this policy allows the developer to simply focus those specific specs and not need to go back and turn the focus off of the container:
|
this policy allows the developer to simply focus those specific specs and not need to go back and turn the focus off of the container:
|
||||||
|
|
||||||
As a common example, consider:
|
As a common example, consider:
|
||||||
|
|
||||||
FDescribe("something to debug", function() {
|
FDescribe("something to debug", function() {
|
||||||
It("works", function() {...})
|
It("works", function() {...})
|
||||||
@ -22,8 +22,8 @@ import (
|
|||||||
It("works", function() {...})
|
It("works", function() {...})
|
||||||
})
|
})
|
||||||
|
|
||||||
here the developer's intent is to focus in on the `"doesn't work"` spec and not to run the adjacent specs in the focused `"something to debug"` container.
|
here the developer's intent is to focus in on the `"doesn't work"` spec and not to run the adjacent specs in the focused `"something to debug"` container.
|
||||||
The nested policy applied by this function enables this behavior.
|
The nested policy applied by this function enables this behavior.
|
||||||
*/
|
*/
|
||||||
func ApplyNestedFocusPolicyToTree(tree *TreeNode) {
|
func ApplyNestedFocusPolicyToTree(tree *TreeNode) {
|
||||||
var walkTree func(tree *TreeNode) bool
|
var walkTree func(tree *TreeNode) bool
|
||||||
@ -44,40 +44,37 @@ func ApplyNestedFocusPolicyToTree(tree *TreeNode) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
Ginkgo supports focussing specs using `FIt`, `FDescribe`, etc. - this is called "programmatic focus"
|
Ginkgo supports focussing specs using `FIt`, `FDescribe`, etc. - this is called "programmatic focus"
|
||||||
It also supports focussing specs using regular expressions on the command line (`-focus=`, `-skip=`) that match against spec text
|
It also supports focussing specs using regular expressions on the command line (`-focus=`, `-skip=`) that match against spec text and file filters (`-focus-files=`, `-skip-files=`) that match against code locations for nodes in specs.
|
||||||
and file filters (`-focus-files=`, `-skip-files=`) that match against code locations for nodes in specs.
|
|
||||||
|
|
||||||
If any of the CLI flags are provided they take precedence. The file filters run first followed by the regex filters.
|
When both programmatic and file filters are provided their results are ANDed together. If multiple kinds of filters are provided, the file filters run first followed by the regex filters.
|
||||||
|
|
||||||
This function sets the `Skip` property on specs by applying Ginkgo's focus policy:
|
This function sets the `Skip` property on specs by applying Ginkgo's focus policy:
|
||||||
- If there are no CLI arguments and no programmatic focus, do nothing.
|
- If there are no CLI arguments and no programmatic focus, do nothing.
|
||||||
- If there are no CLI arguments but a spec somewhere has programmatic focus, skip any specs that have no programmatic focus.
|
- If a spec somewhere has programmatic focus skip any specs that have no programmatic focus.
|
||||||
- If there are CLI arguments parse them and skip any specs that either don't match the focus filters or do match the skip filters.
|
- If there are CLI arguments parse them and skip any specs that either don't match the focus filters or do match the skip filters.
|
||||||
|
|
||||||
*Note:* specs with pending nodes are Skipped when created by NewSpec.
|
*Note:* specs with pending nodes are Skipped when created by NewSpec.
|
||||||
*/
|
*/
|
||||||
func ApplyFocusToSpecs(specs Specs, description string, suiteLabels Labels, suiteConfig types.SuiteConfig) (Specs, bool) {
|
func ApplyFocusToSpecs(specs Specs, description string, suiteLabels Labels, suiteConfig types.SuiteConfig) (Specs, bool) {
|
||||||
focusString := strings.Join(suiteConfig.FocusStrings, "|")
|
focusString := strings.Join(suiteConfig.FocusStrings, "|")
|
||||||
skipString := strings.Join(suiteConfig.SkipStrings, "|")
|
skipString := strings.Join(suiteConfig.SkipStrings, "|")
|
||||||
|
|
||||||
hasFocusCLIFlags := focusString != "" || skipString != "" || len(suiteConfig.SkipFiles) > 0 || len(suiteConfig.FocusFiles) > 0 || suiteConfig.LabelFilter != ""
|
|
||||||
|
|
||||||
type SkipCheck func(spec Spec) bool
|
type SkipCheck func(spec Spec) bool
|
||||||
|
|
||||||
// by default, skip any specs marked pending
|
// by default, skip any specs marked pending
|
||||||
skipChecks := []SkipCheck{func(spec Spec) bool { return spec.Nodes.HasNodeMarkedPending() }}
|
skipChecks := []SkipCheck{func(spec Spec) bool { return spec.Nodes.HasNodeMarkedPending() }}
|
||||||
hasProgrammaticFocus := false
|
hasProgrammaticFocus := false
|
||||||
|
|
||||||
if !hasFocusCLIFlags {
|
|
||||||
// check for programmatic focus
|
|
||||||
for _, spec := range specs {
|
for _, spec := range specs {
|
||||||
if spec.Nodes.HasNodeMarkedFocus() && !spec.Nodes.HasNodeMarkedPending() {
|
if spec.Nodes.HasNodeMarkedFocus() && !spec.Nodes.HasNodeMarkedPending() {
|
||||||
skipChecks = append(skipChecks, func(spec Spec) bool { return !spec.Nodes.HasNodeMarkedFocus() })
|
|
||||||
hasProgrammaticFocus = true
|
hasProgrammaticFocus = true
|
||||||
break
|
break
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if hasProgrammaticFocus {
|
||||||
|
skipChecks = append(skipChecks, func(spec Spec) bool { return !spec.Nodes.HasNodeMarkedFocus() })
|
||||||
}
|
}
|
||||||
|
|
||||||
if suiteConfig.LabelFilter != "" {
|
if suiteConfig.LabelFilter != "" {
|
||||||
|
2
vendor/github.com/onsi/ginkgo/v2/types/version.go
generated
vendored
2
vendor/github.com/onsi/ginkgo/v2/types/version.go
generated
vendored
@ -1,3 +1,3 @@
|
|||||||
package types
|
package types
|
||||||
|
|
||||||
const VERSION = "2.10.0"
|
const VERSION = "2.11.0"
|
||||||
|
2
vendor/modules.txt
vendored
2
vendor/modules.txt
vendored
@ -446,7 +446,7 @@ github.com/modern-go/reflect2
|
|||||||
# github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822
|
# github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822
|
||||||
## explicit
|
## explicit
|
||||||
github.com/munnerz/goautoneg
|
github.com/munnerz/goautoneg
|
||||||
# github.com/onsi/ginkgo/v2 v2.10.0
|
# github.com/onsi/ginkgo/v2 v2.11.0
|
||||||
## explicit; go 1.18
|
## explicit; go 1.18
|
||||||
github.com/onsi/ginkgo/v2
|
github.com/onsi/ginkgo/v2
|
||||||
github.com/onsi/ginkgo/v2/config
|
github.com/onsi/ginkgo/v2/config
|
||||||
|
Loading…
Reference in New Issue
Block a user