From 40a956eb6825f2bbca06e9f24c1fb24dc71a1ecd Mon Sep 17 00:00:00 2001 From: Dmitriy Zaporozhets Date: Thu, 3 Jan 2013 00:01:08 +0200 Subject: [PATCH] Few more fixes after removing roles --- app/models/event.rb | 3 --- app/models/merge_request.rb | 2 +- spec/models/project_spec.rb | 8 -------- spec/models/user_spec.rb | 4 ---- spec/support/stubbed_repository.rb | 4 +--- 5 files changed, 2 insertions(+), 19 deletions(-) diff --git a/app/models/event.rb b/app/models/event.rb index eb88a4e..95075ff 100644 --- a/app/models/event.rb +++ b/app/models/event.rb @@ -15,9 +15,6 @@ # class Event < ActiveRecord::Base - include NoteEvent - include PushEvent - attr_accessible :project, :action, :data, :author_id, :project_id, :target_id, :target_type diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index 052e085..68211ac 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -20,7 +20,7 @@ # require Rails.root.join("app/models/commit") -require Rails.root.join("app/roles/static_model") +require Rails.root.join("lib/static_model") class MergeRequest < ActiveRecord::Base include IssueCommonality diff --git a/spec/models/project_spec.rb b/spec/models/project_spec.rb index 27e68ce..ea1efbb 100644 --- a/spec/models/project_spec.rb +++ b/spec/models/project_spec.rb @@ -133,14 +133,6 @@ describe Project do it { should respond_to(:path_with_namespace) } end - describe 'modules' do - it { should include_module(Repository) } - it { should include_module(PushObserver) } - it { should include_module(Authority) } - it { should include_module(Team) } - it { should include_module(NamespacedProject) } - end - it "should return valid url to repo" do project = Project.new(path: "somewhere") project.url_to_repo.should == Gitlab.config.gitolite.ssh_path_prefix + "somewhere.git" diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 5f6244e..51774e4 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -65,10 +65,6 @@ describe User do it { should ensure_length_of(:bio).is_within(0..255) } end - describe 'modules' do - it { should include_module(Account) } - end - describe "Respond to" do it { should respond_to(:is_admin?) } it { should respond_to(:identifier) } diff --git a/spec/support/stubbed_repository.rb b/spec/support/stubbed_repository.rb index 5bf3ea4..ad88dd7 100644 --- a/spec/support/stubbed_repository.rb +++ b/spec/support/stubbed_repository.rb @@ -1,6 +1,6 @@ # Stubs out all Git repository access done by models so that specs can run # against fake repositories without Grit complaining that they don't exist. -module StubbedRepository +class Project def path_to_repo if new_record? || path == 'newproject' # There are a couple Project specs and features that expect the Project's @@ -27,5 +27,3 @@ module StubbedRepository end end end - -Project.send(:include, StubbedRepository) -- libgit2 0.21.2