Merge pull request #72 from namesco/add-pspell-extension

Add pspell extension

Author: mattheath

manifests/extension/pspell.pp

@@ -0,0 +1,45 @@
+# Installs the pspell extension for a specific version of php.
+#
+# Usage:
+#
+#     php::extension::pspell { 'pspell for 5.4.10':
+#       php => '5.4.10'
+#     }
+#
+define php::extension::pspell(
+  $php,
+) {
+  require php::config
+  require aspell
+
+  # Ensure that the specified version of PHP is installed.
+  php_require($php)
+
+  $extension = 'pspell'
+
+  # Final module install path
+  $module_path = "${php::config::root}/versions/${php}/modules/${extension}.so"
+
+  # Additional options
+  $configure_params = "--with-pspell=${boxen::config::homebrewdir}/opt/aspell"
+
+  php_extension { $name:
+    provider         => php_source,
+
+    extension        => $extension,
+
+    homebrew_path    => $boxen::config::homebrewdir,
+    phpenv_root      => $php::config::root,
+    php_version      => $php,
+
+    configure_params => $configure_params,
+  }
+
+  # Add config file once extension is installed
+
+  file { "${php::config::configdir}/${php}/conf.d/${extension}.ini":
+    content => template('php/extensions/generic.ini.erb'),
+    require => Php_extension[$name],
+  }
+
+}

spec/defines/extensions/php_extension_pspell_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe "php::extension::pspell" do
+  let(:facts) { default_test_facts }
+  let(:title) { "pspell for 5.4.17" }
+  let(:params) do
+    {
+      :php     => "5.4.17",
+    }
+  end
+
+  it do
+    should contain_class("php::config")
+    should contain_class("aspell")
+    should contain_php__version("5.4.17")
+
+    should contain_php_extension("pspell for 5.4.17").with({
+      :provider         => "php_source",
+      :extension        => "pspell",
+      :homebrew_path    => "/test/boxen/homebrew",
+      :phpenv_root      => "/test/boxen/phpenv",
+      :php_version      => "5.4.17",
+      :configure_params => "--with-pspell=/test/boxen/homebrew/opt/aspell",
+    })
+
+    should contain_file("/test/boxen/config/php/5.4.17/conf.d/pspell.ini").with({
+      :content => File.read("spec/fixtures/pspell.ini"),
+      :require => "Php_extension[pspell for 5.4.17]"
+    })
+  end
+end

spec/fixtures/Puppetfile

@@ -12,3 +12,4 @@ mod "mysql",       "1.1.3", :github_tarball => "boxen/puppet-mysql"
 mod "postgresql",  "2.0.1", :github_tarball => "boxen/puppet-postgresql"
 mod "nginx",       "1.4.2", :github_tarball => "boxen/puppet-nginx"
 mod "module_data", "0.0.3", :github_tarball => "ripienaar/puppet-module-data"
+mod "aspell",      "0.0.1", :github_tarball => "namesco/puppet-aspell"

spec/fixtures/Puppetfile.lock

@@ -58,6 +58,11 @@ GITHUBTARBALL
   specs:
     wget (1.0.0)
 
+GITHUBTARBALL
+  remote: namesco/puppet-aspell
+  specs:
+    aspell (0.0.1)
+
 GITHUBTARBALL
   remote: puppetlabs/puppetlabs-stdlib
   specs:
@@ -69,6 +74,7 @@ GITHUBTARBALL
     module_data (0.0.3)
 
 DEPENDENCIES
+  aspell (= 0.0.1)
   autoconf (= 1.0.0)
   boxen (= 3.0.2)
   homebrew (= 1.4.1)

spec/fixtures/pspell.ini

@@ -0,0 +1 @@
+extension=/test/boxen/phpenv/versions/5.4.17/modules/pspell.so