Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tiny integration test change #1311

Merged
merged 1 commit into from
Jun 4, 2014
Merged

Conversation

MagIciaNGTAO
Copy link

It's a tiny change.

I've been following this project for a while as a hobby, also want to contribute to it.

Currently I am going through test cases to learn. Found some tiny problems as we are testing "concat", we should care not only the total number of elements, but also the sequences.

Creating this request mainly want to make sure I am writing/commiting code to the correct place.

Let me know if there is any problem.

Best Regards,
Mingtao

@cloudbees-pull-request-builder

RxJava-pull-requests #1211 SUCCESS
This pull request looks good

@benjchristensen
Copy link
Member

Thanks for getting involved and improving these tests.

benjchristensen added a commit that referenced this pull request Jun 4, 2014
@benjchristensen benjchristensen merged commit 7de1f9b into ReactiveX:master Jun 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants