Don't spread old values into new context value. 😱

Author: Daniel Lehr

src/__tests__/index.test.js

@@ -281,9 +281,7 @@ describe('reactTreeWalker', () => {
 
  describe('react', () => {
  it('supports new context API', () => {
- const { Provider, Consumer } = React.createContext({
- handler: io => io,
- })
+ const { Provider, Consumer } = React.createContext()
 
  class SomeInstance extends React.Component {
  render() {
@@ -292,7 +290,12 @@ describe('reactTreeWalker', () => {
  }
 
  const tree = (
- <Provider value={{ message: 'This is a provider message' }}>
+ <Provider
+ value={{
+ message: 'This is a provider message',
+ handler: io => io,
+ }}
+ >
  <Consumer>
  {({ message, handler }) => (
  <strong>

src/index.js

@@ -116,10 +116,8 @@ export default function reactTreeWalker(
  if (currentElement.type) {
  if (currentElement.type._context) {
  // eslint-disable-next-line no-param-reassign
- currentElement.type._context._currentValue = {
- ...currentElement.type._context._currentValue,
- ...currentElement.props.value,
- }
+ currentElement.type._context._currentValue =
+ currentElement.props.value
  }
  if (currentElement.type.Provider && currentElement.type.Consumer) {
  const el = currentElement.props.children(