Merge pull request #6877 from ryanpetrello/fix-6870

rename "Insights Basic Auth" to "Insights"
This commit is contained in:
Ryan Petrello
2017-07-05 10:43:40 -04:00
committed by GitHub
2 changed files with 4 additions and 4 deletions

View File

@@ -946,16 +946,16 @@ def azure_rm(cls):
def insights(cls): def insights(cls):
return cls( return cls(
kind='insights', kind='insights',
name='Insights Basic Auth', name='Insights',
managed_by_tower=True, managed_by_tower=True,
inputs={ inputs={
'fields': [{ 'fields': [{
'id': 'username', 'id': 'username',
'label': 'Basic Auth Username', 'label': 'Username',
'type': 'string' 'type': 'string'
}, { }, {
'id': 'password', 'id': 'password',
'label': 'Basic Auth Password', 'label': 'Password',
'type': 'string', 'type': 'string',
'secret': True 'secret': True
}], }],

View File

@@ -320,7 +320,7 @@ def test_insights_migration():
'password': 'some-password', 'password': 'some-password',
}) })
assert cred.credential_type.name == 'Insights Basic Auth' assert cred.credential_type.name == 'Insights'
assert cred.inputs['username'] == 'bob' assert cred.inputs['username'] == 'bob'
assert cred.inputs['password'].startswith('$encrypted$') assert cred.inputs['password'].startswith('$encrypted$')