This PR is dependent on the 2.1.0 release of the vagrant_cloud library. It updates the `cloud` command to use the new interface for interacting with the Vagrant Cloud API. It also adds support for direct to backend storage uploads, and defaults to this method. Also included is a bit of cleanup refactoring, addition of method documentation, and fixing up some small issues around custom username usage within the internal client for authentication.
130 lines
4.1 KiB
Ruby
130 lines
4.1 KiB
Ruby
require File.expand_path("../../../../../base", __FILE__)
|
|
require Vagrant.source_root.join("plugins/commands/cloud/version/update")
|
|
|
|
describe VagrantPlugins::CloudCommand::VersionCommand::Command::Update do
|
|
include_context "unit"
|
|
|
|
let(:access_token) { double("token") }
|
|
let(:org_name) { "my-org" }
|
|
let(:box_name) { "my-box" }
|
|
let(:box_version) { double("box_version") }
|
|
let(:account) { double("account") }
|
|
let(:organization) { double("organization") }
|
|
let(:box) { double("box", versions: [version]) }
|
|
let(:version) { double("version", version: box_version) }
|
|
|
|
describe "#update_version" do
|
|
let(:options) { {} }
|
|
let(:env) { double("env", ui: ui) }
|
|
let(:ui) { double("ui") }
|
|
let(:argv) { [] }
|
|
|
|
before do
|
|
allow(ui).to receive(:info)
|
|
allow(ui).to receive(:warn)
|
|
allow(ui).to receive(:success)
|
|
allow(ui).to receive(:error)
|
|
allow(env).to receive(:ui).and_return(ui)
|
|
allow(VagrantCloud::Account).to receive(:new).
|
|
with(custom_server: anything, access_token: access_token).
|
|
and_return(account)
|
|
allow(subject).to receive(:with_version).
|
|
with(account: account, org: org_name, box: box_name, version: box_version).
|
|
and_yield(version)
|
|
allow(version).to receive(:save)
|
|
allow(subject).to receive(:format_box_results)
|
|
end
|
|
|
|
subject { described_class.new(argv, env) }
|
|
|
|
it "should update the version" do
|
|
expect(version).to receive(:save)
|
|
subject.update_version(org_name, box_name, box_version, access_token, options)
|
|
end
|
|
|
|
it "should return 0 on success" do
|
|
result = subject.update_version(org_name, box_name, box_version, access_token, options)
|
|
expect(result).to eq(0)
|
|
end
|
|
|
|
it "should return non-zero result on error" do
|
|
expect(version).to receive(:save).and_raise(VagrantCloud::Error)
|
|
result = subject.update_version(org_name, box_name, box_version, access_token, options)
|
|
expect(result).not_to eq(0)
|
|
expect(result).to be_a(Integer)
|
|
end
|
|
|
|
context "with options set" do
|
|
let(:description) { double("description") }
|
|
let(:options) { {description: description} }
|
|
|
|
it "should set version info before saving" do
|
|
expect(version).to receive(:description=).with(description)
|
|
subject.update_version(org_name, box_name, box_version, access_token, options)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "#execute" do
|
|
let(:argv) { [] }
|
|
let(:iso_env) do
|
|
# We have to create a Vagrantfile so there is a root path
|
|
env = isolated_environment
|
|
env.vagrantfile("")
|
|
env.create_vagrant_env
|
|
end
|
|
|
|
subject { described_class.new(argv, iso_env) }
|
|
|
|
let(:action_runner) { double("action_runner") }
|
|
let(:client) { double("client", token: access_token) }
|
|
|
|
before do
|
|
allow(iso_env).to receive(:action_runner).and_return(action_runner)
|
|
allow(subject).to receive(:client_login).
|
|
and_return(client)
|
|
allow(subject).to receive(:update_version)
|
|
end
|
|
|
|
context "with no arguments" do
|
|
it "shows help" do
|
|
expect { subject.execute }.
|
|
to raise_error(Vagrant::Errors::CLIInvalidUsage)
|
|
end
|
|
end
|
|
|
|
context "with name argument" do
|
|
let(:argv) { ["#{org_name}/#{box_name}"] }
|
|
|
|
it "shows help" do
|
|
expect { subject.execute }.
|
|
to raise_error(Vagrant::Errors::CLIInvalidUsage)
|
|
end
|
|
|
|
context "with version argument" do
|
|
let(:version_arg) { "1.0.0" }
|
|
|
|
before { argv << version_arg }
|
|
|
|
it "should update the version" do
|
|
expect(subject).to receive(:update_version).
|
|
with(org_name, box_name, version_arg, access_token, anything)
|
|
subject.execute
|
|
end
|
|
|
|
context "with description flag" do
|
|
let(:description) { "my-description" }
|
|
|
|
before { argv.push("--description").push(description) }
|
|
|
|
it "should update version with description" do
|
|
expect(subject).to receive(:update_version).
|
|
with(org_name, box_name, version_arg, access_token, hash_including(description: description))
|
|
subject.execute
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|