nunnatsa / ginkgolinter

64%

main: 88%

LAST BUILD ON BRANCH error-async

branch: error-async

CHANGE BRANCH

x

Reset

Sync Branches

• error-async
• 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
• assertion-wrappers
• automation-improvements
• bug-fix
• bug-fixes
• bump-dependencies
• bump-deps
• bump-deps-in-tests
• 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.39.0
• dependabot/go_modules/golang.org/x/tools-0.40.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
• eventually-func
• fake-pr
• feat/constructor-config
• fix-100
• fix-115
• fix-124
• fix-171
• fix-190
• fix-211
• fix-82
• fix-91
• fix-badges
• fix-bug
• fix-comp-type-rule
• fix-false-positive
• fix-interval-bug
• fix-issue-88
• fix-name
• fix-pointer-bug
• fix-pr-template
• fix-sanity
• fix-txtar_updater
• 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-fix
• 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 # 8512491892

Build Type

Pull #146

github

Committed by nunnatsa

Commit Message

Bug fix: missing async wrong error checks

the linter didn't check error with nil assertion also in async
assertions, such as:
```go
Eventually(func() err {return nil}).Should(BeNil())
```
Also, the linter didn't check for MatchError issues in async assertions,
e.g.
```go
Eventually(func() string {return "hello"}).Should(MatchError("hello"))

This PR adds these chacks.
```

Pull Request Pull Request #146: Bug fix: missing async wrong error checks

Run Details

208 of 323 relevant lines covered (64.4%)

7.05 hits per line