From a909ab71e3f5845e2a0b807f55915e2a02b6a610 Mon Sep 17 00:00:00 2001
From: jim1_lin <jim1_lin@asus.com>
Date: Wed, 2 Mar 2016 16:08:39 +0800
Subject: [PATCH] ACL must be specified in _Role

---
 spec/ParseRole.spec.js | 13 ++++++++++---
 src/Schema.js          |  2 +-
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/spec/ParseRole.spec.js b/spec/ParseRole.spec.js
index a217e830..be2de4a2 100644
--- a/spec/ParseRole.spec.js
+++ b/spec/ParseRole.spec.js
@@ -65,8 +65,7 @@ describe('Parse Role testing', () => {
     var rolesNames = ["FooRole", "BarRole", "BazRole"];
 
     var createRole = function(name, parent, user) {
-      var role = new Parse.Object("_Role")
-      role.set("name", name);
+      var role = new Parse.Role(name, new Parse.ACL());
       if (user) {
         var users = role.relation('users');
         users.add(user);
@@ -97,6 +96,7 @@ describe('Parse Role testing', () => {
        })
        done();
      }, function(err){
+       console.log('error?',err);
        fail("should succeed")
        done();
      });
@@ -109,7 +109,14 @@ describe('Parse Role testing', () => {
       fail("_Role object should not save without name.");
     }, (error) => {
       expect(error.code).toEqual(111);
-      done();
+      role.set('name','testRole');
+      role.save(null,{useMasterKey:true})
+      .then((r2)=>{
+        fail("_Role object should not save without ACL.");
+      }, (error2) =>{
+        expect(error2.code).toEqual(111);
+        done();
+      });
     });
   });
 
diff --git a/src/Schema.js b/src/Schema.js
index 70ea57f7..b16dfd18 100644
--- a/src/Schema.js
+++ b/src/Schema.js
@@ -74,7 +74,7 @@ var defaultColumns = {
 
 var requiredColumns = {
   _Product: ["productIdentifier", "icon", "order", "title", "subtitle"],
-  _Role: ["name"]
+  _Role: ["name", "ACL"]
 }
 
 // Valid classes must: