Skip to content
This repository has been archived by the owner on Jun 26, 2022. It is now read-only.

Cleanup test logs. #73

Open
jcblw opened this issue Aug 10, 2019 · 0 comments
Open

Cleanup test logs. #73

jcblw opened this issue Aug 10, 2019 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@jcblw
Copy link
Member

jcblw commented Aug 10, 2019

Introducing React 16.9 and removing some logging mocks brought some verbose logs to tests.

console.error node_modules/react-test-renderer/cjs/react-test-renderer.development.js:102
      Warning: An update to SubscriptionProvider inside a test was not wrapped in act(...).
      
      When testing, code that causes React state updates should be wrapped into act(...):
      
      act(() => {
        /* fire events that update state */
      });
      /* assert on the output */
      
      This ensures that you're testing the behavior the user would see in the browser. Learn more at https://fb.me/react-wrap-tests-with-act
@jcblw jcblw added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed labels Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant