ENGAGE-131: CabinetTests.js is full of cut and pasted logic

Metadata

Source
ENGAGE-131
Type
Bug
Priority
Minor
Status
Closed
Resolution
Fixed
Assignee
Justin Obara
Reporter
Colin Clark
Created
2009-10-16T18:55:34.000-0400
Updated
2009-11-26T15:01:40.000-0500
Versions
  1. 0.1
Fixed Versions
  1. 0.3b
Component
  1. Cabinet

Description

I was going through linting CabinetTests.js and noticed that CabinetTests.js has a fair bit of duplicated logic in each test, related to setting up options for the Cabinet. This should be factored out in one place to avoid duplication and reduce the number of lines of code.

Comments

  • Justin Obara commented 2009-11-26T15:01:40.000-0500

    Cleaned up at r8749, refactored and was able to remove a few hundred lines of code.