updated tests

Dobbias Nan · Dobbias Nan · commit 39ca315d5015 · 2019-08-21T23:24:27.000+02:00
diff --git a/spec/ProtectedFields.spec.js b/spec/ProtectedFields.spec.js
@@ -167,7 +167,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: { '*': ['owner'], 'userField:owner': [] },
           }
         );
@@ -194,7 +193,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: { '*': ['owner'], 'userField:owner': [] },
           }
         );
@@ -221,7 +219,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: { '*': ['owner'], 'userField:owner': [] },
           }
         );
@@ -247,7 +244,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners'],
             protectedFields: { '*': ['owners'], 'userField:owners': [] },
           }
         );
@@ -278,7 +274,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners'],
             protectedFields: { '*': ['owners'], 'userField:owners': [] },
           }
         );
@@ -305,7 +300,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners'],
             protectedFields: { '*': ['owners'], 'userField:owners': [] },
           }
         );
@@ -332,7 +326,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners', 'owner'],
             protectedFields: {
               '*': [],
               'userField:owners': ['owners'],
@@ -350,7 +343,7 @@ describe('ProtectedFields', function() {
         done();
       });
 
-      it('should ignore pointer-permission fields not declared in the readUserFields', async done => {
+      it('should ignore pointer-permission fields not present in object', async done => {
         const config = Config.get(Parse.applicationId);
         const obj = new Parse.Object('AnObject');
 
@@ -366,11 +359,10 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: [],
             protectedFields: {
               '*': [],
-              'userField:owners': ['idontexist'],
-              'userField:owner': ['idontexist2'],
+              'userField:idontexist': ['owner'],
+              'userField:idontexist2': ['owners'],
             },
           }
         );
@@ -403,7 +395,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: { '*': ['owner'], 'userField:owner': [] },
           }
         );
@@ -441,7 +432,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: { '*': ['owner'], 'userField:owner': [] },
           }
         );
@@ -478,7 +468,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: { '*': ['owner'], 'userField:owner': [] },
           }
         );
@@ -514,7 +503,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners'],
             protectedFields: { '*': ['owners'], 'userField:owners': [] },
           }
         );
@@ -564,7 +552,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners'],
             protectedFields: { '*': ['owners'], 'userField:owners': [] },
           }
         );
@@ -601,7 +588,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners'],
             protectedFields: { '*': ['owners'], 'userField:owners': [] },
           }
         );
