HomePhorge

(#7259) Remove ActiveRecord requirement from indirector face spec
c8775f9c5caaUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(#7259) Remove ActiveRecord requirement from indirector face spec

"should be able to return a list of terminuses for a given
indirection" was calling Puppet::Indirector::Face.terminus_classes,
which is just a thin wrapper around
Puppet::Indirector::Terminus.terminus_classes, which would attempt to
load all Terminus classes. This would cause problems if not all of
the prerequisites for all of the Terminus classes were installed (For
example: ActiveRecord).

Now we only test that the thin wrapper appropriately munges the output
from Puppet::Indirector::Terminus.terminus_classes, since the method
being wrapped should have its own tests for the behavior that was
being tested originally.

Paired-with: Nick Lewis <nick@puppetlabs.com>

Details

Event Timeline

Jacob Helwig <jacob@puppetlabs.com> committed rPUc8775f9c5caa: (#7259) Remove ActiveRecord requirement from indirector face spec (authored by Jacob Helwig <jacob@puppetlabs.com>).May 16 2011, 7:06 PM