metamorpher 0.2.4 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c154a782758c9a6ec14e30786ee56348283a5bee
-  data.tar.gz: dba3f527a5d5cca415d86f66fbb57ec39d134a8e
+  metadata.gz: 8735660dd5f257c643ffb8eb2efaa66b3a52c455
+  data.tar.gz: 74a07818202c1ace79822833901584d21632a129
 SHA512:
-  metadata.gz: 2e35036abcb0d0c59fa5c3bd34128b2ac96466da12ca2082ba653b18f70df5c883f16d30c0de5629346ccd2abd3eb72497fd63265c9cae3dddf662f6b87afec6
-  data.tar.gz: 54603a7afb04f3791c3df1e6554c4a8f79a940d505d4afab62848f04cb813ab6732278913604ccedcb8eb306e627a928b0400e6378c5ad0988de5e5306fd46fb
+  metadata.gz: c97328c5e7eb49f4db89913aa66d03cfa6f92820c424548e97492e0f48f6e29a04120a0bfb6db71ea4d3ca7d913a13f68a11656eff47372ac32a45636a4d21c5
+  data.tar.gz: d016159edfb874d20abdfe117c38621f8e36f82eda6ff7b9a6e50ebce2790690954af94df5296adbad7bf69e117a18532140cc71b0abf815df2f951e128ab617

data/.ruby-version

@@ -1 +1 @@
-2.2.2
+2.2.3

data/.travis.yml

@@ -1,7 +1,6 @@
 language: ruby
 rvm:
-  - 2.2.2
-  - jruby
+  - 2.2.3
 addons:
   code_climate:
     repo_token: ccac936a51a09e2bd82f13edee996fc2376304e119df9cbb288a6a7ead7f82ce

data/RELEASES.md

@@ -1,5 +1,8 @@
 # Release History
 
+## v0.2.5 (12 March 2016)
+* Fix bugs that caused mutators to have side-effects when applied to overlapping ASTs.
+
 ## v0.2.4 (23 February 2016)
 * Fix bug in computing the paths of terms that contain term sets.
 

data/lib/metamorpher/builders/ast/literal_builder.rb

@@ -23,7 +23,7 @@ module Metamorpher
         private
 
         def termify(item)
-          item.is_a?(Terms::Term) ? item : literal!(item)
+          item.is_a?(Terms::Term) ? item.dup : literal!(item)
         end
       end
     end

data/lib/metamorpher/rewriter/replacement.rb

@@ -3,7 +3,7 @@ module Metamorpher
     module Replacement
       def replace(path, replacement)
         if path.empty?
