| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
| |
Fix facts added with empty blocks by handling calls to value when setcode not called
Ensure we handle load failures more gracefully
|
|
|
|
|
|
|
|
|
| |
This cleans up xen and vserver detection to enable us to stub out so when we
happen to be running tests on xen we don't report as that.
More cleanup is needed in this area but this should give us a green build.
This renames the tests to be consistent with current naming convention
|
|
|
|
|
|
| |
We already have a network fact it's just missing a test.
Paul
|
|
|
|
|
| |
Solaris orders inet and inet6 seperately. This tests for and fixes this by
uniqueing the list. Will probably need work when we get to ipv6 support.
|
|
|
|
|
|
| |
coverage
switch %x{} call to Facter::Util::Resolution.exec for better testing
|
|
|
|
| |
Add tests and utility for virtual fact detection
|
|
|
|
|
|
|
|
|
|
|
|
| |
This seems to have been fixed functionally on master
(in /Users/pnasrat/Development/facter/spec)
Finished in 1.604955 seconds
183 examples, 0 failures
However I want to take the opportunity to add tests to prevent regressions
Paul
|
|
|
|
| |
Added a test to pickup typo and fix
|
|
|
|
|
|
|
|
| |
The interface collection regex was leaving trailing
':' characters on the interface names, which meant
individual interfaces weren't quite right.
Signed-off-by: Luke Kanies <luke@madstop.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
this ensures we can compare all kind of objects and
not only instances of strings.
It compares strings in a case-insensitive manner and converts
symbols to strings.
introducing this behavior required that we introduce
a convert util method, to ensure that we convert the value
correctly. Introduced this method in other places as well.
This behavior change requires that we drop one test, which have
become anyway deprecated.
|
|
|
|
|
|
| |
Facts that raise exceptions just return nil now.
Signed-off-by: Luke Kanies <luke@madstop.com>
|
|
|
|
|
|
| |
This name change just reflects the file location.
Signed-off-by: Luke Kanies <luke@madstop.com>
|
|
|
|
|
|
|
| |
Also added tests to the macosx code; it's much cleaner and
actually tested now.
Signed-off-by: Luke Kanies <luke@madstop.com>
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I've made the IPMess stuff a lot less messy, and refactored
a lot of the util/ip module, including naming it more sensibly.
The biggest changes are that I moved the big case statement into
a constant and then used a bit of dispatch-style logic to use it,
and I eliminated a bunch of duplicate code in the ipmess.rb file.
Added some test data for FreeBSD and fixed a bug in my map
logic pointed out by Paul Nasrat.
I've also fixed #1846, in that the interface list now s/:/_/g.
Signed-off-by: Luke Kanies <luke@madstop.com>
|
|
|
|
| |
Signed-off-by: Luke Kanies <luke@madstop.com>
|
|
|
|
| |
Without this call, every time there's a timeout, we'll get a zombie.
|
|\ |
|
| | |
|
| | |
|
| | |
|
| | |
|
|/ |
|
|
|
|
|
|
|
| |
I don't like using Fact.add(:fact, :limit => 5), so I
added a little extra code to allow us to continue using
':timeout => 5', but use 'limit' for retrieving the value,
to avoid the conflict Steve Hajducko is seeing.
|
|
|
|
|
| |
This was done to avoid a possible variable scoping issue
that was occuring on some AIX systems.
|
|
|
|
|
|
| |
but the default is still 0.5.
Set the timeout on the AIX kernelrelease fact to 5 seconds.
|
|
|
|
|
| |
The timeout is currently 0.5 seconds, which should generally
be enough time.
|
|
|
|
|
| |
Facter no longer knows anything about Puppet, so there's
no inter-dependency issue.
|
|
|
|
|
|
|
|
|
| |
I was previously creating a mock Puppet class, and this
actually uses Puppet's code.
Note that this use of Puppet creates some additional ordering
problems, which I'm going to fix in the next commit by
undoing all of this work. :)
|
|
|
|
|
| |
I was only autoloading in certain circumstances, but it
now autoloads any time you try to load a fact directly.
|
|
|
|
|
| |
This should make it possible to move all facts to separate
files and only load them on demand.
|
|
|
|
|
|
| |
It creates a constant that affects loader behaviour,
which causes other tests to break if the constant is
present but not functional.
|
|
This makes it easier for our loader to distinguish between
code that Facter uses and new facts.
|