feat: add Svg prefix to exports that starts with a number (#383)

Closes #379

Author: TrySound

__fixtures__/numeric/2.file.svg

@@ -1,5 +1,10 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`cli should add Svg prefix to index.js exports staring with number 1`] = `
+"export { default as Svg2File } from './2File'
+export { default as File } from './File'"
+`;
+
 exports[`cli should not override config with cli defaults 1`] = `
 "import React from \\"react\\";
 
@@ -412,6 +417,8 @@ __fixtures__/complex/telegram.svg -> __fixtures_build__/whole/complex/Telegram.j
 __fixtures__/nesting/a/c/three.svg -> __fixtures_build__/whole/nesting/a/c/Three.js
 __fixtures__/nesting/a/two.svg -> __fixtures_build__/whole/nesting/a/Two.js
 __fixtures__/nesting/one.svg -> __fixtures_build__/whole/nesting/One.js
+__fixtures__/numeric/2.file.svg -> __fixtures_build__/whole/numeric/2File.js
+__fixtures__/numeric/file.svg -> __fixtures_build__/whole/numeric/File.js
 __fixtures__/simple/file.svg -> __fixtures_build__/whole/simple/File.js
 __fixtures__/withPrettierRc/file.svg -> __fixtures_build__/whole/withPrettierRc/File.js
 __fixtures__/withSvgoYml/file.svg -> __fixtures_build__/whole/withSvgoYml/File.js

__fixtures__/numeric/file.svg

@@ -38,7 +38,8 @@ export function isCompilable(filename) {
 function defaultIndexTemplate(files) {
   const exportEntries = files.map(file => {
     const basename = path.basename(file, path.extname(file))
-    return `export { default as ${basename} } from './${basename}'`
+    const exportName = /^\d/.test(basename) ? `Svg${basename}` : basename
+    return `export { default as ${exportName} } from './${basename}'`
   })
   return exportEntries.join('\n')
 }

packages/cli/src/__snapshots__/index.test.js.snap

@@ -167,11 +167,22 @@ describe('cli', () => {
     expect(result).toMatchSnapshot()
   }, 10000)
 
+  it('should add Svg prefix to index.js exports staring with number', async () => {
+    const inDir = '__fixtures__/numeric'
+    const outDir = `__fixtures_build__/prefix-exports`
+    await del(outDir)
+    await cli(`${inDir} --out-dir=${outDir}`)
+    const content = fs.readFileSync(path.join(outDir, 'index.js'), 'utf-8')
+    expect(content).toMatchSnapshot()
+  }, 10000)
+
   it('should support custom index.js with directory output', async () => {
     const inDir = '__fixtures__/simple'
     const outDir = `__fixtures_build__/custom-index`
     await del(outDir)
-    await cli(`${inDir} --out-dir=${outDir} --config-file=__fixtures__/custom-index.config.js`)
+    await cli(
+      `${inDir} --out-dir=${outDir} --config-file=__fixtures__/custom-index.config.js`,
+    )
     const content = fs.readFileSync(path.join(outDir, 'index.js'), 'utf-8')
     expect(content).toMatchSnapshot()
   }, 10000)
@@ -180,7 +191,9 @@ describe('cli', () => {
     const inDir = '__fixtures__/simple'
     const outDir = `__fixtures_build__/custom-index-arg`
     await del(outDir)
-    await cli(`${inDir} --out-dir=${outDir} --index-template=__fixtures__/custom-index-template.js`)
+    await cli(
+      `${inDir} --out-dir=${outDir} --index-template=__fixtures__/custom-index-template.js`,
+    )
     const content = fs.readFileSync(path.join(outDir, 'index.js'), 'utf-8')
     expect(content).toMatchSnapshot()
   }, 10000)