add Stdlib::Filesource

Author: b4ldr

README.md

@@ -409,8 +409,40 @@ C:\\
 
 Unacceptable input example:
 
+#### `Stdlib::Filesource`
+
+Matches paths valid values for the source parameter of the puppet file type
+
+Acceptable input example:
+
 ```shell
-/usr2/username/bin:/usr/local/bin:/usr/bin:.
+http://example.com
+
+https://example.com
+
+file:///hello/bla
+
+/usr2/username/bin
+
+C:\foo\bar
+
+/var/opt/../lib/puppet
+
+puppet:///modules/foo/bar.log
+```
+
+Unacceptable input example:
+
+```shell
+*/Users//nope
+
+\Users/hc/wksp/stdlib
+
+C:noslashes
+
+puppet:///foo/bar.log
+
+ftp://ftp.example.com
 ```
 
 ### Facts

spec/aliases/filesource_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+if Puppet::Util::Package.versioncmp(Puppet.version, '4.5.0') >= 0
+  describe 'test::filesource', type: :class do
+    describe 'valid handling' do
+      %w(
+        https://hello.com
+        https://notcreative.org
+        https://canstillaccepthttps.co.uk
+        http://anhttp.com
+        http://runningoutofideas.gov
+        file:///hello/bla
+        file:///foo/bar.log
+        puppet:///modules/foo/bar.log
+        /usr2/username/bin:/usr/local/bin:/usr/bin:.
+        C:/
+        C:\\
+        C:\\WINDOWS\\System32
+        C:/windows/system32
+        X:/foo/bar
+        X:\\foo\\bar
+        \\\\host\\windows
+        //host/windows
+        /var/tmp
+        /var/opt/../lib/puppet
+      ).each do |value|
+        describe value.inspect do
+          let(:params) { { value: value } }
+          it { is_expected.to compile }
+        end
+      end
+    end
+
+    describe 'invalid path handling' do
+      context 'garbage inputs' do
+        [
+          nil,
+          [nil],
+          [nil, nil],
+          { 'foo' => 'bar' },
+          {},
+          '',
+          '*/Users//nope',
+          '\\Users/hc/wksp/stdlib',
+          'C:noslashes',
+          '\\var\\tmp',
+          'puppet:///foo/bar.log'
+        ].each do |value|
+          describe value.inspect do
+            let(:params) { { value: value } }
+            it do
+              is_expected.to compile.and_raise_error(
+                %r{parameter 'value' expects a Stdlib::Filesource}
+              )
+            end
+          end
+        end
+      end
+    end
+  end
+end

spec/fixtures/test/manifests/filesource.pp

@@ -0,0 +1,6 @@
+# Class to test the Stdlib::Filesource type alias
+class test::filesource (
+  Stdlib::Filesource $value,
+) {
+  notice('Success')
+}

types/filesource.pp

@@ -0,0 +1,9 @@
+# Validate the source parameter on file types
+type Stdlib::Filesource = Variant[
+  Stdlib::Absolutepath,
+  Stdlib::HTTPUrl,
+  Pattern[
+    /^file:\/\/\/([^\/\0]+(\/)?)+$/,
+    /^puppet:\/\/\/modules\/([^\/\0]+(\/)?)+$/,
+  ],
+]