AMBARI-8003 UI changes for Ambari Datanode directories need to allow for heterogeneous storage tags 2. (ababiichuk)

ambari-web/app/models/service_config.js

@@ -808,7 +808,7 @@ App.ServiceConfigProperty = Ember.Object.extend({
           break;
         case 'datanodedirs':
           if (!validator.isValidDataNodeDir(value)) {
-            this.set('errorMessage', 'dir format is wrong, can be "[{storage type}] /{dir name}"');
+            this.set('errorMessage', 'dir format is wrong, can be "[{storage type}]/{dir name}"');
             isError = true;
           }
           else {

ambari-web/app/utils/validator.js

@@ -62,7 +62,7 @@ module.exports = {
    * @returns {boolean}
    */
   isValidDataNodeDir: function(value) {
-    var floatRegex = /(^\[[0-9a-z]+\]$)|(^\/[0-9a-z]*)/;
+    var floatRegex = /^(\[[0-9a-z]+\])?(\/[0-9a-z]*)/;
     var dirs = value.replace(/,/g,' ').trim().split(new RegExp("\\s+", "g"));
     for(var i = 0; i < dirs.length; i++){
       if(!floatRegex.test(dirs[i])){

ambari-web/test/utils/validator_test.js

@@ -296,7 +296,7 @@ describe('validator', function () {
       })
     });
   });
-  describe.only('#isValidDataNodeDir(value)', function() {
+  describe('#isValidDataNodeDir(value)', function() {
     var tests = [
       {m:'"dir" - invalid',i:'dir',e:false},
       {m:'"/dir" - valid',i:'/dir',e:true},
@@ -305,9 +305,9 @@ describe('validator', function () {
       {m:'"/123" - valid',i:'/111',e:true},
       {m:'"/abc" - valid',i:'/abc',e:true},
       {m:'"/1a2b3c" - valid',i:'/1a2b3c',e:true},
-      {m:'"[ssd] /1a2b3c" - valid',i:'[ssd] /1a2b3c',e:true},
+      {m:'"[ssd]/1a2b3c" - valid',i:'[ssd]/1a2b3c',e:true},
       {m:'"[] /1a2b3c" - invalid',i:'[] /1a2b3c',e:false},
-      {m:'"[ssd]/1a2b3c" - invalid',i:'[ssd]/1a2b3c',e:false},
+      {m:'"[ssd] /1a2b3c" - invalid',i:'[ssd] /1a2b3c',e:false},
       {m:'"[/1a2b3c]" - invalid',i:'[/1a2b3c]',e:false},
       {m:'"[s]ss /sd" - invalid',i:'[s]ss /sd',e:false}