cexpect 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ccc43da0141c2168408e3a9f931f58e37c0a71edab8b2bcd40066396ed8dfbe
-  data.tar.gz: 543082cfb8016426f41aad970df5cadee7109f285aec39b8062cb097662ab49a
+  metadata.gz: 8c069d8b016446307cbdc97331682a9aac2a0bdb0a70fefb40c5fdd946f71cef
+  data.tar.gz: 8f0b4dabc8fd1d75b223c3a55822ab26400b3e6fa96ef0178f4b18f4e6fc1e7a
 SHA512:
-  metadata.gz: faf35c685141bbe365000a483594932614fc9b5873b152a5fff7fdbdc961b769f62c91f69b8553f6265b31cf4d9beaf5913f0736e5d21c4f9e644d48b27c98cb
-  data.tar.gz: '0383c442a569ac777e57f9bd38b2c63025ad2d1bfb2748ddc6adf0859f32cdc85c755a66d9dfc6c15a336ed5258bafd2566652299954d56d357d3fcb6d9f025f'
+  metadata.gz: 6a73e2690ed2625d7be89045cf65ed3601301fb3f8afd581464231daba94cfed43cc51d2185159ea5d2df64585db9f24580e307f7af8b721be7aa42c2dfb1e3c
+  data.tar.gz: e2713329c4fb95c1e78d4107dfebd052548b7db5a6211fbdabdb38b51ef374afa934479a742d903ce886575d86e7ce710ce48006b9b71fd18358c60e11b37f88

data/.rspec

@@ -1,3 +1,4 @@
 --format documentation
 --color
 --require spec_helper
+-Ilib

data/.rubocop.yml

@@ -0,0 +1,43 @@
+require: rubocop-rspec
+
+AllCops:
+  Exclude:
+    - 'bin/{console,setup}' # not mine
+    - 'Gemfile'             # maintained through "bundle" command
+
+# TODO: Clean up tests
+RSpec/ExpectInHook:
+  Enabled: false
+RSpec/FilePath:
+  Exclude:
+    - spec/cexpect_spec.rb
+RSpec/MessageSpies:
+  Enabled: false
+RSpec/NestedGroups:
+  Enabled: false
+RSpec/VerifiedDoubles:
+  Enabled: false
+
+# Personal preferences
+Layout/DotPosition:
+  EnforcedStyle: trailing
+Style/PercentLiteralDelimiters:
+  PreferredDelimiters:
+    '%i': '()'
+
+# New stuff in rubocop, not yet enabled/disabled by default
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+Lint/RaiseException:
+  Enabled: true
+Lint/StructNewOverride:
+  Enabled: true
+Style/HashEachMethods:
+  Enabled: true
+Style/HashTransformKeys:
+  Enabled: true
+Style/HashTransformValues:
+  Enabled: true
+
+Style/ExponentialNotation:
+  Enabled: false

data/Gemfile

@@ -5,3 +5,6 @@ gemspec
 
 gem "rake", "~> 12.0"
 gem "rspec", "~> 3.0"
+
+gem "rubocop", "~> 0.82.0"
+gem "rubocop-rspec", "~> 1.38"

data/Gemfile.lock

@@ -1,13 +1,20 @@
 PATH
   remote: .
   specs:
-    cexpect (0.1.0)
+    cexpect (0.1.1)
 
 GEM
   remote: https://rubygems.org/
   specs:
+    ast (2.4.0)
     diff-lcs (1.3)
+    jaro_winkler (1.5.4)
+    parallel (1.19.1)
+    parser (2.7.1.1)
+      ast (~> 2.4.0)
+    rainbow (3.0.0)
     rake (12.3.3)
+    rexml (3.2.4)
     rspec (3.9.0)
       rspec-core (~> 3.9.0)
       rspec-expectations (~> 3.9.0)
@@ -21,6 +28,18 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.9.0)
     rspec-support (3.9.2)
+    rubocop (0.82.0)
+      jaro_winkler (~> 1.5.1)
+      parallel (~> 1.10)
+      parser (>= 2.7.0.1)
+      rainbow (>= 2.2.2, < 4.0)
+      rexml
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 1.4.0, < 2.0)
+    rubocop-rspec (1.38.1)
+      rubocop (>= 0.68.1)
+    ruby-progressbar (1.10.1)
+    unicode-display_width (1.7.0)
 
 PLATFORMS
   ruby
@@ -29,6 +48,8 @@ DEPENDENCIES
   cexpect!
   rake (~> 12.0)
   rspec (~> 3.0)
+  rubocop (~> 0.82.0)
+  rubocop-rspec (~> 1.38)
 
 BUNDLED WITH
    2.1.4

data/README.md

@@ -62,6 +62,13 @@ end
 
 wr.puts('cat very-large-file')
 contents = rd.fexpect('$ ')
