What's that smell? Tidying Up our Test Code by Angie Jones
Report
Sauce LabsFollow
May. 3, 2019•0 likes•318 views
What's that smell? Tidying Up our Test Code by Angie Jones
May. 3, 2019•0 likes•318 views
Download to read offline
Report
Software
We are often reminded by those experienced in writing test automation that code is code. The sentiment being conveyed is that test code should be written with the same care and rigor with which production code is written. However, many people who write test code may not have experience writing production code, so it’s not exactly clear what is meant by this sentiment. And even those who write production code find that there are unique design patterns and code smells that are specific to test code in which they are not aware. Given a smelly test automation code base that is littered with several bad coding practices, Angie walks through each of the smells, discusses why it is considered a violation, and demonstrates a cleaner approach by refactoring the code live on stage. Key takeaways include how to identify code smells within test code, understanding the reasons why an approach is considered problematic, and how to implement clean coding practices within test automation
Sauce LabsFollow
Recommended
NIWeek 2011: Five Clever Debugging Techniques for Every LabVIEW DeveloperJKI
The Android GunslingerSaúl Díaz González
Its the Product. Not the Project May 17 2017Kathleen Leach, PMP
Evident Secrets of Successful Application - Max GoncharovRuby Meditation
Thinking Lean HardwareHAX
Being a professional software testerAnton Keks
More Related Content
Similar to What's that smell? Tidying Up our Test Code by Angie Jones
Rugged Software Using Rugged Driven DevelopmentJames Wickett
Chaos engineering - The art of breaking stuff in production on purposeGeert van der Cruijsen
My Road To Test Driven DevelopmentGerard Sychay
TDD and Getting PaidRowan Merewood
Barriers to Container Security and How to Overcome ThemWhiteSource
Why Remote Research Is (or isn't) Right for your projectGreenlight Guru
More from Sauce Labs
Simplify Salesforce Testing with AI-Driven Codeless ToolsSauce Labs
Testing on Mobile Devices with Location ServicesSauce Labs
Your Framework for Success: introduction to JavaScript Testing at ScaleSauce Labs
Automating Hybrid Applications with AppiumSauce Labs
Quality at Speed: More API Testing, Less UI TestingSauce Labs
Creating Digital Confidence with Test AutomationSauce Labs
![[Deu] Test Automatisierung Mit Web Driver.io](https://cdn.slidesharecdn.com/ss_thumbnails/deutestautomatisierungmitwebdriverio-191120180803-thumbnail.jpg?width=1600&height=908&fit=bounds)
Recently uploaded
MicroK8s 1.28 - MicroCeph on MicroK8s.pdfKonstantinos Tsakalozos
From Code to Cosmos: Mastering Microservices in the Distributed WonderlandBogdan Sucaciu
The Future of AI-Based Test AutomationApplitools
Expo - Zero to App.pptx😎 Anthony Kariuki
Five years of self-selection reteaming at Redgate - Agile Cambridge 2023Chris Smith
The Next Era of CRM.pdfPatrickYANG48
What's that smell? Tidying Up our Test Code by Angie Jones
- 1. What's That Smell?What's That Smell? Angie JonesAngie Jones http://angiejones.techhttp://angiejones.tech http://TestAutomationU.comhttp://TestAutomationU.com @techgirl1908@techgirl1908 Senior Developer AdvocateSenior Developer Advocate Director of Test Automation UniversityDirector of Test Automation University Applitools, San Francisco, CA, USAApplitools, San Francisco, CA, USA Tidying Up Our Test CodeTidying Up Our Test Code
- 2. @techgirl1
- 3. aa CODE SMELLCODE SMELL is an implementation thatis an implementation that violates fundamental designviolates fundamental design principles in a way that mayprinciples in a way that may slow down development andslow down development and increase the risk of future issues.increase the risk of future issues.
- 6. @techgirl1908 LONG CLASSLONG CLASS The purpose of the class isThe purpose of the class is multifold.multifold. To find anything, you mustTo find anything, you must scroll and scroll.scroll and scroll.
- 7. ISSUESISSUES @techgirl1908 1. No single responsibility1. No single responsibility 2. Hard to find things2. Hard to find things 3. Difficult to maintain3. Difficult to maintain
- 9. @techgirl1908 LONG METHODLONG METHOD This guy does it all!This guy does it all! But it makes it hard to knowBut it makes it hard to know when to call.when to call.
- 10. ISSUESISSUES @techgirl1908 1. No single responsibility1. No single responsibility 2. Confusing for callers2. Confusing for callers 3. Difficult to read and understan3. Difficult to read and understan
- 11. @techgirl1908 WE HAVE A PROBLEM!WE HAVE A PROBLEM! How come our second testHow come our second test cannot utilize the searchcannot utilize the search method?method?
- 13. @techgirl1908 DUPLICATE CODEDUPLICATE CODE When you find yourselfWhen you find yourself coding in haste,coding in haste, it's very temptingit's very tempting to copy and paste.to copy and paste.
- 14. ISSUESISSUES @techgirl1908 1. Any change needed has to1. Any change needed has to take place in multiple spotstake place in multiple spots 2. Can lead to other smells2. Can lead to other smells
- 16. FLAKY LOCATOR STRATEGYFLAKY LOCATOR STRATEGY At this moment,At this moment, your locators work fine.your locators work fine. But can theyBut can they stand the test of time?stand the test of time?
- 17. ISSUESISSUES @techgirl1908 1. Fragility1. Fragility 2. Unreliability2. Unreliability
- 19. @techgirl1908 INDECENT EXPOSUREINDECENT EXPOSURE Too many see you.Too many see you. Your scope is too wide.Your scope is too wide. All who should know youAll who should know you
- 20. ISSUESISSUES 1. Violates encapsulation1. Violates encapsulation 2. Allows tests to directly access2. Allows tests to directly access and manipulate the DOM,and manipulate the DOM, which is not their responsibilitywhich is not their responsibility
- 22. @techgirl1908 INEFFICIENT WAITSINEFFICIENT WAITS The speed of machineThe speed of machine is faster than manis faster than man so you slow it downso you slow it down any way you can.any way you can.
- 23. ISSUESISSUES @techgirl1908 1. Slows down runtime1. Slows down runtime 2. Different environments may2. Different environments may require different wait timesrequire different wait times
- 25. @techgirl1908 MULTIPLE POINTS OF FAILUREMULTIPLE POINTS OF FAILURE The framework's jobThe framework's job is to force state,is to force state, not to determinenot to determine a test's fate.a test's fate.
- 26. ISSUESISSUES @techgirl1908 1. Violates single responsibility1. Violates single responsibility 2. Limits reusuability2. Limits reusuability
- 29. What's That Smell?What's That Smell? Angie JonesAngie Jones http://angiejones.techhttp://angiejones.tech http://TestAutomationU.comhttp://TestAutomationU.com @techgirl1908@techgirl1908 Senior Developer AdvocateSenior Developer Advocate Director of Test Automation UniversityDirector of Test Automation University Applitools, San Francisco, CA, USAApplitools, San Francisco, CA, USA Tidying Up Our Test CodeTidying Up Our Test Code