Disable bucketservice integration by default

[dankolbman]

Following kids-first/kf-api-bucketservice#35, we will need to start authenticating with the bucket service using auth0 service tokens. Due to kids-first/kf-api-study-creator#288, however, we will no longer be needing to create new buckets from the dataservice, so it will be easier to simply disable the bucket service integration entirely.

[znatty22]

Remove bucket creation code and testing

• kf-api-dataservice/dataservice/api/study/models.py

  Lines 71 to 87 in 36a0437

  	@event.listens_for(Study, 'after_insert', propagate=True)
  	def make_bucket(mapper, connection, target):
  	"""
  	Invokes the new bucket service to create a bucket in s3 for this study.
  	Only creates a new bucket on insertion of a study.
  	Will not attempt to remove the bucket on deletion to prevent any unwanted
  	data loss.
  	"""
  	url = current_app.config['BUCKET_SERVICE_URL']
  	if url is not None:
  	header = {}
  	token = current_app.config['BUCKET_SERVICE_TOKEN']
  	if token:
  	header['Authorization'] = 'Bearer {}'.format(token)
  	resp = requests.post(url+'/buckets',
  	json={'study_id': target.kf_id},
  	headers=header)

• kf-api-dataservice/tests/study/test_study_models.py

  Lines 33 to 45 in 36a0437

  	def test_bucket_service(self):
  	"""
  	Test that a request is sent to create a new bucket
  	"""
  	s = Study(external_id='phs002', short_code='KF-ST0')
  	db.session.add(s)
  	db.session.commit()
  	assert self.bucket_service.post.call_count == 1
  	headers = {'Authorization': 'Bearer test123'}
  	self.bucket_service.post.assert_called_with('/buckets',
  	json={'study_id': s.kf_id},
  	headers=headers)

• kf-api-dataservice/tests/utils.py

  Lines 9 to 27 in 36a0437

  	class WithBucketService():
  	""" Mocks responses from the bucket service """
  	def setUpBucketMock(self):
  	mod = 'dataservice.api.study.models.requests'
  	self.patch_bucket_service = patch(mod)
  	self.bucket_service = self.patch_bucket_service.start()
  	mock_resp_get = Mock()
  	mock_resp_get.status_code = 200
  	mock_resp_post = Mock()
  	mock_resp_post.status_code = 201
  	self.bucket_service.Session().get.side_effect = mock_resp_get
  	self.bucket_service.Session().post.side_effect = mock_resp_post
  	def tearDownBucketMock(self):
  	self.bucket_service.stop()

• kf-api-dataservice/tests/utils.py

  Line 29 in 36a0437

  class FlaskTestCase(unittest.TestCase, WithBucketService):