Skip to content

Commit

Permalink
Rename AdminSetCreateService to avoid confusion with CC::AdminSetService
Browse files Browse the repository at this point in the history
  • Loading branch information
jcoyne committed Oct 31, 2016
1 parent 6d6f8ae commit fca33fb
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion app/controllers/sufia/admin/admin_sets_controller.rb
Expand Up @@ -83,7 +83,7 @@ def list_search_builder
end

def create_admin_set
AdminSetService.new(@admin_set, current_user).create
AdminSetCreateService.new(@admin_set, current_user).create
end

def setup_form
Expand Down
@@ -1,6 +1,6 @@
module Sufia
# Creates AdminSets
class AdminSetService
class AdminSetCreateService
# @param admin_set [AdminSet] the admin set to operate on
# @param creating_user [User] the user who created the admin set.
def initialize(admin_set, creating_user)
Expand Down
4 changes: 2 additions & 2 deletions spec/controllers/sufia/admin/admin_sets_controller_spec.rb
Expand Up @@ -55,9 +55,9 @@
end

describe "#create" do
let(:service) { instance_double(Sufia::AdminSetService) }
let(:service) { instance_double(Sufia::AdminSetCreateService) }
before do
allow(Sufia::AdminSetService).to receive(:new)
allow(Sufia::AdminSetCreateService).to receive(:new)
.with(AdminSet, user)
.and_return(service)
end
Expand Down
@@ -1,6 +1,6 @@
require 'spec_helper'

RSpec.describe Sufia::AdminSetService do
RSpec.describe Sufia::AdminSetCreateService do
let(:admin_set) { AdminSet.new(title: ['test']) }
let(:service) { described_class.new(admin_set, user) }
let(:user) { create(:user) }
Expand Down

0 comments on commit fca33fb

Please sign in to comment.