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

Respect DB_IP and DB_PORT in mongodb tests #2910

Merged
merged 1 commit into from
Feb 13, 2013
Merged

Respect DB_IP and DB_PORT in mongodb tests #2910

merged 1 commit into from
Feb 13, 2013

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Feb 10, 2013

It is usually preferred to run tests on a test-local database with dedicated mongod instance rather than using the system-wide database. In order to support that, an easy way of providing the proper port is handy.

For example, pymongo test suite uses DB_IP and DB_PORT environment variables to allow providing an alternate mongod instance. Using the same variable names in IPython's test suite seems like a good idea.

It is usually preferred to run tests on a test-local database with
dedicated mongod instance rather than using the system-wide database.
In order to support that, an easy way of providing the proper port
is handy.

For example, pymongo test suite uses DB_IP and DB_PORT environment
variables to allow providing an alternate mongod instance. Using
the same variable names in IPython's test suite seems a good idea.
@minrk
Copy link
Member

minrk commented Feb 13, 2013

I don't see a reason why not. Merging.

minrk added a commit that referenced this pull request Feb 13, 2013
Respect DB_IP and DB_PORT in mongodb tests

It is usually preferred to run tests on a test-local database with dedicated mongod instance rather than using the system-wide database. In order to support that, an easy way of providing the proper port is handy.

For example, pymongo test suite uses DB_IP and DB_PORT environment variables to allow providing an alternate mongod instance. Using the same variable names in IPython's test suite seems like a good idea.
@minrk minrk merged commit 64c76a1 into ipython:master Feb 13, 2013
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
Respect DB_IP and DB_PORT in mongodb tests

It is usually preferred to run tests on a test-local database with dedicated mongod instance rather than using the system-wide database. In order to support that, an easy way of providing the proper port is handy.

For example, pymongo test suite uses DB_IP and DB_PORT environment variables to allow providing an alternate mongod instance. Using the same variable names in IPython's test suite seems like a good idea.
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.

2 participants