Skip to content

Commit

Permalink
Tried to clarify "rules" about changing tests.
Browse files Browse the repository at this point in the history
@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?
  • Loading branch information
NicMcPhee committed Feb 9, 2015
1 parent 8e6e98a commit cac082c
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,11 @@ they are in any way complete. Obviously you want your code to pass at
least these tests, but you shouldn't assume that passing these tests
guarantees any kind of correctness. You're welcome to read the tests
and extend them if
you'd like. Don't change any of the given tests, though; we'll test
your code against *those* tests, and it's a Bad
Thing if they don't pass those.
you'd like. You may even need to make changes to the
test code to handle memory leaks
[(see below)](https://github.com/UMM-CSci-Systems/C-Lab-Starter#fixing-memory-problems).
Do be careful to not remove or weaken the tests, though; at a minimum you definitely
want to be able to pass the tests as given.

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

0 comments on commit cac082c

Please sign in to comment.