@kravc/dos 1.11.18 → 1.11.20

package/examples/ReadProfile.js

@@ -3,4 +3,16 @@
 const Read    = require('../src/operations/Read')
 const Profile = require('./Profile')
 
-module.exports = Read(Profile)
+class ReadProfile extends Read(Profile) {
+  static get query() {
+    return {
+      id: {
+        description: 'Profile ID',
+        required: true,
+        example: 'PRO_1'
+      }
+    }
+  }
+}
+
+module.exports = ReadProfile

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kravc/dos",
-  "version": "1.11.18",
+  "version": "1.11.20",
   "description": "Convention-based, easy-to-use library for building API-driven serverless services.",
   "keywords": [
     "Service",

package/src/Document.js

@@ -9,6 +9,8 @@ const DocumentNotFoundError = require('./errors/DocumentNotFoundError')
 const { get, omit, capitalize, cloneDeep } = require('lodash')
 
 const STORE = {}
+const SYSTEM = 'SYSTEM'
+const IDENTITY_SUBJECT_PATH = 'identity.sub'
 
 class Document extends Component {
   static get idKey() {
@@ -88,12 +90,7 @@ class Document extends Component {
 
     const { validator } = context
     mutation = validator.normalize(mutation, this.id)
-
-    const identitySubjectId = get(context, 'identity.sub')
-
-    if (identitySubjectId) {
-      mutation.createdBy = identitySubjectId
-    }
+    mutation.createdBy = get(context, IDENTITY_SUBJECT_PATH, SYSTEM)
 
     const timestamp = new Date().toJSON()
     mutation.createdAt = timestamp
@@ -191,12 +188,7 @@ class Document extends Component {
 
   static async update(context, query, mutation, originalDocument = null) {
     mutation = omit(mutation, [ this.idKey, 'createdAt', 'createdBy' ])
-
-    const identitySubjectId = get(context, 'identity.sub')
-
-    if (identitySubjectId) {
-      mutation.updatedBy = identitySubjectId
-    }
+    mutation.updatedBy = get(context, IDENTITY_SUBJECT_PATH, SYSTEM)
 
     const timestamp = new Date().toJSON()
     mutation.updatedAt = timestamp

package/src/Document.spec.js

@@ -58,14 +58,14 @@ describe('Document', () => {
       expect(profile.attributes.name).to.eql('Olga')
     })
 
-    it('creates document without identity in context', async () => {
+    it('creates document without identity in context as SYSTEM', async () => {
       const context = { validator }
       const profile = await Profile.create(context, { name: 'Oleg' })
 
       expect(profile.id).to.exist
       expect(profile.attributes.name).to.eql('Oleg')
       expect(profile.attributes.createdAt).to.exist
-      expect(profile.attributes.createdBy).to.not.exist
+      expect(profile.attributes.createdBy).to.eql('SYSTEM')
     })
 
     it('returns exiting document from the context', async () => {
@@ -115,14 +115,14 @@ describe('Document', () => {
       expect(profile.attributes.updatedBy).to.exist
     })
 
-    it('updates document without identity in context', async () => {
+    it('updates document without identity in context as SYSTEM', async () => {
       const context = { validator }
       const { id } = await Profile.create(context, { name: 'Gustav' })
 
       const profile = await Profile.update(context, { id }, { name: 'Jack' })
 
       expect(profile.attributes.updatedAt).to.exist
-      expect(profile.attributes.updatedBy).to.not.exist
+      expect(profile.attributes.updatedBy).to.eql('SYSTEM')
     })
 
     it('throws "DocumentNotFoundError" if document not found', async () => {

package/src/Service.spec.js

@@ -291,7 +291,7 @@ describe('Service', () => {
       expect(response.statusCode).to.eql(201)
 
       request = {
-        url:    'http://localhost:3000/api/UpdateProfile?id=HELLO_WORLD',
+        url:    'http://localhost:3000/api/UpdateProfile?id=HELLO_WORLD&tags=["tag1","tag2"]',
         body:    JSON.stringify({ name: 'HTTP test!' }),
         method:  'PATCH',
         headers: { authorization }

package/src/helpers/createContext.js

@@ -65,6 +65,18 @@ const createContext = (service, request, extraContext = {}) => {
     context.query = queryStringParameters
   }
 
+  const queryKeys = Object.keys(context.query)
+
+  for (const queryKey of queryKeys) {
+    const value = context.query[queryKey]
+
+    const isJsonArray = `${value}`.startsWith('["')
+
+    if (isJsonArray) {
+      context.query[queryKey] = JSON.parse(value)
+    }
+  }
+
   if (body) {
     const isJSON = isString(body)
 

package/src/helpers/createSpec.js

@@ -80,7 +80,11 @@ const createSpec = (operations, schemasMap, url) => {
 
     for (const name in query) {
       const queryParameter = { in: 'query', name, type: 'string', ...query[name] }
-      delete queryParameter.example
+
+      if (queryParameter.example) {
+        queryParameter['x-example'] = queryParameter.example
+        delete queryParameter.example
+      }
 
       parameters.push(queryParameter)
     }