1
53%
main: 53%

Ran 24 Apr 2024 11:57AM UTC

Files 176

Run time 3s

Badge

Embed ▾

Committed 24 Apr 2024 11:49AM UTC coverage: 49.754% (-0.005%) from 49.759%

Job # 8816020947.1

Build Type

push

github

Committed by

web-flow

Commit Message

Use `go generate` directives for mock generation (#3159)

Instead of adding commands to the Makefile, use go generate directives in the files which contain the interfaces we want to mock. This approach has the following advantages:

    Less context switching: you can declare the need for a mock in the
    code and not have to edit the makefiles.
    The same directive can be reused everywhere - you no longer need to
    specify the filepath, interface name and the path to the generated
    mock.
    If the interface is renamed or moved, the code generation will still
    work without modification.

I have left some commands in the makefile for generating mocks from code in pkg, and from autogenerated code. We could in theory get rid of these usages as well, but I did not feel there was much point at this time since they are edge cases.

You may note that there are a lot of additional lines of code: this is because I'm using the "source" mode of mockgen instead of the "reflect" mode. The source mode will generate mocks for all interfaces in the file, and not just the specified ones. However, it does so much quicker than the reflect mode - make gen now takes about half the time it did previously.

Coverage Stats

8784 of 17655 relevant lines covered (49.75%)

37.52 hits per line

stacklok / minder / 8816020947 / 1
53%
main: 53%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8816020947.1

stacklok / minder / 8816020947 / 1 53% main: 53%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 8816020947.1

stacklok / minder / 8816020947 / 1
53%
main: 53%

README BADGES
x