+
+# If CExpect.spawn doesn't suit you
+orig_rd, wr = IO.pipe
+rd = CExpect::Reader.new(orig_rd)
+
+wr.puts "Hello, World!\n"
+greeting = rd.fexpect("\n")
 ```
 
 ## Development

data/Rakefile

@@ -1,6 +1,10 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
+RuboCop::RakeTask.new
 
-task :default => :spec
+task default: %i(spec rubocop)

data/cexpect.gemspec

@@ -1,27 +1,32 @@
+# frozen_string_literal: true
+
 require_relative 'lib/cexpect/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "cexpect"
+  spec.name          = 'cexpect'
   spec.version       = Cexpect::VERSION
-  spec.authors       = ["Christer Jansson"]
-  spec.email         = ["christer@janssons.org"]
+  spec.authors       = ['Christer Jansson']
+  spec.email         = ['christer@janssons.org']
 
-  spec.summary       = %q{An improved expect method}
-  spec.description   = %q{An expect method with more reasonable return values and logging functionality}
-  spec.homepage      = "https://github.com/kondensatorn/cexpect"
-  spec.license       = "BSD 2-clause"
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.3.0")
+  spec.summary       = 'An improved expect method'
+  spec.description   = 'An expect method with more reasonable return' \
+                       ' values and logging functionality'
+  spec.homepage      = 'https://github.com/kondensatorn/cexpect'
+  spec.license       = 'BSD 2-clause'
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.3.0')
 
-  spec.metadata["homepage_uri"] = spec.homepage
-  spec.metadata["source_code_uri"] = spec.homepage
-  spec.metadata["changelog_uri"] = "#{spec.homepage}/CHANGELOG.md"
+  spec.metadata['homepage_uri'] = spec.homepage
+  spec.metadata['source_code_uri'] = spec.homepage
+  spec.metadata['changelog_uri'] = "#{spec.homepage}/CHANGELOG.md"
 
   # Specify which files should be added to the gem when it is released.
-  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files         = Dir.chdir(File.expand_path('..', __FILE__)) do
-    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  # The `git ls-files -z` loads the files in the RubyGem that have
+  # been added into git.
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
+    `git ls-files -z`.
+      split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 end

data/lib/cexpect.rb

@@ -1,7 +1,12 @@
+# frozen_string_literal: true
+
 require 'delegate'
 require 'observer'
 require 'pty'
 
+#
+# A module for supplying a different expect method
+#
 module CExpect
   def self.spawn(*args)
     original_rd, wr, pid = PTY.spawn(*args)
@@ -13,6 +18,10 @@ module CExpect
     end
   end
 
+  #
+  # A class delegating normal operations to a wrapped IO, adding an
+  # expect method
+  #
   class Reader < SimpleDelegator
     include Observable
 
@@ -21,11 +30,11 @@ module CExpect
       super
     end
 
-    def expect(pat, timeout = nil, match_method: :_expect_re_match)
+    def expect(pat, timeout = nil, match_method: :re_match)
       buf = +''
 
       result = catch(:result) do
-        loop { _expect_try(pat, buf, timeout, match_method) }
+        loop { expect_try(pat, buf, timeout, match_method) }
       end
 
       if block_given?
@@ -36,13 +45,13 @@ module CExpect
     end
 
     def fexpect(pat, timeout = nil)
-      expect(pat, timeout, match_method: :_expect_string_match)
+      expect(pat, timeout, match_method: :string_match)
     end
 
     private
 
-    def _expect_try(pat, buf, timeout, match_method)
-      c = _expect_getc(timeout)
+    def expect_try(pat, buf, timeout, match_method)
+      c = getc(timeout)
 
       if c.nil?
         @leftovers = buf
@@ -51,32 +60,34 @@ module CExpect
 
       buf << c
 
-      _expect_log(pat, buf)
+      log(pat, buf)
 
       result = send(match_method, buf, pat)
       throw(:result, result) if result
     end
 
-    def _expect_getc(timeout)
+    def getc(timeout)
       return @leftovers.slice!(0).chr unless @leftovers.empty?
 
-      return nil if !IO.select([self], nil, nil, timeout) || eof?
+      rd = __getobj__
+
+      return nil if !IO.select([rd], nil, nil, timeout) || eof?
 
-      getc.chr
+      rd.getc.chr
     end
 
-    def _expect_log(pat, buf)
+    def log(pat, buf)
       return if count_observers.zero?
 
       changed
-      notify_observers("pat: #{pat.inspect}\nbuf: #{buf.inspect}\n")
+      notify_observers(pat, buf)
     end
 
-    def _expect_re_match(buf, pat)
+    def re_match(buf, pat)
       buf.match(pat)
     end
 
-    def _expect_string_match(buf, pat)
+    def string_match(buf, pat)
       buf[0, buf.size - pat.size] if buf.end_with?(pat)
     end
   end

data/lib/cexpect/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Cexpect
-  VERSION = "0.1.0"
+  VERSION = '0.1.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cexpect
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Christer Jansson
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-04-28 00:00:00.000000000 Z
+date: 2020-04-29 00:00:00.000000000 Z
 dependencies: []
 description: An expect method with more reasonable return values and logging functionality
 email:
@@ -19,6 +19,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile