Actions
Bug #37273
closedInvalid kwargs handling in FindCommon
Status:
Closed
Priority:
Normal
Assignee:
Category:
Database
Target version:
Difficulty:
Triaged:
Yes
Description
foreman_salt fails its unit test. The code it uses doesn't override related parts.
/home/runner/work/foreman_salt/foreman_salt/app/controllers/concerns/find_common.rb:50: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
/home/runner/work/foreman_salt/foreman_salt/app/models/host.rb:2: warning: The called method `method_missing' is defined here
[MinitestRetry] retry 'test_0006_salt variable matching host with hostgroup specific value' count: 1, msg: NoMethodError: undefined method `empty?' for #<ActiveRecord::QueryMethods::WhereChain:0x00005e8b1602df28>
app/controllers/concerns/find_common.rb:12:in `resource_finder'
app/controllers/concerns/find_common.rb:8:in `find_resource'
app/controllers/concerns/foreman/controller/timezone.rb:10:in `set_timezone'
app/models/concerns/foreman/thread_session.rb:32:in `clear_thread'
app/controllers/concerns/foreman/controller/topbar_sweeper.rb:12:in `set_topbar_sweeper_controller'
foreman_salt/test/functional/minions_controller_test.rb:99:in `block in <class:MinionsControllerTest>'
There appears to be a difference in calling resource.where([]) and resource.where(*[]).
Updated by The Foreman Bot almost 2 years ago
- Status changed from New to Ready For Testing
- Assignee set to Ewoud Kohl van Wijngaarden
- Pull request https://github.com/theforeman/foreman/pull/10097 added
Updated by Ewoud Kohl van Wijngaarden over 1 year ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman|6c77d09b44456bc1770f0264f41c1238557c64fa.
Updated by The Foreman Bot over 1 year ago
- Pull request https://github.com/theforeman/foreman/pull/10103 added
Updated by Ewoud Kohl van Wijngaarden over 1 year ago
- Triaged changed from No to Yes
- Fixed in Releases deleted (
3.11.0)
Actions