FLUID-3397: Infusion Builder's javascript unit tests are poorly factored, making them hard to debug and difficult to read

Metadata

Source
FLUID-3397
Type
Bug
Priority
Major
Status
Closed
Resolution
Won't Fix
Assignee
N/A
Reporter
Colin Clark
Created
2009-12-01T18:57:08.000-0500
Updated
2014-03-03T11:20:13.368-0500
Versions
  1. 1.4
Fixed Versions
  1. 1.5
Component
  1. Infusion Builder

Description

The Infusion Builder's unit tests have been designed in an awkward way, causing state to be deeply coupled to all portions of the test. This makes the tests hard to read and understand, and gives them a monolithic structure that can be difficult to work with. Because we've factored the test functions themselves out into "creator functions," it's also very difficult to debug a particular instance of a test run correctly.

While it's tempting to just rewrite the tests from scratch, my suggestion is to start splitting pieces of the tests out into simpler chunks with less dependence on closure-derived state. With time, we can break out the tests into a few different files to ensure that they're correctly modularized.

Comments

  • Laurel Williams commented 2009-12-02T13:41:47.000-0500

    Updated title of this issue. There were some words missing.

  • Justin Obara commented 2014-03-03T11:20:13.367-0500

    builder.fluidproject.org has been taken down and there are no immediate plans to replace it. Currently all builds are expected to be generated directly from the grunt build scripts.

    http://fluid.2324889.n4.nabble.com/Infusion-Builder-issues-tp8926.html