From 5149d52794ac1a3df0263374f789cc39dc1580d2 Mon Sep 17 00:00:00 2001
From: Ben Keen <ben.keen@gmail.com>
Date: Wed, 20 May 2015 09:23:11 -0700
Subject: [PATCH] Small fix for Permissions page and tests

The mocha tests were failing (correctly!) due to extra check to
prevent adding blank entries.
---
 app/addons/permissions/components.react.jsx   |  3 +--
 .../tests/componentsSpec.react.jsx            | 20 +++++++++++++++----
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/app/addons/permissions/components.react.jsx b/app/addons/permissions/components.react.jsx
index 7b07a0f2e..97d704a81 100644
--- a/app/addons/permissions/components.react.jsx
+++ b/app/addons/permissions/components.react.jsx
@@ -65,7 +65,6 @@ function (app, FauxtonAPI, React, Components, Stores, Actions) {
       if (!_.isEmpty(value)) {
         return false;
       }
-
       FauxtonAPI.addNotification({
         msg: 'Cannot add an empty value for ' + type + '.',
         type: 'warning'
@@ -76,7 +75,7 @@ function (app, FauxtonAPI, React, Components, Stores, Actions) {
 
     addNames: function (e) {
       e.preventDefault();
-      if (this.isEmptyValue(this.state.newRole, 'names')) {
+      if (this.isEmptyValue(this.state.newName, 'names')) {
         return;
       }
       this.props.addItem({
diff --git a/app/addons/permissions/tests/componentsSpec.react.jsx b/app/addons/permissions/tests/componentsSpec.react.jsx
index acbb6686e..795f4cbdf 100644
--- a/app/addons/permissions/tests/componentsSpec.react.jsx
+++ b/app/addons/permissions/tests/componentsSpec.react.jsx
@@ -85,8 +85,14 @@ define([
         });
 
         it('adds user on submit', function () {
-          var dom = $(el.getDOMNode()).find('.permission-item-form')[0];
-          TestUtils.Simulate.submit(dom);
+          var input = $(el.getDOMNode()).find('input')[0];
+          TestUtils.Simulate.change(input, {
+            target: {
+              value: 'newusername'
+            }
+          });
+          var form = $(el.getDOMNode()).find('.permission-item-form')[0];
+          TestUtils.Simulate.submit(form);
 
           var options = addSpy.args[0][0];
           assert.ok(addSpy.calledOnce);
@@ -95,8 +101,14 @@ define([
         });
 
         it('adds role on submit', function () {
-          var dom = $(el.getDOMNode()).find('.permission-item-form')[1];
-          TestUtils.Simulate.submit(dom);
+          var input = $(el.getDOMNode()).find('input')[1];
+          TestUtils.Simulate.change(input, {
+            target: {
+              value: 'newrole'
+            }
+          });
+          var form = $(el.getDOMNode()).find('.permission-item-form')[1];
+          TestUtils.Simulate.submit(form);
 
           var options = addSpy.args[0][0];
           assert.ok(addSpy.calledOnce);