@@ -639,7 +625,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owners', 'owner'],
             protectedFields: {
               '*': [],
               'userField:owners': ['owners'],
@@ -666,7 +651,7 @@ describe('ProtectedFields', function() {
         done();
       });
 
-      it('should ignore pointer-permission fields not declared in the readUserFields', async done => {
+      it('should ignore pointer-permission fields not present in object', async done => {
         const config = Config.get(Parse.applicationId);
         const obj = new Parse.Object('AnObject');
         const obj2 = new Parse.Object('AnObject');
@@ -686,11 +671,10 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: [],
             protectedFields: {
               '*': [],
-              'userField:owners': ['idontexist'],
-              'userField:owner': ['idontexist2'],
+              'userField:idontexist': ['owner'],
+              'userField:idontexist2': ['owners'],
             },
           }
         );
@@ -733,7 +717,6 @@ describe('ProtectedFields', function() {
           {
             get: { '*': true },
             find: { '*': true },
-            readUserFields: ['owner'],
             protectedFields: {
               '*': ['owner'],
               'userField:owner': [],

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,6 @@ describe('ProtectedFields', function() {`
`167`	`167`	`{`
`168`	`168`	`get: { '*': true },`
`169`	`169`	`find: { '*': true },`
`170`		`- readUserFields: ['owner'],`
`171`	`170`	`protectedFields: { '*': ['owner'], 'userField:owner': [] },`
`172`	`171`	`}`
`173`	`172`	`);`
`@@ -194,7 +193,6 @@ describe('ProtectedFields', function() {`
`194`	`193`	`{`
`195`	`194`	`get: { '*': true },`
`196`	`195`	`find: { '*': true },`
`197`		`- readUserFields: ['owner'],`
`198`	`196`	`protectedFields: { '*': ['owner'], 'userField:owner': [] },`
`199`	`197`	`}`
`200`	`198`	`);`
`@@ -221,7 +219,6 @@ describe('ProtectedFields', function() {`
`221`	`219`	`{`
`222`	`220`	`get: { '*': true },`
`223`	`221`	`find: { '*': true },`
`224`		`- readUserFields: ['owner'],`
`225`	`222`	`protectedFields: { '*': ['owner'], 'userField:owner': [] },`
`226`	`223`	`}`
`227`	`224`	`);`
`@@ -247,7 +244,6 @@ describe('ProtectedFields', function() {`
`247`	`244`	`{`
`248`	`245`	`get: { '*': true },`
`249`	`246`	`find: { '*': true },`
`250`		`- readUserFields: ['owners'],`
`251`	`247`	`protectedFields: { '*': ['owners'], 'userField:owners': [] },`
`252`	`248`	`}`
`253`	`249`	`);`
`@@ -278,7 +274,6 @@ describe('ProtectedFields', function() {`
`278`	`274`	`{`
`279`	`275`	`get: { '*': true },`
`280`	`276`	`find: { '*': true },`
`281`		`- readUserFields: ['owners'],`
`282`	`277`	`protectedFields: { '*': ['owners'], 'userField:owners': [] },`
`283`	`278`	`}`
`284`	`279`	`);`
`@@ -305,7 +300,6 @@ describe('ProtectedFields', function() {`
`305`	`300`	`{`
`306`	`301`	`get: { '*': true },`
`307`	`302`	`find: { '*': true },`
`308`		`- readUserFields: ['owners'],`
`309`	`303`	`protectedFields: { '*': ['owners'], 'userField:owners': [] },`
`310`	`304`	`}`
`311`	`305`	`);`
`@@ -332,7 +326,6 @@ describe('ProtectedFields', function() {`
`332`	`326`	`{`
`333`	`327`	`get: { '*': true },`
`334`	`328`	`find: { '*': true },`
`335`		`- readUserFields: ['owners', 'owner'],`
`336`	`329`	`protectedFields: {`
`337`	`330`	`'*': [],`
`338`	`331`	`'userField:owners': ['owners'],`
`@@ -350,7 +343,7 @@ describe('ProtectedFields', function() {`
`350`	`343`	`done();`
`351`	`344`	`});`
`352`	`345`
`353`		`- it('should ignore pointer-permission fields not declared in the readUserFields', async done => {`
	`346`	`+ it('should ignore pointer-permission fields not present in object', async done => {`
`354`	`347`	`const config = Config.get(Parse.applicationId);`
`355`	`348`	`const obj = new Parse.Object('AnObject');`
`356`	`349`
`@@ -366,11 +359,10 @@ describe('ProtectedFields', function() {`
`366`	`359`	`{`
`367`	`360`	`get: { '*': true },`
`368`	`361`	`find: { '*': true },`
`369`		`- readUserFields: [],`
`370`	`362`	`protectedFields: {`
`371`	`363`	`'*': [],`
`372`		`- 'userField:owners': ['idontexist'],`
`373`		`- 'userField:owner': ['idontexist2'],`
	`364`	`+ 'userField:idontexist': ['owner'],`
	`365`	`+ 'userField:idontexist2': ['owners'],`
`374`	`366`	`},`
`375`	`367`	`}`
`376`	`368`	`);`
`@@ -403,7 +395,6 @@ describe('ProtectedFields', function() {`
`403`	`395`	`{`
`404`	`396`	`get: { '*': true },`
`405`	`397`	`find: { '*': true },`
`406`		`- readUserFields: ['owner'],`
`407`	`398`	`protectedFields: { '*': ['owner'], 'userField:owner': [] },`
`408`	`399`	`}`
`409`	`400`	`);`
`@@ -441,7 +432,6 @@ describe('ProtectedFields', function() {`
`441`	`432`	`{`
`442`	`433`	`get: { '*': true },`
`443`	`434`	`find: { '*': true },`
`444`		`- readUserFields: ['owner'],`
`445`	`435`	`protectedFields: { '*': ['owner'], 'userField:owner': [] },`
`446`	`436`	`}`
`447`	`437`	`);`
`@@ -478,7 +468,6 @@ describe('ProtectedFields', function() {`
`478`	`468`	`{`
`479`	`469`	`get: { '*': true },`
`480`	`470`	`find: { '*': true },`
`481`		`- readUserFields: ['owner'],`
`482`	`471`	`protectedFields: { '*': ['owner'], 'userField:owner': [] },`
`483`	`472`	`}`
`484`	`473`	`);`
`@@ -514,7 +503,6 @@ describe('ProtectedFields', function() {`
`514`	`503`	`{`
`515`	`504`	`get: { '*': true },`
`516`	`505`	`find: { '*': true },`
`517`		`- readUserFields: ['owners'],`
`518`	`506`	`protectedFields: { '*': ['owners'], 'userField:owners': [] },`
`519`	`507`	`}`
`520`	`508`	`);`
`@@ -564,7 +552,6 @@ describe('ProtectedFields', function() {`
`564`	`552`	`{`
`565`	`553`	`get: { '*': true },`
`566`	`554`	`find: { '*': true },`
`567`		`- readUserFields: ['owners'],`
`568`	`555`	`protectedFields: { '*': ['owners'], 'userField:owners': [] },`
`569`	`556`	`}`
`570`	`557`	`);`
`@@ -601,7 +588,6 @@ describe('ProtectedFields', function() {`
`601`	`588`	`{`
`602`	`589`	`get: { '*': true },`
`603`	`590`	`find: { '*': true },`
`604`		`- readUserFields: ['owners'],`
`605`	`591`	`protectedFields: { '*': ['owners'], 'userField:owners': [] },`
`606`	`592`	`}`
`607`	`593`	`);`
`@@ -639,7 +625,6 @@ describe('ProtectedFields', function() {`
`639`	`625`	`{`
`640`	`626`	`get: { '*': true },`
`641`	`627`	`find: { '*': true },`
`642`		`- readUserFields: ['owners', 'owner'],`
`643`	`628`	`protectedFields: {`
`644`	`629`	`'*': [],`
`645`	`630`	`'userField:owners': ['owners'],`
`@@ -666,7 +651,7 @@ describe('ProtectedFields', function() {`
`666`	`651`	`done();`
`667`	`652`	`});`
`668`	`653`
`669`		`- it('should ignore pointer-permission fields not declared in the readUserFields', async done => {`
	`654`	`+ it('should ignore pointer-permission fields not present in object', async done => {`
`670`	`655`	`const config = Config.get(Parse.applicationId);`
`671`	`656`	`const obj = new Parse.Object('AnObject');`
`672`	`657`	`const obj2 = new Parse.Object('AnObject');`
`@@ -686,11 +671,10 @@ describe('ProtectedFields', function() {`
`686`	`671`	`{`
`687`	`672`	`get: { '*': true },`
`688`	`673`	`find: { '*': true },`
`689`		`- readUserFields: [],`
`690`	`674`	`protectedFields: {`
`691`	`675`	`'*': [],`
`692`		`- 'userField:owners': ['idontexist'],`
`693`		`- 'userField:owner': ['idontexist2'],`
	`676`	`+ 'userField:idontexist': ['owner'],`
	`677`	`+ 'userField:idontexist2': ['owners'],`
`694`	`678`	`},`
`695`	`679`	`}`
`696`	`680`	`);`
`@@ -733,7 +717,6 @@ describe('ProtectedFields', function() {`
`733`	`717`	`{`
`734`	`718`	`get: { '*': true },`
`735`	`719`	`find: { '*': true },`
`736`		`- readUserFields: ['owner'],`
`737`	`720`	`protectedFields: {`
`738`	`721`	`'*': ['owner'],`
`739`	`722`	`'userField:owner': [],`