neovim 0.7.0 → 0.7.1

data/spec/neovim/connection_spec.rb

@@ -30,16 +30,16 @@ module Neovim
       it "registers a msgpack ext type" do
         ext_class = Struct.new(:id) do
           def self.from_msgpack_ext(data)
-            new(data.unpack('N')[0])
+            new(data.unpack("N")[0])
           end
 
           def to_msgpack_ext
-            [self.id].pack('C')
+            [id].pack("C")
           end
         end
 
         client_rd, server_wr = IO.pipe
-        server_rd, client_wr = IO.pipe
+        _, client_wr = IO.pipe
 
         connection = Connection.new(client_rd, client_wr)
 
@@ -56,7 +56,6 @@ module Neovim
       end
     end
 
-
     describe "#close" do
       it "closes IO handles" do
         rd, wr = ::IO.pipe
@@ -69,10 +68,10 @@ module Neovim
       it "kills spawned processes" do
         io = ::IO.popen("cat", "rb+")
         pid = io.pid
-        expect(pid).to respond_to(:to_int)
+        thr = Process.detach(pid)
 
         Connection.new(io, nil_io).close
-        expect { Process.kill(0, pid) }.to raise_error(Errno::ESRCH)
+        expect(thr.join.pid).to eq(pid)
       end
     end
   end

data/spec/neovim/current_spec.rb

@@ -35,9 +35,9 @@ module Neovim
         initial_index = current.buffer.index
         client.command("vnew")
 
-        expect {
+        expect do
           current.buffer = initial_index
-        }.to change { current.buffer.index }.to(initial_index)
+        end.to change { current.buffer.index }.to(initial_index)
       end
 
       it "sets the current buffer from a Buffer" do
@@ -45,9 +45,9 @@ module Neovim
         client.command("vnew")
         b1 = current.buffer
 
-        expect {
+        expect do
           current.buffer = b0
-        }.to change { current.buffer }.from(b1).to(b0)
+        end.to change { current.buffer }.from(b1).to(b0)
       end
 
       it "returns an integer" do
@@ -72,9 +72,9 @@ module Neovim
         start_index = current.window.index
         client.command("vsp")
 
-        expect {
+        expect do
           current.window = start_index
-        }.to change { current.window.index }.to(start_index)
+        end.to change { current.window.index }.to(start_index)
       end
 
       it "sets the current window from a Window" do
@@ -82,9 +82,9 @@ module Neovim
         client.command("vsp")
         w1 = current.window
 
-        expect {
+        expect do
           current.window = w0
-        }.to change { current.window }.from(w1).to(w0)
+        end.to change { current.window }.from(w1).to(w0)
       end
 
       it "returns an integer" do
@@ -110,9 +110,9 @@ module Neovim
         client.command("tabnew")
         expect(current.tabpage.index).not_to eq(initial_index)
 
-        expect {
+        expect do
           current.tabpage = initial_index
-        }.to change { current.tabpage.index }.to(initial_index)
+        end.to change { current.tabpage.index }.to(initial_index)
       end
 
       it "sets the current tabpage from a Tabpage" do
@@ -120,9 +120,9 @@ module Neovim
         client.command("tabnew")
         tp1 = current.tabpage
 
-        expect {
+        expect do
           current.tabpage = tp0
-        }.to change { current.tabpage }.from(tp1).to(tp0)
+        end.to change { current.tabpage }.from(tp1).to(tp0)
       end
 
       it "returns an integer" do

data/spec/neovim/executable_spec.rb

@@ -23,9 +23,9 @@ module Neovim
       it "raises with an invalid executable path" do
         executable = Executable.new(File::NULL)
 
-        expect {
+        expect do
           executable.version
-        }.to raise_error(Executable::Error, Regexp.new(File::NULL))
+        end.to raise_error(Executable::Error, Regexp.new(File::NULL))
       end
     end
   end

data/spec/neovim/host/loader_spec.rb

@@ -45,9 +45,9 @@ module Neovim
 
         it "doesn't leak the overidden Neovim.plugin method" do
           loader.load([plugin_path])
-          expect {
+          expect do
             Neovim.plugin
-          }.to raise_error(/outside of a plugin host/)
+          end.to raise_error(/outside of a plugin host/)
         end
       end
     end

data/spec/neovim/host_spec.rb

@@ -64,7 +64,7 @@ module Neovim
       context "after poll" do
         before do
           nvim_wr.write([0, 1, :poll, []]).flush
-          _, reqid, _ = nvim_rd.read
+          _, reqid, = nvim_rd.read
 
           session = Session.new(EventLoop.child(Support.child_argv))
           api_info = session.request(:nvim_get_api_info)
@@ -87,22 +87,22 @@ module Neovim
                   "type" => "command",
                   "name" => "Echo",
                   "sync" => true,
