From bf4ea4e76c99da449d25e928200fad64b6b5b6fc Mon Sep 17 00:00:00 2001
From: Brian Cain <bcain@hashicorp.com>
Date: Tue, 9 Jan 2018 15:42:26 -0800
Subject: [PATCH] Fix sticky Travis CI test

this commit removes the webserver block within the box add unit test.
It's not actually needed since the test is just ensuring that if a user
adds a box name as a URL it prints a warning to the user.
---
 .../vagrant/action/builtin/box_add_test.rb    | 50 +++++++++----------
 1 file changed, 23 insertions(+), 27 deletions(-)

diff --git a/test/unit/vagrant/action/builtin/box_add_test.rb b/test/unit/vagrant/action/builtin/box_add_test.rb
index ef33328a3..427635d80 100644
--- a/test/unit/vagrant/action/builtin/box_add_test.rb
+++ b/test/unit/vagrant/action/builtin/box_add_test.rb
@@ -283,47 +283,43 @@ describe Vagrant::Action::Builtin::BoxAdd, :skip_windows do
     context "with a box name accidentally set as a URL" do
       it "displays a warning to the user" do
         box_path = iso_env.box2_file(:virtualbox)
-        with_web_server(box_path) do |port|
+        port = 9999
 
-          box_url_name = "http://127.0.0.1:#{port}/#{box_path.basename}"
-          env[:box_name] = box_url_name
+        box_url_name = "http://127.0.0.1:#{port}/#{box_path.basename}"
+        env[:box_name] = box_url_name
 
-          expect(box_collection).to receive(:add).with(any_args) { |path, name, version, **opts|
-            expect(name).to eq(box_url_name)
-            expect(version).to eq("0")
-            expect(opts[:metadata_url]).to be_nil
-            true
-          }.and_return(box)
+        expect(box_collection).to receive(:add).with(any_args) { |path, name, version, **opts|
+          expect(name).to eq(box_url_name)
+          expect(version).to eq("0")
+          expect(opts[:metadata_url]).to be_nil
+          true
+        }.and_return(box)
 
-          expect(app).to receive(:call).with(env)
+        expect(app).to receive(:call).with(env)
 
-          expect(env[:ui]).to receive(:warn)
-            .with(/It looks like you attempted to add a box with a URL for the name/)
+        expect(env[:ui]).to receive(:warn)
+          .with(/It looks like you attempted to add a box with a URL for the name/)
 
-          subject.call(env)
-        end
+        subject.call(env)
       end
     end
 
     context "with a box name containing invalid URI characters" do
       it "should not raise an error" do
         box_path = iso_env.box2_file(:virtualbox)
-        with_web_server(box_path) do |port|
+        box_url_name = "box name with spaces"
+        env[:box_name] = box_url_name
 
-          box_url_name = "box name with spaces"
-          env[:box_name] = box_url_name
+        expect(box_collection).to receive(:add).with(any_args) { |path, name, version, **opts|
+          expect(name).to eq(box_url_name)
+          expect(version).to eq("0")
+          expect(opts[:metadata_url]).to be_nil
+          true
+        }.and_return(box)
 
-          expect(box_collection).to receive(:add).with(any_args) { |path, name, version, **opts|
-            expect(name).to eq(box_url_name)
-            expect(version).to eq("0")
-            expect(opts[:metadata_url]).to be_nil
-            true
-          }.and_return(box)
+        expect(app).to receive(:call).with(env)
 
-          expect(app).to receive(:call).with(env)
-
-          subject.call(env)
-        end
+        subject.call(env)
       end
     end