seeing_is_believing 3.4.0 → 3.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: '05637967d8e40131f85a42ba2246cca7e28c9c53fe02a1ab802dab93cd368a73'
-  data.tar.gz: cb89541eec8388b26643ee4c4adcdbadbe60673ad309fb1df2eef1d8fb5e7c5c
+SHA1:
+  metadata.gz: 2191c3427344ea922d31c319fe5442b7cc548d35
+  data.tar.gz: c9d3e073bbeb2d093a69ffbea07f287750b42f73
 SHA512:
-  metadata.gz: c3e1fe60f931cfbee373c8bf0cc96a1442e92db587866434ecc35d3e5b638282553265d7576a572206aad91f2b8db73c5c0c01be4b956255e98823f743bc4ee5
-  data.tar.gz: b7304a173678eeb157bc6ce35c4fe16777f9c511eae18240862668b15facd4fa7e4153dc13de0c0630e73fe4da257c8dd5a5a1fc1826bdecec6b317014f241fe
+  metadata.gz: bbfb8141efc9ca5a3ec5eaaae28588439b593911eb411305843d39d5b42ab0d670386c0934e95d2b709f90e2d5a80133637b7b6e2181ef37387530387b8caa31
+  data.tar.gz: 37148dff2dd6e2231a1a8be45437f41160e747eef14e52f5df2bb68258ec630ffe2345ea310b47b1b432507cf9253c9c2b7dca26d59d98f0c786df4a55087ad2

data/.travis.yml

@@ -8,3 +8,4 @@ rvm:
   - 2.2.0
   - 2.3.1
   - 2.4.1
+  - 2.5.0

data/appveyor.yml

@@ -29,3 +29,5 @@ environment:
     - ruby_version: '21'
     - ruby_version: '22'
     - ruby_version: '23'
+    # - ruby_version: '24'
+    # - ruby_version: '25'

data/features/flags.feature

@@ -268,7 +268,7 @@ Feature: Using flags
 
 
   Scenario: --alignment-strategy file