-                  "opts" => {"nargs" => 1},
+                  "opts" => {"nargs" => 1}
                 },
                 {
                   "type" => "command",
                   "name" => "Boom",
                   "sync" => true,
-                  "opts" => {},
+                  "opts" => {}
                 },
                 {
                   "type" => "command",
                   "name" => "BoomAsync",
                   "sync" => false,
-                  "opts" => {},
-                },
-              ],
-            ],
+                  "opts" => {}
+                }
+              ]
+            ]
           )
         end
 

data/spec/neovim/line_range_spec.rb

@@ -66,9 +66,9 @@ module Neovim
         expect(line_range[-2, 2]).to eq(["3", "4"])
         expect(line_range[-2, 3]).to eq(["3", "4"])
 
-        expect {
+        expect do
           line_range[2, 3]
-        }.to raise_error(/out of bounds/)
+        end.to raise_error(/out of bounds/)
       end
 
       it "accepts a range" do
@@ -83,9 +83,9 @@ module Neovim
         expect(line_range[0...-4]).to eq([])
         expect(line_range[-2..-3]).to eq([])
 
-        expect {
+        expect do
           line_range[2..4]
-        }.to raise_error(/out of bounds/)
+        end.to raise_error(/out of bounds/)
       end
     end
 
@@ -97,9 +97,9 @@ module Neovim
         expect(line_range[-1] = "bar").to eq("bar")
         expect(line_range.to_a).to eq(["foo", "2", "3", "bar"])
 
-        expect {
+        expect do
           line_range[-5] = "foo"
-        }.to raise_error(/out of bounds/)
+        end.to raise_error(/out of bounds/)
       end
 
       it "accepts an index and length" do
@@ -112,9 +112,9 @@ module Neovim
         expect(line_range[0, 2] = "baz").to eq("baz")
         expect(line_range.to_a).to eq(["baz"])
 
-        expect {
+        expect do
           line_range[0, 5] = "foo"
-        }.to raise_error(/out of bounds/)
+        end.to raise_error(/out of bounds/)
       end
 
       it "accepts a range" do
@@ -141,17 +141,17 @@ module Neovim
 
     describe "#delete" do
       it "deletes the line at the given index" do
-        expect {
+        expect do
           line_range.delete(0)
-        }.to change { line_range.to_a }.to(["2", "3", "4"])
+        end.to change { line_range.to_a }.to(["2", "3", "4"])
 
-        expect {
+        expect do
           line_range.delete(-1)
-        }.to change { line_range.to_a }.to(["2", "3"])
+        end.to change { line_range.to_a }.to(["2", "3"])
 
-        expect {
+        expect do
           line_range.delete(-2)
-        }.to change { line_range.to_a }.to(["3"])
+        end.to change { line_range.to_a }.to(["3"])
       end
 
       it "returns the line deleted" do
@@ -160,9 +160,9 @@ module Neovim
       end
 
       it "returns nil if provided a non-integer" do
-        expect {
+        expect do
           expect(line_range.delete(:foo)).to eq(nil)
-        }.not_to change { line_range.to_a }
+        end.not_to change { line_range.to_a }
       end
     end
   end

data/spec/neovim/logging_spec.rb

@@ -16,21 +16,21 @@ module Neovim
 
     describe ".logger" do
       it "fetches the output from $NVIM_RUBY_LOG_FILE" do
-        logger = instance_double(Logger, "level=": nil, "formatter=": nil)
+        logger = instance_double(Logger, :level= => nil, :formatter= => nil)
         expect(Logger).to receive(:new).with("/tmp/nvim.log").and_return(logger)
         Logging.logger("NVIM_RUBY_LOG_FILE" => "/tmp/nvim.log")
         expect(Logging.logger).to be(logger)
       end
 
       it "defaults the output to STDERR" do
-        logger = instance_double(Logger, "level=": nil, "formatter=": nil)
+        logger = instance_double(Logger, :level= => nil, :formatter= => nil)
         expect(Logger).to receive(:new).with(STDERR).and_return(logger)
         Logging.logger({})
         expect(Logging.logger).to be(logger)
       end
 
       it "fetches the level from $NVIM_RUBY_LOG_LEVEL as a string" do
-        logger = instance_double(Logger, "formatter=": nil)
+        logger = instance_double(Logger, :formatter= => nil)
         expect(Logger).to receive(:new).and_return(logger)
         expect(logger).to receive(:level=).with(Logger::DEBUG)
         Logging.logger("NVIM_RUBY_LOG_LEVEL" => "DEBUG")
@@ -38,7 +38,7 @@ module Neovim
       end
 
       it "fetches the level from $NVIM_RUBY_LOG_LEVEL as an integer" do
-        logger = instance_double(Logger, "formatter=": nil)
+        logger = instance_double(Logger, :formatter= => nil)
         expect(Logger).to receive(:new).and_return(logger)
         expect(logger).to receive(:level=).with(0)
         Logging.logger("NVIM_RUBY_LOG_LEVEL" => "0")
@@ -46,7 +46,7 @@ module Neovim
       end
 
       it "defaults the level to WARN" do
-        logger = instance_double(Logger, "formatter=": nil)
+        logger = instance_double(Logger, :formatter= => nil)
         expect(Logger).to receive(:new).and_return(logger)
         expect(logger).to receive(:level=).with(Logger::WARN)
         Logging.logger({})