-          replacement
+          replacement.dup
         else
           Terms::Literal.new(
             name: name,

data/lib/metamorpher/rewriter/substitution.rb

@@ -16,7 +16,7 @@ module Metamorpher
       end
 
       def visit_variable(variable)
-        substitution_for_variable(variable.name)
+        substitution_for_variable(variable.name).dup
       end
 
       def visit_literal(literal)

data/lib/metamorpher/version.rb

@@ -1,3 +1,3 @@
 module Metamorpher
-  VERSION = "0.2.4"
+  VERSION = "0.2.5"
 end

data/metamorpher.gemspec

@@ -22,7 +22,7 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "parser", "~> 2.2.2"
   spec.add_runtime_dependency "unparser", "~> 0.2.4"
 
-  spec.add_development_dependency "bundler", "~> 1.10.3"
+  spec.add_development_dependency "bundler", "~> 1.11"
   spec.add_development_dependency "rake", "~> 10.4.2"
   spec.add_development_dependency "rspec", "~> 3.3.0"
   spec.add_development_dependency "codeclimate-test-reporter", "~> 0.4.6"

data/spec/integration/ruby/mutator_spec.rb

@@ -20,31 +20,41 @@ describe "Mutator" do
 
     let(:mutatable) do
       "def compare\n" \
-      "  foo < bar\n" \
-      "  bar < baz\n" \
+      "  foo < bar < baz\n" \
+      "  baaz < qux\n" \
       "end"
     end
 
     let(:mutated) do
       [
         "def compare\n" \
-        "  foo > bar\n" \
-        "  bar < baz\n" \
+        "  (foo < bar) > baz\n" \
+        "  baaz < qux\n" \
         "end",
 
         "def compare\n" \
-        "  foo == bar\n" \
-        "  bar < baz\n" \
+        "  (foo < bar) == baz\n" \
+        "  baaz < qux\n" \
         "end",
 
         "def compare\n" \
-        "  foo < bar\n" \
-        "  bar > baz\n" \
+        "  foo < bar < baz\n" \
+        "  baaz > qux\n" \
         "end",
 
         "def compare\n" \
-        "  foo < bar\n" \
-        "  bar == baz\n" \
+        "  foo < bar < baz\n" \
+        "  baaz == qux\n" \
+        "end",
+
+        "def compare\n" \
+        "  foo > bar < baz\n" \
+        "  baaz < qux\n" \
+        "end",
+
+        "def compare\n" \
+        "  foo == bar < baz\n" \
+        "  baaz < qux\n" \
         "end"
       ]
     end
@@ -56,15 +66,6 @@ describe "Mutator" do
         it "should return the mutated code" do
           expect(subject.mutate(mutatable)).to eq(mutated)
         end
-
-        it "should yield for each mutation site" do
-          expect { |b| subject.mutate(mutatable, &b) }.to yield_successive_args(
-            site_for(14..22, "foo < bar", "foo > bar"),
-            site_for(14..22, "foo < bar", "foo == bar"),
-            site_for(26..34, "bar < baz", "bar > baz"),
-            site_for(26..34, "bar < baz", "bar == baz")
-          )
-        end
       end
 
       describe "for code that cannot be mutated" do
@@ -88,10 +89,12 @@ describe "Mutator" do
 
         it "should yield for each mutating site" do
           expect { |b| subject.mutate_file(mutatable_file, &b) }.to yield_successive_args(
+            site_for(14..28, "foo < bar < baz", "(foo < bar) > baz"),
+            site_for(14..28, "foo < bar < baz", "(foo < bar) == baz"),
+            site_for(32..41, "baaz < qux", "baaz > qux"),
+            site_for(32..41, "baaz < qux", "baaz == qux"),
             site_for(14..22, "foo < bar", "foo > bar"),
-            site_for(14..22, "foo < bar", "foo == bar"),
-            site_for(26..34, "bar < baz", "bar > baz"),
-            site_for(26..34, "bar < baz", "bar == baz")
+            site_for(14..22, "foo < bar", "foo == bar")
           )
         end
       end
@@ -154,10 +157,12 @@ describe "Mutator" do
           mutatable_file,
           mutated,
           [
+            site_for(14..28, "foo < bar < baz", "(foo < bar) > baz"),
+            site_for(14..28, "foo < bar < baz", "(foo < bar) == baz"),
+            site_for(32..41, "baaz < qux", "baaz > qux"),
+            site_for(32..41, "baaz < qux", "baaz == qux"),
             site_for(14..22, "foo < bar", "foo > bar"),
-            site_for(14..22, "foo < bar", "foo == bar"),
-            site_for(26..34, "bar < baz", "bar > baz"),
-            site_for(26..34, "bar < baz", "bar == baz")
+            site_for(14..22, "foo < bar", "foo == bar")
           ]
         ]
 
@@ -165,10 +170,12 @@ describe "Mutator" do
           clone_of_mutatable_file,
           mutated,
           [
+            site_for(14..28, "foo < bar < baz", "(foo < bar) > baz"),
+            site_for(14..28, "foo < bar < baz", "(foo < bar) == baz"),
+            site_for(32..41, "baaz < qux", "baaz > qux"),
+            site_for(32..41, "baaz < qux", "baaz == qux"),
             site_for(14..22, "foo < bar", "foo > bar"),
-            site_for(14..22, "foo < bar", "foo == bar"),
-            site_for(26..34, "bar < baz", "bar > baz"),
-            site_for(26..34, "bar < baz", "bar == baz")
+            site_for(14..22, "foo < bar", "foo == bar")
           ]
         ]
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: metamorpher
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors:
 - Louis Rose
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-23 00:00:00.000000000 Z
+date: 2016-03-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: attributable
@@ -58,14 +58,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.10.3
+        version: '1.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.10.3
+        version: '1.11'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -242,7 +242,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: A term rewriting library for transforming (Ruby) programs