Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Commit

Permalink
Sync out latest changes
Browse files Browse the repository at this point in the history
  • Loading branch information
zpao committed Oct 9, 2015
1 parent c970cd4 commit 5a9b886
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 12 deletions.
1 change: 1 addition & 0 deletions scripts/jest/preprocessor.js
Expand Up @@ -10,6 +10,7 @@ var babelOpts = {
plugins: [babelPluginModules],
_moduleMap: {
'React': 'react',
'ReactDOM': 'react-dom',
}
};

Expand Down
2 changes: 1 addition & 1 deletion src/StaticContainer.react.js
Expand Up @@ -34,7 +34,7 @@ class StaticContainer extends React.Component {
return !!nextProps.shouldUpdate;
}

render() {
render(): ?ReactElement {
var child = this.props.children;
if (child === null || child === false) {
return null;
Expand Down
20 changes: 9 additions & 11 deletions src/__tests__/StaticContainer.react-test.js
Expand Up @@ -11,37 +11,35 @@

jest.dontMock('StaticContainer.react');

describe('StaticContainer', () => {
var React;
var StaticContainer;
var React = require('React');
var ReactDOM = require('ReactDOM');
var StaticContainer = require('StaticContainer.react');

describe('StaticContainer', () => {
var container;

beforeEach(() => {
jest.resetModuleRegistry();

React = require('React');
StaticContainer = require('StaticContainer.react');

container = document.createElement('div');
});

it('does not update static content', () => {
var mockRender = jest.genMockFunction().mockReturnValue(<div />);
var MyComponent = React.createClass({render: mockRender});

React.render(<StaticContainer><MyComponent /></StaticContainer>, container);
ReactDOM.render(<StaticContainer><MyComponent /></StaticContainer>, container);

expect(mockRender.mock.calls.length).toBe(1);

React.render(<StaticContainer><MyComponent /></StaticContainer>, container);
ReactDOM.render(<StaticContainer><MyComponent /></StaticContainer>, container);

expect(mockRender.mock.calls.length).toBe(1);
});

it('allows null children', () => {
expect(() => {
React.render(
ReactDOM.render(
<StaticContainer>{null}</StaticContainer>,
container
);
Expand All @@ -50,14 +48,14 @@ describe('StaticContainer', () => {

it('requires exactly one child', () => {
expect(() => {
React.render(<StaticContainer />, container);
ReactDOM.render(<StaticContainer />, container);
}).toThrow(
'Invariant Violation: ' +
'onlyChild must be passed a children with exactly one child.'
);

expect(() => {
React.render(<StaticContainer><a /><a /></StaticContainer>, container);
ReactDOM.render(<StaticContainer><a /><a /></StaticContainer>, container);
}).toThrow(
'Invariant Violation: ' +
'onlyChild must be passed a children with exactly one child.'
Expand Down

0 comments on commit 5a9b886

Please sign in to comment.