Skip to content
This repository was archived by the owner on Sep 30, 2021. It is now read-only.

Commit cac082c

Browse files
committed
Tried to clarify "rules" about changing tests.
@kklamberty pointed out that we still had text that essentially told people not to make changes to the test code, which clearly isn't consistent with the need to make changes to the test code to fix memory leaks. Hopefully this helps?
1 parent 8e6e98a commit cac082c

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

Diff for: README.md

+5-3
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,11 @@ they are in any way complete. Obviously you want your code to pass at
3838
least these tests, but you shouldn't assume that passing these tests
3939
guarantees any kind of correctness. You're welcome to read the tests
4040
and extend them if
41-
you'd like. Don't change any of the given tests, though; we'll test
42-
your code against *those* tests, and it's a Bad
43-
Thing if they don't pass those.
41+
you'd like. You may even need to make changes to the
42+
test code to handle memory leaks
43+
[(see below)](https://github.com/UMM-CSci-Systems/C-Lab-Starter#fixing-memory-problems).
44+
Do be careful to not remove or weaken the tests, though; at a minimum you definitely
45+
want to be able to pass the tests as given.
4446

4547
On a related note, don't over focus on the tests. CMockery frankly
4648
doesn't give you super useful error messages or info when things fail.

0 commit comments

Comments
 (0)