Dan Bode [Tue, 2 Apr 2013 00:27:10 +0000 (17:27 -0700)]
fix unit test failures
the previous tests were stubbing global class instances
in a way that was causing other tests to fail.
This commit updates the tests, and introdcuces child classes which
can safely be stubbed (and more closely test the real behavior intended to
be tested here)
Dan Bode [Mon, 7 Jan 2013 23:54:41 +0000 (15:54 -0800)]
Add purging support to ini file
This commit adds purging to ini file native types.
Purging will only work for child providers that
implement the method: self.file_path. This is
because collecting all instances of the type (
which is a requirement for purging) is only possible
when the path of the file that is being managed
can be accessed by the class instance (in the
method self.instances.)
This commit adds the following method to the internal
of the ini_file:
- get_settings - has of all settings/values for
a given section
It also adds the following method to the section
class:
- setting_names - list of all setting names in a
section.
These methods are required for the instances method
to be able to list the values for all settings of
each section.
Chris Price [Sat, 27 Oct 2012 00:24:37 +0000 (17:24 -0700)]
Add detection for commented versions of settings
This commit adds support for detecting commented versions of
settings in an existing version of an inifile. If you are
setting a value for a setting that isn't currently set
in the file, but a commented version is found, then we
add the new setting immediately following the commented
version, rather than at the end of the section.
Chris Price [Sun, 21 Oct 2012 06:14:39 +0000 (23:14 -0700)]
Refactor to clarify implementation of `save`
The `save` method was previously relying on some really
specific implementation details of the `section` class
(when the start/end_line would be nil, etc.). This made
the code a bit hard to follow.
This commit introduces a few utility methods in the
`section` class (`is_new_section?`, `is_global?`), and
refactors the `save` method to use them... this makes
the logic a little easier to follow and should hopefully
make it easier to maintain.
Chris Price [Sat, 20 Oct 2012 07:24:37 +0000 (00:24 -0700)]
Better handling of whitespace lines at ends of sections
This is another bit of cosmetic functionality; prior to
this commit, when adding a new setting to a section, we'd
write it on the very last line before the next section,
even if there was a chunk of trailing whitespace lines
at the end of the existing section. This was functional,
but the output was not always particularly pleasant for
human consumption.
This commit tweaks things so that we insert new settings
just before the final chunk of whitespace lines in an
existing section. This keeps things a bit cleaner.
Chris Price [Sat, 20 Oct 2012 07:08:06 +0000 (00:08 -0700)]
Respect indentation / spacing for existing sections and settings
This commit adds some cosmetic functionality. The main two
improvements are:
* We'll now pay attention to indentation within existing
sections, and when we write new settings or update
existing ones, we'll match the existing indentation.
* When modifying existing settings, the regex now captures
a greater portion of the original line and preserves it.
Basically, the original whitespacing in the line should
remain intact and only the value should be modified.
Chris Price [Wed, 17 Oct 2012 20:27:28 +0000 (13:27 -0700)]
Minor tweaks to handling of removing settings
This commit makes some minor changes to how we handle removing
settings. In particular, it updates all of the line numbers
of the various 'section' objects to correspond to the new
state of the world based on the removal of a line that appeared
before them.
Also adds one more test related to setting removal.
Dan Bode [Wed, 3 Oct 2012 01:52:34 +0000 (18:52 -0700)]
Make ruby provider a better parent.
In order to allow the provider to be a parent for
other providers, I have implemented the following
methods: section, setting, file_path, separator so
that they can be overridden by child providers and
decouple this provider from its type.
Chris Price [Wed, 19 Sep 2012 22:42:16 +0000 (15:42 -0700)]
Allow overriding separator string between key/val pairs
This introduces a new parameter, 'key_val_separator', which
can be set in order to override the string that is used
as a separator between the key/value pair of a setting line.
The default is ' = ', but you could set the param to '=' if
you don't want to include whitespace in your settings file.
Chris Price [Fri, 17 Aug 2012 11:48:28 +0000 (04:48 -0700)]
Add support for "global" section at beginning of file
This commit does the following:
* Fixes a bug in ExternalIterator
* Adds support for a "global" section before the first named
section at the beginning of the INI file
* Improves test coverage
Stephen [Tue, 14 Aug 2012 12:38:08 +0000 (13:38 +0100)]
Fixed regex to match sections and settings with non alphanumeric
characters. Fixed writing to file without any sections at all.
Fixed exists checking for variable type by always casting to string
and added all the tests for the above items.
projects / puppet-inifile.git / log