Closed burtlo closed 11 years ago
jcasimir
commented
11 years ago I think this is a good plan in the long run. For the next week, though, let's pare it down to just one or two iterations using tests. Maybe leave the phone number bit untested, but add testing for the zipcode since it's almost the same process.
steveklabnik
commented
11 years ago :+1: on doing it for zipcodes
burtlo
commented
11 years ago Alright I have a pull request up for the new event manager tutorial. None of it is test driven. Essentially the tests could be bolted on after in an iteration.
Easy things to define tests for in the current version of event manager is really just the clean_zipcodes method.
jcasimir
commented
11 years ago Fa
On Monday, January 21, 2013, Franklin Webber wrote:
Alright I have a pull request up for the new event manager tutorial. None of it is test driven. Essentially the tests could be bolted on after in an iteration.
Easy things to define tests for in the current version of event manager is really just the clean_zipcodes method.
— Reply to this email directly or view it on GitHubhttps://github.com/JumpstartLab/curriculum/issues/404#issuecomment-12523177.
Jeff Casimir Principal, Jumpstart Lab http://jumpstartlab.com
burtlo
commented
11 years ago Oh I could write tests. I just think they would blow minds.
jcasimir
commented
11 years ago I think Violet was the author of my previous, insightful post.
On Monday, January 21, 2013, Franklin Webber wrote:
Oh I could write tests. I just think they would blow minds.
— Reply to this email directly or view it on GitHubhttps://github.com/JumpstartLab/curriculum/issues/404#issuecomment-12528436.
Jeff Casimir Principal, Jumpstart Lab http://jumpstartlab.com
burtlo
commented
11 years ago Hah, I thought it was a defiant statement against the common cry "I can't test all of that, it's not possible" that you thought I was making.
The Event Manager tutorial needs to have a test drive approach added.