Dashboard backend: set current user in job.post

src/dashboard/server/api/controllers/cluster/jobs.post.js

@@ -3,13 +3,16 @@ const uuid = require('uuid')
 /**
  * @typedef {Object} State
  * @property {import('../../services/cluster')} cluster
+ * @property {import('../../services/user')} user
  */
 
 /** @type {import('koa').Middleware<State>} */
 module.exports = async context => {
-  const { cluster } = context.state
+  const { cluster, user } = context.state
 
   const job = Object.assign({}, context.request.body)
+
+  job['userName'] = user.email
   job['familyToken'] = uuid()
   job['isParent'] = 1
 

src/dashboard/server/test/controllers/cluster/jobs.post.js

@@ -1,6 +1,8 @@
 const axiosist = require('axiosist')
 const sinon = require('sinon')
 const nock = require('nock')
+const _ = require('lodash')
+
 const User = require('../../../api/services/user')
 const api = require('../../../api').callback()
 
@@ -45,4 +47,17 @@ describe('POST /clusters/:clusterid/jobs', () => {
       {}, {params: userParams})
     response.status.should.equal(400)
   })
+
+  it('should forcely set userName as current user in the submitted job', async () => {
+    nock('http://universe')
+      .post('/PostJob', _.matches({ userName: '[email protected]' }))
+      .reply(200, {
+        message: 'job adding succeeded'
+      })
+    sinon.stub(User.prototype, 'fillIdFromWinbind').resolves()
+
+    const response = await axiosist(api).post('/clusters/Universe/jobs',
+      { vcName: 'test', userName: 'foo' }, { params: userParams })
+    response.status.should.equal(200)
+  })
 })