CSC/ECE 517 Spring 2013/OSS E608

From Expertiza_Wiki
Revision as of 22:50, 8 March 2013 by Hliu11 (talk | contribs)
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.

Retrieved from "https://wiki.expertiza.ncsu.edu/index.php?title=CSC/ECE_517_Spring_2013/OSS_E608&oldid=73855"