Fix transforming via API

README.md

@@ -80,7 +80,9 @@ $ babel --plugins inline-react-svg script.js
 
 ```javascript
 require('@babel/core').transform('code', {
-  plugins: ['inline-react-svg']
+  plugins: [
+    ['inline-react-svg', { filename: 'filename representing the code' }],
+  ]
 }) // => { code, map, ast };
 ```
 

src/index.js

@@ -32,8 +32,8 @@ export default declare(({
     if (typeof importPath !== 'string') {
       throw new TypeError('`applyPlugin` `importPath` must be a string');
     }
-    const { ignorePattern, caseSensitive } = state.opts;
-    const { file } = state;
+    const { ignorePattern, caseSensitive, filename: providedFilename } = state.opts;
+    const { file, filename } = state;
     if (ignorePattern) {
       // Only set the ignoreRegex once:
       ignoreRegex = ignoreRegex || new RegExp(ignorePattern);
@@ -44,7 +44,7 @@ export default declare(({
     }
     // This plugin only applies for SVGs:
     if (extname(importPath) === '.svg') {
-      const iconPath = state.file.opts.filename;
+      const iconPath = filename || providedFilename;
       const svgPath = resolve.sync(importPath, { basedir: dirname(iconPath) });
       if (caseSensitive && !fileExistsWithCaseSync(svgPath)) {
         throw new Error(`File path didn't match case of file on disk: ${svgPath}`);
@@ -105,7 +105,13 @@ export default declare(({
   return {
     visitor: {
       Program: {
-        enter({ scope, node }, { file }) {
+        enter({ scope, node }, { file, opts, filename }) {
+          if (typeof filename === 'string' && typeof opts.filename !== 'undefined') {
+            throw new TypeError('the "filename" option may only be provided when transforming code');
+          }
+          if (typeof filename === 'undefined' && typeof opts.filename !== 'string') {
+            throw new TypeError('the "filename" option is required when transforming code');
+          }
           if (!scope.hasBinding('React')) {
             const reactImportDeclaration = t.importDeclaration([
               t.importDefaultSpecifier(t.identifier('React')),

test/fixtures/test-import-read-file.jsx

@@ -0,0 +1,12 @@
+import React from 'react';
+import MySvg from './close.svg';
+
+export default function MyFunctionIcon() {
+  return <MySvg />;
+}
+
+export class MyClassIcon extends React.Component {
+  render() {
+    return <MySvg />;
+  }
+}

test/sanity.js

@@ -1,4 +1,4 @@
-import { transformFile } from '@babel/core';
+import { transformFile, transform } from '@babel/core';
 import fs from 'fs';
 import path from 'path';
 import inlineReactSvgPlugin from '../src';
@@ -71,7 +71,7 @@ if (fs.existsSync(path.resolve('./PACKAGE.JSON'))) {
     if (err && err.message.indexOf('match case') !== -1) {
       console.log('test/fixtures/test-case-sensitive.jsx', 'Test passed: Expected case sensitive error was thrown');
     } else {
-      throw new Error('Test failed: Expected case sensitive error wasn‘t thrown');
+      throw new Error('Test failed: Expected case sensitive error wasn‘t thrown, got: ' + err.message);
     }
   });
 } else {
@@ -122,3 +122,14 @@ transformFile('test/fixtures/test-dynamic-require.jsx', {
   if (err) throw err;
   console.log('test/fixtures/test-dynamic-require.jsx', result.code);
 });
+
+const filename = 'test/fixtures/test-import-read-file.jsx';
+transform(fs.readFileSync(filename), {
+  presets: ['airbnb'],
+  plugins: [
+    [inlineReactSvgPlugin, { filename }],
+  ],
+}, (err, result) => {
+  if (err) throw err;
+  console.log('test/fixtures/test-import-read-file.jsx', result.code);
+});