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

Test Driven Development #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Test Driven Development #3

wants to merge 1 commit into from

Conversation

hiattp
Copy link
Contributor

@hiattp hiattp commented Jun 19, 2018

I'm going to start chipping away at this PR over the next week but comments/contributions are welcome. In my opinion TDD (or some offshoot/variation) is an incredibly important, necessary and powerful tool not just for protection against regressions but also for velocity in the initial implementations if done properly, refactoring ability and for turnaround time for hotfixes (regression specs are much easier to write in the context of a rich and comprehensive test suite).

Despite my efforts to evangelize this over the years we haven't really internalized it at Lockstep—I still regularly see PRs even on greenfield projects that have no specs or specs that were clearly added after the fact. Even low-hanging regression specs are frequently omitted. But the best firms in our industry stand by this as a cornerstone of their engineering practices (Pivotal, Thoughbot, etc), and it has been an important driver of stability/productivity/velocity for my own work over the years.

I'd like to begin the process of becoming more serious and formal about this as part of our development practice, but I also want to standardize our philosophy and approach because there are many ways to do XTDD poorly and I think they contribute to the failure of adoption. I'll be outlining my thoughts in this new playbook section.

@citizen428
Copy link
Contributor

Nice effort! Looking forward to your thoughts on this one.

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

Successfully merging this pull request may close these issues.

2 participants