Merge branch 'adnasa-adnasa-remove-console-log-test'

Author: nitin42

__tests__/__snapshots__/hostEffects.test.js.snap

@@ -0,0 +1,20 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Committing Host Effects Sanity 1`] = `
+Object {
+  "Committing Host Effects": Object {
+    "timeSpent": Array [
+      1.14,
+      0.09,
+    ],
+    "totalEffects": Array [
+      7,
+      1,
+    ],
+  },
+}
+`;
+
+exports[`Committing Host Effects Total effects 1`] = `8`;
+
+exports[`Committing Host Effects Total time taken in committing host effects 1`] = `1.23`;

__tests__/__snapshots__/lifecycle.test.js.snap

@@ -0,0 +1,20 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Calling Lifecycle Methods Sanity 1`] = `
+Object {
+  "Calling Lifecycle Methods": Object {
+    "timeSpent": Array [
+      4.38,
+      0.89,
+    ],
+    "totalMethods": Array [
+      6,
+      1,
+    ],
+  },
+}
+`;
+
+exports[`Calling Lifecycle Methods Total effects 1`] = `7`;
+
+exports[`Calling Lifecycle Methods Total time taken in calling lifecycle methods 1`] = `5.27`;

__tests__/commitChanges.test.js

@@ -7,8 +7,6 @@ describe('Committing changes time duration', () => {
   test('Committing changes with and without warning for cascading updates in earlier commits', () => {
     expect(getCommitChangesTime(MEASURES)).toMatchSnapshot()
   })
-
-  console.log(getTotalTime(getCommitChangesTime(MEASURES)))
   test('Returns total time taken in committing changes', () => {
     expect(getTotalTime(getCommitChangesTime(MEASURES))).toMatchSnapshot()
   })

__tests__/hostEffects.test.js

@@ -8,14 +8,16 @@ var { getTotalTime } = require('../src/shared/totalTime')
 
 describe('Committing Host Effects', () => {
   test('Sanity', () => {
-    console.log(getCommitHostEffectsTime(MEASURES))
+    expect(getCommitHostEffectsTime(MEASURES)).toMatchSnapshot()
   })
 
   test('Total time taken in committing host effects', () => {
-    console.log(getTotalTime(getCommitHostEffectsTime(MEASURES)))
+    expect(getTotalTime(getCommitHostEffectsTime(MEASURES))).toMatchSnapshot()
   })
 
   test('Total effects', () => {
-    console.log(getTotalEffects(getCommitHostEffectsTime(MEASURES)))
+    expect(
+      getTotalEffects(getCommitHostEffectsTime(MEASURES))
+    ).toMatchSnapshot()
   })
 })

__tests__/lifecycle.test.js

@@ -5,14 +5,14 @@ var { getTotalTime } = require('../src/shared/totalTime')
 
 describe('Calling Lifecycle Methods', () => {
   test('Sanity', () => {
-    console.log(getLifecycleTime(MEASURES))
+    expect(getLifecycleTime(MEASURES)).toMatchSnapshot()
   })
 
   test('Total time taken in calling lifecycle methods', () => {
-    console.log(getTotalTime(getLifecycleTime(MEASURES)))
+    expect(getTotalTime(getLifecycleTime(MEASURES))).toMatchSnapshot()
   })
 
   test('Total effects', () => {
-    console.log(getTotalMethods(getLifecycleTime(MEASURES)))
+    expect(getTotalMethods(getLifecycleTime(MEASURES))).toMatchSnapshot()
   })
 })

__tests__/logMeasures.js

@@ -4,7 +4,6 @@ var MEASURES = require('../samples/measures')
 
 describe('Map Data from React Performance Data', () => {
   test('Returns an object containing information about the component measure', () => {
-    console.log(generateDataFromMeasures(getReactPerformanceData(MEASURES)))
     expect(
       generateDataFromMeasures(getReactPerformanceData(MEASURES))
     ).toMatchSnapshot()

__tests__/mapData.test.js

@@ -2,11 +2,6 @@ var { getComponentAndPhaseName } = require('../src/shared/parseMeasures')
 
 describe('Measure names', () => {
   test('Component name and phase name', () => {
-    console.log(
-      getComponentAndPhaseName({
-        name: '⚛ App [mount]'
-      })
-    )
     expect(
       getComponentAndPhaseName({
         name: '⚛ App [mount]'
@@ -15,11 +10,6 @@ describe('Measure names', () => {
   })
 
   test('Component name and lifecycle hook without warning for cascading updates', () => {
-    console.log(
-      getComponentAndPhaseName({
-        name: '⚛ App.componentWillMount'
-      })
-    )
     expect(
       getComponentAndPhaseName({
         name: '⚛ App.componentWillMount'
@@ -28,11 +18,6 @@ describe('Measure names', () => {
   })
 
   test('Component name and lifecycle hook with warning for cascading updates', () => {
-    console.log(
-      getComponentAndPhaseName({
-        name: '⛔ App.componentDidMount Warning: Scheduled a cascading update'
-      })
-    )
     expect(
       getComponentAndPhaseName({
         name: '⛔ App.componentDidMount Warning: Scheduled a cascading update'

__tests__/measures.test.js

@@ -3,7 +3,6 @@ var MEASURES = require('../samples/measures')
 
 describe('React Performance Data', () => {
   test('Parse component name and phase name', () => {
-    console.log(getReactPerformanceData(MEASURES))
     expect(getReactPerformanceData(MEASURES)).toMatchSnapshot()
   })
 })