-    Given the file "file_alignments.rb":
+     Given the file "file_alignments.rb":
     """
     # comment
     1
@@ -614,3 +614,239 @@ Feature: Using flags
     And stdout is '__FILE__  # => "cwd_of_file_test.rb"'
 
 
+  Scenario: --toggle-mark adds a mark to the line if it is unmarked and exists
+    Given the file "unmarked.rb":
+    """
+    1
+    2
+    """
+    When I run "seeing_is_believing unmarked.rb --toggle-mark 1"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is '{{"1  # => \n2"}}'
+    When I run "seeing_is_believing unmarked.rb --toggle-mark 2"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is '{{"1\n2  # => "}}'
+    When I run "seeing_is_believing unmarked.rb --toggle-mark 3"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is '{{"1\n2"}}'
+
+
+  Scenario: --toggle-mark doesn't add a mark to the line if it has a comment or can't be commented
+    Given the file "no_toggle_line_here.rb":
+    """
+    1 # comment
+    "2
+    "
+    """
+    When I run "seeing_is_believing no_toggle_line_here.rb --toggle-mark 1"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1 # comment
+    "2
+    "
+    """
+    When I run "seeing_is_believing no_toggle_line_here.rb --toggle-mark 2"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1 # comment
+    "2
+    "
+    """
+    When I run "seeing_is_believing no_toggle_line_here.rb --toggle-mark 3"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1 # comment
+    "2
+    "  # =>{{" "}}
+    """
+
+
+  Scenario: --toggle-mark removes a mark from the line if it is marked and exists
+    Given the file "marked.rb":
+    """
+    1      # =>
+    2 + 2  # =>
+    """
+    When I run "seeing_is_believing marked.rb --toggle-mark 1"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is '{{"1\n2 + 2  # => "}}'
+    When I run "seeing_is_believing marked.rb --toggle-mark 2"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is '{{"1      # => \n2 + 2"}}'
+    When I run "seeing_is_believing marked.rb --toggle-mark 3"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is '{{"1      # => \n2 + 2  # => "}}'
+
+
+  Scenario: --toggle-mark removes error lines, and stdout/stderr output
+    Given the file "output_to_toggle_off.rb":
+    """
+    1 + "a" # ~> b
+    # >> c
+    # !> d
+    # ~> e
+    """
+    When I run "seeing_is_believing output_to_toggle_off.rb --toggle-mark 1"
+    Then stdout is:
+    """
+    1 + "a"
+    # >> c
+    # !> d
+    # ~> e
+    """
+    When I run "seeing_is_believing output_to_toggle_off.rb --toggle-mark 2"
+    Then stdout is:
+    """
+    1 + "a" # ~> b
+
+    # !> d
+    # ~> e
+    """
+    When I run "seeing_is_believing output_to_toggle_off.rb --toggle-mark 3"
+    Then stdout is:
+    """
+    1 + "a" # ~> b
+    # >> c
+
+    # ~> e
+    """
+    When I run "seeing_is_believing output_to_toggle_off.rb --toggle-mark 4"
+    Then stdout is:
+    """
+    1 + "a" # ~> b
+    # >> c
+    # !> d
+    """
+
+
+  Scenario: --toggle-mark respects the --alignment-strategy of file and updates any existing annotations to respect it
+    Given the file "toggle_mark_with_file_alignment.rb":
+    """
+    1
+
+    1 + 1 # => 2
+    1 + 1 + 1 # => 3
+    1 + 1 + 1 + 1
+    """
+    When I run "seeing_is_believing --toggle-mark 1 --alignment-strategy file toggle_mark_with_file_alignment.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1              # =>{{" "}}
+
+    1 + 1          # => 2
+    1 + 1 + 1      # => 3
+    1 + 1 + 1 + 1
+    """
+    When I run "seeing_is_believing --toggle-mark 3 --alignment-strategy file toggle_mark_with_file_alignment.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1
+
+    1 + 1
+    1 + 1 + 1      # => 3
+    1 + 1 + 1 + 1
+    """
+
+
+  Scenario: --toggle-mark respects the --alignment-strategy of chunk and updates any existing annotations to respect it
+    Given the file "toggle_mark_with_chunk_alignments.rb":
+    """
+    1+1# => 2
+    1+1+1
+    1+1+1+1
+
+    1 + 1
+    # comment in the middle!
+    1
+    """
+    When I run "seeing_is_believing --toggle-mark 2 --alignment-strategy chunk toggle_mark_with_chunk_alignments.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1+1      # => 2
+    1+1+1    # =>{{" "}}
+    1+1+1+1
+
+    1 + 1
+    # comment in the middle!
+    1
+    """
+    When I run "seeing_is_believing --toggle-mark 5 --alignment-strategy chunk toggle_mark_with_chunk_alignments.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1+1      # => 2
+    1+1+1
+    1+1+1+1
+
+    1 + 1  # =>{{" "}}
+    # comment in the middle!
+    1
+    """
+    When I run "seeing_is_believing --toggle-mark 1 --alignment-strategy chunk toggle_mark_with_chunk_alignments.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1+1
+    1+1+1
+    1+1+1+1
+
+    1 + 1
+    # comment in the middle!
+    1
+    """
+
+
+  Scenario: --toggle-mark respects the --alignment-strategy of line and updates any existing annotations to respect it
+    Given the file "toggle_mark_with_line_alignments.rb":
+    """
+    1
+    1 + 1# => x
+    1 + 1 + 1
+    """
+    When I run "seeing_is_believing --toggle-mark 1 --alignment-strategy line toggle_mark_with_line_alignments.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1  # =>{{" "}}
+    1 + 1  # => x
+    1 + 1 + 1
+    """
+    When I run "seeing_is_believing --toggle-mark 2 --alignment-strategy line toggle_mark_with_line_alignments.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1
+    1 + 1
+    1 + 1 + 1
+    """
+    When I run "seeing_is_believing --toggle-mark 3 --alignment-strategy line toggle_mark_with_line_alignments.rb"
+    Then stderr is empty
+    And the exit status is 0
+    And stdout is:
+    """
+    1
+    1 + 1  # => x
+    1 + 1 + 1  # =>{{" "}}
+    """

data/features/regression.feature

@@ -471,6 +471,7 @@ Feature:
   # Not going to get too detailed on what it prints, b/c that message seems pretty fragile,
   # but just generally that it doesn't fkn blow up
   @not-2.4
+  @not-2.5
   Scenario: Old JSON bug
     Given the file "json_and_encodings.rb":
     """
