mirror of
https://github.com/openfoodfoundation/openfoodnetwork
synced 2026-02-28 01:53:25 +00:00
Enterprises.visible attribute is set to only_through_links by default
+ update enterprise_factory.rb to create "public" enterprise through specs via its factory
This commit is contained in:
@@ -0,0 +1,9 @@
|
||||
class ChangeDefaultvalueForVisibleEnterprise < ActiveRecord::Migration[7.0]
|
||||
def up
|
||||
change_column :enterprises, :visible, :string, default: "only_through_links"
|
||||
end
|
||||
|
||||
def down
|
||||
change_column :enterprises, :visible, :string, default: "public"
|
||||
end
|
||||
end
|
||||
@@ -10,7 +10,7 @@
|
||||
#
|
||||
# It's strongly recommended that you check this file into your version control system.
|
||||
|
||||
ActiveRecord::Schema[7.0].define(version: 2023_07_06_033212) do
|
||||
ActiveRecord::Schema[7.0].define(version: 2023_07_20_080504) do
|
||||
# These are extensions that must be enabled in order to support this database
|
||||
enable_extension "pg_stat_statements"
|
||||
enable_extension "plpgsql"
|
||||
@@ -233,7 +233,7 @@ ActiveRecord::Schema[7.0].define(version: 2023_07_06_033212) do
|
||||
t.boolean "enable_subscriptions", default: false, null: false
|
||||
t.integer "business_address_id"
|
||||
t.boolean "show_customer_names_to_suppliers", default: false, null: false
|
||||
t.string "visible", limit: 255, default: "public", null: false
|
||||
t.string "visible", default: "only_through_links", null: false
|
||||
t.string "whatsapp_phone", limit: 255
|
||||
t.boolean "hide_ofn_navigation", default: false, null: false
|
||||
t.text "white_label_logo_link"
|
||||
|
||||
@@ -47,6 +47,15 @@ describe Admin::EnterprisesController, type: :controller do
|
||||
expect(distributor_manager.enterprise_roles.where(enterprise_id: enterprise).first).to be
|
||||
end
|
||||
|
||||
it "set the `visible` attribute to `hidden`" do
|
||||
allow(controller).to receive_messages spree_current_user: distributor_manager
|
||||
enterprise_params[:enterprise][:owner_id] = distributor_manager
|
||||
|
||||
spree_put :create, enterprise_params
|
||||
enterprise = Enterprise.find_by name: 'zzz'
|
||||
expect(enterprise.visible).to eq 'only_through_links'
|
||||
end
|
||||
|
||||
context "when I already own a hub" do
|
||||
before { distributor }
|
||||
|
||||
|
||||
@@ -14,6 +14,7 @@ FactoryBot.define do
|
||||
description { 'enterprise' }
|
||||
long_description { '<p>Hello, world!</p><p>This is a paragraph.</p>' }
|
||||
address
|
||||
visible { 'public' }
|
||||
|
||||
after(:create) do |enterprise, proxy|
|
||||
proxy.users.each do |user|
|
||||
|
||||
@@ -63,6 +63,9 @@ describe '
|
||||
|
||||
click_button 'Create'
|
||||
expect(flash_message).to eq('Enterprise "Eaterprises" has been successfully created!')
|
||||
|
||||
# `Visible in search` radio button should be set to `Hide all references` by default
|
||||
expect(page).to have_checked_field "enterprise_visible_only_through_links"
|
||||
end
|
||||
|
||||
it "editing an existing enterprise" do
|
||||
|
||||
Reference in New Issue
Block a user