Skip to content

Commit

Permalink
Use lower cased names for groups: (getredash#3221)
Browse files Browse the repository at this point in the history
Otherwise sorting depends on the Postgres collation and causes tests
to fail on different envoirnments.
  • Loading branch information
arikfr authored Dec 26, 2018
1 parent 9e156c1 commit f62d0e1
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions tests/test_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -217,8 +217,8 @@ def test_change_permissions(self):

def test_list(self):
self.factory.create_group(name='test', permissions=['list_dashboards'])
self.factory.create_group(name='Agroup', permissions=['list_dashboards'])
self.factory.create_group(name='Bgroup', permissions=['list_dashboards'])
self.factory.create_group(name='agroup', permissions=['list_dashboards'])
self.factory.create_group(name='bgroup', permissions=['list_dashboards'])

self.factory.create_user(name='Fred Foobar',
email=u'foobar@example.com',
Expand All @@ -230,27 +230,27 @@ def test_list(self):
self.assertFalse(result.exception)
self.assertEqual(result.exit_code, 0)
output = """
Id: 1
Name: admin
Type: builtin
Organization: default
Permissions: [admin,super_admin]
Users:
--------------------
Id: 4
Name: Agroup
Name: agroup
Type: regular
Organization: default
Permissions: [list_dashboards]
Users:
--------------------
Id: 5
Name: Bgroup
Name: bgroup
Type: regular
Organization: default
Permissions: [list_dashboards]
Users:
--------------------
Id: 1
Name: admin
Type: builtin
Organization: default
Permissions: [admin,super_admin]
Users:
--------------------
Id: 2
Name: default
Type: builtin
Expand Down

0 comments on commit f62d0e1

Please sign in to comment.