Published May 26, 2020

Google's Engineering Practices - What to Look for in a Code Review

Delve into Google's engineering practices as Joe Zack unpacks the intricacies of code reviews, emphasizing design, architecture, and complexity management. Discover the importance of robust unit testing and learn how positive feedback and effective commenting can shape a more productive code review culture.
Episode Highlights
Coding Blocks logo

Popular Clips

Episode Highlights

  • Unit Testing

    Effective unit testing is crucial for maintaining code quality and ensuring that changes do not introduce new bugs. emphasizes the importance of simplicity and clarity in tests, advocating for one assertion per test to maintain focus and effectiveness 1. agrees, noting that overly complex tests can obscure their purpose and reduce their usefulness 2. He suggests that tests should be added in the same change list unless it's an emergency, ensuring they are relevant and up-to-date with the code changes.

       

    Test Challenges

    Writing unit tests presents several challenges, particularly in maintaining simplicity and avoiding redundancy. highlights the common pitfall of copy-pasting test code, which can lead to errors and false positives 3. He stresses the importance of ensuring tests will fail if the code is broken, as this is the primary purpose of unit testing. also discusses the difficulty of tracking test coverage changes without automated tools, which can complicate maintaining high-quality tests 4.

Related Episodes