Skip to content
  • Chris Salzberg's avatar
    42a8340a
    Add validation to subclass in tests to avoid polluting parent class · 42a8340a
    Chris Salzberg authored
    These two tests currently both define acceptance validators on the same
    class, Topic. This means that in either one test or the other, there are
    not one but *two* instances of the LazilyDefineAttributes module
    builder in the class' ancestors, which can result in unpredictable
    results.
    
    Subclassing Topic in each test avoids conflicts.
    42a8340a
    Add validation to subclass in tests to avoid polluting parent class
    Chris Salzberg authored
    These two tests currently both define acceptance validators on the same
    class, Topic. This means that in either one test or the other, there are
    not one but *two* instances of the LazilyDefineAttributes module
    builder in the class' ancestors, which can result in unpredictable
    results.
    
    Subclassing Topic in each test avoids conflicts.
Loading