Project

General

Profile

Actions
Copy link

Bug #25602

closed

audit failures with Rails 5.2.2

Added by Anonymous over 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomer Brisker
Category:
Rails
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/7200
Fixed in Releases:
2.0.0
Found in Releases:
Nightly
Red Hat JIRA:

Description

The following two failures can be seen, starting with https://github.com/rails/rails/commit/7794d00c7a966a63d98331cfd986edb1d2ccdb2c

AuditAssociationsTest#test_0002_Should audit associations on update [.../test/models/concerns/audit_associations_test.rb:27]:
Expected: [[], [15]]
  Actual: [[7], [15]]

UserTest::audit user roles#test_0001_should audit when a role is assigned to a user [.../test/models/user_test.rb:1214]:
Expected [7] to be empty.

Related issues 1 (0 open — 1 closed)

Blocks Foreman - Tracker #25601: Rails 5.2.4 trackerRejected

Actions
Actions
Copy link
 #1

Updated by Anonymous over 5 years ago

Actions
Copy link
 #2

Updated by Kavita Gaikwad almost 5 years ago

  • Assignee set to Kavita Gaikwad
Actions
Copy link
 #3

Updated by Kavita Gaikwad over 4 years ago

  • Status changed from New to Resolved

Marking it as resolved as not able to reproduce with the changes from https://projects.theforeman.org/issues/27185.

Actions
Copy link
 #4

Updated by Kavita Gaikwad over 4 years ago

  • Related to Bug #27185: Reset changed? state after saving with DirtyAssociations added
Actions
Copy link
 #5

Updated by Kavita Gaikwad over 4 years ago

  • Status changed from Resolved to Assigned

Reopening this as this test is failing on 5.2.3. My apologies!

Actions
Copy link
 #6

Updated by Kavita Gaikwad over 4 years ago

  • Related to deleted (Bug #27185: Reset changed? state after saving with DirtyAssociations)
Actions
Copy link
 #7

Updated by Tomer Brisker over 4 years ago

  • Category set to Rails
Actions
Copy link
 #8

Updated by The Foreman Bot over 4 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7200 added
Actions
Copy link
 #9

Updated by Tomer Brisker over 4 years ago

  • Assignee changed from Kavita Gaikwad to Tomer Brisker
Actions
Copy link
 #10

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases 2.0.0 added
Actions
Copy link
 #11

Updated by Tomer Brisker over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link

Also available in: Atom PDF