Download to read offline
![package HelloWorld;
public class HelloWorld {
public static void main(String[] args) {
System.out.println("Hello World");
}
}
@BagmarAnand
Hello World](https://image.slidesharecdn.com/rewritevsrefactor-agileindia2021-211120063431/75/Rewrite-vs-Refactor-AgileIndia-2021-20-2048.jpg)
![@BagmarAnand
Start afresh
package HelloWorld;
public class HelloWorld {
public static void main(String[] args) {
System.out.println("Hello World");
}
}
http://coastalbend.edu/uploadedImages/CBC/Content/Instructional_Services/Dual_Credit/hard_working_on_computer_anim_500_clr_7364.gif](https://image.slidesharecdn.com/rewritevsrefactor-agileindia2021-211120063431/75/Rewrite-vs-Refactor-AgileIndia-2021-24-2048.jpg)
![package HelloWorld;
public class HelloWorld {
public static void main(String[] args) {
System.out.println("Hello World");
}
}
@BagmarAnand
Hello World](https://crownmelresort.com/image.slidesharecdn.com/rewritevsrefactor-agileindia2021-211120063431/75/Rewrite-vs-Refactor-AgileIndia-2021-20-2048.jpg)
![@BagmarAnand
Start afresh
package HelloWorld;
public class HelloWorld {
public static void main(String[] args) {
System.out.println("Hello World");
}
}
http://coastalbend.edu/uploadedImages/CBC/Content/Instructional_Services/Dual_Credit/hard_working_on_computer_anim_500_clr_7364.gif](https://crownmelresort.com/image.slidesharecdn.com/rewritevsrefactor-agileindia2021-211120063431/75/Rewrite-vs-Refactor-AgileIndia-2021-24-2048.jpg)
Uploaded byAnand Bagmar
Rewrite vs Refactor (AgileIndia 2021)
The document discusses the differences between rewriting and refactoring code, particularly in the context of test automation. It highlights the importance of understanding existing code, assessing challenges, and ensuring that automation adds value, while also addressing common anti-patterns in coding practices. Strategies for improving test automation practices and achieving objectives are outlined, emphasizing the need for clear objectives, increased awareness, and proper coding standards.
More Related Content
Similar to Rewrite vs Refactor (AgileIndia 2021)
![[DevDay2018] Let’s all get along. Clean Code please! - By: Christophe K. Ngo,...](https://cdn.slidesharecdn.com/ss_thumbnails/christophek-180419103038-thumbnail.jpg?width=640&height=640&fit=bounds)
Rewrite vs Refactor (AgileIndia 2021)
- 1. Rewrite Vs Refactor @BagmarAnand AnandBagmar Software Quality Evangelist, Essence Of Testing
- 2.
- 3.
- 4. We will focuson code for Test Automation @BagmarAnand
- 5. Applicable for anytype of code you write @BagmarAnand
- 6.
- 7.
- 8. • Revamp TestingPractices • Automate everything! @BagmarAnand Charter https://i.pinimg.com/474x/ec/8a/17/ec8a176036490a3338f806172a23e4b3.jpg
- 9. @BagmarAnand Assessment https://media.istockphoto.com/vectors/assessment-infographic-with-8-steps-parts-options-vector-id640310008 • Validate thecharter! • Understand by investigation: • What is working well, • The challenges, and, • The (real & false) constraints
- 10. @BagmarAnand Assessment: Investigation https://media.istockphoto.com/vectors/assessment-infographic-with-8-steps-parts-options-vector-id640310008 • Speakwith the team members • Code reviews • Run the tests yourself! • Validate / Challenge Assumptions • Challenges • Setup / Infra issues
- 11. @BagmarAnand Assessment: Finding https://media.istockphoto.com/vectors/assessment-infographic-with-8-steps-parts-options-vector-id640310008 • NoCI • Automation working with lot of manual interventions • Setup / execution all over the place • Large & separate codebase
- 12. • Revamp TestingQuality Practices • Automate everything that makes sense to automate! @BagmarAnand Evolved Charter https://storage.needpix.com/rsynced_images/smile-2352472_1280.png
- 13.
- 14. • Fast feedback(from 1000s of automated UI tests) • Clarity of intent – know what is being tested • Maintainable & Scalable Automation @BagmarAnand Charter https://i.pinimg.com/474x/ec/8a/17/ec8a176036490a3338f806172a23e4b3.jpg
- 15. @BagmarAnand Assessment: Investigation https://media.istockphoto.com/vectors/assessment-infographic-with-8-steps-parts-options-vector-id640310008 • Speakwith the team members • Code reviews • Run the tests yourself! • Validate / Challenge Assumptions • Challenges • Setup / Infra issues
- 16. @BagmarAnand Assessment: Finding • Automationcode quality is sub-par • Excessive duplication, massive files, no encapsulation • Excessive use of “sleep” • Extremely complex code • No test execution trend analysis • Too much focus on UI / end-2-end automation • No idea what is being tested (without reading a lot of code)
- 17. • Fast feedback(from 1000s of automated UI tests) • Clarity of intent – know what is being tested • Maintainable & Scalable Automation • Focus on automating tests at lower level • Unit Test Automation • API / Component Test Automation • API Workflow Automation @BagmarAnand Evolved Charter https://storage.needpix.com/rsynced_images/smile-2352472_1280.png
- 18. Details of theAnalysis @BagmarAnand
- 19. Evolution of thecode-base @BagmarAnand
- 20. package HelloWorld; public classHelloWorld { public static void main(String[] args) { System.out.println("Hello World"); } } @BagmarAnand Hello World
- 21.
- 22.
- 23. In few cases…. @BagmarAnand
- 24. @BagmarAnand Start afresh package HelloWorld; publicclass HelloWorld { public static void main(String[] args) { System.out.println("Hello World"); } } http://coastalbend.edu/uploadedImages/CBC/Content/Instructional_Services/Dual_Credit/hard_working_on_computer_anim_500_clr_7364.gif
- 25. In most cases…. @BagmarAnand
- 26. https://149351115.v2.pressablecdn.com/wp-content/uploads/2018/08/codebase_project_management-1200x675.png @BagmarAnand Work on existingcode-base http://2.bp.blogspot.com/-ntk4QrnZT48/UoYoBG6XNlI/AAAAAAAANx0/Rtjng1VZFQY/s1600/LabVIEW_spaghetti_code.png
- 27. Where you will…. @BagmarAnand
- 28. https://149351115.v2.pressablecdn.com/wp-content/uploads/2018/08/codebase_project_management-1200x675.png @BagmarAnand Extend & evolvesuch code http://2.bp.blogspot.com/-ntk4QrnZT48/UoYoBG6XNlI/AAAAAAAANx0/Rtjng1VZFQY/s1600/LabVIEW_spaghetti_code.png
- 29.
- 30. https://149351115.v2.pressablecdn.com/wp-content/uploads/2018/08/codebase_project_management-1200x675.png @BagmarAnand Multiple copies ofsuch code http://2.bp.blogspot.com/-ntk4QrnZT48/UoYoBG6XNlI/AAAAAAAANx0/Rtjng1VZFQY/s1600/LabVIEW_spaghetti_code.png • Multiple repos • Multiple and long living branches
- 31.
- 32. What is thetypical reaction when you see existing code? @BagmarAnand
- 33.
- 34.
- 35.
- 36.
- 37.
- 38.
- 39. https://40rik02ft2xye26xv2i0y0yc-wpengine.netdna-ssl.com/wp-content/uploads/2016/04/AdobeStock_99227409-1110x699.jpeg Documentation – sparse/ outdated @BagmarAnand • It is important to have lean and meaningful documentation • Do not just add comments in code for the sake of “documenting the implementation logic” – your code should be readable and understandable
- 40.
- 41.
- 42.
- 43.
- 44.
- 45.
- 46.
- 47. What was goodyesterday, may not necessarily be good today as well! @BagmarAnand http://www.hptx.org/images/InfoAdvanced/70/brain%20gears.png
- 48.
- 49.
- 50. https://empxtrack.com/wp-content/uploads/2016/05/come_join_the_team_500_wht_10876.gif @BagmarAnand @BagmarAnand Changes in TeamDynamics • People leave • New people join
- 51.
- 52. http://2.bp.blogspot.com/-ntk4QrnZT48/UoYoBG6XNlI/AAAAAAAANx0/Rtjng1VZFQY/s1600/LabVIEW_spaghetti_code.png If you needto work on such code …. @BagmarAnand
- 53.
- 54. When you findwhere change needs to happen, then… @BagmarAnand
- 55. @BagmarAnand What is theimpact of change going to be? http://textbook.stpauls.br/Business_Organization/images/pic311.jpg
- 56. Pressure of gettingit done on Time! https://newsatjama.files.wordpress.com/2018/07/hourglass-clock-on-light-blue-background-thinkstockphotos-479876312.jpg?w=1200 https://miro.medium.com/max/5600/1*frbsX4eY4Sj2mla2vszKPA.jpeg @BagmarAnand
- 57.
- 58. @BagmarAnand Anti-pattern: Forget pastlessons learnt https://storage.needpix.com/rsynced_images/reminder-23771_1280.png
- 59.
- 60. @BagmarAnand Anti-pattern: Reinvent thewheel https://external-preview.redd.it/ZVPjiFo_Ubl4JeiU63SaTjdIoq5zveSnNZimKpgn2I8.png?auto=webp&s=bf32c94b630befa121075c1ae99b2599af6dedc5 • Don’t want to modify existing code
- 61. @BagmarAnand Anti-pattern: Reinvent thewheel https://storage.needpix.com/rsynced_images/traveller-3213992_1280.jpg • Unable to find what you were looking for
- 62.
- 63. @BagmarAnand Anti-pattern: Sleep toreduce flakiness https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Zzz_sleep.svg/1024px-Zzz_sleep.svg.png https://upload.wikimedia.org/wikipedia/commons/thumb/7/76/Evil_red.svg/1200px-Evil_red.svg.png https://storage.needpix.com/rsynced_images/snail-2383080_1280.jpg
- 64. Why do weend up using these anti-patters? @BagmarAnand
- 65. Because …. It(still) works! https://www.linkedin.com/posts/bhagyaitrecruiteratgmail_activity-6642620864221274112-NnWI @BagmarAnand
- 66. @BagmarAnand Let someone elsefigure out a better way… later https://lh3.googleusercontent.com/proxy/MuzrRuna5ts_O-FX_KPUcsC-2SA_blkGTlwPUBfLS9uWo11NN5RLhI8dUBucVrGUtTvQs5FP-AGlBrLTXI0VnZzBT0C30E7HvWtOHyqd8TP3QIeJCg
- 67.
- 68.
- 69. Refactoring is anessential practice when evolving existing code @BagmarAnand
- 70. But … wemay end up creating another mess IF @BagmarAnand
- 71. @BagmarAnand Refactoring without clearobjective https://encrypted-tbn0.gstatic.com/images?q=tbn%3AANd9GcRPV-74mTtTdR8VcenNpVhYM5AQq7PWuEkr2hW5jTlrqnaFfBrQ
- 72.
- 73.
- 74. @BagmarAnand Automation does notprovide enough value! https://4.bp.blogspot.com/-dgoCWivfsb8/XDcqaFvQC2I/AAAAAAAAF2o/qz5zcKCf9lcD6ExCo3p16YsFCwkQsCxjgCLcBGAs/s1600/DRC-falling-robot.png • Difficult to add new features / tests • Difficult to scale • Supposedly simple changes take long time, and have unexpected behavior
- 75. @BagmarAnand Manual, repeated effortkeeps increasing http://logico4576.blogspot.com/2012/03/bertrand-russell-para-que-repetir-los.html
- 76.
- 77. @BagmarAnand Assessment – TheSummary! • Testing was QA team responsibility • Automation not adding any value
- 78.
- 79.
- 80. @BagmarAnand Overall challenges fromTesting perspective • Process / Way of Working & Ownership of Testing • Understanding of Automation • Experience & Skillset • Coding skills • Time pressure • CI
- 81. • Device &Driver management • Run on demand, on QA machine • Slow (100s of sleeps) • Static methods, no-OOPS • Sequential • Branching nightmare! @BagmarAnand Reality of existing Automated Tests https://i.stack.imgur.com/SDmtn.png
- 82. Code Analysis &Execution @BagmarAnand
- 83. @BagmarAnand Test Code Snippets •Methods returning default value • Extremely complex method (deep nesting of loops) • Lack of naming conventions • Swallowing of exceptions, returning Null • Sleeps, static members / methods
- 84.
- 85. @BagmarAnand Excessive cyclomatic complexity inmethod: 46 warnings Java analysis: 10 errors, 6212 warnings Test Code Analysis
- 86. @BagmarAnand Excessive cyclomatic complexity inmethod: 126 warnings Java analysis: 20 errors, 10355 warnings Test Code Analysis
- 87.
- 88. How to turnthe ship around in such cases? @BagmarAnand https://upload.wikimedia.org/wikipedia/commons/7/72/Container_ship_Reecon_Whale_on_Black_Sea_near_Constan%C8%9Ba_Romania.jpg
- 89.
- 90. @BagmarAnand How do weachieve our objectives? https://www.expandgh.com/wp-content/uploads/2018/04/Set-Objectives-for-Each-Person.jpg
- 91. @BagmarAnand How do weachieve our objectives? https://www.expandgh.com/wp-content/uploads/2018/04/Set-Objectives-for-Each-Person.jpg Know your objectives first!
- 92. • Fast inexecution and feedback • Independent tests • Executed on every change • Know what is being tested, as the customer / consumer • Able to release with confidence! @BagmarAnand Expectations from Automated Tests
- 93. @BagmarAnand Approach • Increase awareness •Upskill (pairing, training) • Evolved way-of-working • In initial stages, you may want to consider Automation rewrite as a separate, independent project
- 94. @BagmarAnand Approach - Automation •Identify Design Patterns that will help • Setup Coding guidelines and coding styles • Use powerful IDEs • Automate Infrastructure setup for test execution • Keep Tests Independent
- 95.
- 96. Criteria for automating testsscenarios @BagmarAnand https://chiefexecutive.net/wp-content/uploads/2019/02/AdobeStock_82846171.jpeg
- 97. @BagmarAnand What are youautomating? • Know your users • Simulate user behavior
- 98. @BagmarAnand Criteria for ImplementingAutomation • Clear & visible intent of the automated test • Multi-browser & multi-device support • Run on every (local & deployed) change (local, CI & cloud) • Fast feedback (parallel execution) • Rich execution reports & Trend Analysis • https://www.infoq.com/articles/test-automation-ai-ml/
- 99.
- 100. Will team beable to refactor, while other projects are going on, and get value from automation, the way it is supposed to be? @BagmarAnand https://lh3.googleusercontent.com/proxy/3JFWJGTlmTym_g9xCZpHIFNLN3rQZu3js8f7iJDHFc6CCeTDb4hpLJaaQm1YCe2BfxyuA_7Lv3B4Clt7MpICbe_Ay09TY8k
- 101.
- 102.
- 103.
- 104. @BagmarAnand Approach for Rewritewith Reuse … • Formed team of 2 people initially • Automated test execution environment setup script • Created new repository • Implemented 2 tests • Parallel execution • Triggered automatically via CI • No manual setup / intervention
- 105. @BagmarAnand Approach for Rewritewith Reuse • Used tools / frameworks / libraries that can achieve the objectives of the test • Build Vs Buy / Reuse • Hard discussions around duplication Vs complexity of specific modules – ex: test data setup for product • Rewrite everything, or reuse some existing implementation?
- 106.
- 107. • IDEs provideyou lot of information about the code automatically • Additional plugins can be installed / used • Free / Open-Source plugins may also be available • Static Code Analysis can be setup in VCS / CI pipelines @BagmarAnand Maintain Code Quality
- 108. @BagmarAnand Remember, and watchout for … https://i.pinimg.com/originals/0e/0b/1f/0e0b1f8c2b36404e4740b6f733097f19.jpg
- 109.