Refactor shout/unshout mutations

- Remove unrecommended auto-commit transactions from code base
- Favor transaction functions

backend/src/schema/resolvers/shout.js

@@ -1,3 +1,5 @@
+import log from './helpers/databaseLogger'
+
 export default {
   Mutation: {
     shout: async (_object, params, context, _resolveInfo) => {
@@ -5,22 +7,24 @@ export default {
 
       const session = context.driver.session()
       try {
-        const transactionRes = await session.run(
+        const shoutWriteTxResultPromise = session.writeTransaction(async transaction => {
-          `MATCH (node {id: $id})<-[:WROTE]-(userWritten:User), (user:User {id: $userId})
+          const shoutTransactionResponse = await transaction.run(
+            `
+              MATCH (node {id: $id})<-[:WROTE]-(userWritten:User), (user:User {id: $userId})
               WHERE $type IN labels(node) AND NOT userWritten.id = $userId
               MERGE (user)-[relation:SHOUTED{createdAt:toString(datetime())}]->(node)
-          RETURN COUNT(relation) > 0 as isShouted`,
+              RETURN COUNT(relation) > 0 as isShouted
+            `,
             {
               id,
               type,
               userId: context.user.id,
             },
           )
-
+          log(shoutTransactionResponse)
-        const [isShouted] = transactionRes.records.map(record => {
+          return shoutTransactionResponse.records.map(record => record.get('isShouted'))
-          return record.get('isShouted')
         })
-
+        const [isShouted] = await shoutWriteTxResultPromise
         return isShouted
       } finally {
         session.close()
@@ -31,20 +35,24 @@ export default {
       const { id, type } = params
       const session = context.driver.session()
       try {
-        const transactionRes = await session.run(
+        const unshoutWriteTxResultPromise = session.writeTransaction(async transaction => {
-          `MATCH (user:User {id: $userId})-[relation:SHOUTED]->(node {id: $id})
+          const unshoutTransactionResponse = await transaction.run(
+            `
+              MATCH (user:User {id: $userId})-[relation:SHOUTED]->(node {id: $id})
               WHERE $type IN labels(node)
               DELETE relation
-          RETURN COUNT(relation) > 0 as isShouted`,
+              RETURN COUNT(relation) > 0 as isShouted
+            `,
             {
               id,
               type,
               userId: context.user.id,
             },
           )
-        const [isShouted] = transactionRes.records.map(record => {
+          log(unshoutTransactionResponse)
-          return record.get('isShouted')
+          return unshoutTransactionResponse.records.map(record => record.get('isShouted'))
         })
+        const [isShouted] = await unshoutWriteTxResultPromise
         return isShouted
       } finally {
         session.close()