@@ -826,3 +827,10 @@ Feature:
     end
     str  # => a string
     """
+
+
+  Scenario: The last line can end in a semicolon
+    When I run "seeing_is_believing -e '1'"
+    Then stdout is "1  # => 1"
+    When I run "seeing_is_believing -e '1;'"
+    Then stdout is "1;  # => 1"

data/lib/seeing_is_believing/binary.rb

@@ -32,6 +32,11 @@ class SeeingIsBelieving
         return SUCCESS_STATUS
       end
 
+      if config.toggle_mark?
+        stdout.print engine.toggled_mark
+        return SUCCESS_STATUS
+      end
+
       if engine.syntax_error?
         stderr.puts engine.syntax_error
         return NONDISPLAYABLE_ERROR_STATUS

data/lib/seeing_is_believing/binary/annotate_end_of_file.rb

@@ -12,7 +12,7 @@ class SeeingIsBelieving
                  exception_output_for(results, options)
 
         code = Code.new(new_body)
-        code.rewriter.insert_after_multi code.body_range, output
+        code.rewriter.insert_after code.body_range, output
         new_body.replace code.rewriter.process
       end
 

data/lib/seeing_is_believing/binary/annotate_marked_lines.rb

@@ -47,8 +47,6 @@ class SeeingIsBelieving
           WrapExpressions.call \
             program,
             before_each: -> line_number {
-              # 74 b/c pretty print_defaults to 79 (guessing 80 chars with 1 reserved for newline), and
-              # 79 - "# => ".length # => 4
               inspect        = "$SiB.record_result(:inspect, #{line_number}, ("
               pp             = "$SiB.record_result(:pp, #{line_number}, ("
 
@@ -62,6 +60,10 @@ class SeeingIsBelieving
               end
             },
             after_each:  -> line_number {
+              # 74 b/c pretty print_defaults to 79 (guessing 80 chars with 1 reserved for newline), and
+              # 79 - "# => ".length # => 4
+              # ALSO: This should be configurable, b/c otherwise you have to go into the guts of `pp`
+              #       https://gist.github.com/JoshCheek/6472c8f334ae493f4ab1f7865e2470e5
               inspect = "))"
               pp      = ")) { |v| PP.pp v, '', 74 }"
 

data/lib/seeing_is_believing/binary/comment_lines.rb

@@ -22,7 +22,7 @@ class SeeingIsBelieving
             first_index -= 1 while first_index > 0 && raw_code[first_index-1] != "\n"
             comment_text = commenter.call raw_code[first_index...last_index], line_number
             range        = Parser::Source::Range.new(commentable_lines.buffer, first_index, last_index)
-            commentable_lines.rewriter.insert_after_multi range, comment_text
+            commentable_lines.rewriter.insert_after range, comment_text
           end
           commentable_lines.rewriter.process
         end

data/lib/seeing_is_believing/binary/config.rb

@@ -30,6 +30,7 @@ class SeeingIsBelieving
       predicate(:remove_value_prefixes) { true  }
       predicate(:debug)                 { false }
       predicate(:ignore_unknown_flags)  { false }
+      predicate(:toggle_mark)           { nil }
       attribute(:body)                  { nil }
       attribute(:filename)              { nil }
       attribute(:errors)                { [] }
@@ -109,6 +110,11 @@ class SeeingIsBelieving
             self.lib_options.require_files << 'pp'
             self.lib_options.require_files << 'seeing_is_believing/customize_pp'
 
+          when '--toggle-mark'
+            extract_positive_int_for.call arg do |n|
+              self.toggle_mark = n
+            end
+
           when '-i', '--inherit-exitstatus', '--inherit-exit-status'
             self.inherit_exitstatus = true
             arg.include?("exit-status") &&
@@ -252,6 +258,13 @@ class SeeingIsBelieving
         print_event_stream? && (result_as_json? || annotator == AnnotateMarkedLines) &&
           add_error("can only have one output format, --stream is not compatible with --json, -x, and --xmpfilter-style")
 
+        toggle_mark? && print_event_stream? &&
+          add_error("--toggle-mark and --stream are mutually exclusive")
+
+        toggle_mark? && result_as_json? &&
+          add_error("--toggle-mark and --json are mutually exclusive")
+
+
         self.filename                  = filenames.first
         self.lib_options.filename      = as || filename
         self.lib_options.debugger      = debugger
@@ -342,6 +355,7 @@ Options:
   -g,  --debug                   # print debugging information
        --debug-to FILE           # print debugging information to FILE
   -x,  --xmpfilter-style         # annotate marked lines instead of every line
+       --toggle-mark n           # add / remove annotation on line n
   -j,  --json                    # print results in json format (i.e. so another program can consume them)
   -i,  --inherit-exitstatus      # exit with the exit status of the program being evaluated
        --stream                  # a JSON stream of every event ias it is seen (such as recording a line)

data/lib/seeing_is_believing/binary/engine.rb

@@ -18,12 +18,69 @@ class SeeingIsBelieving
 
       def cleaned_body
         @cleaned_body ||= if missing_newline?
-                            normalized_cleaned_body.chomp!
+                            normalized_cleaned_body.chomp
                           else
                             normalized_cleaned_body
                           end
       end
 
+
+      require 'seeing_is_believing/binary/rewrite_comments'
+      require 'seeing_is_believing/binary/format_comment'
+      module ToggleMark
+        def self.call(options)
+          options = options.dup
+          body    = options.delete :body
+          line    = options.delete :line
+          markers = options.delete :markers
+          alignment_strategy = options.delete :alignment_strategy
+
+          marker_regexes = markers.values.map(&:regex)
+          RewriteComments.call body, include_lines: [line] do |comment|
+            if line == comment.line_number && marker_regexes.any? { |r| r =~ comment.text }
+              new_comment = ''
+            elsif line == comment.line_number && comment.text.empty?
+              new_comment = FormatComment.call(
+                comment.whitespace_col,
+                markers.value.prefix,
+                '',
+                options.merge(
+                  pad_to: alignment_strategy.line_length_for(comment.line_number)
+                ),
+              )
+            elsif line == comment.line_number
+              new_comment = comment.whitespace + comment.text
+            elsif match = markers.value.regex.match(comment.text)
+              new_comment = FormatComment.call(
+                comment.whitespace_col,
+                markers.value.prefix,
+                match.post_match,
+                options.merge(
+                  pad_to: alignment_strategy.line_length_for(comment.line_number)
+                )
+              )
+            else
+              new_comment = comment.whitespace + comment.text
+            end
+            [new_comment[/^\s*/], new_comment.lstrip]
+          end
+        end
+      end
+
+      def toggled_mark
+        body = config.body
+        body         += "\n" if missing_newline?
+        toggled = ToggleMark.call(
+          body:               body,
+          line:               config.toggle_mark,
+          markers:            config.markers,
+          alignment_strategy: config.annotator_options.alignment_strategy.new(normalized_cleaned_body),
+          options:            config.annotator_options.to_h,
+        )
+        toggled.chomp! if missing_newline?
+        toggled
+      end
+
       def syntax_error?
         code.syntax.invalid?
       end

data/lib/seeing_is_believing/code.rb

@@ -25,7 +25,7 @@ class SeeingIsBelieving
       @raw             = raw_code
       @buffer          = Parser::Source::Buffer.new(name||"SeeingIsBelieving")
       @buffer.source   = raw
-      @rewriter        = Parser::Source::Rewriter.new buffer
+      @rewriter        = Parser::Source::TreeRewriter.new buffer
       builder          = Parser::Builders::Default.new.tap { |b| b.emit_file_line_as_literals = false }
       @parser          = Parser::CurrentRuby.new builder
       @syntax          = Syntax.new
@@ -109,7 +109,7 @@ class SeeingIsBelieving
       (name, (_, range)) = tokens.max_by { |name, (data, range)| range.end_pos } ||
                             [nil, [nil, range_for(0, 1)]]
       end_pos = range.end_pos
-      end_pos += 1 if name == :tCOMMENT || name == :tSTRING_END #
+      end_pos += 1 if name == :tCOMMENT || name == :tSTRING_END || name == :tSEMI
       range_for 0, end_pos
     end
 

data/lib/seeing_is_believing/evaluate_by_moving_files.rb

@@ -102,8 +102,13 @@ class SeeingIsBelieving
 
       # send stdin (char at a time b/c input could come from a stream)
       Thread.new do
-        provided_input.each_char { |char| child.io.stdin.write char }
-        child.io.stdin.close
+        begin
+          provided_input.each_char { |char| child.io.stdin.write char }
+        rescue
+          # don't explode if child closes IO
+        ensure
+          child.io.stdin.close
+        end
       end
 
       # set up the event consumer

data/lib/seeing_is_believing/version.rb

@@ -1,3 +1,3 @@
 class SeeingIsBelieving
-  VERSION = '3.4.0'
+  VERSION = '3.6.0'
 end

data/lib/seeing_is_believing/wrap_expressions.rb

@@ -30,20 +30,20 @@ class SeeingIsBelieving
           when :total_fucking_failure
             rewriter.replace range,  '.....TOTAL FUCKING FAILURE!.....'
           when :match_current_line
-            rewriter.insert_before_multi range, '~' # Regexp#~
+            rewriter.insert_before range, '~' # Regexp#~
           end
         end
 
         wrappings.each do |line_num, (range, last_col, meta)|
-          rewriter.insert_before_multi range, before_each.call(line_num)
+          rewriter.insert_before range, before_each.call(line_num)
         end
 
         wrappings.each do |line_num, (range, last_col, meta)|
-          rewriter.insert_after_multi range, after_each.call(line_num)
+          rewriter.insert_after range, after_each.call(line_num)
         end
 
-        rewriter.insert_before_multi root_range, before_all.call
-        rewriter.insert_after_multi  root_range, after_all_text
+        rewriter.insert_before root_range, before_all.call
+        rewriter.insert_after  root_range, after_all_text
         rewriter.process
       end
     end

data/seeing_is_believing.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |s|
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_dependency             "parser",      "~> 2.4.0"
+  s.add_dependency             "parser",      "~> 2.5.0"
   s.add_dependency             "childprocess","~> 0.8.0"
 
   s.add_development_dependency "pry"

data/spec/binary/config_spec.rb

@@ -611,40 +611,58 @@ RSpec.describe SeeingIsBelieving::Binary::Config do
       it('defaults :stdout    to "# >> "') { expect(default_markers.stdout   .prefix).to eq "# >> " }
       it('defaults :stderr    to "# !> "') { expect(default_markers.stderr   .prefix).to eq "# !> " }
     end
-  end
 
-  describe 'print_event_stream?' do
-    it 'print_event_stream? is false by default' do
-      expect(parse([]).print_event_stream?).to eq false
-    end
-    it 'print_event_stream? can be turned on with --stream' do
-      expect(parse(['--stream']).print_event_stream?).to eq true
-    end
-    it 'adds an error if --stream is used with --json' do
-      expect(parse(['--stream'])).to_not have_error '--stream'
-      expect(parse(['--stream', '--json'])).to have_error '--stream'
-      expect(parse(['--json', '--stream'])).to have_error '--stream'
-    end
-    it 'adds an error if --stream is used with -x or --xmpfilter-style' do
-      expect(parse(['--stream'])).to_not have_error '--stream'
-      expect(parse(['--stream', '-x'])).to have_error '--stream'
-      expect(parse(['-x', '--stream'])).to have_error '--stream'
-      expect(parse(['--xmpfilter-style', '--stream'])).to have_error '--stream'
+    describe 'print_event_stream?' do
+      it 'print_event_stream? is false by default' do
+        expect(parse([]).print_event_stream?).to eq false
+      end
+      it 'print_event_stream? can be turned on with --stream' do
+        expect(parse(['--stream']).print_event_stream?).to eq true
+      end
+      it 'adds an error if --stream is used with --json' do
+        expect(parse(['--stream'])).to_not have_error '--stream'
+        expect(parse(['--stream', '--json'])).to have_error '--stream'
+        expect(parse(['--json', '--stream'])).to have_error '--stream'
+      end
+      it 'adds an error if --stream is used with -x or --xmpfilter-style' do
+        expect(parse(['--stream'])).to_not have_error '--stream'
+        expect(parse(['--stream', '-x'])).to have_error '--stream'
+        expect(parse(['-x', '--stream'])).to have_error '--stream'
+        expect(parse(['--xmpfilter-style', '--stream'])).to have_error '--stream'
+      end
     end
-  end
-
 
-  describe 'align_results?' do
-    it 'defaults to true' do
-      expect(parse([]).annotator_options.interline_align?).to eq true
-    end
-    it 'can be turned on with --interline-align' do
-      expect(parse(['--interline-align'])).to_not have_error '--interline-align'
-      expect(parse(['--interline-align']).annotator_options.interline_align?).to eq true
+    describe 'align_results?' do
+      it 'defaults to true' do
+        expect(parse([]).annotator_options.interline_align?).to eq true
+      end
+      it 'can be turned on with --interline-align' do
+        expect(parse(['--interline-align'])).to_not have_error '--interline-align'
+        expect(parse(['--interline-align']).annotator_options.interline_align?).to eq true
+      end
+      it 'can be turned off with --no-interline-align' do
+        expect(parse(['--no-interline-align'])).to_not have_error '--no-interline-align'
+        expect(parse(['--no-interline-align']).annotator_options.interline_align?).to eq false
+      end
     end
-    it 'can be turned off with --no-interline-align' do
-      expect(parse(['--no-interline-align'])).to_not have_error '--no-interline-align'
-      expect(parse(['--no-interline-align']).annotator_options.interline_align?).to eq false
+
+    describe 'toggle_mark/toggle_mark?' do
+      it 'defaults to nil/false' do
+        expect(parse([]).toggle_mark).to eq nil
+        expect(parse([]).toggle_mark?).to eq false
+      end
+      it 'can be set to true/linenum with --toggle-mark' do
+        expect(parse(['--toggle-mark', '123']).toggle_mark).to eq 123
+        expect(parse(['--toggle-mark', '123']).toggle_mark?).to eq true
+      end
+      it_behaves_like 'it requires a positive int argument', ['--toggle-mark']
+      it 'adds an error if used with --json or --stream' do
+        # expect(parse(['--toggle-mark', '123'])).to_not have_error /--toggle-mark/
+        expect(parse(['--stream', '--toggle-mark', '123'])).to have_error /--toggle-mark/
+        expect(parse(['--toggle-mark', '123', '--stream'])).to have_error /--toggle-mark/
+        expect(parse(['--json', '--toggle-mark', '123'])).to have_error /--toggle-mark/
+        expect(parse(['--toggle-mark', '123', '--json'])).to have_error /--toggle-mark/
+      end
     end
   end
 

data/spec/binary/engine_spec.rb

@@ -7,7 +7,8 @@ class SeeingIsBelieving
       def call(body, options={})
         timeout  = options.fetch :timeout, 0
         filename = options.fetch :filename, "program.rb"
-        config   = Config.new body: body, timeout_seconds: timeout
+        toggle   = options.fetch :toggled_mark, nil
+        config   = Config.new body: body, timeout_seconds: timeout, toggle_mark: toggle
         config.lib_options.timeout_seconds = timeout
         config.lib_options.filename        = filename
         Engine.new config
@@ -50,6 +51,15 @@ class SeeingIsBelieving
         end
       end
 
+      context 'toggled_mark' do
+        it 'has the mark toggled and doesn\'t change the newline' do
+          expect(call("1",        toggled_mark: 1).toggled_mark).to eq "1  # => "
+          expect(call("1 # => ",  toggled_mark: 1).toggled_mark).to eq "1"
+          expect(call("1\n",      toggled_mark: 1).toggled_mark).to eq "1  # => \n"
+          expect(call("1 # =>\n", toggled_mark: 1).toggled_mark).to eq "1\n"
+        end
+      end
+
       context 'before evaluating it raises if asked for' do
         specify('result')          { assert_must_evaluate :result }
         specify('exitstatus')      { assert_must_evaluate :exitstatus }

data/spec/seeing_is_believing_spec.rb

@@ -531,6 +531,10 @@ RSpec.describe SeeingIsBelieving do
     expect(result.exception.message).to match /recursive/i
   end
 
+  it 'does not blow up when the first line looks like it might have a magic comment in it (#126)' do
+    expect(values_for "1+1 # and a comment with 'Accept-Encoding: gzip' in it").to eq [['2']]
+  end
+
   it 'makes the SeeingIsBelieving::VERSION available to the program' do
     expect(values_for "SeeingIsBelieving::VERSION").to eq [[SeeingIsBelieving::VERSION.inspect]]
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: seeing_is_believing
 version: !ruby/object:Gem::Version
-  version: 3.4.0
+  version: 3.6.0
 platform: ruby
 authors:
 - Josh Cheek
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-18 00:00:00.000000000 Z
+date: 2018-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parser
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.4.0
+        version: 2.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.4.0
+        version: 2.5.0
 - !ruby/object:Gem::Dependency
   name: childprocess
   requirement: !ruby/object:Gem::Requirement
@@ -287,7 +287,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: seeing_is_believing
-rubygems_version: 2.7.2
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: Records results of every line of code in your file