@@ -110,7 +110,7 @@ module Neovim
             "_method" => "some_method",
             "_time" => /\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d+/,
             "exception" => "RuntimeError",
-            "message" => "BOOM",
+            "message" => "BOOM"
           )
 
           expect(debug).to match(
@@ -120,7 +120,7 @@ module Neovim
             "_time" => /\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d+/,
             "exception" => "RuntimeError",
             "message" => "BOOM",
-            "backtrace" => ["one", "two"],
+            "backtrace" => ["one", "two"]
           )
         end
       end

data/spec/neovim/plugin_spec.rb

@@ -33,8 +33,8 @@ module Neovim
             nargs: 1,
             range: "",
             bang: "",
-            register: "",
-          },
+            register: ""
+          }
         )
       end
 
@@ -56,7 +56,7 @@ module Neovim
           type: :autocmd,
           name: "BufEnter",
           sync: false,
-          opts: {pattern: "*.rb"},
+          opts: {pattern: "*.rb"}
         )
       end
 
@@ -78,7 +78,7 @@ module Neovim
           type: :function,
           name: "Foo",
           sync: false,
-          opts: {range: "", nargs: 1},
+          opts: {range: "", nargs: 1}
         )
       end
 
@@ -95,9 +95,9 @@ module Neovim
           end
         end
 
-        expect {
+        expect do
           plugin.setup(:client)
-        }.to change { yielded }.from([]).to([:client, :other])
+        end.to change { yielded }.from([]).to([:client, :other])
       end
 
       it "registers a top level RPC" do

data/spec/neovim/ruby_provider/buffer_ext_spec.rb

@@ -19,9 +19,9 @@ module Neovim
 
     describe ".count" do
       it "returns the current buffer count from the global Vim client" do
-        expect {
+        expect do
           nvim.command("new")
-        }.to change { Buffer.count }.by(1)
+        end.to change { Buffer.count }.by(1)
       end
     end
 

data/spec/neovim/ruby_provider/vim_spec.rb

@@ -58,13 +58,13 @@ RSpec.describe Vim do
       Vim.__client = client
       Vim.__refresh_globals(client)
 
-      expect {
+      expect do
         Vim.command("wincmd n")
-      }.to change { $curwin.index }.by(1)
+      end.to change { $curwin.index }.by(1)
 
-      expect {
+      expect do
         Vim.command("vs bar")
-      }.to change { $curbuf.index }.by(1)
+      end.to change { $curbuf.index }.by(1)
     end
   end
 end

data/spec/neovim/ruby_provider/window_ext_spec.rb

@@ -19,31 +19,31 @@ module Neovim
 
     describe ".count" do
       it "returns the current window count from the global Vim client" do
-        expect {
+        expect do
           nvim.command("new")
-        }.to change { Window.count }.by(1)
+        end.to change { Window.count }.by(1)
       end
 
       it "only includes windows within a tabpage" do
-        expect {
+        expect do
           nvim.command("tabnew")
-        }.not_to change { Window.count }.from(1)
+        end.not_to change { Window.count }.from(1)
       end
     end
 
     describe ".[]" do
       it "returns the window at the given index" do
-        expect {
+        expect do
           nvim.command("new")
-        }.to change { Window[1] }.from(nil).to(kind_of(Window))
+        end.to change { Window[1] }.from(nil).to(kind_of(Window))
       end
 
       it "only includes windows within a tabpage" do
         nvim.command("new")
 
-        expect {
+        expect do
           nvim.command("tabnew")
-        }.to change { Window[1] }.from(kind_of(Window)).to(nil)
+        end.to change { Window[1] }.from(kind_of(Window)).to(nil)
       end
     end
   end

data/spec/neovim/session_spec.rb

@@ -13,9 +13,9 @@ module Neovim
       end
 
       it "raises an exception when there are errors" do
-        expect {
+        expect do
           session.request(:nvim_strwidth, "too", "many")
-        }.to raise_error(/wrong number of arguments/i)
+        end.to raise_error(/wrong number of arguments/i)
       end
 
       it "handles large data" do
@@ -25,17 +25,17 @@ module Neovim
       end
 
       it "fails outside of the main thread", :silence_thread_exceptions do
-        expect {
+        expect do
           Thread.new { session.request(:nvim_strwidth, "foo") }.join
-        }.to raise_error(/outside of the main thread/)
+        end.to raise_error(/outside of the main thread/)
       end
     end
 
     describe "#notify" do
       it "doesn't raise exceptions" do
-        expect {
+        expect do
           session.notify(:nvim_strwidth, "too", "many")
-        }.not_to raise_error
+        end.not_to raise_error
       end
 
       it "handles large data" do
@@ -45,9 +45,9 @@ module Neovim
       end
 
       it "succeeds outside of the main thread" do
-        expect {
+        expect do
           Thread.new { session.notify(:nvim_set_current_line, "foo") }.join
-        }.not_to raise_error
+        end.not_to raise_error
       end
     end