nunnatsa / ginkgolinter

78%

main: 88%

LAST BUILD ON BRANCH fix-190

branch: fix-190

CHANGE BRANCH

x

Reset

Sync Branches

• fix-190
• Add-Enhancement-Suggestion-template
• add-badges
• add-bug-report-template
• add-contrib-guide
• add-is
• add-linter-cover
• add-pr-template
• add-revive-linter
• add-rules
• add-to-readme
• allow-multi-edit
• allow-underscore
• automation-improvements
• bug-fix
• bug-fixes
• bump-dependencies
• bump-deps
• bump-deps-testdata
• bump-go-1.21
• bump-go-1.22
• bump-go-to-1.24
• bump-golangci-lint
• bump-lint
• bump-tools-v0.28.0
• bump-typeparams
• bump-x-tools
• bump_golang.org/x/tools_to_v0.31.0
• cap-equal
• check-wrong-bool-asert
• chor-update-deps
• cleanup
• code-improvements
• compare-len
• compare-types
• cont-refactoring
• dependabot/go_modules/github.com/go-toolsmith/astcopy-1.1.0
• dependabot/go_modules/golang.org/x/tools-0.10.0
• dependabot/go_modules/golang.org/x/tools-0.11.0
• dependabot/go_modules/golang.org/x/tools-0.11.1
• dependabot/go_modules/golang.org/x/tools-0.12.0
• dependabot/go_modules/golang.org/x/tools-0.13.0
• dependabot/go_modules/golang.org/x/tools-0.14.0
• dependabot/go_modules/golang.org/x/tools-0.15.0
• dependabot/go_modules/golang.org/x/tools-0.16.0
• dependabot/go_modules/golang.org/x/tools-0.16.1
• dependabot/go_modules/golang.org/x/tools-0.17.0
• dependabot/go_modules/golang.org/x/tools-0.18.0
• dependabot/go_modules/golang.org/x/tools-0.19.0
• dependabot/go_modules/golang.org/x/tools-0.20.0
• dependabot/go_modules/golang.org/x/tools-0.21.0
• dependabot/go_modules/golang.org/x/tools-0.22.0
• dependabot/go_modules/golang.org/x/tools-0.23.0
• dependabot/go_modules/golang.org/x/tools-0.24.0
• dependabot/go_modules/golang.org/x/tools-0.26.0
• dependabot/go_modules/golang.org/x/tools-0.27.0
• dependabot/go_modules/golang.org/x/tools-0.28.0
• dependabot/go_modules/golang.org/x/tools-0.29.0
• dependabot/go_modules/golang.org/x/tools-0.31.0
• dependabot/go_modules/golang.org/x/tools-0.32.0
• dependabot/go_modules/golang.org/x/tools-0.33.0
• dependabot/go_modules/golang.org/x/tools-0.35.0
• dependabot/go_modules/golang.org/x/tools-0.36.0
• dependabot/go_modules/golang.org/x/tools-0.38.0
• dependabot/go_modules/golang.org/x/tools-0.6.0
• dependabot/go_modules/golang.org/x/tools-0.7.0
• dependabot/go_modules/golang.org/x/tools-0.8.0
• dependabot/go_modules/golang.org/x/tools-0.9.1
• dependabot/go_modules/golang.org/x/tools-0.9.3
• doc-improve
• docs-fix-typos
• double-neg
• enable-dependabot
• error-async
• eventually-func
• fake-pr
• feat/constructor-config
• fix-100
• fix-115
• fix-124
• fix-171
• fix-211
• fix-82
• fix-91
• fix-badges
• fix-bug
• fix-comp-type-rule
• fix-false-positive
• fix-issue-88
• fix-name
• fix-pointer-bug
• fix-pr-template
• fix-sanity
• fix/go-mod
• fix/unalias
• focus-spec
• force-expect-to
• ginkgohandler-unit
• git-ignore-bin
• gitignore
• global-expect
• go-downgrade-1.20
• go1.20
• go119
• golangci-lint
• gomega-only
• havelen0
• helper-methods
• improve-gomega-detection
• improve-tests
• internal
• main
• matcherror
• more-bins
• nav/enforce-correct-succeed-usage
• new-bool-check
• no-assertion-rule
• nogo-support
• optional_description_rule
• patch-1
• pointer-val
• prevent-focus
• readme-opening
• refactor
• refactor-ginkgo-handler
• refactor/fix-typos
• simplify-error-msg
• small-refactor
• smaller-files
• sprintf-msg
• succeed-rule
• support-hold-label
• test-pr
• testscript
• timing
• tiny-cleanup
• tiny-fix
• to-and-not
• update-CONTRIBUTING
• update-dependencies
• update-deps
• var-unit-tests
• vars-in-containers
• version

Build # 13390718887

Build Type

Pull #191

github

Committed by nunnatsa

Commit Message

Fix issue: failed to recognize ginkgo.SpecContext in Eventually

ginkgolinter fails to recognized ginkgo.SpecContext as a valid context in
Eventually, and so wrongly identify it as the actual value, instead of
the following function parameter.

This PR fixes the issue and now this scenarion works

```go
It("should work", func(ctx SpecContext) {
	Eventually(func() error {
		return fmt.Errorf("foo")
	}).WithContext(ctx).Should(MatchError("foo"))
}, SpecTimeout(time.Second))
```

Pull Request Pull Request #191: Fix issue: failed to recognize ginkgo.SpecContext in Eventually

Run Details

12 of 12 new or added lines in 2 files covered. (100.0%)

2200 of 2833 relevant lines covered (77.66%)

1015.26 hits per line