Rename DisplayAdHelper to BillboardHelper (#19750)
This commit is contained in:
parent
ef5d126a0f
commit
be1e57d85c
5 changed files with 9 additions and 9 deletions
|
|
@ -1,6 +1,6 @@
|
|||
class BillboardsController < ApplicationController
|
||||
before_action :set_cache_control_headers, only: %i[show], unless: -> { current_user }
|
||||
include DisplayAdHelper
|
||||
include BillboardHelper
|
||||
CACHE_EXPIRY_FOR_DISPLAY_ADS = 15.minutes.to_i.freeze
|
||||
|
||||
def show
|
||||
|
|
|
|||
|
|
@ -1,5 +1,5 @@
|
|||
module DisplayAdHelper
|
||||
def display_ads_placement_area_options_array
|
||||
module BillboardHelper
|
||||
def billboards_placement_area_options_array
|
||||
DisplayAd::ALLOWED_PLACEMENT_AREAS_HUMAN_READABLE.zip(DisplayAd::ALLOWED_PLACEMENT_AREAS)
|
||||
end
|
||||
|
||||
|
|
@ -1,6 +1,6 @@
|
|||
module DisplayAds
|
||||
class FilteredAdsQuery
|
||||
include DisplayAdHelper
|
||||
include BillboardHelper
|
||||
def self.call(...)
|
||||
new(...).call
|
||||
end
|
||||
|
|
|
|||
|
|
@ -24,7 +24,7 @@
|
|||
|
||||
<div class="crayons-field">
|
||||
<%= label_tag :placement_area, "Placement Area:", class: "crayons-field__label" %>
|
||||
<%= select_tag :placement_area, options_for_select(display_ads_placement_area_options_array, selected: @display_ad.placement_area), include_blank: "Select...", class: "crayons-select js-placement-area" %>
|
||||
<%= select_tag :placement_area, options_for_select(billboards_placement_area_options_array, selected: @display_ad.placement_area), include_blank: "Select...", class: "crayons-select js-placement-area" %>
|
||||
</div>
|
||||
|
||||
<div class="crayons-field">
|
||||
|
|
|
|||
|
|
@ -1,10 +1,10 @@
|
|||
require "rails_helper"
|
||||
|
||||
describe DisplayAdHelper do
|
||||
describe ".display_ads_placement_area_options_array" do
|
||||
describe BillboardHelper do
|
||||
describe ".billboards_placement_area_options_array" do
|
||||
it "returns proper human value" do
|
||||
expect(helper.display_ads_placement_area_options_array[1]).to eq ["Sidebar Left (Second Position)",
|
||||
"sidebar_left_2"]
|
||||
expect(helper.billboards_placement_area_options_array[1]).to eq ["Sidebar Left (Second Position)",
|
||||
"sidebar_left_2"]
|
||||
end
|
||||
end
|
||||
|
||||
Loading…
Add table
Reference in a new issue