add parameter $manage_nodejs_package

README.md

@@ -380,6 +380,10 @@ applied before the local installation of npm packages using `nodejs::npm`.
 Path to cmd.exe on Windows. Defaults to C:\Windows\system32\cmd.exe. You may
 need to change this parameter for certain versions of Windows Server.
 
+#### `manage_nodejs_package`
+
+Whether to manage the nodejs and nodejs-dev packages. Defaults to `true`.
+
 #### `manage_package_repo`
 
 Whether to manage an external repository and use it as the source of the

manifests/init.pp

@@ -1,6 +1,7 @@
 # == Class: nodejs: See README.md for documentation.
 class nodejs (
  $cmd_exe_path = $nodejs::params::cmd_exe_path,
+ Boolean $manage_nodejs_package = true,
  Boolean $manage_package_repo = $nodejs::params::manage_package_repo,
  $nodejs_debug_package_ensure = $nodejs::params::nodejs_debug_package_ensure,
  Optional[String] $nodejs_debug_package_name = $nodejs::params::nodejs_debug_package_name,

manifests/install.pp

@@ -8,7 +8,7 @@
  }
 
  # npm is a Gentoo USE flag
- if $facts['os']['name'] == 'Gentoo' {
+ if $facts['os']['name'] == 'Gentoo' and $nodejs::manage_nodejs_package {
  package_use { $nodejs::nodejs_package_name:
  ensure => present,
  target => 'nodejs-flags',
@@ -20,13 +20,15 @@
  Package { provider => $nodejs::package_provider }
 
  # nodejs
- package { $nodejs::nodejs_package_name:
- ensure => $nodejs::nodejs_package_ensure,
- tag => 'nodesource_repo',
+ if $nodejs::manage_nodejs_package {
+ package { $nodejs::nodejs_package_name:
+ ensure => $nodejs::nodejs_package_ensure,
+ tag => 'nodesource_repo',
+ }
  }
 
  # nodejs-development
- if $nodejs::nodejs_dev_package_name {
+ if $nodejs::manage_nodejs_package and $nodejs::nodejs_dev_package_name {
  package { $nodejs::nodejs_dev_package_name:
  ensure => $nodejs::nodejs_dev_package_ensure,
  tag => 'nodesource_repo',

spec/classes/nodejs_spec.rb

@@ -59,6 +59,27 @@
  }
  end
 
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ default_params.merge!(manage_nodejs_package: true)
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ default_params.merge!(manage_nodejs_package: false)
+ end
+
+ it 'the nodejs and dev package resources should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ is_expected.not_to contain_package(native_debian_devel_package)
+ end
+ end
+
  context 'and repo_class set to ::nodejs::repo::nodesource' do
  let :params do
  default_params.merge!(repo_class: 'nodejs::repo::nodesource')
@@ -351,6 +372,27 @@
  }
  end
 
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ default_params.merge!(manage_nodejs_package: true)
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ default_params.merge!(manage_nodejs_package: false)
+ end
+
+ it 'the nodejs and dev package resources should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ is_expected.not_to contain_package('nodejs-devel')
+ end
+ end
+
  context 'and repo_class set to ::nodejs::repo::nodesource' do
  let :params do
  default_params.merge!(repo_class: 'nodejs::repo::nodesource')
@@ -686,6 +728,32 @@
  end
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs and dev package resources should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ is_expected.not_to contain_package('nodejs-devel')
+ end
+ end
+
  # nodejs_package_ensure
  context 'with nodejs_package_ensure set to present' do
  let :params do
@@ -747,6 +815,31 @@
  }
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs package resource should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ end
+ end
+
  # nodejs_package_ensure
  context 'with nodejs_package_ensure set to present' do
  let :params do
@@ -783,6 +876,32 @@
  }
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('www/node')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs and dev package resources should not be present' do
+ is_expected.not_to contain_package('www/node')
+ is_expected.not_to contain_package('www/node-devel')
+ end
+ end
+
  # nodejs_dev_package_ensure
  context 'with nodejs_dev_package_ensure set to present' do
  let :params do
@@ -869,6 +988,31 @@
  }
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('node')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs package resource should not be present' do
+ is_expected.not_to contain_package('node')
+ end
+ end
+
  # nodejs_package_ensure
  context 'with nodejs_package_ensure set to present' do
  let :params do
@@ -905,6 +1049,32 @@
  }
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs and dev package resources should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ is_expected.not_to contain_package('nodejs-devel')
+ end
+ end
+
  # nodejs_dev_package_ensure
  context 'with nodejs_dev_package_ensure set to present' do
  let :params do
@@ -1012,6 +1182,31 @@
  }
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs package resource should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ end
+ end
+
  # nodejs_package_ensure
  context 'with nodejs_package_ensure set to present' do
  let :params do
@@ -1073,6 +1268,31 @@
  }
  end
 
+ # manage_nodejs_package
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ {
+ manage_nodejs_package: true
+ }
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('net-libs/nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ {
+ manage_nodejs_package: false
+ }
+ end
+
+ it 'the nodejs package resource should not be present' do
+ is_expected.not_to contain_package('net-libs/nodejs')
+ end
+ end
+
  # nodejs_package_ensure
  context 'with nodejs_package_ensure set to present' do
  let :params do
@@ -1139,6 +1359,27 @@
  }
  end
 
+ context 'and manage_nodejs_package set to true' do
+ let :params do
+ default_params.merge!(manage_nodejs_package: true)
+ end
+
+ it 'the nodejs package resource should be present' do
+ is_expected.to contain_package('nodejs')
+ end
+ end
+
+ context 'and manage_nodejs_package set to false' do
+ let :params do
+ default_params.merge!(manage_nodejs_package: false)
+ end
+
+ it 'the nodejs and dev package resources should not be present' do
+ is_expected.not_to contain_package('nodejs')
+ is_expected.not_to contain_package('nodejs-devel')
+ end
+ end
+
  context 'and repo_class set to ::nodejs::repo::nodesource' do
  let :params do
  default_params.merge!(repo_class: 'nodejs::repo::nodesource')