CSC/ECE 517 Spring 2013/OSS E608

From Expertiza_Wiki
Revision as of 22:49, 8 March 2013 by Hliu11 (talk | contribs) (Created page with "E608. Refactoring and Testing - Course Classes: *models/course.rb (76 lines) *models/course_participant.rb (94 lines) *models/course_team.rb (116 lines) *models/course_node.rb (9...")
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigation Jump to search

E608. Refactoring and Testing - Course Classes:

  • models/course.rb (76 lines)
  • models/course_participant.rb (94 lines)
  • models/course_team.rb (116 lines)
  • models/course_node.rb (95 lines)

What it does: Involves creation and management of courses in Expertiza What needs to be done It is not clear what a method named 'create_node' is doing in the 'course' class. Refactor by renaming this method appropriately. In the course_team class the import method seems to be doing too many things. See if this method can be broken down into smaller methods with specific functionality. The 'get' method in course_node.rb appears to be doing a lot. Refactor this method by breaking it up into smaller methods. Look for any unused methods or variables in these files. Also apply other refactorings such as Rename variable, Rename method to give the variables and methods more meaningful names. Write unit tests for all the methods in each of the listed model classes.