Skip to content

Database: Rename test helpers#280

Merged
schmidt-sebastian merged 1 commit intomasterfrom
mrschmidt-rename
Mar 15, 2019
Merged

Database: Rename test helpers#280
schmidt-sebastian merged 1 commit intomasterfrom
mrschmidt-rename

Conversation

@schmidt-sebastian
Copy link
Contributor

While working on the component registration for the RTDB, I got super confused because we have two TestHelper files in the same package. One is used for Unit Tests and one for Integration Tests. They used to be part of the same package, but I split them up when we moved to GitHub.

This PR:

  • Renames them to IntegrationTestHelpers and UnitTestHelpers respectivley
  • Moves ReadFuture and WriteFuture from testUtil to androidTest, since they require the IntegrationTestHelpers
  • Removes some unused methods from the test helpers

@googlebot googlebot added the cla: yes Override cla label Mar 14, 2019
@schmidt-sebastian
Copy link
Contributor Author

/test connected-check-changed

Copy link
Contributor

@mikelehen mikelehen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, that's one way to pad your CL stats I guess.... 😛

@schmidt-sebastian schmidt-sebastian merged commit 50e8932 into master Mar 15, 2019
@schmidt-sebastian
Copy link
Contributor Author

schmidt-sebastian commented Mar 15, 2019

Padded (and probably a better way than #281 :/)!

@vkryachko vkryachko deleted the mrschmidt-rename branch March 19, 2019 14:09
@firebase firebase locked and limited conversation to collaborators Oct 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants