Skip to content

Commit 2cd0bcf

Browse files
committed
🚒 Fixing namespace declaration
* Made sure namespaces are unique within each service's test ; * Fixed GITHUB namespace leaking into gitlab and bitbucket cassettes ; * Added special case for bitbucket namespace handling (which is still part of the token) Signed-off-by: Guyzmo <[email protected]>
1 parent 3c74df8 commit 2cd0bcf

28 files changed

+109
-107
lines changed

tests/conftest.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@
1919
token_name = 'PRIVATE_KEY_{}'.format(service.upper())
2020
namespace_name = '{}_NAMESPACE'.format(service.upper())
2121
if token_name not in os.environ:
22-
os.environ[token_name] = '_token_{}_:_private_'.format(s) # using a : for bitbucket's case
22+
os.environ[token_name] = '_namespace_{}_:_private_'.format(s) # using a : for bitbucket's case
2323
if namespace_name not in os.environ:
24-
os.environ[namespace_name] = '_not_a_namespace_'
24+
os.environ[namespace_name] = '_namespace_{}_'.format(s)
2525
else:
2626
# if running tests "locally" and not in travis, let's try to extract the keys from
2727
# the local configuration if there is some local configuration. And exposes them as
@@ -34,16 +34,18 @@
3434
get_token = lambda s: config.get_value(get_section(s), 'token',
3535
config.get_value(get_section(s), 'private_token',
3636
config.get_value(get_section(s), 'privatekey',
37-
'_token_{}_:_private_'.format(s) # using a : for bitbucket's case
37+
'_namespace_{}_:_private_'.format(s) # using a : for bitbucket's case
3838
)))
39+
# XXX temporary fix that should not be necessary when refactoring with pybitbucket
40+
get_default_namespace = lambda s: os.environ[token_name].split(':')[0] if s == 'bitbucket' else '_namespace_{}_'.format(s)
3941

4042
for service in services:
4143
token_name = 'PRIVATE_KEY_{}'.format(service.upper())
4244
namespace_name = '{}_NAMESPACE'.format(service.upper())
4345
if token_name not in os.environ:
4446
os.environ[token_name] = get_token(service)
4547
if namespace_name not in os.environ:
46-
os.environ[namespace_name] = os.environ.get('GITREPO_NAMESPACE', getpass.getuser())
48+
os.environ[namespace_name] = os.environ.get('GITREPO_NAMESPACE', get_default_namespace(service))
4749

4850
betamax.Betamax.register_serializer(pretty_json.PrettyJSONSerializer)
4951

tests/integration/cassettes/test_bitbucket_test_00_fork.json

Lines changed: 5 additions & 5 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_01_create.json

Lines changed: 5 additions & 5 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_02_delete.json

Lines changed: 6 additions & 6 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_03_delete_nouser.json

Lines changed: 6 additions & 6 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_04_clone.json

Lines changed: 4 additions & 4 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_05_add.json

Lines changed: 4 additions & 4 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_06_add__name.json

Lines changed: 4 additions & 4 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_07_add__alone.json

Lines changed: 4 additions & 4 deletions
Large diffs are not rendered by default.

tests/integration/cassettes/test_bitbucket_test_08_add__alone_name.json

Lines changed: 4 additions & 4 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)