Project

General

Profile

Bug #27347

Pin audited gem to <4.9

Added by Adam Ruzicka 2 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Packaging
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Looking into the issue I found out that audited changes is a hash with strings as keys, but the tests expect symbols as keys.

UsergroupTest::audit usergroup::roles.test_0002_should audit when a role is removed/de-assigned from a usergroup (from UsergroupTest__audit usergroup__roles)

Error Message

No audits found for usergroup-roles

Stacktrace

No audits found for usergroup-roles (Minitest::Assertion)
/home/jenkins/workspace/foreman_salt-pull-request/database/postgresql/label/fast/ruby/2.6/foreman/test/models/usergroup_test.rb:364   

Related issues

Related to Foreman - Refactor #27359: Update audited gem to 4.9.xClosed

Associated revisions

Revision 2fc6163d (diff)
Added by Adam Ruzicka 2 months ago

Fixes #27347 - Pin audited to < 4.9.0 (#6915)

Audited 4.9.0 was released on 18.7.2019. This release changed
audited_changes from hash with indifferent access to hash, where keys
are strings. This change made all tests which accessed parts of the hash
using symbols as keys fail.

History

#1 Updated by The Foreman Bot 2 months ago

  • Assignee set to Adam Ruzicka
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6913 added

#2 Updated by The Foreman Bot 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6915 added

#3 Updated by Adam Ruzicka 2 months ago

Audited 4.9.0 was released on 18.7.2019. This release changed audited_changes from hash with indifferent access to hash, where keys are strings. This change made all tests which accessed parts of the hash using symbols as keys fail.

#4 Updated by Michael Moll 2 months ago

#5 Updated by The Foreman Bot 2 months ago

  • Fixed in Releases 1.23.0 added

#6 Updated by Tomer Brisker 2 months ago

  • Subject changed from User and UserGroup tests fail with no audits found to Pin audited gem to <4.9
  • Fixed in Releases 1.21.4, 1.22.1 added

#7 Updated by Tomer Brisker 2 months ago

  • Category changed from Tests to Packaging

#8 Updated by Adam Ruzicka 2 months ago

  • Status changed from Ready For Testing to Closed

#9 Updated by Adam Ruzicka 2 months ago

#10 Updated by Adam Ruzicka 2 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/6913)

#11 Updated by Michael Moll 2 months ago

Also available in: Atom PDF