Merge pull request #12 from modestjoust/master

allow async to be set via custom attributes

Author: jakubkottnauer

src/index.jsx

@@ -89,7 +89,11 @@ export default class Script extends React.Component {
     }
 
     script.src = url;
-    script.async = 1;
+    
+    // default async to true if not set with custom attributes
+    if (!script.hasAttribute('async')) {
+      script.async = 1;
+    }
 
     const callObserverFuncAndRemoveObserver = (shouldRemoveObserver) => {
       const observers = this.constructor.scriptObservers[url];

src/index.test.jsx

@@ -17,6 +17,7 @@ beforeEach(() => {
       id: 'dummyId',
       dummy: 'non standard',
       'data-dummy': 'standard',
+      async: false,
     },
   };
   wrapper = shallow(<Script {...props} />);
@@ -88,4 +89,5 @@ test('custom attributes should be set on the script tag', () => {
   expect(script.getAttribute('id')).toBe('dummyId');
   expect(script.getAttribute('dummy')).toBe('non standard');
   expect(script.getAttribute('data-dummy')).toBe('standard');
+  expect(script.getAttribute('async')).toBe(false);
 });