CSC/ECE 517 Spring 2023 - E2336. Reimplement response.rb: Difference between revisions
Jump to navigation
Jump to search
No edit summary |
|||
Line 2: | Line 2: | ||
There are several issues we plan to address as part of this project. | There are several issues we plan to address as part of this project. | ||
* Refactor the get_all_review_comments method as an instance method in response.rb file | |||
* Refactor the self.get_all_responses method as an instance method in response_map.rb file. | * Refactor the self.get_all_responses method as an instance method in response_map.rb file. | ||
* Add more descriptive comments for the tests. | * Add more descriptive comments for the tests. | ||
===Issue 1: Refactor the self.get_all_responses method=== | |||
===Issue 5: Refactor the self.get_all_responses method=== | ===Issue 5: Refactor the self.get_all_responses method=== |
Revision as of 23:07, 7 April 2023
Project Overview
There are several issues we plan to address as part of this project.
- Refactor the get_all_review_comments method as an instance method in response.rb file
- Refactor the self.get_all_responses method as an instance method in response_map.rb file.
- Add more descriptive comments for the tests.
Issue 1: Refactor the self.get_all_responses method
Issue 5: Refactor the self.get_all_responses method
The self.get_all_responses method which is currently in the response.rb file would be much cleaner as an instance method in response_map.rb. We plan to refactor the self.get_all_responses method in response.rb to an instance method in response_map.rb.
Issue 6: Add more descriptive comments for the tests
The existing tests need more descriptive comments. We plan to add more detailed comments for all the test cases.