Skip to content

Commit a909ab7

Browse files
committed
ACL must be specified in _Role
1 parent 5f9b5d5 commit a909ab7

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

spec/ParseRole.spec.js

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,7 @@ describe('Parse Role testing', () => {
6565
var rolesNames = ["FooRole", "BarRole", "BazRole"];
6666

6767
var createRole = function(name, parent, user) {
68-
var role = new Parse.Object("_Role")
69-
role.set("name", name);
68+
var role = new Parse.Role(name, new Parse.ACL());
7069
if (user) {
7170
var users = role.relation('users');
7271
users.add(user);
@@ -97,6 +96,7 @@ describe('Parse Role testing', () => {
9796
})
9897
done();
9998
}, function(err){
99+
console.log('error?',err);
100100
fail("should succeed")
101101
done();
102102
});
@@ -109,7 +109,14 @@ describe('Parse Role testing', () => {
109109
fail("_Role object should not save without name.");
110110
}, (error) => {
111111
expect(error.code).toEqual(111);
112-
done();
112+
role.set('name','testRole');
113+
role.save(null,{useMasterKey:true})
114+
.then((r2)=>{
115+
fail("_Role object should not save without ACL.");
116+
}, (error2) =>{
117+
expect(error2.code).toEqual(111);
118+
done();
119+
});
113120
});
114121
});
115122

src/Schema.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ var defaultColumns = {
7474

7575
var requiredColumns = {
7676
_Product: ["productIdentifier", "icon", "order", "title", "subtitle"],
77-
_Role: ["name"]
77+
_Role: ["name", "ACL"]
7878
}
7979

8080
// Valid classes must:

0 commit comments

Comments
 (0)