chore: up

waitingsong · Apr 23, 2024 · 0ad51eb · 0ad51eb
1 parent 1fa5ec8
commit 0ad51eb
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 4 deletions.
diff --git a/packages/midway-component-kmore/src/decorator/transactional.handler.ts b/packages/midway-component-kmore/src/decorator/transactional.handler.ts
@@ -3,7 +3,7 @@ import { MConfig, DecoratorExecutorParamBase, DecoratorHandlerBase } from '@mwcp
 
 import { ConfigKey, KmorePropagationConfig } from '##/lib/types.js'
 
-import { decoratorExecutor, genDecoratorExecutorOptionsAsync } from './transactional.helper.js'
+import { decoratorExecutorAsync, decoratorExecutorSync, genDecoratorExecutorOptionsAsync } from './transactional.helper.js'
 import { DecoratorExecutorOptions, GenDecoratorExecutorOptionsExt } from './transactional.types.js'
 
 
@@ -20,7 +20,14 @@ export class DecoratorHandlerTransactional extends DecoratorHandlerBase {
   }
 
   override async executorAsync(options: DecoratorExecutorOptions) {
-    return decoratorExecutor(options)
+    return decoratorExecutorAsync(options)
+  }
+
+  /**
+   * @Caution Will not execute any method including database operations, just return the result
+   */
+  override executorSync(options: DecoratorExecutorOptions) {
+    return decoratorExecutorSync(options)
   }
 }
 
diff --git a/packages/midway-component-kmore/src/decorator/transactional.helper.ts b/packages/midway-component-kmore/src/decorator/transactional.helper.ts
@@ -50,7 +50,7 @@ export async function genDecoratorExecutorOptionsAsync<T extends object>(
 }
 
 
-export async function decoratorExecutor(options: DecoratorExecutorOptions): Promise<unknown> {
+export async function decoratorExecutorAsync(options: DecoratorExecutorOptions): Promise<unknown> {
   const {
     instanceName,
     mergedDecoratorParam,
@@ -140,3 +140,12 @@ async function processEx(options: ProcessExOptions): Promise<never> {
   await trxStatusSvc.trxRollbackEntry(callerKey)
   throw error
 }
+
+
+
+export function decoratorExecutorSync(options: DecoratorExecutorOptions): unknown {
+  const { method, methodArgs } = options
+  const resp = method(...methodArgs)
+  return resp
+}
+
diff --git a/packages/midway-component-kmore/test/cache/70.cache.test.ts b/packages/midway-component-kmore/test/cache/70.cache.test.ts
@@ -21,7 +21,7 @@ describe(fileShortPath(import.meta.url), () => {
   describe('Should ignore @Cacheable combined with @Transactional', () => {
     const prefix = apiPrefix.cache
 
-    it(apiRoute.get, async () => {
+    it.only(apiRoute.get, async () => {
       const { httpRequest } = testConfig
       const url = `${prefix}/${apiRoute.get}`