Remove expect.assertions

Author: alexeyr

node_package/tests/Authenticity.test.js

@@ -9,25 +9,21 @@ document.head.appendChild(meta);
 
 describe('authenticityToken', () => {
   it('exists in ReactOnRails API', () => {
-    expect.assertions(1);
     expect(typeof ReactOnRails.authenticityToken).toBe('function');
   });
 
   it('can read Rails CSRF token from <meta>', () => {
-    expect.assertions(1);
     const realToken = ReactOnRails.authenticityToken();
     expect(realToken).toEqual(testToken);
   });
 });
 
 describe('authenticityHeaders', () => {
   it('exists in ReactOnRails API', () => {
-    expect.assertions(1);
     expect(typeof ReactOnRails.authenticityHeaders).toBe('function');
   });
 
   it('returns valid header with CSRF token', () => {
-    expect.assertions(1);
     const realHeader = ReactOnRails.authenticityHeaders();
     expect(realHeader).toEqual({ 'X-CSRF-Token': testToken, 'X-Requested-With': 'XMLHttpRequest' });
   });

node_package/tests/ComponentRegistry.test.js

@@ -37,7 +37,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('registers and retrieves React function components', () => {
-    expect.assertions(1);
     const C1 = () => <div>HELLO</div>;
     ComponentRegistry.register({ C1 });
     const actual = ComponentRegistry.get('C1');
@@ -46,7 +45,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('registers and retrieves Render-Function components where property renderFunction is set and zero params', () => {
-    expect.assertions(1);
     const C1 = () => <div>HELLO</div>;
     C1.renderFunction = true;
     ComponentRegistry.register({ C1 });
@@ -56,7 +54,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('registers and retrieves ES5 class components', () => {
-    expect.assertions(1);
     const C2 = createReactClass({
       render() {
         return <div> WORLD </div>;
@@ -69,7 +66,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('registers and retrieves ES6 class components', () => {
-    expect.assertions(1);
     class C3 extends React.Component {
       render() {
         return <div>Wow!</div>;
@@ -82,7 +78,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('registers and retrieves renderers if 3 params', () => {
-    expect.assertions(1);
     const C4 = (a1, a2, a3) => null;
     ComponentRegistry.register({ C4 });
     const actual = ComponentRegistry.get('C4');
@@ -95,7 +90,6 @@ describe('ComponentRegistry', () => {
    * Thus, tests are cumulative.
    */
   it('registers and retrieves multiple components', () => {
-    expect.assertions(4);
     // Plain react stateless functional components
     const C5 = () => <div>WHY</div>;
     const C6 = () => <div>NOW</div>;
@@ -127,7 +121,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('only detects a renderer function if it has three arguments', () => {
-    expect.assertions(2);
     const C7 = (a1, a2) => null;
     const C8 = (a1) => null;
     ComponentRegistry.register({ C7 });
@@ -148,20 +141,17 @@ describe('ComponentRegistry', () => {
   });
 
   it('throws error for retrieving unregistered component', () => {
-    expect.assertions(1);
     expect(() => ComponentRegistry.get('foobar')).toThrow(
       /Could not find component registered with name foobar/,
     );
   });
 
   it('throws error for setting null component', () => {
-    expect.assertions(1);
     const C9 = null;
     expect(() => ComponentRegistry.register({ C9 })).toThrow(/Called register with null component named C9/);
   });
 
   it('retrieves component asynchronously when registered later', async () => {
-    expect.assertions(1);
     const C1 = () => <div>HELLO</div>;
     const componentPromise = ComponentRegistry.getOrWaitForComponent('C1');
     ComponentRegistry.register({ C1 });
@@ -175,7 +165,6 @@ describe('ComponentRegistry', () => {
   });
 
   it('handles timeout for unregistered components', async () => {
-    expect.assertions(1);
     let error;
     try {
       await ComponentRegistry.getOrWaitForComponent('NonExistent');

node_package/tests/ReactOnRails.test.jsx

@@ -8,7 +8,6 @@ import ReactOnRails from '../src/ReactOnRails.client.ts';
 
 describe('ReactOnRails', () => {
   it('render returns a virtual DOM element for component', () => {
-    expect.assertions(1);
     const R1 = createReactClass({
       render() {
         return <div> WORLD </div>;
@@ -29,37 +28,31 @@ describe('ReactOnRails', () => {
 
   it('accepts traceTurbolinks as an option true', () => {
     ReactOnRails.resetOptions();
-    expect.assertions(1);
     ReactOnRails.setOptions({ traceTurbolinks: true });
     const actual = ReactOnRails.option('traceTurbolinks');
     expect(actual).toBe(true);
   });
 
   it('accepts traceTurbolinks as an option false', () => {
     ReactOnRails.resetOptions();
-    expect.assertions(1);
     ReactOnRails.setOptions({ traceTurbolinks: false });
     const actual = ReactOnRails.option('traceTurbolinks');
     expect(actual).toBe(false);
   });
 
   it('not specified has traceTurbolinks as false', () => {
     ReactOnRails.resetOptions();
-    expect.assertions(1);
     ReactOnRails.setOptions({});
     const actual = ReactOnRails.option('traceTurbolinks');
     expect(actual).toBe(false);
   });
 
   it('setOptions method throws error for invalid options', () => {
     ReactOnRails.resetOptions();
-    expect.assertions(1);
     expect(() => ReactOnRails.setOptions({ foobar: true })).toThrow(/Invalid option/);
   });
 
   it('registerStore throws if passed a falsey object (null, undefined, etc)', () => {
-    expect.assertions(3);
-
     expect(() => ReactOnRails.registerStore(null)).toThrow(/null or undefined/);
 
     expect(() => ReactOnRails.registerStore(undefined)).toThrow(/null or undefined/);
@@ -68,7 +61,6 @@ describe('ReactOnRails', () => {
   });
 
   it('register store and getStoreGenerator allow registration', () => {
-    expect.assertions(2);
     function reducer() {
       return {};
     }
@@ -86,7 +78,6 @@ describe('ReactOnRails', () => {
   });
 
   it('setStore and getStore', () => {
-    expect.assertions(2);
     function reducer() {
       return {};
     }
@@ -108,7 +99,6 @@ describe('ReactOnRails', () => {
   });
 
   it('clearHydratedStores', () => {
-    expect.assertions(2);
     function reducer() {
       return {};
     }

node_package/tests/StoreRegistry.test.js

@@ -20,7 +20,6 @@ describe('StoreRegistry', () => {
   });
 
   it('StoreRegistry throws error for registering null or undefined store', () => {
-    expect.assertions(2);
     expect(() => StoreRegistry.register({ storeGenerator: null })).toThrow(
       /Called ReactOnRails.registerStoreGenerators with a null or undefined as a value/,
     );
@@ -30,14 +29,12 @@ describe('StoreRegistry', () => {
   });
 
   it('StoreRegistry throws error for retrieving unregistered store', () => {
-    expect.assertions(1);
     expect(() => StoreRegistry.getStore('foobar')).toThrow(
       /There are no stores hydrated and you are requesting the store/,
     );
   });
 
   it('StoreRegistry registers and retrieves Render-Function stores', () => {
-    expect.assertions(2);
     StoreRegistry.register({ storeGenerator, storeGenerator2 });
     const actual = StoreRegistry.getStoreGenerator('storeGenerator');
     const expected = storeGenerator;
@@ -48,22 +45,19 @@ describe('StoreRegistry', () => {
   });
 
   it('StoreRegistry throws error for retrieving unregistered store generator', () => {
-    expect.assertions(1);
     expect(() => StoreRegistry.getStoreGenerator('foobar')).toThrow(
       /Could not find store generator registered with name foobar\. Registered store generator names include/,
     );
   });
 
   it('StoreRegistry returns undefined for retrieving unregistered store, passing throwIfMissing = false', () => {
-    expect.assertions(1);
     StoreRegistry.setStore('foobarX', {});
     const actual = StoreRegistry.getStore('foobar', false);
     const expected = undefined;
     expect(actual).toEqual(expected);
   });
 
   it('StoreRegistry getStore, setStore', () => {
-    expect.assertions(1);
     const store = storeGenerator({});
     StoreRegistry.setStore('storeGenerator', store);
     const actual = StoreRegistry.getStore('storeGenerator');
@@ -72,14 +66,12 @@ describe('StoreRegistry', () => {
   });
 
   it('StoreRegistry throws error for retrieving unregistered hydrated store', () => {
-    expect.assertions(1);
     expect(() => StoreRegistry.getStore('foobar')).toThrow(
       /Could not find hydrated store registered with name foobar\. Registered hydrated store names include/,
     );
   });
 
   it('StoreRegistry clearHydratedStores', () => {
-    expect.assertions(2);
     StoreRegistry.clearHydratedStores();
 
     const result = storeGenerator({});

node_package/tests/buildConsoleReplay.test.js

@@ -2,27 +2,23 @@ import buildConsoleReplay, { consoleReplay } from '../src/buildConsoleReplay.ts'
 
 describe('consoleReplay', () => {
   it('does not throw an exception if no console.history object', () => {
-    expect.assertions(1);
     expect(() => consoleReplay()).not.toThrow(/Error/);
   });
 
   it('returns empty string if no console.history object', () => {
-    expect.assertions(1);
     const actual = consoleReplay();
     const expected = '';
     expect(actual).toEqual(expected);
   });
 
   it('returns empty string if console.history is empty', () => {
-    expect.assertions(1);
     console.history = [];
     const actual = consoleReplay();
     const expected = '';
     expect(actual).toEqual(expected);
   });
 
   it('replays multiple history messages', () => {
-    expect.assertions(1);
     console.history = [
       { arguments: ['a', 'b'], level: 'log' },
       { arguments: ['c', 'd'], level: 'warn' },
@@ -33,7 +29,6 @@ describe('consoleReplay', () => {
   });
 
   it('replays converts console param objects to JSON', () => {
-    expect.assertions(1);
     console.history = [
       { arguments: ['some message', { a: 1, b: 2 }], level: 'log' },
       { arguments: ['other message', { c: 3, d: 4 }], level: 'warn' },
@@ -46,7 +41,6 @@ console.warn.apply(console, ["other message","{\\"c\\":3,\\"d\\":4}"]);`;
   });
 
   it('replays converts script tag inside of object string to be safe', () => {
-    expect.assertions(1);
     console.history = [
       {
         arguments: [
@@ -67,7 +61,6 @@ console.warn.apply(console, ["other message","{\\"c\\":3,\\"d\\":4}"]);`;
   });
 
   it('buildConsoleReplay wraps console replay in a script tag', () => {
-    expect.assertions(1);
     console.history = [
       { arguments: ['some message', { a: 1, b: 2 }], level: 'log' },
       { arguments: ['other message', { c: 3, d: 4 }], level: 'warn' },

node_package/tests/renderFunction.test.jsx

@@ -9,8 +9,6 @@ import isRenderFunction from '../src/isRenderFunction.ts';
 
 describe('isRenderFunction', () => {
   it('returns false for a ES5 React Component', () => {
-    expect.assertions(1);
-
     const es5Component = createReactClass({
       render() {
         return <div>ES5 React Component</div>;
@@ -21,8 +19,6 @@ describe('isRenderFunction', () => {
   });
 
   it('returns false for a ES6 React class', () => {
-    expect.assertions(1);
-
     class ES6Component extends React.Component {
       render() {
         return <div>ES6 Component</div>;
@@ -33,8 +29,6 @@ describe('isRenderFunction', () => {
   });
 
   it('returns false for a ES6 React subclass', () => {
-    expect.assertions(1);
-
     class ES6Component extends React.Component {
       render() {
         return <div>ES6 Component</div>;
@@ -51,33 +45,25 @@ describe('isRenderFunction', () => {
   });
 
   it('returns false for a stateless functional component with zero params', () => {
-    expect.assertions(1);
-
     const pureComponent = () => <h1>Hello</h1>;
 
     expect(isRenderFunction(pureComponent)).toBe(false);
   });
 
   it('returns false for a stateless functional component with one param', () => {
-    expect.assertions(1);
-
     const pureComponent = (props) => <h1>{props.title}</h1>;
 
     expect(isRenderFunction(pureComponent)).toBe(false);
   });
 
   it('returns true for a Render-Function (containing two params)', () => {
-    expect.assertions(1);
-
     const foobarComponent = () => <div>Component for Render-Function</div>;
     const foobarrenderFunction = (_props, _railsContext) => foobarComponent;
 
     expect(isRenderFunction(foobarrenderFunction)).toBe(true);
   });
 
   it('returns false for simple object', () => {
-    expect.assertions(1);
-
     const foobarComponent = {
       hello() {
         return 'world';

node_package/tests/scriptSanitizedVal.test.js

@@ -2,39 +2,34 @@ import scriptSanitizedVal from '../src/scriptSanitizedVal.ts';
 
 describe('scriptSanitizedVal', () => {
   it('returns no </script if spaces, uppercase 1', () => {
-    expect.assertions(1);
     const input = '[SERVER] This is a script:"</div>"</script> <script>alert(\'WTF\')</  SCRIPT >';
     const actual = scriptSanitizedVal(input);
     const expected = '[SERVER] This is a script:"</div>"(/script> <script>alert(\'WTF\')(/script >';
     expect(actual).toEqual(expected);
   });
 
   it('returns no </script> 2', () => {
-    expect.assertions(1);
     const input = 'Script2:"</div>"</script xx> <script>alert(\'WTF2\')</script xx>';
     const actual = scriptSanitizedVal(input);
     const expected = 'Script2:"</div>"(/script xx> <script>alert(\'WTF2\')(/script xx>';
     expect(actual).toEqual(expected);
   });
 
   it('returns no </script> 3', () => {
-    expect.assertions(1);
     const input = 'Script3:"</div>"</  SCRIPT xx> <script>alert(\'WTF3\')</script xx>';
     const actual = scriptSanitizedVal(input);
     const expected = 'Script3:"</div>"(/script xx> <script>alert(\'WTF3\')(/script xx>';
     expect(actual).toEqual(expected);
   });
 
   it('returns no </script> 4', () => {
-    expect.assertions(1);
     const input = 'Script4"</div>"</script <script>alert(\'WTF4\')</script>';
     const actual = scriptSanitizedVal(input);
     const expected = 'Script4"</div>"(/script <script>alert(\'WTF4\')(/script>';
     expect(actual).toEqual(expected);
   });
 
   it('returns no </script> 5', () => {
-    expect.assertions(1);
     const input = 'Script5:"</div>"</ script> <script>alert(\'WTF5\')</script>';
     const actual = scriptSanitizedVal(input);
     const expected = 'Script5:"</div>"(/script> <script>alert(\'WTF5\')(/script>';