Treat securestring case insentive

fixes https://bugzilla.redhat.com/show_bug.cgi?id=1543629
bzwei · Feb 9, 2018 · 4fe6037 · 4fe6037
1 parent 386b14d
commit 4fe6037
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/app/models/manageiq/providers/azure/cloud_manager/orchestration_template.rb b/app/models/manageiq/providers/azure/cloud_manager/orchestration_template.rb
@@ -20,7 +20,7 @@ def parameters
         :label         => key.titleize,
         :data_type     => val['type'],
         :default_value => val['defaultValue'],
-        :hidden        => val['type'] == 'securestring',
+        :hidden        => val['type'].casecmp('securestring').zero?,
         :required      => true
       )
 

diff --git a/spec/fixtures/orchestration_templates/azure_parameters.json b/spec/fixtures/orchestration_templates/azure_parameters.json
@@ -9,7 +9,7 @@
             }
         },
         "adminPassword": {
-            "type": "securestring",
+            "type": "secureString",
             "metadata": {
               "description": "Admin password"
             }

diff --git a/spec/models/manageiq/providers/azure/cloud_manager/orchestration_template_spec.rb b/spec/models/manageiq/providers/azure/cloud_manager/orchestration_template_spec.rb
@@ -28,7 +28,7 @@ def assert_secret_type(parameter)
       :name          => "adminPassword",
       :label         => "Admin Password",
       :description   => "Admin password",
-      :data_type     => "securestring",
+      :data_type     => "secureString",
       :default_value => nil,
       :hidden        => true,
       :required      => true,