Add phony target for test in Makefile

[tommyschaefer]

Hi there!

I noticed while looking at the Travis CI output yesterday that the tests don't appear to be running.

From Travis CI log output:

$ make test

make: `test' is up to date.

The command "make test" exited with 0.

This is happening because we added a test directory, and the Makefile target is also named test.

If this looks good to you all and gets merged, I will rebase and push #514, #510, and #509 to run CI with the tests.

Thanks so much for your time!

[tommyschaefer]

With this change the Travis CI log looks like this:

$ make test

----------------------------------------------------------------
running tests for: accumulate
Run options: --seed 16500

# Running:

.......

Finished in 0.001917s, 3652.0113 runs/s, 3652.0113 assertions/s.

7 runs, 7 assertions, 0 failures, 0 errors, 0 skips


----------------------------------------------------------------
running tests for: acronym
Run options: --seed 26000

# Running:

.......

Finished in 0.001685s, 4154.4654 runs/s, 4154.4654 assertions/s.

7 runs, 7 assertions, 0 failures, 0 errors, 0 skips


----------------------------------------------------------------

...

[Insti]

Looks good @tommyschaefer, thanks. I really appreciate the detailed problem description/solution information you include in your PRs ❤️

For reference, documentation for Phony targets in make: https://www.gnu.org/software/make/manual/html_node/Phony-Targets.html

[Insti]

Thanks. I didn't notice that I'd broken the tests when I created the test directory.
Sorry about that.

I was just glad that the tests of the exercise tests did not run needlessly all the time. So perhaps there is something there that can be improved. Only running the tests of the exercise tests when they have been changed.