Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'spec/models/project_wiki_spec.rb')
-rw-r--r--spec/models/project_wiki_spec.rb453
1 files changed, 20 insertions, 433 deletions
diff --git a/spec/models/project_wiki_spec.rb b/spec/models/project_wiki_spec.rb
index 1b121b7dee1..a4181e3be9a 100644
--- a/spec/models/project_wiki_spec.rb
+++ b/spec/models/project_wiki_spec.rb
@@ -1,448 +1,35 @@
# frozen_string_literal: true
-require "spec_helper"
+require 'spec_helper'
describe ProjectWiki do
- let(:user) { create(:user, :commit_email) }
- let(:project) { create(:project, :wiki_repo, namespace: user.namespace) }
- let(:repository) { project.repository }
- let(:gitlab_shell) { Gitlab::Shell.new }
- let(:project_wiki) { described_class.new(project, user) }
- let(:raw_repository) { Gitlab::Git::Repository.new(project.repository_storage, subject.disk_path + '.git', 'foo', 'group/project.wiki') }
- let(:commit) { project_wiki.repository.head_commit }
+ it_behaves_like 'wiki model' do
+ let(:wiki_container) { create(:project, :wiki_repo, namespace: user.namespace) }
+ let(:wiki_container_without_repo) { create(:project, namespace: user.namespace) }
- subject { project_wiki }
+ it { is_expected.to delegate_method(:storage).to(:container) }
+ it { is_expected.to delegate_method(:repository_storage).to(:container) }
+ it { is_expected.to delegate_method(:hashed_storage?).to(:container) }
- it { is_expected.to delegate_method(:repository_storage).to :project }
- it { is_expected.to delegate_method(:hashed_storage?).to :project }
-
- describe "#full_path" do
- it "returns the project path with namespace with the .wiki extension" do
- expect(subject.full_path).to eq(project.full_path + '.wiki')
- end
-
- it 'returns the same value as #full_path' do
- expect(subject.full_path).to eq(subject.full_path)
- end
- end
-
- describe '#web_url' do
- it 'returns the full web URL to the wiki' do
- expect(subject.web_url).to eq(Gitlab::UrlBuilder.build(subject))
- end
- end
-
- describe "#url_to_repo" do
- it "returns the correct ssh url to the repo" do
- expect(subject.url_to_repo).to eq(Gitlab::RepositoryUrlBuilder.build(subject.repository.full_path, protocol: :ssh))
- end
- end
-
- describe "#ssh_url_to_repo" do
- it "equals #url_to_repo" do
- expect(subject.ssh_url_to_repo).to eq(subject.url_to_repo)
- end
- end
-
- describe "#http_url_to_repo" do
- it "returns the correct http url to the repo" do
- expect(subject.http_url_to_repo).to eq(Gitlab::RepositoryUrlBuilder.build(subject.repository.full_path, protocol: :http))
- end
- end
-
- describe "#wiki_base_path" do
- it "returns the wiki base path" do
- wiki_base_path = "#{Gitlab.config.gitlab.relative_url_root}/#{project.full_path}/-/wikis"
-
- expect(subject.wiki_base_path).to eq(wiki_base_path)
- end
- end
-
- describe "#wiki" do
- it "contains a Gitlab::Git::Wiki instance" do
- expect(subject.wiki).to be_a Gitlab::Git::Wiki
- end
-
- it "creates a new wiki repo if one does not yet exist" do
- expect(project_wiki.create_page("index", "test content")).to be_truthy
- end
-
- it "creates a new wiki repo with a default commit message" do
- expect(project_wiki.create_page("index", "test content", :markdown, "")).to be_truthy
-
- page = project_wiki.find_page('index')
-
- expect(page.last_version.message).to eq("#{user.username} created page: index")
- end
-
- it "raises CouldNotCreateWikiError if it can't create the wiki repository" do
- # Create a fresh project which will not have a wiki
- project_wiki = described_class.new(create(:project), user)
- expect(project_wiki.repository).to receive(:create_if_not_exists) { false }
-
- expect { project_wiki.send(:wiki) }.to raise_exception(ProjectWiki::CouldNotCreateWikiError)
- end
- end
-
- describe "#empty?" do
- context "when the wiki repository is empty" do
- describe '#empty?' do
- subject { super().empty? }
-
- it { is_expected.to be_truthy }
- end
- end
-
- context "when the wiki has pages" do
- before do
- project_wiki.create_page("index", "This is an awesome new Gollum Wiki")
- project_wiki.create_page("another-page", "This is another page")
- end
-
- describe '#empty?' do
- subject { super().empty? }
-
- it { is_expected.to be_falsey }
-
- it 'only instantiates a Wiki page once' do
- expect(WikiPage).to receive(:new).once.and_call_original
-
- subject
- end
- end
- end
- end
-
- describe "#list_pages" do
- let(:wiki_pages) { subject.list_pages }
-
- before do
- create_page("index", "This is an index")
- create_page("index2", "This is an index2")
- create_page("an index3", "This is an index3")
- end
-
- after do
- wiki_pages.each do |wiki_page|
- destroy_page(wiki_page.page)
- end
- end
-
- it "returns an array of WikiPage instances" do
- expect(wiki_pages.first).to be_a WikiPage
- end
-
- it 'does not load WikiPage content by default' do
- wiki_pages.each do |page|
- expect(page.content).to be_empty
- end
- end
-
- it 'returns all pages by default' do
- expect(wiki_pages.count).to eq(3)
- end
-
- context "with limit option" do
- it 'returns limited set of pages' do
- expect(subject.list_pages(limit: 1).count).to eq(1)
- end
- end
-
- context "with sorting options" do
- it 'returns pages sorted by title by default' do
- pages = ['an index3', 'index', 'index2']
-
- expect(subject.list_pages.map(&:title)).to eq(pages)
- expect(subject.list_pages(direction: "desc").map(&:title)).to eq(pages.reverse)
- end
-
- it 'returns pages sorted by created_at' do
- pages = ['index', 'index2', 'an index3']
-
- expect(subject.list_pages(sort: 'created_at').map(&:title)).to eq(pages)
- expect(subject.list_pages(sort: 'created_at', direction: "desc").map(&:title)).to eq(pages.reverse)
- end
- end
-
- context "with load_content option" do
- let(:pages) { subject.list_pages(load_content: true) }
-
- it 'loads WikiPage content' do
- expect(pages.first.content).to eq("This is an index3")
- expect(pages.second.content).to eq("This is an index")
- expect(pages.third.content).to eq("This is an index2")
- end
- end
- end
-
- describe "#find_page" do
- before do
- create_page("index page", "This is an awesome Gollum Wiki")
- end
-
- after do
- subject.list_pages.each { |page| destroy_page(page.page) }
- end
-
- it "returns the latest version of the page if it exists" do
- page = subject.find_page("index page")
- expect(page.title).to eq("index page")
- end
-
- it "returns nil if the page does not exist" do
- expect(subject.find_page("non-existent")).to eq(nil)
- end
-
- it "can find a page by slug" do
- page = subject.find_page("index-page")
- expect(page.title).to eq("index page")
- end
-
- it "returns a WikiPage instance" do
- page = subject.find_page("index page")
- expect(page).to be_a WikiPage
- end
-
- context 'pages with multibyte-character title' do
- before do
- create_page("autre pagé", "C'est un génial Gollum Wiki")
- end
-
- it "can find a page by slug" do
- page = subject.find_page("autre pagé")
- expect(page.title).to eq("autre pagé")
- end
- end
-
- context 'pages with invalidly-encoded content' do
- before do
- create_page("encoding is fun", "f\xFCr".b)
- end
-
- it "can find the page" do
- page = subject.find_page("encoding is fun")
- expect(page.content).to eq("fr")
+ describe '#disk_path' do
+ it 'returns the repository storage path' do
+ expect(subject.disk_path).to eq("#{subject.container.disk_path}.wiki")
end
end
- end
-
- describe '#find_sidebar' do
- before do
- create_page(described_class::SIDEBAR, 'This is an awesome Sidebar')
- end
-
- after do
- subject.list_pages.each { |page| destroy_page(page.page) }
- end
-
- it 'finds the page defined as _sidebar' do
- page = subject.find_page('_sidebar')
-
- expect(page.content).to eq('This is an awesome Sidebar')
- end
- end
- describe '#find_file' do
- let(:image) { File.open(Rails.root.join('spec', 'fixtures', 'big-image.png')) }
+ describe '#update_container_activity' do
+ it 'updates project activity' do
+ wiki_container.update!(
+ last_activity_at: nil,
+ last_repository_updated_at: nil
+ )
- before do
- subject.wiki # Make sure the wiki repo exists
+ subject.create_page('Test Page', 'This is content')
+ wiki_container.reload
- repo_path = Gitlab::GitalyClient::StorageSettings.allow_disk_access do
- subject.repository.path_to_repo
+ expect(wiki_container.last_activity_at).to be_within(1.minute).of(Time.now)
+ expect(wiki_container.last_repository_updated_at).to be_within(1.minute).of(Time.now)
end
-
- BareRepoOperations.new(repo_path).commit_file(image, 'image.png')
- end
-
- it 'returns the latest version of the file if it exists' do
- file = subject.find_file('image.png')
- expect(file.mime_type).to eq('image/png')
- end
-
- it 'returns nil if the page does not exist' do
- expect(subject.find_file('non-existent')).to eq(nil)
- end
-
- it 'returns a Gitlab::Git::WikiFile instance' do
- file = subject.find_file('image.png')
- expect(file).to be_a Gitlab::Git::WikiFile
- end
-
- it 'returns the whole file' do
- file = subject.find_file('image.png')
- image.rewind
-
- expect(file.raw_data.b).to eq(image.read.b)
- end
- end
-
- describe "#create_page" do
- after do
- destroy_page(subject.list_pages.first.page)
- end
-
- it "creates a new wiki page" do
- expect(subject.create_page("test page", "this is content")).not_to eq(false)
- expect(subject.list_pages.count).to eq(1)
- end
-
- it "returns false when a duplicate page exists" do
- subject.create_page("test page", "content")
- expect(subject.create_page("test page", "content")).to eq(false)
end
-
- it "stores an error message when a duplicate page exists" do
- 2.times { subject.create_page("test page", "content") }
- expect(subject.error_message).to match(/Duplicate page:/)
- end
-
- it "sets the correct commit message" do
- subject.create_page("test page", "some content", :markdown, "commit message")
- expect(subject.list_pages.first.page.version.message).to eq("commit message")
- end
-
- it 'sets the correct commit email' do
- subject.create_page('test page', 'content')
-
- expect(user.commit_email).not_to eq(user.email)
- expect(commit.author_email).to eq(user.commit_email)
- expect(commit.committer_email).to eq(user.commit_email)
- end
-
- it 'updates project activity' do
- subject.create_page('Test Page', 'This is content')
-
- project.reload
-
- expect(project.last_activity_at).to be_within(1.minute).of(Time.now)
- expect(project.last_repository_updated_at).to be_within(1.minute).of(Time.now)
- end
- end
-
- describe "#update_page" do
- before do
- create_page("update-page", "some content")
- @gitlab_git_wiki_page = subject.wiki.page(title: "update-page")
- subject.update_page(
- @gitlab_git_wiki_page,
- content: "some other content",
- format: :markdown,
- message: "updated page"
- )
- @page = subject.list_pages(load_content: true).first.page
- end
-
- after do
- destroy_page(@page)
- end
-
- it "updates the content of the page" do
- expect(@page.raw_data).to eq("some other content")
- end
-
- it "sets the correct commit message" do
- expect(@page.version.message).to eq("updated page")
- end
-
- it 'sets the correct commit email' do
- expect(user.commit_email).not_to eq(user.email)
- expect(commit.author_email).to eq(user.commit_email)
- expect(commit.committer_email).to eq(user.commit_email)
- end
-
- it 'updates project activity' do
- subject.update_page(
- @gitlab_git_wiki_page,
- content: 'Yet more content',
- format: :markdown,
- message: 'Updated page again'
- )
-
- project.reload
-
- expect(project.last_activity_at).to be_within(1.minute).of(Time.now)
- expect(project.last_repository_updated_at).to be_within(1.minute).of(Time.now)
- end
- end
-
- describe "#delete_page" do
- before do
- create_page("index", "some content")
- @page = subject.wiki.page(title: "index")
- end
-
- it "deletes the page" do
- subject.delete_page(@page)
- expect(subject.list_pages.count).to eq(0)
- end
-
- it 'sets the correct commit email' do
- subject.delete_page(@page)
-
- expect(user.commit_email).not_to eq(user.email)
- expect(commit.author_email).to eq(user.commit_email)
- expect(commit.committer_email).to eq(user.commit_email)
- end
-
- it 'updates project activity' do
- subject.delete_page(@page)
-
- project.reload
-
- expect(project.last_activity_at).to be_within(1.minute).of(Time.now)
- expect(project.last_repository_updated_at).to be_within(1.minute).of(Time.now)
- end
- end
-
- describe '#ensure_repository' do
- let(:project) { create(:project) }
-
- it 'creates the repository if it not exist' do
- expect(raw_repository.exists?).to eq(false)
-
- subject.ensure_repository
-
- expect(raw_repository.exists?).to eq(true)
- end
-
- it 'does not create the repository if it exists' do
- subject.wiki
- expect(raw_repository.exists?).to eq(true)
-
- expect(subject).not_to receive(:create_repo!)
-
- subject.ensure_repository
- end
- end
-
- describe '#hook_attrs' do
- it 'returns a hash with values' do
- expect(subject.hook_attrs).to be_a Hash
- expect(subject.hook_attrs.keys).to contain_exactly(:web_url, :git_ssh_url, :git_http_url, :path_with_namespace, :default_branch)
- end
- end
-
- private
-
- def create_temp_repo(path)
- FileUtils.mkdir_p path
- system(*%W(#{Gitlab.config.git.bin_path} init --quiet --bare -- #{path}))
- end
-
- def remove_temp_repo(path)
- FileUtils.rm_rf path
- end
-
- def commit_details
- Gitlab::Git::Wiki::CommitDetails.new(user.id, user.username, user.name, user.commit_email, "test commit")
- end
-
- def create_page(name, content)
- subject.wiki.write_page(name, :markdown, content, commit_details)
- end
-
- def destroy_page(page)
- subject.delete_page(page, "test commit")
end
end