From dbcd04ab535f5a56139d671e4aeef9b05d07983f Mon Sep 17 00:00:00 2001
From: Iristyle <Iristyle@github>
Date: Wed, 3 Jun 2015 11:15:23 -0700
Subject: [PATCH] (maint) fix future parser acceptance failures

 - Previously during SQL Server install as part of the pre-suite, a
   manifest was generated with:

    features => SQL

   Future parser does not like this, and expects an actual array with
   quoted values like

    features => [ 'SQL' ]
---
 spec/sql_testing_helpers.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/spec/sql_testing_helpers.rb b/spec/sql_testing_helpers.rb
index b398c0a1..33b5fe39 100644
--- a/spec/sql_testing_helpers.rb
+++ b/spec/sql_testing_helpers.rb
@@ -21,11 +21,11 @@ def mount_iso(host, opts = {})
 
 def install_sqlserver(host, opts = {})
   # this method installs SQl server on a given host
-  features = opts[:features]
+  features = opts[:features].map{ |x| "'#{x}'"}.join(', ')
   pp = <<-MANIFEST
     sqlserver_instance{'MSSQLSERVER':
       source => 'H:',
-      features => #{features},
+      features => [ #{features} ],
       security_mode => 'SQL',
       sa_pwd => 'Pupp3t1@',
       sql_sysadmin_accounts => ['Administrator'],
@@ -94,7 +94,7 @@ def base_install(sql_version)
   # Mount the ISO on the agent
   mount_iso(host, iso_opts)
   # Install Microsoft SQL on the agent before running any tests
-  install_sqlserver(host, {:features => 'SQL'})
+  install_sqlserver(host, {:features => ['SQL']})
 end
 
 def validate_sql_install(host, opts = {}, &block)