Refactor #18976: HostsControllerTest REMOTE_USER tests should not assume format of subsequent requests - Foreman

Actions

Copy link

Refactor #18976

closed

HostsControllerTest REMOTE_USER tests should not assume format of subsequent requests

Added by Dominic Cleal about 7 years ago. Updated almost 6 years ago.

Status:

Closed

Priority:

Low

Assignee:

Dominic Cleal

Category:

Tests

Target version:

1.15.0

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/4395

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

Under Rails 5.0, two HostsControllerTest tests fails:

HostsControllerTest#test_0037_if only authorize_login_delegation is set, REMOTE_USER should be
            ignored for API requests [test/controllers/hosts_controller_test.rb:617]:
    Expected response to be a &lt;2XX: success&gt;, but was a &lt;401: Unauthorized&gt;

When making a JSON format request followed by a request with no explicit format, Rails 5 is now memorising that it's requesting application/json instead of resetting to HTML or similar.

This is not a bug in Rails 5, as it shouldn't be assumed that the state will be reset between requests in a single test case: https://github.com/rails/rails/commit/33db42168425866b714203e35a3a36693983b3f9

The format should be set explicitly or moved into separate tests with a common setup.