fix: removed auth on organization api (#395)

Author: BrRenat

src/actions/organization/create.js

@@ -107,7 +107,6 @@ async function createOrganizationAction({ params, locals }) {
   };
 }
 
-createOrganizationAction.auth = 'bearer';
 createOrganizationAction.transports = [ActionTransport.amqp, ActionTransport.internal];
 createOrganizationAction.allowed = async function checkOrganizationExistsConflict(request) {
   const { name } = request.params;

src/actions/organization/delete.js

@@ -49,7 +49,6 @@ async function deleteOrganization({ params }) {
   return pipeline.exec().then(handlePipeline);
 }
 
-deleteOrganization.auth = 'bearer';
 deleteOrganization.allowed = checkOrganizationExists;
 deleteOrganization.transports = [ActionTransport.amqp, ActionTransport.internal];
 module.exports = deleteOrganization;

test/helpers/organization.js

@@ -1,8 +1,6 @@
-const Promise = require('bluebird');
 const faker = require('faker');
 const times = require('lodash/times');
 const { inspectPromise } = require('@makeomatic/deploy');
-const jwt = require('../../src/utils/jwt');
 
 async function createMembers(totalUsers = 1) {
   this.userNames = [];
@@ -23,29 +21,6 @@ exports.createOrganization = async function (customOpts = {}, totalUsers = 1) {
     await createMembers.call(this, totalUsers);
   }
 
-  await this.users.dispatch('register', {
-    params: {
-      username: 'v@makeomatic.ru',
-      password: '123456',
-      audience: 'test',
-      metadata: {
-        fine: true,
-      },
-    },
-  }).catch(() => null);
-
-  const [bearer] = await Promise.all([
-    this.users.dispatch('token.create', {
-      params: {
-        username: 'v@makeomatic.ru',
-        name: 'sample',
-      },
-    }),
-    jwt.login.call(this.users, 'v@makeomatic.ru', 'test'),
-  ]);
-
-  this.bearerAuthHeaders = { authorization: `Bearer ${bearer}` };
-
   const params = {
     name: faker.company.companyName(),
     metadata: {
@@ -55,8 +30,7 @@ exports.createOrganization = async function (customOpts = {}, totalUsers = 1) {
     members: this.userNames.slice(0, totalUsers),
     ...customOpts,
   };
-  const organization = await this.users
-    .dispatch('organization.create', { params, headers: this.bearerAuthHeaders })
+  const organization = await this.dispatch('users.organization.create', params)
     .reflect()
     .then(inspectPromise(true));
 

test/suites/organization/create.js

@@ -12,7 +12,7 @@ describe('#create organization', function registerSuite() {
   afterEach(global.clearRedis);
 
   it('must reject invalid organization params and return detailed error', function test() {
-    return this.users.dispatch('organization.create', { headers: this.bearerAuthHeaders })
+    return this.dispatch('users.organization.create', {})
       .reflect()
       .then(inspectPromise(false))
       .then((response) => {
@@ -30,7 +30,7 @@ describe('#create organization', function registerSuite() {
       members: this.userNames.slice(0, 2),
     };
 
-    return this.users.dispatch('organization.create', { params, headers: this.bearerAuthHeaders })
+    return this.dispatch('users.organization.create', params)
       .reflect()
       .then(inspectPromise(true))
       .then((response) => {
@@ -48,7 +48,7 @@ describe('#create organization', function registerSuite() {
       name: this.organization.name,
     };
 
-    return this.users.dispatch('organization.create', { params, headers: this.bearerAuthHeaders })
+    return this.dispatch('users.organization.create', params)
       .reflect()
       .then(inspectPromise(false))
       .then((response) => {

test/suites/organization/delete.js

@@ -12,7 +12,7 @@ describe('#delete organization', function registerSuite() {
   afterEach(global.clearRedis);
 
   it('must reject invalid organization params and return detailed error', function test() {
-    return this.users.dispatch('organization.delete', { headers: this.bearerAuthHeaders })
+    return this.dispatch('users.organization.delete', {})
       .reflect()
       .then(inspectPromise(false))
       .then((response) => {
@@ -25,7 +25,7 @@ describe('#delete organization', function registerSuite() {
     const params = {
       organizationId: this.organization.id,
     };
-    return this.users.dispatch('organization.delete', { params, headers: this.bearerAuthHeaders })
+    return this.dispatch('users.organization.delete', params)
       .reflect()
       .then(inspectPromise());
   });
@@ -35,7 +35,7 @@ describe('#delete organization', function registerSuite() {
       organizationId: faker.company.companyName(),
     };
 
-    return this.users.dispatch('organization.delete', { params, headers: this.bearerAuthHeaders })
+    return this.dispatch('users.organization.delete', params)
       .reflect()
       .then(inspectPromise(false))
       .then((response) => {