[#120] Fix makefile - Githubissues

serokell / coffer

Multi-backend password store with multiple frontends

4 stars 2 forks source link

[#120] Fix makefile #121

Closed dcastro closed 2 years ago

dcastro commented 2 years ago

Description

There are a few issues with the makefile:

The test-all and all targets do the same thing as test and coffer, so they're redundant
The test target does not include golden tests
The test target calls the doctests executable with --color, but doctests does not recognize this option.

This PR:

removes those redundant targets
fixes make test so that it runs all test suites
does not call doctest with --color
adds makefile targets for each individual test suite
updates the readme to reflect these changes

Related issue(s)

Fixed #120

:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Tests
- [x] If I added new functionality, I added tests covering it.
- [x] If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
Documentation
- [x] I checked whether I should update the docs and did so if necessary:
- README
- Haddock

Stylistic guide (mandatory)

[x] My commits comply with the policy used in Serokell.
[x] My code complies with the style guide.