bterkuile/fizzy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't require encrypted credentials in test

Browse Source
This commit is contained in:
Mike Dalessio
2025-06-24 12:09:41 -04:00
parent 92383902fb
commit bb54aa1c7e
2 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/controllers/signup/base_controller.rb
+3 -2
View File
@@ -2,9 +2,10 @@ class Signup::BaseController < ApplicationController
require_untenanted_access
before_action :redirect_if_local_auth
# TODO: Remove this auth before launch.
http_basic_authenticate_with(
name: Rails.application.credentials.dig(:account_signup_http_basic_auth, :name),
password: Rails.application.credentials.dig(:account_signup_http_basic_auth, :password)
name: Rails.env.test? ? "testname" : Rails.application.credentials.account_signup_http_basic_auth.name,
password: Rails.env.test? ? "testpassword" : Rails.application.credentials.account_signup_http_basic_auth.password
)
attr_reader :authenticated_identity
test/controllers/signup/accounts_controller_test.rb
+1 -6
View File
@@ -81,12 +81,7 @@ class Signup::AccountsControllerTest < ActionDispatch::IntegrationTest
private
def auth_headers
{
"HTTP_AUTHORIZATION" => ActionController::HttpAuthentication::Basic.encode_credentials(
Rails.application.credentials.dig(:account_signup_http_basic_auth, :name),
Rails.application.credentials.dig(:account_signup_http_basic_auth, :password)
)
}
{ "HTTP_AUTHORIZATION" => ActionController::HttpAuthentication::Basic.encode_credentials("testname", "testpassword") }
end
def assert_authentication_requested_for(identity)