Merge pull request #259 from CKarper/master

Change owner to proposer for whitelist checks

lib/reviewer.js

@@ -101,11 +101,11 @@ class Reviewer {
   }
 
   renderComment(results, state) {
-    const { isOwnerWhitelisted, owner, ...remain } = state;
+    const { isProposerWhitelisted, proposer, ...remain } = state;
     const stateMd = this.renderCommentState(remain);
 
-    const body = isOwnerWhitelisted ?
-          `@${owner} is whitelisted, no review required.` :
+    const body = isProposerWhitelisted ?
+          `@${proposer} is whitelisted, no review required.` :
           results.map(({ comment }) => comment).join('');
 
     return dedent`${stateMd}
@@ -171,11 +171,11 @@ class Reviewer {
       const previousComment = [...shaComments].pop();
 
       const whitelist = this.config('users_whitelist');
-      const isOwnerWhitelisted = whitelist && whitelist.includes(pr.owner);
+      const isProposerWhitelisted = whitelist && whitelist.includes(pr.proposer);
 
-      const providers = isOwnerWhitelisted ? [] : await this.prepareProviders();
+      const providers = isProposerWhitelisted ? [] : await this.prepareProviders();
 
-      if (!isOwnerWhitelisted && providers.length === 0) {
+      if (!isProposerWhitelisted && providers.length === 0) {
         this.log.warn('No providers run.');
         return pr;
       }
@@ -198,8 +198,8 @@ class Reviewer {
       const newState = {
         sha: pr.sha,
         providers: instances.map(({ name }) => name),
-        owner: pr.owner,
-        isOwnerWhitelisted,
+        proposer: pr.proposer,
+        isProposerWhitelisted,
       };
 
       const newBody = this.renderComment(results, newState).trim();

lib/reviewer.test.js

@@ -201,7 +201,7 @@ test('Reviewer.review - whitelisted user', async t => {
 
   const providers = { test_provider: TestProvider };
   const config = createTestConfig({
-    users_whitelist: ['paultyng'],
+    users_whitelist: ['proposer'],
     review: {
       test_provider: true,
     },