fix(schema-compiler): ExportNamedDeclaration for variables object

[arkadiusz-swierczek-pm]

Check List

• Tests has been run in packages where changes made if available
• Linter has been run for changed code
• Tests for the changes have been added if not covered yet
• Docs have been added / updated if required

Class ImportExportTranspiler contains a bug that prevent using exported object syntax from cubes.

Related: https://cube.dev/docs/product/data-modeling/dynamic/code-reusability-export-and-import#import-from-parent-directories

The changes were tesed using https://astexplorer.net/ and content:

export const helperFunction = () => 'hello'
export { helperFunction as alias }
export default helperFunction

Without the fix, the Compiler throws error TypeError: Cannot read properties of null (reading 'declarations'). After the fix the babel transpiles the file to:

const helperFunction = () => 'hello';

addExport({
  helperFunction: helperFunction
})
addExport({
  alias: helperFunction
});
setExport(helperFunction);

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

8 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
examples-angular-dashboard	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-react-d3	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-react-dashboard	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-react-data-table	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-react-highcharts	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-react-material-ui	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-react-pivot-table	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm
examples-vue-query-builder	⬜️ Ignored (Inspect)	Visit Preview		Oct 3, 2024 3:34pm

[KSDaemon]

👍🏻 Looks good! @arkadiusz-swierczek-pm thnx for fixing this!

[KSDaemon]

@arkadiusz-swierczek-pm, it would be awesome if you could add a kind of basic test for this in transpilers.test.ts

[ovr]

As I can see, path.get('declaration') is still used in path.replaceWithMultiple.
Two different nodes in if condition and replaceWithMultiple call.