Uh oh!
There was an error while loading. Please reload this page.
Add an integration test for WebSocket - #11433
Conversation
facebook-github-bot
commented
Dec 13, 2016
By analyzing the blame information on this pull request, we identified @fkgozali and @bestander to be potential reviewers. |
douglowder
commented
Dec 13, 2016
@satya164 Here's a network test that will hopefully run in Travis CI, per our discussion in Slack. Please take a look and see what you think. |
| var connectSucceeded = true; | ||
| var component = this; | ||
| component._connect(); | ||
| component._waitFor(component._socketIsConnected,5,function(result) { |
There was a problem hiding this comment.
number This type is incompatible with the expected param type of Number
| ]; | ||
| class WebSocketTest extends React.Component { | ||
| state: State = { |
There was a problem hiding this comment.
identifier State Could not resolve name
| callback(true); | ||
| return; | ||
| } | ||
| remaining--; |
There was a problem hiding this comment.
Number The operand of an arithmetic operation must be a number.
| } | ||
| var disconnectSucceeded = true; | ||
| component._disconnect(); | ||
| component._waitFor(component._socketIsDisconnected,5,function(result2) { |
There was a problem hiding this comment.
number This type is incompatible with the expected param type of Number
satya164
commented
Dec 16, 2016
@facebook-github-bot shipit |
1 similar comment
satya164
commented
Dec 19, 2016
@facebook-github-bot shipit |
satya164
commented
Dec 19, 2016
@facebook-github-bot shipit |
2 similar comments
janicduplessis
commented
Dec 19, 2016
@facebook-github-bot shipit |
mkonicek
commented
Dec 19, 2016
@facebook-github-bot shipit |
facebook-github-bot
commented
Dec 20, 2016
I tried to merge this pull request into the Facebook internal repo but some checks failed. To unblock yourself please check the following: Does this pull request pass all open source tests on GitHub? If not please fix those. Does the code still apply cleanly on top of GitHub master? If not can please rebase. In all other cases this means some internal test failed, for example a part of a fb app won't work with this pull request. I've added the Import Failed label to this pull request so it is easy for someone at fb to find the pull request and check what failed. If you don't see anyone comment in a few days feel free to comment mentioning one of the core contributors to the project so they get a notification. |
facebook-github-bot
commented
Jan 7, 2017
Something went wrong when importing this pull request. Please cc someone from the team at fb to help with importing this. |
facebook-github-bot
commented
Jan 13, 2017
@ericvicenti has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
facebook-github-bot
commented
Jan 14, 2017
I tried to merge this pull request into the Facebook internal repo but some checks failed. To unblock yourself please check the following: Does this pull request pass all open source tests on GitHub? If not please fix those. Does the code still apply cleanly on top of GitHub master? If not can please rebase. In all other cases this means some internal test failed, for example a part of a fb app won't work with this pull request. I've added the Import Failed label to this pull request so it is easy for someone at fb to find the pull request and check what failed. If you don't see anyone comment in a few days feel free to comment mentioning one of the core contributors to the project so they get a notification. |
facebook-github-bot
commented
Jan 16, 2017
@javache has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: **Motivation** See if we can safely run a WebSocket test in Travis CI Closesreact/react-native#11433 Differential Revision: D4342024 Pulled By: ericvicenti fbshipit-source-id: 137fb0c39ed7ea3726e2778d5c0bdac4cef6ab89
Motivation
See if we can safely run a WebSocket test in Travis CI