Skip to content

Commit

Permalink
Merge pull request openedx-unsupported#124 from edx/elasticsearch-port
Browse files Browse the repository at this point in the history
Run test ES server on different port to avoid conflict with edx-platform devstack
  • Loading branch information
thallada committed Jun 29, 2016
2 parents 93cf3a0 + e9207b8 commit b165ff0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion Makefile
Expand Up @@ -3,6 +3,7 @@ COVERAGE_DIR = $(ROOT)/build/coverage
PACKAGES = analyticsdataserver analytics_data_api
DATABASES = default analytics
ELASTICSEARCH_VERSION = 1.5.2
ELASTICSEARCH_PORT = 9223
TEST_SETTINGS = analyticsdataserver.settings.test

.PHONY: requirements develop clean diff.report view.diff.report quality
Expand All @@ -13,9 +14,10 @@ requirements:
test.install_elasticsearch:
curl -L -O https://download.elastic.co/elasticsearch/elasticsearch/elasticsearch-$(ELASTICSEARCH_VERSION).zip
unzip elasticsearch-$(ELASTICSEARCH_VERSION).zip
echo "http.port: $(ELASTICSEARCH_PORT)" >> elasticsearch-$(ELASTICSEARCH_VERSION)/config/elasticsearch.yml

test.run_elasticsearch:
cd elasticsearch-$(ELASTICSEARCH_VERSION) && ./bin/elasticsearch -d
cd elasticsearch-$(ELASTICSEARCH_VERSION) && ./bin/elasticsearch -d --http.port=$(ELASTICSEARCH_PORT)

test.requirements: requirements
pip install -q -r requirements/test.txt
Expand Down
2 changes: 1 addition & 1 deletion analyticsdataserver/settings/test.py
Expand Up @@ -26,6 +26,6 @@
TEST_RUNNER = 'django_nose.NoseTestSuiteRunner'

# Default elasticsearch port when running locally
ELASTICSEARCH_LEARNERS_HOST = 'http://localhost:9200/'
ELASTICSEARCH_LEARNERS_HOST = 'http://localhost:9223/'
ELASTICSEARCH_LEARNERS_INDEX = 'roster_test'
ELASTICSEARCH_LEARNERS_UPDATE_INDEX = 'index_update_test'

0 comments on commit b165ff0

Please sign in to comment.