require 'spec_helper' describe Import::BitbucketController do let(:user) { create(:user, bitbucket_access_token: 'asd123', bitbucket_access_token_secret: "sekret") } before do sign_in(user) controller.stub(:bitbucket_import_enabled?).and_return(true) end describe "GET callback" do before do session[:oauth_request_token] = {} end it "updates access token" do token = "asdasd12345" secret = "sekrettt" access_token = double(token: token, secret: secret) Gitlab::BitbucketImport::Client.any_instance.stub(:get_token).and_return(access_token) Gitlab.config.omniauth.providers << OpenStruct.new(app_id: "asd123", app_secret: "asd123", name: "bitbucket") get :callback expect(user.reload.bitbucket_access_token).to eq(token) expect(user.reload.bitbucket_access_token_secret).to eq(secret) expect(controller).to redirect_to(status_import_bitbucket_url) end end describe "GET status" do before do @repo = OpenStruct.new(slug: 'vim', owner: 'asd') end it "assigns variables" do @project = create(:project, import_type: 'bitbucket', creator_id: user.id) controller.stub_chain(:client, :projects).and_return([@repo]) get :status expect(assigns(:already_added_projects)).to eq([@project]) expect(assigns(:repos)).to eq([@repo]) end it "does not show already added project" do @project = create(:project, import_type: 'bitbucket', creator_id: user.id, import_source: 'asd/vim') controller.stub_chain(:client, :projects).and_return([@repo]) get :status expect(assigns(:already_added_projects)).to eq([@project]) expect(assigns(:repos)).to eq([]) end end describe "POST create" do let(:bitbucket_username) { user.username } let(:bitbucket_user) { { user: { username: bitbucket_username } }.with_indifferent_access } let(:bitbucket_repo) { { slug: "vim", owner: bitbucket_username }.with_indifferent_access } before do allow(Gitlab::BitbucketImport::KeyAdder). to receive(:new).with(bitbucket_repo, user). and_return(double(execute: true)) controller.stub_chain(:client, :user).and_return(bitbucket_user) controller.stub_chain(:client, :project).and_return(bitbucket_repo) end context "when the repository owner is the Bitbucket user" do context "when the Bitbucket user and GitLab user's usernames match" do it "takes the current user's namespace" do expect(Gitlab::BitbucketImport::ProjectCreator). to receive(:new).with(bitbucket_repo, user.namespace, user). and_return(double(execute: true)) post :create, format: :js end end context "when the Bitbucket user and GitLab user's usernames don't match" do let(:bitbucket_username) { "someone_else" } it "takes the current user's namespace" do expect(Gitlab::BitbucketImport::ProjectCreator). to receive(:new).with(bitbucket_repo, user.namespace, user). and_return(double(execute: true)) post :create, format: :js end end end context "when the repository owner is not the Bitbucket user" do let(:other_username) { "someone_else" } before do bitbucket_repo["owner"] = other_username end context "when a namespace with the Bitbucket user's username already exists" do let!(:existing_namespace) { create(:namespace, name: other_username, owner: user) } context "when the namespace is owned by the GitLab user" do it "takes the existing namespace" do expect(Gitlab::BitbucketImport::ProjectCreator). to receive(:new).with(bitbucket_repo, existing_namespace, user). and_return(double(execute: true)) post :create, format: :js end end context "when the namespace is not owned by the GitLab user" do before do existing_namespace.owner = create(:user) existing_namespace.save end it "doesn't create a project" do expect(Gitlab::BitbucketImport::ProjectCreator). not_to receive(:new) post :create, format: :js end end end context "when a namespace with the Bitbucket user's username doesn't exist" do it "creates the namespace" do expect(Gitlab::BitbucketImport::ProjectCreator). to receive(:new).and_return(double(execute: true)) post :create, format: :js expect(Namespace.where(name: other_username).first).not_to be_nil end it "takes the new namespace" do expect(Gitlab::BitbucketImport::ProjectCreator). to receive(:new).with(bitbucket_repo, an_instance_of(Group), user). and_return(double(execute: true)) post :create, format: :js end end end end end