Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Car.toString() && Car.decelerate() w/Tests #37

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

dezren39
Copy link

@dezren39 dezren39 commented Feb 14, 2018

I implemented .toString() and .decelerate().
Attached are the changes required and tests to show how they work.

Cheers.

@Vandise
Copy link
Owner

Vandise commented Feb 21, 2018

5 points per:

  • You clone this repository and perform the work specified in your own repository.
  • You submit a pull request on Github
  • Your pull request contains modified versions of the class and the unit test
  • The test cases pass.
  • You have implemented new methods to make the tests pass.
  • Your code is free from errors
  • Your code is well formed, correct, and efficient.

10 points per

  • Your Git commit history in your project shows that you developed the tests first. That means you will have multiple commits. You must use Git to demonstrate to me that tests were written before code was implemented.

  • Make sure you follow the rules of TDD:

  • Do not write any code unless it’s to make a failing test pass.

  • Do not write any more of a unit test than is sufficient to fail, and not compiling is failing.

  • Do not write more code than you need to make the current unit test pass.

Instructor comments:

https://github.com/Vandise/agile_fall_2017_tdd/pull/37/commits/82dd2b3f3ba8f315c4b29e388482ffc0c7d9677c

Very clever. Excellent work.

Check E360 for your Grade as an unchecked box does not mean you were not given any points,
a checked box doesn't mean you were given full points.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants