diff --git a/coveralls/api.py b/coveralls/api.py index 73d51166..a05de8cd 100644 --- a/coveralls/api.py +++ b/coveralls/api.py @@ -118,9 +118,10 @@ def load_config_from_travis(): @staticmethod def load_config_from_semaphore(): - job = os.environ.get('SEMAPHORE_BUILD_NUMBER') - pr = os.environ.get('PULL_REQUEST_NUMBER') - return 'semaphore-ci', job, None, pr + job = os.environ.get('SEMAPHORE_CURRENT_JOB') + number = os.environ.get('SEMAPHORE_BUILD_NUMBER') + pr = os.environ.get('SEMAPHORE_BRANCH_ID') + return 'semaphore-ci', job, number, pr @staticmethod def load_config_from_unknown(): diff --git a/tests/api/configuration_test.py b/tests/api/configuration_test.py index 985bc925..71bb295e 100644 --- a/tests/api/configuration_test.py +++ b/tests/api/configuration_test.py @@ -173,12 +173,14 @@ def test_travis_no_config(self): @mock.patch.dict(os.environ, {'SEMAPHORE': 'True', 'SEMAPHORE_BUILD_NUMBER': '888', - 'PULL_REQUEST_NUMBER': '9999'}, + 'SEMAPHORE_CURRENT_JOB': '1', + 'SEMAPHORE_BRANCH_ID': '9999'}, clear=True) def test_semaphore_no_config(self): cover = Coveralls(repo_token='xxx') assert cover.config['service_name'] == 'semaphore-ci' - assert cover.config['service_job_id'] == '888' + assert cover.config['service_job_id'] == '1' + assert cover.config['service_number'] == '888' assert cover.config['service_pull_request'] == '9999' @mock.patch.dict(os.environ, {'COVERALLS_SERVICE_NAME': 'xxx'}, clear=True)