From b6c6102e85196151aecb42e0e54cd561747dfe41 Mon Sep 17 00:00:00 2001 From: Brian Cain Date: Wed, 5 Dec 2018 12:19:51 -0800 Subject: [PATCH] Redirect output to stderr instead of using Basic UI class This preserves the bolding for the version check rather than using the no format Basic class. --- lib/vagrant/util/checkpoint_client.rb | 6 +++--- test/unit/vagrant/util/checkpoint_client_test.rb | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/vagrant/util/checkpoint_client.rb b/lib/vagrant/util/checkpoint_client.rb index 3013e4e6a..e8ff3da54 100644 --- a/lib/vagrant/util/checkpoint_client.rb +++ b/lib/vagrant/util/checkpoint_client.rb @@ -159,11 +159,11 @@ module Vagrant def version_check latest_version = Gem::Version.new(result["current_version"]) installed_version = Gem::Version.new(VERSION) - ui = Vagrant::UI::Prefixed.new(Vagrant::UI::Basic.new, "vagrant") + ui = Vagrant::UI::Prefixed.new(env.ui, "vagrant") if latest_version > installed_version @logger.info("new version of Vagrant available - #{latest_version}") - ui.error(I18n.t("vagrant.version_upgrade_available", latest_version: latest_version, installed_version: installed_version)) - env.ui.error("") + ui.info(I18n.t("vagrant.version_upgrade_available", latest_version: latest_version, installed_version: installed_version), channel: :error) + env.ui.info("", channel: :error) else @logger.debug("vagrant is currently up to date") end diff --git a/test/unit/vagrant/util/checkpoint_client_test.rb b/test/unit/vagrant/util/checkpoint_client_test.rb index a4fa99165..0e38c54d8 100644 --- a/test/unit/vagrant/util/checkpoint_client_test.rb +++ b/test/unit/vagrant/util/checkpoint_client_test.rb @@ -175,7 +175,7 @@ describe Vagrant::Util::CheckpointClient do let(:result) { {"current_version" => new_version} } it "should not display upgrade information" do - expect(prefixed_ui).not_to receive(:error).at_least(:once) + expect(prefixed_ui).not_to receive(:info).at_least(:once) subject.version_check end end