diff options
-rw-r--r-- | .rubocop.yml | 24 | ||||
-rw-r--r-- | .travis.yml | 2 | ||||
-rwxr-xr-x | src/update_assets.rb | 26 | ||||
-rwxr-xr-x | test/test_go.rb | 617 |
4 files changed, 348 insertions, 321 deletions
diff --git a/.rubocop.yml b/.rubocop.yml new file mode 100644 index 00000000..496b049e --- /dev/null +++ b/.rubocop.yml @@ -0,0 +1,24 @@ +Layout/LineLength: + Enabled: false +Metrics: + Enabled: false +Lint/ShadowingOuterLocalVariable: + Enabled: false +Style/MethodCallWithArgsParentheses: + Enabled: true + IgnoredMethods: + - assert + - exit + - paste + - puts + - raise + - refute + - require + - send_keys + IgnoredPatterns: + - ^assert_ + - ^refute_ +Style/NumericPredicate: + Enabled: false +Style/WordArray: + MinSize: 1 diff --git a/.travis.yml b/.travis.yml index 32b3b691..8cd544d8 100644 --- a/.travis.yml +++ b/.travis.yml @@ -14,9 +14,11 @@ addons: - fish - tmux update: true +install: gem install rubocop rubocop-performance script: - make test # LC_ALL=C to avoid escape codes in # printf %q $'\355\205\214\354\212\244\355\212\270' on macOS. Bash on # macOS is built without HANDLE_MULTIBYTE? - make install && ./install --all && LC_ALL=C tmux new-session -d && ruby test/test_go.rb --verbose + - rubocop --require rubocop-performance diff --git a/src/update_assets.rb b/src/update_assets.rb index b4aaba54..531a8eeb 100755 --- a/src/update_assets.rb +++ b/src/update_assets.rb @@ -1,19 +1,20 @@ #!/usr/bin/env ruby +# frozen_string_literal: true # http://www.rubydoc.info/github/rest-client/rest-client/RestClient require 'rest_client' require 'json' if ARGV.length < 3 - puts "usage: #$0 <token> <version> <files...>" + puts "usage: #{$PROGRAM_NAME} <token> <version> <files...>" exit 1 end token, version, *files = ARGV -base = "https://api.github.com/repos/junegunn/fzf-bin/releases" +base = 'https://api.github.com/repos/junegunn/fzf-bin/releases' # List releases -rels = JSON.parse(RestClient.get(base, :authorization => "token #{token}")) +rels = JSON.parse(RestClient.get(base, authorization: "token #{token}")) rel = rels.find { |r| r['tag_name'] == version } unless rel puts "#{version} not found" @@ -21,25 +22,26 @@ unless rel end # List assets -assets = Hash[rel['assets'].map { |a| a.values_at *%w[name id] }] +assets = Hash[rel['assets'].map { |a| a.values_at('name', 'id') }] -files.select { |f| File.exists? f }.map do |file| +files.select { |f| File.exist?(f) }.map do |file| Thread.new do - name = File.basename file + name = File.basename(file) - if asset_id = assets[name] + if asset_id = assets[name] # rubocop:todo Lint/AssignmentInCondition puts "#{name} found. Deleting asset id #{asset_id}." - RestClient.delete "#{base}/assets/#{asset_id}", - :authorization => "token #{token}" + RestClient.delete("#{base}/assets/#{asset_id}", + authorization: "token #{token}") else puts "#{name} not found" end puts "Uploading #{name}" RestClient.post( - "#{base.sub 'api', 'uploads'}/#{rel['id']}/assets?name=#{name}", + "#{base.sub('api', 'uploads')}/#{rel['id']}/assets?name=#{name}", File.read(file), - :authorization => "token #{token}", - :content_type => "application/octet-stream") + authorization: "token #{token}", + content_type: 'application/octet-stream' + ) end end.each(&:join) diff --git a/test/test_go.rb b/test/test_go.rb index 420af34a..6f31dcf2 100755 --- a/test/test_go.rb +++ b/test/test_go.rb @@ -1,10 +1,6 @@ #!/usr/bin/env ruby -# encoding: utf-8 # frozen_string_literal: true -# rubocop:disable Metrics/LineLength -# rubocop:disable Metrics/MethodLength - require 'minitest/autorun' require 'fileutils' require 'English' @@ -45,7 +41,8 @@ def wait since = Time.now while Time.now - since < DEFAULT_TIMEOUT return if yield - sleep 0.05 + + sleep(0.05) end raise 'timeout' end @@ -57,7 +54,7 @@ class Shell begin bashrc = '/tmp/fzf.bash' File.open(bashrc, 'w') do |f| - f.puts(ERB.new(TEMPLATE).result(binding)) + f.puts ERB.new(TEMPLATE).result(binding) end "bash --rcfile #{bashrc}" @@ -71,7 +68,7 @@ class Shell FileUtils.rm_rf(zdotdir) FileUtils.mkdir_p(zdotdir) File.open("#{zdotdir}/.zshrc", 'w') do |f| - f.puts(ERB.new(TEMPLATE).result(binding)) + f.puts ERB.new(TEMPLATE).result(binding) end "ZDOTDIR=#{zdotdir} zsh" end @@ -91,7 +88,7 @@ class Tmux go(%W[set-window-option -t #{@win} pane-base-index 0]) return unless shell == :fish - send_keys('function fish_prompt; end; clear', :Enter) + send_keys 'function fish_prompt; end; clear', :Enter self.until(&:empty?) end @@ -123,7 +120,7 @@ class Tmux class << lines def counts lazy - .map { |l| l.scan %r{^. ([0-9]+)\/([0-9]+)( \(([0-9]+)\))?} } + .map { |l| l.scan(%r{^. ([0-9]+)\/([0-9]+)( \(([0-9]+)\))?}) } .reject(&:empty?) .first&.first&.map(&:to_i)&.values_at(0, 1, 3) || [0, 0, 0] end @@ -142,7 +139,7 @@ class Tmux def any_include?(val) method = val.is_a?(Regexp) ? :match : :include? - select { |line| line.send method, val }.first + find { |line| line.send(method, val) } end end yield(lines).tap do |ok| @@ -175,7 +172,7 @@ class Tmux private def go(args) - IO.popen(['tmux'] + args) { |io| io.readlines(chomp: true) } + IO.popen(%w[tmux] + args) { |io| io.readlines(chomp: true) } end end @@ -188,11 +185,11 @@ class TestBase < Minitest::Test @temp_suffix ||= 0 [TEMPNAME, caller_locations.map(&:label).find { |l| l =~ /^test_/ }, - @temp_suffix].join '-' + @temp_suffix].join('-') end def writelines(path, lines) - File.unlink path while File.exist? path + File.unlink(path) while File.exist?(path) File.open(path, 'w') { |f| f << lines.join($INPUT_RECORD_SEPARATOR) + $INPUT_RECORD_SEPARATOR } end @@ -200,7 +197,7 @@ class TestBase < Minitest::Test wait { File.exist?(tempname) } File.read(tempname) ensure - File.unlink tempname while File.exist?(tempname) + File.unlink(tempname) while File.exist?(tempname) @temp_suffix += 1 tmux.prepare end @@ -219,7 +216,7 @@ class TestBase < Minitest::Test o.to_s end end.compact - "#{FZF} #{opts.join ' '}" + "#{FZF} #{opts.join(' ')}" end end @@ -245,10 +242,10 @@ class TestGoFZF < TestBase # Testing basic key bindings tmux.send_keys '99', 'C-a', '1', 'C-f', '3', 'C-b', 'C-h', 'C-u', 'C-e', 'C-y', 'C-k', 'Tab', 'BTab' tmux.until do |lines| - '> 3910' == lines[-4] && - ' 391' == lines[-3] && - ' 856/100000' == lines[-2] && - '> 391' == lines[-1] + lines[-4] == '> 3910' && + lines[-3] == ' 391' && + lines[-2] == ' 856/100000' && + lines[-1] == '> 391' end tmux.send_keys :Enter @@ -348,13 +345,13 @@ class TestGoFZF < TestBase end def test_multi_order - tmux.send_keys "seq 1 10 | #{fzf :multi}", :Enter + tmux.send_keys "seq 1 10 | #{fzf(:multi)}", :Enter tmux.until { |lines| lines.last =~ /^>/ } tmux.send_keys :Tab, :Up, :Up, :Tab, :Tab, :Tab, # 3, 2 'C-K', 'C-K', 'C-K', 'C-K', :BTab, :BTab, # 5, 6 :PgUp, 'C-J', :Down, :Tab, :Tab # 8, 7 - tmux.until { |lines| lines[-2].include? '(6)' } + tmux.until { |lines| lines[-2].include?('(6)') } tmux.send_keys 'C-M' assert_equal %w[3 2 5 6 8 7], readonce.split($INPUT_RECORD_SEPARATOR) end @@ -422,7 +419,7 @@ class TestGoFZF < TestBase [true, false].each do |multi| tmux.send_keys "(echo ' 1st 2nd 3rd/'; echo ' first second third/') | - #{fzf multi && :multi, :x, :nth, 2, :with_nth, '2,-1,1'}", + #{fzf(multi && :multi, :x, :nth, 2, :with_nth, '2,-1,1')}", :Enter tmux.until { |lines| lines[-2].include?('2/2') } @@ -448,28 +445,28 @@ class TestGoFZF < TestBase def test_scroll [true, false].each do |rev| - tmux.send_keys "seq 1 100 | #{fzf rev && :reverse}", :Enter - tmux.until { |lines| lines.include? ' 100/100' } + tmux.send_keys "seq 1 100 | #{fzf(rev && :reverse)}", :Enter + tmux.until { |lines| lines.include?(' 100/100') } tmux.send_keys(*Array.new(110) { rev ? :Down : :Up }) - tmux.until { |lines| lines.include? '> 100' } + tmux.until { |lines| lines.include?('> 100') } tmux.send_keys :Enter assert_equal '100', readonce.chomp end end def test_select_1 - tmux.send_keys "seq 1 100 | #{fzf :with_nth, '..,..', :print_query, :q, 5555, :'1'}", :Enter + tmux.send_keys "seq 1 100 | #{fzf(:with_nth, '..,..', :print_query, :q, 5555, :'1')}", :Enter assert_equal %w[5555 55], readonce.split($INPUT_RECORD_SEPARATOR) end def test_exit_0 - tmux.send_keys "seq 1 100 | #{fzf :with_nth, '..,..', :print_query, :q, 555_555, :'0'}", :Enter - assert_equal ['555555'], readonce.split($INPUT_RECORD_SEPARATOR) + tmux.send_keys "seq 1 100 | #{fzf(:with_nth, '..,..', :print_query, :q, 555_555, :'0')}", :Enter + assert_equal %w[555555], readonce.split($INPUT_RECORD_SEPARATOR) end def test_select_1_exit_0_fail [:'0', :'1', %i[1 0]].each do |opt| - tmux.send_keys "seq 1 100 | #{fzf :print_query, :multi, :q, 5, *opt}", :Enter + tmux.send_keys "seq 1 100 | #{fzf(:print_query, :multi, :q, 5, *opt)}", :Enter tmux.until { |lines| lines.last =~ /^> 5/ } tmux.send_keys :BTab, :BTab, :BTab tmux.until { |lines| lines[-2].include?('(3)') } @@ -479,14 +476,14 @@ class TestGoFZF < TestBase end def test_query_unicode - tmux.paste "(echo abc; echo $'\\352\\260\\200\\353\\202\\230\\353\\213\\244') | #{fzf :query, "$'\\352\\260\\200\\353\\213\\244'"}" - tmux.until { |lines| lines[-2].include? '1/2' } + tmux.paste "(echo abc; echo $'\\352\\260\\200\\353\\202\\230\\353\\213\\244') | #{fzf(:query, "$'\\352\\260\\200\\353\\213\\244'")}" + tmux.until { |lines| lines[-2].include?('1/2') } tmux.send_keys :Enter - assert_equal ['가나다'], readonce.split($INPUT_RECORD_SEPARATOR) + assert_equal %w[가나다], readonce.split($INPUT_RECORD_SEPARATOR) end def test_sync - tmux.send_keys "seq 1 100 | #{fzf! :multi} | awk '{print $1 $1}' | #{fzf :sync}", :Enter + tmux.send_keys "seq 1 100 | #{fzf!(:multi)} | awk '{print $1 $1}' | #{fzf(:sync)}", :Enter tmux.until { |lines| lines[-1] == '>' } tmux.send_keys 9 tmux.until { |lines| lines[-2] == ' 19/100' } @@ -495,12 +492,12 @@ class TestGoFZF < TestBase tmux.send_keys :Enter tmux.until { |lines| lines[-1] == '>' } tmux.send_keys 'C-K', :Enter - assert_equal ['9090'], readonce.split($INPUT_RECORD_SEPARATOR) + assert_equal %w[9090], readonce.split($INPUT_RECORD_SEPARATOR) end def test_tac - tmux.send_keys "seq 1 1000 | #{fzf :tac, :multi}", :Enter - tmux.until { |lines| lines[-2].include? '1000/1000' } + tmux.send_keys "seq 1 1000 | #{fzf(:tac, :multi)}", :Enter + tmux.until { |lines| lines[-2].include?('1000/1000') } tmux.send_keys :BTab, :BTab, :BTab tmux.until { |lines| lines[-2].include?('(3)') } tmux.send_keys :Enter @@ -508,10 +505,10 @@ class TestGoFZF < TestBase end def test_tac_sort - tmux.send_keys "seq 1 1000 | #{fzf :tac, :multi}", :Enter - tmux.until { |lines| lines[-2].include? '1000/1000' } + tmux.send_keys "seq 1 1000 | #{fzf(:tac, :multi)}", :Enter + tmux.until { |lines| lines[-2].include?('1000/1000') } tmux.send_keys '99' - tmux.until { |lines| lines[-2].include? '28/1000' } + tmux.until { |lines| lines[-2].include?('28/1000') } tmux.send_keys :BTab, :BTab, :BTab tmux.until { |lines| lines[-2].include?('(3)') } tmux.send_keys :Enter @@ -519,10 +516,10 @@ class TestGoFZF < TestBase end def test_tac_nosort - tmux.send_keys "seq 1 1000 | #{fzf :tac, :no_sort, :multi}", :Enter - tmux.until { |lines| lines[-2].include? '1000/1000' } + tmux.send_keys "seq 1 1000 | #{fzf(:tac, :no_sort, :multi)}", :Enter + tmux.until { |lines| lines[-2].include?('1000/1000') } tmux.send_keys '00' - tmux.until { |lines| lines[-2].include? '10/1000' } + tmux.until { |lines| lines[-2].include?('10/1000') } tmux.send_keys :BTab, :BTab, :BTab tmux.until { |lines| lines[-2].include?('(3)') } tmux.send_keys :Enter @@ -531,69 +528,69 @@ class TestGoFZF < TestBase def test_expect test = lambda do |key, feed, expected = key| - tmux.send_keys "seq 1 100 | #{fzf :expect, key}; sync", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 1 100 | #{fzf(:expect, key)}; sync", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys '55' - tmux.until { |lines| lines[-2].include? '1/100' } + tmux.until { |lines| lines[-2].include?('1/100') } tmux.send_keys(*feed) tmux.prepare assert_equal [expected, '55'], readonce.split($INPUT_RECORD_SEPARATOR) end - test.call 'ctrl-t', 'C-T' - test.call 'ctrl-t', 'Enter', '' - test.call 'alt-c', %i[Escape c] - test.call 'f1', 'f1' - test.call 'f2', 'f2' - test.call 'f3', 'f3' - test.call 'f2,f4', 'f2', 'f2' - test.call 'f2,f4', 'f4', 'f4' - test.call 'alt-/', %i[Escape /] + test.call('ctrl-t', 'C-T') + test.call('ctrl-t', 'Enter', '') + test.call('alt-c', %i[Escape c]) + test.call('f1', 'f1') + test.call('f2', 'f2') + test.call('f3', 'f3') + test.call('f2,f4', 'f2', 'f2') + test.call('f2,f4', 'f4', 'f4') + test.call('alt-/', %i[Escape /]) %w[f5 f6 f7 f8 f9 f10].each do |key| - test.call 'f5,f6,f7,f8,f9,f10', key, key + test.call('f5,f6,f7,f8,f9,f10', key, key) end - test.call '@', '@' + test.call('@', '@') end def test_expect_print_query - tmux.send_keys "seq 1 100 | #{fzf '--expect=alt-z', :print_query}", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 1 100 | #{fzf('--expect=alt-z', :print_query)}", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys '55' - tmux.until { |lines| lines[-2].include? '1/100' } + tmux.until { |lines| lines[-2].include?('1/100') } tmux.send_keys :Escape, :z - assert_equal ['55', 'alt-z', '55'], readonce.split($INPUT_RECORD_SEPARATOR) + assert_equal %w[55 alt-z 55], readonce.split($INPUT_RECORD_SEPARATOR) end def test_expect_printable_character_print_query - tmux.send_keys "seq 1 100 | #{fzf '--expect=z --print-query'}", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 1 100 | #{fzf('--expect=z --print-query')}", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys '55' - tmux.until { |lines| lines[-2].include? '1/100' } + tmux.until { |lines| lines[-2].include?('1/100') } tmux.send_keys 'z' assert_equal %w[55 z 55], readonce.split($INPUT_RECORD_SEPARATOR) end def test_expect_print_query_select_1 - tmux.send_keys "seq 1 100 | #{fzf '-q55 -1 --expect=alt-z --print-query'}", :Enter + tmux.send_keys "seq 1 100 | #{fzf('-q55 -1 --expect=alt-z --print-query')}", :Enter assert_equal ['55', '', '55'], readonce.split($INPUT_RECORD_SEPARATOR) end def test_toggle_sort ['--toggle-sort=ctrl-r', '--bind=ctrl-r:toggle-sort'].each do |opt| - tmux.send_keys "seq 1 111 | #{fzf "-m +s --tac #{opt} -q11"}", :Enter - tmux.until { |lines| lines[-3].include? '> 111' } + tmux.send_keys "seq 1 111 | #{fzf("-m +s --tac #{opt} -q11")}", :Enter + tmux.until { |lines| lines[-3].include?('> 111') } tmux.send_keys :Tab - tmux.until { |lines| lines[-2].include? '4/111 -S (1)' } + tmux.until { |lines| lines[-2].include?('4/111 -S (1)') } tmux.send_keys 'C-R' - tmux.until { |lines| lines[-3].include? '> 11' } + tmux.until { |lines| lines[-3].include?('> 11') } tmux.send_keys :Tab - tmux.until { |lines| lines[-2].include? '4/111 +S (2)' } + tmux.until { |lines| lines[-2].include?('4/111 +S (2)') } tmux.send_keys :Enter assert_equal %w[111 11], readonce.split($INPUT_RECORD_SEPARATOR) end end def test_unicode_case - writelines tempname, %w[строКА1 СТРОКА2 строка3 Строка4] + writelines(tempname, %w[строКА1 СТРОКА2 строка3 Строка4]) assert_equal %w[СТРОКА2 Строка4], `#{FZF} -fС < #{tempname}`.split($INPUT_RECORD_SEPARATOR) assert_equal %w[строКА1 СТРОКА2 строка3 Строка4], `#{FZF} -fс < #{tempname}`.split($INPUT_RECORD_SEPARATOR) end @@ -605,7 +602,7 @@ class TestGoFZF < TestBase ----foobar-- -------foobar- ] - writelines tempname, input + writelines(tempname, input) assert_equal input, `#{FZF} -ffoobar --tiebreak=index < #{tempname}`.split($INPUT_RECORD_SEPARATOR) @@ -638,14 +635,14 @@ class TestGoFZF < TestBase end def test_tiebreak_index_begin - writelines tempname, [ - 'xoxxxxxoxx', - 'xoxxxxxox', - 'xxoxxxoxx', - 'xxxoxoxxx', - 'xxxxoxox', - ' xxoxoxxx' - ] + writelines(tempname, [ + 'xoxxxxxoxx', + 'xoxxxxxox', + 'xxoxxxoxx', + 'xxxoxoxxx', + 'xxxxoxox', + ' xxoxoxxx' + ]) assert_equal [ 'xxxxoxox', @@ -687,10 +684,10 @@ class TestGoFZF < TestBase end def test_tiebreak_begin_algo_v2 - writelines tempname, [ - 'baz foo bar', - 'foo bar baz' - ] + writelines(tempname, [ + 'baz foo bar', + 'foo bar baz' + ]) assert_equal [ 'foo bar baz', 'baz foo bar' @@ -698,14 +695,14 @@ class TestGoFZF < TestBase end def test_tiebreak_end - writelines tempname, [ - 'xoxxxxxxxx', - 'xxoxxxxxxx', - 'xxxoxxxxxx', - 'xxxxoxxxx', - 'xxxxxoxxx', - ' xxxxoxxx' - ] + writelines(tempname, [ + 'xoxxxxxxxx', + 'xxoxxxxxxx', + 'xxxoxxxxxx', + 'xxxxoxxxx', + 'xxxxxoxxx', + ' xxxxoxxx' + ]) assert_equal [ ' xxxxoxxx', @@ -742,7 +739,7 @@ class TestGoFZF < TestBase 12345:he 1234567:h ] - writelines tempname, input + writelines(tempname, input) output = %w[ 1:hell @@ -757,12 +754,12 @@ class TestGoFZF < TestBase end def test_invalid_cache - tmux.send_keys "(echo d; echo D; echo x) | #{fzf '-q d'}", :Enter - tmux.until { |lines| lines[-2].include? '2/3' } + tmux.send_keys "(echo d; echo D; echo x) | #{fzf('-q d')}", :Enter + tmux.until { |lines| lines[-2].include?('2/3') } tmux.send_keys :BSpace - tmux.until { |lines| lines[-2].include? '3/3' } + tmux.until { |lines| lines[-2].include?('3/3') } tmux.send_keys :D - tmux.until { |lines| lines[-2].include? '1/3' } + tmux.until { |lines| lines[-2].include?('1/3') } tmux.send_keys :Enter end @@ -803,26 +800,26 @@ class TestGoFZF < TestBase end def test_bind - tmux.send_keys "seq 1 1000 | #{fzf '-m --bind=ctrl-j:accept,u:up,T:toggle-up,t:toggle'}", :Enter - tmux.until { |lines| lines[-2].end_with? '/1000' } + tmux.send_keys "seq 1 1000 | #{fzf('-m --bind=ctrl-j:accept,u:up,T:toggle-up,t:toggle')}", :Enter + tmux.until { |lines| lines[-2].end_with?('/1000') } tmux.send_keys 'uuu', 'TTT', 'tt', 'uu', 'ttt', 'C-j' assert_equal %w[4 5 6 9], readonce.split($INPUT_RECORD_SEPARATOR) end def test_bind_print_query - tmux.send_keys "seq 1 1000 | #{fzf '-m --bind=ctrl-j:print-query'}", :Enter - tmux.until { |lines| lines[-2].end_with? '/1000' } + tmux.send_keys "seq 1 1000 | #{fzf('-m --bind=ctrl-j:print-query')}", :Enter + tmux.until { |lines| lines[-2].end_with?('/1000') } tmux.send_keys 'print-my-query', 'C-j' assert_equal %w[print-my-query], readonce.split($INPUT_RECORD_SEPARATOR) end def test_bind_replace_query - tmux.send_keys "seq 1 1000 | #{fzf '--print-query --bind=ctrl-j:replace-query'}", :Enter + tmux.send_keys "seq 1 1000 | #{fzf('--print-query --bind=ctrl-j:replace-query')}", :Enter tmux.send_keys '1' - tmux.until { |lines| lines[-2].end_with? '272/1000' } + tmux.until { |lines| lines[-2].end_with?('272/1000') } tmux.send_keys 'C-k', 'C-j' - tmux.until { |lines| lines[-2].end_with? '29/1000' } - tmux.until { |lines| lines[-1].end_with? '> 10' } + tmux.until { |lines| lines[-2].end_with?('29/1000') } + tmux.until { |lines| lines[-1].end_with?('> 10') } end def test_long_line @@ -836,37 +833,36 @@ class TestGoFZF < TestBase def test_read0 lines = `find .`.split($INPUT_RECORD_SEPARATOR) assert_equal lines.last, `find . | #{FZF} -e -f "^#{lines.last}$"`.chomp - assert_equal( + assert_equal \ lines.last, `find . -print0 | #{FZF} --read0 -e -f "^#{lines.last}$"`.chomp - ) end def test_select_all_deselect_all_toggle_all - tmux.send_keys "seq 100 | #{fzf '--bind ctrl-a:select-all,ctrl-d:deselect-all,ctrl-t:toggle-all --multi'}", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 100 | #{fzf('--bind ctrl-a:select-all,ctrl-d:deselect-all,ctrl-t:toggle-all --multi')}", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys :BTab, :BTab, :BTab - tmux.until { |lines| lines[-2].include? '(3)' } + tmux.until { |lines| lines[-2].include?('(3)') } tmux.send_keys 'C-t' - tmux.until { |lines| lines[-2].include? '(97)' } + tmux.until { |lines| lines[-2].include?('(97)') } tmux.send_keys 'C-a' - tmux.until { |lines| lines[-2].include? '(100)' } + tmux.until { |lines| lines[-2].include?('(100)') } tmux.send_keys :Tab, :Tab - tmux.until { |lines| lines[-2].include? '(98)' } + tmux.until { |lines| lines[-2].include?('(98)') } tmux.send_keys '100' tmux.until { |lines| lines.match_count == 1 } tmux.send_keys 'C-d' - tmux.until { |lines| lines[-2].include? '(97)' } + tmux.until { |lines| lines[-2].include?('(97)') } tmux.send_keys 'C-u' tmux.until { |lines| lines.match_count == 100 } tmux.send_keys 'C-d' - tmux.until { |lines| !lines[-2].include? '(' } + tmux.until { |lines| !lines[-2].include?('(') } tmux.send_keys :BTab, :BTab - tmux.until { |lines| lines[-2].include? '(2)' } + tmux.until { |lines| lines[-2].include?('(2)') } tmux.send_keys 0 - tmux.until { |lines| lines[-2].include? '10/100' } + tmux.until { |lines| lines[-2].include?('10/100') } tmux.send_keys 'C-a' - tmux.until { |lines| lines[-2].include? '(12)' } + tmux.until { |lines| lines[-2].include?('(12)') } tmux.send_keys :Enter assert_equal %w[1 2 10 20 30 40 50 60 70 80 90 100], readonce.split($INPUT_RECORD_SEPARATOR) @@ -877,59 +873,59 @@ class TestGoFZF < TestBase # History with limited number of entries begin - File.unlink history_file - rescue + File.unlink(history_file) + rescue StandardError nil end opts = "--history=#{history_file} --history-size=4" input = %w[00 11 22 33 44].map { |e| e + $INPUT_RECORD_SEPARATOR } input.each do |keys| - tmux.send_keys "seq 100 | #{fzf opts}", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 100 | #{fzf(opts)}", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys keys - tmux.until { |lines| lines[-2].include? '1/100' } + tmux.until { |lines| lines[-2].include?('1/100') } tmux.send_keys :Enter readonce end assert_equal input[1..-1], File.readlines(history_file) # Update history entries (not changed on disk) - tmux.send_keys "seq 100 | #{fzf opts}", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 100 | #{fzf(opts)}", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys 'C-p' - tmux.until { |lines| lines[-1].end_with? '> 44' } + tmux.until { |lines| lines[-1].end_with?('> 44') } tmux.send_keys 'C-p' - tmux.until { |lines| lines[-1].end_with? '> 33' } + tmux.until { |lines| lines[-1].end_with?('> 33') } tmux.send_keys :BSpace - tmux.until { |lines| lines[-1].end_with? '> 3' } + tmux.until { |lines| lines[-1].end_with?('> 3') } tmux.send_keys 1 - tmux.until { |lines| lines[-1].end_with? '> 31' } + tmux.until { |lines| lines[-1].end_with?('> 31') } tmux.send_keys 'C-p' - tmux.until { |lines| lines[-1].end_with? '> 22' } + tmux.until { |lines| lines[-1].end_with?('> 22') } tmux.send_keys 'C-n' - tmux.until { |lines| lines[-1].end_with? '> 31' } + tmux.until { |lines| lines[-1].end_with?('> 31') } tmux.send_keys 0 - tmux.until { |lines| lines[-1].end_with? '> 310' } + tmux.until { |lines| lines[-1].end_with?('> 310') } tmux.send_keys :Enter readonce assert_equal %w[22 33 44 310].map { |e| e + $INPUT_RECORD_SEPARATOR }, File.readlines(history_file) # Respect --bind option - tmux.send_keys "seq 100 | #{fzf opts + ' --bind ctrl-p:next-history,ctrl-n:previous-history'}", :Enter - tmux.until { |lines| lines[-2].include? '100/100' } + tmux.send_keys "seq 100 | #{fzf(opts + ' --bind ctrl-p:next-history,ctrl-n:previous-history')}", :Enter + tmux.until { |lines| lines[-2].include?('100/100') } tmux.send_keys 'C-n', 'C-n', 'C-n', 'C-n', 'C-p' tmux.until { |lines| lines[-1].end_with?('33') } tmux.send_keys :Enter ensure - File.unlink history_file + File.unlink(history_file) end def test_execute output = '/tmp/fzf-test-execute' opts = %[--bind "alt-a:execute(echo /{}/ >> #{output}),alt-b:execute[echo /{}{}/ >> #{output}],C:execute:echo /{}{}{}/ >> #{output}"] - wait = ->(exp) { tmux.until { |lines| lines[-2].include? exp } } - writelines tempname, %w[foo'bar foo"bar foo$bar] - tmux.send_keys "cat #{tempname} | #{fzf opts}; sync", :Enter + wait = ->(exp) { tmux.until { |lines| lines[-2].include?(exp) } } + writelines(tempname, %w[foo'bar foo"bar foo$bar]) + tmux.send_keys "cat #{tempname} | #{fzf(opts)}; sync", :Enter wait['3/3'] tmux.send_keys :Escape, :a wait['/3'] @@ -957,8 +953,8 @@ class TestGoFZF < TestBase File.readlines(output).map(&:chomp) ensure begin - File.unlink output - rescue + File.unlink(output) + rescue StandardError nil end end @@ -966,17 +962,17 @@ class TestGoFZF < TestBase def test_execute_multi output = '/tmp/fzf-test-execute-multi' opts = %[--multi --bind "alt-a:execute-multi(echo {}/{+} >> #{output}; sync)"] - writelines tempname, %w[foo'bar foo"bar foo$bar foobar] - tmux.send_keys "cat #{tempname} | #{fzf opts}", :Enter - tmux.until { |lines| lines[-2].include? '4/4' } + writelines(tempname, %w[foo'bar foo"bar foo$bar foobar]) + tmux.send_keys "cat #{tempname} | #{fzf(opts)}", :Enter + tmux.until { |lines| lines[-2].include?('4/4') } tmux.send_keys :Escape, :a - tmux.until { |lines| lines[-2].include? '/4' } + tmux.until { |lines| lines[-2].include?('/4') } tmux.send_keys :BTab, :BTab, :BTab tmux.send_keys :Escape, :a - tmux.until { |lines| lines[-2].include? '/4' } + tmux.until { |lines| lines[-2].include?('/4') } tmux.send_keys :Tab, :Tab tmux.send_keys :Escape, :a - tmux.until { |lines| lines[-2].include? '/4' } + tmux.until { |lines| lines[-2].include?('/4') } tmux.send_keys :Enter tmux.prepare readonce @@ -986,8 +982,8 @@ class TestGoFZF < TestBase File.readlines(output).map(&:chomp) ensure begin - File.unlink output - rescue + File.unlink(output) + rescue StandardError nil end end @@ -995,22 +991,22 @@ class TestGoFZF < TestBase def test_execute_plus_flag output = tempname + '.tmp' begin - File.unlink output - rescue + File.unlink(output) + rescue StandardError nil end - writelines tempname, ['foo bar', '123 456'] + writelines(tempname, ['foo bar', '123 456']) tmux.send_keys "cat #{tempname} | #{FZF} --multi --bind 'x:execute-silent(echo {+}/{}/{+2}/{2} >> #{output})'", :Enter execute = lambda do tmux.send_keys 'x', 'y' - tmux.until { |lines| lines[-2].include? '0/2' } + tmux.until { |lines| lines[-2].include?('0/2') } tmux.send_keys :BSpace - tmux.until { |lines| lines[-2].include? '2/2' } + tmux.until { |lines| lines[-2].include?('2/2') } end - tmux.until { |lines| lines[-2].include? '2/2' } + tmux.until { |lines| lines[-2].include?('2/2') } execute.call tmux.send_keys :Up @@ -1029,10 +1025,10 @@ class TestGoFZF < TestBase %(123 456/foo bar/456/bar), %(123 456 foo bar/foo bar/456 bar/bar) ], File.readlines(output).map(&:chomp) - rescue + rescue StandardError begin - File.unlink output - rescue + File.unlink(output) + rescue StandardError nil end end @@ -1041,50 +1037,50 @@ class TestGoFZF < TestBase # Custom script to use as $SHELL output = tempname + '.out' begin - File.unlink output - rescue + File.unlink(output) + rescue StandardError nil end - writelines tempname, - ['#!/usr/bin/env bash', "echo $1 / $2 > #{output}", 'sync'] - system "chmod +x #{tempname}" + writelines(tempname, + ['#!/usr/bin/env bash', "echo $1 / $2 > #{output}", 'sync']) + system("chmod +x #{tempname}") tmux.send_keys "echo foo | SHELL=#{tempname} fzf --bind 'enter:execute:{}bar'", :Enter - tmux.until { |lines| lines[-2].include? '1/1' } + tmux.until { |lines| lines[-2].include?('1/1') } tmux.send_keys :Enter - tmux.until { |lines| lines[-2].include? '1/1' } + tmux.until { |lines| lines[-2].include?('1/1') } tmux.send_keys 'C-c' tmux.prepare assert_equal ["-c / 'foo'bar"], File.readlines(output).map(&:chomp) ensure begin - File.unlink output - rescue + File.unlink(output) + rescue StandardError nil end end def test_cycle - tmux.send_keys "seq 8 | #{fzf :cycle}", :Enter - tmux.until { |lines| lines[-2].include? '8/8' } + tmux.send_keys "seq 8 | #{fzf(:cycle)}", :Enter + tmux.until { |lines| lines[-2].include?('8/8') } tmux.send_keys :Down - tmux.until { |lines| lines[-10].start_with? '>' } + tmux.until { |lines| lines[-10].start_with?('>') } tmux.send_keys :Down - tmux.until { |lines| lines[-9].start_with? '>' } + tmux.until { |lines| lines[-9].start_with?('>') } tmux.send_keys :Up - tmux.until { |lines| lines[-10].start_with? '>' } + tmux.until { |lines| lines[-10].start_with?('>') } tmux.send_keys :PgUp - tmux.until { |lines| lines[-10].start_with? '>' } + tmux.until { |lines| lines[-10].start_with?('>') } tmux.send_keys :Up - tmux.until { |lines| lines[-3].start_with? '>' } + tmux.until { |lines| lines[-3].start_with?('>') } tmux.send_keys :PgDn - tmux.until { |lines| lines[-3].start_with? '>' } + tmux.until { |lines| lines[-3].start_with?('>') } tmux.send_keys :Down - tmux.until { |lines| lines[-10].start_with? '>' } + tmux.until { |lines| lines[-10].start_with?('>') } end def test_header_lines - tmux.send_keys "seq 100 | #{fzf '--header-lines=10 -q 5'}", :Enter + tmux.send_keys "seq 100 | #{fzf('--header-lines=10 -q 5')}", :Enter 2.times do tmux.until do |lines| lines[-2].include?('/90') && @@ -1099,7 +1095,7 @@ class TestGoFZF < TestBase end def test_header_lines_reverse - tmux.send_keys "seq 100 | #{fzf '--header-lines=10 -q 5 --reverse'}", :Enter + tmux.send_keys "seq 100 | #{fzf('--header-lines=10 -q 5 --reverse')}", :Enter 2.times do tmux.until do |lines| lines[1].include?('/90') && @@ -1114,7 +1110,7 @@ class TestGoFZF < TestBase end def test_header_lines_reverse_list - tmux.send_keys "seq 100 | #{fzf '--header-lines=10 -q 5 --layout=reverse-list'}", :Enter + tmux.send_keys "seq 100 | #{fzf('--header-lines=10 -q 5 --layout=reverse-list')}", :Enter 2.times do tmux.until do |lines| lines[0] == '> 50' && @@ -1129,7 +1125,7 @@ class TestGoFZF < TestBase end def test_header_lines_overflow - tmux.send_keys "seq 100 | #{fzf '--header-lines=200'}", :Enter + tmux.send_keys "seq 100 | #{fzf('--header-lines=200')}", :Enter tmux.until do |lines| lines[-2].include?('0/0') && lines[-3].include?(' 1') @@ -1139,7 +1135,7 @@ class TestGoFZF < TestBase end def test_header_lines_with_nth - tmux.send_keys "seq 100 | #{fzf '--header-lines 5 --with-nth 1,1,1,1,1'}", :Enter + tmux.send_keys "seq 100 | #{fzf('--header-lines 5 --with-nth 1,1,1,1,1')}", :Enter tmux.until do |lines| lines[-2].include?('95/95') && lines[-3] == ' 11111' && @@ -1151,7 +1147,7 @@ class TestGoFZF < TestBase end def test_header - tmux.send_keys "seq 100 | #{fzf "--header \"$(head -5 #{FILE})\""}", :Enter + tmux.send_keys "seq 100 | #{fzf("--header \"$(head -5 #{FILE})\"")}", :Enter header = File.readlines(FILE).take(5).map(&:strip) tmux.until do |lines| lines[-2].include?('100/100') && @@ -1161,7 +1157,7 @@ class TestGoFZF < TestBase end def test_header_reverse - tmux.send_keys "seq 100 | #{fzf "--header \"$(head -5 #{FILE})\" --reverse"}", :Enter + tmux.send_keys "seq 100 | #{fzf("--header \"$(head -5 #{FILE})\" --reverse")}", :Enter header = File.readlines(FILE).take(5).map(&:strip) tmux.until do |lines| lines[1].include?('100/100') && @@ -1171,7 +1167,7 @@ class TestGoFZF < TestBase end def test_header_reverse_list - tmux.send_keys "seq 100 | #{fzf "--header \"$(head -5 #{FILE})\" --layout=reverse-list"}", :Enter + tmux.send_keys "seq 100 | #{fzf("--header \"$(head -5 #{FILE})\" --layout=reverse-list")}", :Enter header = File.readlines(FILE).take(5).map(&:strip) tmux.until do |lines| lines[-2].include?('100/100') && @@ -1181,7 +1177,7 @@ class TestGoFZF < TestBase end def test_header_and_header_lines - tmux.send_keys "seq 100 | #{fzf "--header-lines 10 --header \"$(head -5 #{FILE})\""}", :Enter + tmux.send_keys "seq 100 | #{fzf("--header-lines 10 --header \"$(head -5 #{FILE})\"")}", :Enter header = File.readlines(FILE).take(5).map(&:strip) tmux.until do |lines| lines[-2].include?('90/90') && @@ -1191,7 +1187,7 @@ class TestGoFZF < TestBase end def test_header_and_header_lines_reverse - tmux.send_keys "seq 100 | #{fzf "--reverse --header-lines 10 --header \"$(head -5 #{FILE})\""}", :Enter + tmux.send_keys "seq 100 | #{fzf("--reverse --header-lines 10 --header \"$(head -5 #{FILE})\"")}", :Enter header = File.readlines(FILE).take(5).map(&:strip) tmux.until do |lines| lines[1].include?('90/90') && @@ -1201,7 +1197,7 @@ class TestGoFZF < TestBase end def test_header_and_header_lines_reverse_list - tmux.send_keys "seq 100 | #{fzf "--layout=reverse-list --header-lines 10 --header \"$(head -5 #{FILE})\""}", :Enter + tmux.send_keys "seq 100 | #{fzf("--layout=reverse-list --header-lines 10 --header \"$(head -5 #{FILE})\"")}", :Enter header = File.readlines(FILE).take(5).map(&:strip) tmux.until do |lines| line |