fix(define-macros-order): update rule to support interface/type expor…

…t in <script lang="ts" setup/> (#2040)
vuejs · Nov 19, 2022 · 8dc3cd9 · 8dc3cd9
1 parent cc1b8d9
commit 8dc3cd9
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/lib/rules/define-macros-order.js b/lib/rules/define-macros-order.js
@@ -45,7 +45,8 @@ function getTargetStatementPosition(scriptSetup, program) {
     'TSInterfaceDeclaration',
     'TSTypeAliasDeclaration',
     'DebuggerStatement',
-    'EmptyStatement'
+    'EmptyStatement',
+    'ExportNamedDeclaration'
   ])
 
   for (const [index, item] of program.body.entries()) {

diff --git a/tests/lib/rules/define-macros-order.js b/tests/lib/rules/define-macros-order.js
@@ -106,6 +106,28 @@ tester.run('define-macros-order', rule, {
         </script>
       `,
       options: optionsEmitsFirst
+    },
+    {
+      filename: 'test.vue',
+      code: `
+        <script setup lang="ts">
+          import { bar } from 'foo'
+          export interface Props {
+            msg?: string
+            labels?: string[]
+          }
+          defineEmits(['update:test'])
+          const props = withDefaults(defineProps<Props>(), {
+            msg: 'hello',
+            labels: () => ['one', 'two']
+          })
+          console.log('test')
+        </script>
+      `,
+      options: optionsEmitsFirst,
+      parserOptions: {
+        parser: require.resolve('@typescript-eslint/parser')
+      }
     }
   ],
   invalid: [