Project

General

Profile

Bug #5391

Authorizer test should use testing permission

Added by Marek Hulán about 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Authorization
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

:view_hosts permission is granted by anonymous role by default, so it would be safer to use custom permissions for testing purposes
in fact it breaks tests for https://github.com/theforeman/foreman/pull/1371

Associated revisions

Revision 29a52c6c (diff)
Added by Marek Hulán about 5 years ago

Fixes #5391 - Authorizer test should use testing permission

Also change order of seed so admin user can be saved and
existing Anonymous role is assigned.

Revision 4dd8cc7d (diff)
Added by Marek Hulán about 5 years ago

Fixes #5391 - Authorizer test should use testing permission

Also change order of seed so admin user can be saved and
existing Anonymous role is assigned.

(cherry picked from commit 29a52c6c51afc55f36e7135a8c6794604173a5aa)

History

#1 Updated by Marek Hulán about 5 years ago

  • Status changed from Assigned to Ready For Testing

#2 Updated by Dmitri Dolguikh about 5 years ago

  • Target version changed from 1.8.4 to 1.8.3

#3 Updated by Dmitri Dolguikh about 5 years ago

  • Target version changed from 1.8.3 to 1.8.4

#4 Updated by Dmitri Dolguikh about 5 years ago

  • Target version changed from 1.8.4 to 1.8.3

#5 Updated by Dominic Cleal about 5 years ago

  • Legacy Backlogs Release (now unused) set to 4

#6 Updated by Marek Hulán about 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF