Skip to content

Commit

Permalink
PR FIXUP - Remove mock regen from ci
Browse files Browse the repository at this point in the history
I'm still in favour of this in the short term, but it is too controversial and is bogging down the highly desired `make mock` command.
  • Loading branch information
AndrewSisley committed Jul 6, 2023
1 parent 1c16e70 commit b71916b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,6 @@ test\:build:

.PHONY: test\:ci
test\:ci:
@$(MAKE) mock
DEFRA_BADGER_MEMORY=true DEFRA_BADGER_FILE=true $(MAKE) test:names

.PHONY: test\:go
Expand Down
2 changes: 1 addition & 1 deletion tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ These tests focus on small, isolated parts of the code to ensure each part is wo
For unit tests, we sometimes use mocks. Mocks are automatically generated from Go interfaces using the mockery tool.
This helps to isolate the code being tested and provide more focused and reliable tests.

To regenerate the mocks, run `make mock`. `make test:ci` will also do this.
To regenerate the mocks, run `make mock`.

The mocks are typically generated into a separate mocks directory.

Expand Down

0 comments on commit b71916b

Please sign in to comment.