Merge pull request #487 from voxpupuli/version

replace repo_url_suffix with repo_version

README.md

@@ -57,11 +57,11 @@ rather than 20.x on Debian/RHEL platforms:
 
 ```puppet
 class { 'nodejs':
-  repo_url_suffix => '21.x',
+  repo_version => '21',
 }
 ```
 
-See the `repo_url_suffix` parameter entry below for possible values.
+See the `repo_version` parameter entry below for possible values.
 
 ## Usage
 
@@ -91,15 +91,15 @@ class { '::nodejs':
 ### Upgrades
 
 The parameter `nodejs_package_ensure` defaults to `installed`. Changing the
-`repo_url_suffix` will not result in a new version being installed. Changing
+`repo_version` will not result in a new version being installed. Changing
 the `nodejs_package_ensure` parameter should provide the desired effect.
 
 For example:
 
 ```puppet
 # Upgrade from nodejs 5.x to 6.x
 class { 'nodejs':
-  repo_url_suffix       => '6.x',
+  repo_version          => '6',
   nodejs_package_ensure => '6.12.2',
 }
 ```

manifests/init.pp

@@ -23,7 +23,7 @@
   $repo_proxy_password                                 = $nodejs::params::repo_proxy_password,
   $repo_proxy_username                                 = $nodejs::params::repo_proxy_username,
   Optional[String] $repo_release                       = $nodejs::params::repo_release,
-  $repo_url_suffix                                     = $nodejs::params::repo_url_suffix,
+  String[1] $repo_version                              = $nodejs::params::repo_version,
   Array $use_flags                                     = $nodejs::params::use_flags,
   Optional[String] $package_provider                   = $nodejs::params::package_provider,
 ) inherits nodejs::params {

manifests/params.pp

@@ -11,9 +11,9 @@
   $repo_proxy_password         = 'absent'
   $repo_proxy_username         = 'absent'
   $repo_release                = undef
-  $repo_url_suffix             = ($facts['os']['family'] == 'RedHat' and $facts['os']['release']['major'] == '7') ? {
-    true    => '16.x',
-    default => '20.x',
+  $repo_version                = ($facts['os']['family'] == 'RedHat' and $facts['os']['release']['major'] == '7') ? {
+    true    => '16',
+    default => '20',
   }
   $use_flags                   = ['npm', 'snapshot']
 

manifests/repo/nodesource.pp

@@ -8,7 +8,8 @@
   $proxy_password = $nodejs::repo_proxy_password
   $proxy_username = $nodejs::repo_proxy_username
   $release        = $nodejs::repo_release
-  $url_suffix     = $nodejs::repo_url_suffix
+
+  $url_suffix     = "${nodejs::repo_version}.x"
 
   case $facts['os']['family'] {
     'RedHat': {

spec/classes/nodejs_spec.rb

@@ -134,9 +134,9 @@
           end
         end
 
-        context 'and repo_url_suffix set to 9.x' do
+        context 'and repo_version set to 9' do
           let :params do
-            default_params.merge!(repo_url_suffix: '9.x')
+            default_params.merge!(repo_version: '9')
           end
 
           it 'the repo apt::source resource should contain location = https://deb.nodesource.com/node_9.x' do
@@ -372,9 +372,9 @@
           end
         end
 
-        context 'and repo_url_suffix set to 5.x' do
+        context 'and repo_version set to 5' do
           let :params do
-            default_params.merge!(repo_url_suffix: '5.x')
+            default_params.merge!(repo_version: '5')
           end
 
           it "the yum nodesource repo resource should contain baseurl = https://rpm.nodesource.com/pub_5.x/#{dist_type}/#{operatingsystemmajrelease}/$basearch" do