1
88%
main: 88%

Ran 03 Oct 2023 08:04AM UTC

Files 6

Run time 0s

Badge

Embed ▾

Committed 03 Oct 2023 08:02AM UTC coverage: 84.034% (+0.05%) from 83.982%

Job # 6390277556.1

Build Type

push

github

Committed by

nunnatsa

Commit Message

New Rule: should not compare two different types

The `Equal` and the `BeIdentical` matchers also check the type, not only
the value.

The following code will fail in runtime:
```go
x := 5 // x is int
Expect(x).Should(Eqaul(uint(5)) // x and uint(5) are with different
types
```

When using negative checks, it's even worse, because we get a fale
positive:
```
x := 5
Expect(x).ToNot(Equal(uint(5))
```

To solve this problem, we want to find these errors before running the
tests and failing on runtime. This is even more important for e2e or
functional tests, where we sometimes can't run them on the local
development environment.

This PR adds a new rule to the linter, to find comaprison of values
from different types.

Run Details

1000 of 1190 relevant lines covered (84.03%)

798.8 hits per line

nunnatsa / ginkgolinter / 6390277556 / 1
88%
main: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 6390277556.1

nunnatsa / ginkgolinter / 6390277556 / 1 88% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 6390277556.1

nunnatsa / ginkgolinter / 6390277556 / 1
88%
main: 88%

README BADGES
x