transmogrifier 0.0.1 → 0.0.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NGEzMmI4YjQ5ODEyZGE3NDJiYWE4OGU5MTg1MDY4NDk5YmI2MDFmNA==
+    MDE3MWU3Yjc2NTU5ZjQ5NTNmMzhmZDU0YWU0N2NjZWExMjQzNmQwMA==
   data.tar.gz: !binary |-
-    NjA2ZGNlOGQzMWQ2MmYyMWZmMjVlNzg3MjZjYWMwYjY1YTQ4ZTliOQ==
-!binary "U0hBNTEy":
+    ZTk1NTkyN2JlZTRkMzI2MTgxZTM3ZjY2OWFmZTY4YjRjYmUxZTQwOA==
+SHA512:
   metadata.gz: !binary |-
-    ZDg5MWEwOGE4YTkwMmUzNzA0ZjliMjJiZDJjYzhlOGEwNzI0OGRkMTMyYTM4
-    NjI0YjE1N2NhZmZkYTE1MWEwYmEwYjBiZGVmNzUyNDlmODQyZDQ0YmI2ZWVm
-    NTNmZTFlOGRjMzZkOWU1NWRlZjQwZGZlNzVlMWZhOTFhZWVhYjA=
+    ZDQ3MDFkYzkyZTE2NDk1M2RlYmM1MjFlOTRhNzIxN2JmZWE4MWFlNDk5MDcw
+    NjhkMWVlZTBjZDllZTNhZjg2MWU1OWU2N2QxNDNmYWQyYzk3MWE2YjVjNDIy
+    YzJjNTM3NWVjNmU1YTQ3YjgxNGRhNTBlZTIxMzY4NTM4NGQzYTk=
   data.tar.gz: !binary |-
-    Y2JiMTQ5ZDhjNjU1Y2YwYzk4ZWYwYzlmMmQxMTkzNTU3MGFlY2M5ZGQ0NTU3
-    NjE4NDYyNDRmZDRlMmMwMDU3ZGEwMDEwZTZlN2Q2Mjk3MzMyYjljMDJhNDMy
-    MDY4OTEyZWZlZWUyNGFkYWEwMjgwM2ZhMmE0ZDU5NmU2MjNkMTE=
+    MDdhNWI1ODUwNDcxNDQ0YTFmZDBmMjI4MzMwNTgzMjFiNDhlMGYxZDQ2MTVl
+    NjJhMDhkZTMzOTE3NDBhMjk5YzgwNjY0OTMyYWI1MWIxYTFkZmE0NjllOTgy
+    ZjdjZWM5YThjY2ZiMWYzNmNkNmJiZWE0MzA4ZDRjNzFhZDg5MTQ=

data/README.md

@@ -17,6 +17,19 @@ output_hash = engine.run(input_hash)
 # output_hash => {"key" => "value", "new_key" => "new_value"}
 ```
 
+#### Copying a key
+```ruby
+engine = Transmogrifier::Engine.new
+move = Transmogrifier::Rules::Copy.new("", "key", "nested")
+
+engine.add_rule(:copy, "", "key", "nested")
+
+input_hash = {"key" => "value", "nested" => {"nested_key" => "nested_value"}}
+output_hash = transmogrifier.run(input_hash)
+
+# output_hash => {"key" => "value", "nested" => {"nested_key" => "nested_value", "key" => "value"}}
+```
+
 #### Deleting  a key
 ```ruby
 engine = Transmogrifier::Engine.new
@@ -30,6 +43,19 @@ output_hash = engine.run(input_hash)
 # output_hash => {"key" => "value"}
 ```
 
+#### Modifying a value
+```ruby
+engine = Transmogrifier::Engine.new
+move = Transmogrifier::Rules::Modify.new("", "key", "nested")
+
+engine.add_rule(:modify, "key", "al", "og")
+
+input_hash = {"key" => "value", "nested" => {"nested_key" => "nested_value"}}
+output_hash = transmogrifier.run(input_hash)
+
+# output_hash => {"key" => "vogue", "nested" => {"nested_key" => "nested_value"}}
+```
+
 #### Moving a key
 ```ruby
 engine = Transmogrifier::Engine.new
@@ -109,4 +135,4 @@ the selector `nested.second_level.deep` will apply to `buried_value`. Rules can
   ],
 }
 ```
-the hash with the name `this one!` can be operated on with `array.[name=this one!]`. Arrays can also wildcard match all children. For example to match both hashes in the array above, use the selector `array.[]`. 
+the hash with the name `this one!` can be operated on with `array.[name=this one!]` or `array.[name!=not me]`. Arrays can also wildcard match all children. For example to match both hashes in the array above, use the selector `array.[]`.

data/lib/transmogrifier/engine.rb

@@ -5,7 +5,14 @@ module Transmogrifier
         rules_array.map do |rule|
           type = rule["type"].capitalize
           selector = rule["selector"]
-          options = [rule["object"], rule["from"], rule["to"], rule["name"]].compact
+          options = [
+            rule["object"],
+            rule["from"],
+            rule["to"],
+            rule["name"],
+            rule["pattern"],
+            rule["replacement"],
+          ].compact
           Transmogrifier::Rules.const_get(type).new(selector, *options)
         end
       )

data/lib/transmogrifier/nodes/array_node.rb

@@ -18,10 +18,17 @@ module Transmogrifier
       end
     end
 
+    def clone(key)
+      matching_nodes = find_nodes(key)
+      raise "Multiple nodes match #{key}, clone criteria ambiguous" if matching_nodes.length > 1
+
+      Marshal.load(Marshal.dump(matching_nodes.first))
+    end
+
     def delete(key)
       matching_nodes = find_nodes(key)
-      raise "Multiple nodes match #{key}, deletion criteria ambiguous" if matching_nodes.length > 1
-      @array.delete(matching_nodes.first)
+      deleted_nodes = matching_nodes.each { |n| @array.delete(n) }
+      deleted_nodes.length > 1 ? deleted_nodes : deleted_nodes.first
     end
 
     def_delegator :@array, :<<, :append
@@ -30,7 +37,20 @@ module Transmogrifier
     private
 
     def find_nodes(attributes)
-      @array.select { |node| node.merge(Hash[attributes]) == node }
+      return @array if attributes.empty?
+
+      filtered = @array.clone
+      attributes.each do |attr|
+        case attr[0]
+          when "="
+            filtered.select! { |node| node.merge(Hash[*attr[1..-1]]) == node }
+          when "!="
+            filtered.reject! { |node| node.merge(Hash[*attr[1..-1]]) == node }
+          else
+            raise "Unsupported attribute filter #{attr.inspect}"
+        end
+      end
+      filtered
     end
   end
 end

data/lib/transmogrifier/nodes/hash_node.rb

@@ -20,6 +20,13 @@ module Transmogrifier
       end
     end
 
+    def clone(key)
+      matching_node = @hash[key]
+      raise "Multiple nodes match #{key}, clone criteria ambiguous" if matching_node.nil?
+
+      Marshal.load(Marshal.dump(matching_node))
+    end
+
     def_delegator :@hash, :delete
     def_delegator :@hash, :merge!, :append
     def_delegator :@hash, :to_hash, :raw

data/lib/transmogrifier/nodes/node.rb

@@ -19,6 +19,10 @@ module Transmogrifier
       raise NotImplementedError
     end
 
+    def clone(key_or_name)
+      raise NotImplementedError
+    end
+
     def delete(key_or_name)
       raise NotImplementedError
     end
@@ -26,5 +30,9 @@ module Transmogrifier
     def append(node)
       raise NotImplementedError
     end
+
+    def modify(pattern, replacement)
+      raise NotImplementedError
+    end
   end
 end

data/lib/transmogrifier/nodes/value_node.rb

@@ -11,6 +11,11 @@ module Transmogrifier
       raise "cannot find children of ValueNode satisfying non-empty selector #{keys}"
     end
 
+    def modify(pattern, replacement)
+      raise "Value is not modifiable using pattern matching" unless @value.respond_to?(:gsub)
+      return @value.gsub!(Regexp.new(pattern), replacement)
+    end
+
     def raw
       @value
     end

data/lib/transmogrifier/rules/copy.rb

@@ -0,0 +1,29 @@
+module Transmogrifier
+  module Rules
+    class Copy
+      def initialize(parent_selector, from, to)
+        @parent_selector, @from, @to = parent_selector, from, to
+      end
+
+      def apply!(input_hash)
+        top = Node.for(input_hash)
+        *from_keys, from_key = Selector.from_string(@from).keys
+        *to_keys, to_key = Selector.from_string(@to).keys
+
+        parents = top.find_all(Selector.from_string(@parent_selector).keys)
+        parents.each do |parent|
+          to_parent = parent.find_all(to_keys).first
+          object = parent.find_all(from_keys).first.clone(from_key)
+
+          if to_child = to_parent.find_all([to_key]).first
+            to_child.append(object)
+          else
+            to_parent.append({to_key => object})
+          end
+        end
+
+        top.raw
+      end
+    end
+  end
+end

data/lib/transmogrifier/rules/modify.rb

@@ -0,0 +1,19 @@
+module Transmogrifier
+  module Rules
+    class Modify
+      def initialize(parent_selector, pattern, replacement)
+        @parent_selector, @pattern, @replacement = parent_selector, pattern, replacement
+      end
+
+      def apply!(input_hash)
+        top = Node.for(input_hash)
+        parent_keys = Selector.from_string(@parent_selector).keys
+
+        parents = top.find_all(parent_keys)
+        parents.each { |parent| parent.modify(@pattern, @replacement) }
+
+        top.raw
+      end
+    end
+  end
+end

data/lib/transmogrifier/selector.rb

@@ -1,12 +1,13 @@
 module Transmogrifier
   class Selector
     FILTER_REGEX = /\[(.*)\]/
+    OPERATORS = ["!=", "="]
 
     def self.from_string(string)
       new(
         string.split(".").map do |str|
           match = str.scan(FILTER_REGEX).flatten.first
-          match ? match.split(",").map { |s| s.split("=") } : str
+          match ? match.split(",").map {|s| to_array(s)}  : str
         end
       )
     end
@@ -16,5 +17,15 @@ module Transmogrifier
     def initialize(keys)
       @keys = keys
     end
+
+    private
+
+    def self.to_array(string)
+      OPERATORS.each do |op|
+        next unless string.include?(op)
+        return string.split(op).insert(0, op)
+      end
+      [string]
+    end
   end
 end

data/lib/transmogrifier/version.rb

@@ -1,3 +1,3 @@
 module Transmogrifier
-  VERSION = "0.0.1"
+  VERSION = "0.0.3"
 end

data/spec/engine_spec.rb

@@ -23,6 +23,20 @@ describe Transmogrifier::Engine do
           "type" => "delete",
           "selector" => "top",
           "name" => "key3"
+        },
+
+        {
+          "type" => "copy",
+          "selector" => "top",
+          "from" => "key4",
+          "to" => "key5",
+        },
+
+        {
+          "type" => "modify",
+          "selector" => "top.key5",
+          "pattern" => "\\d+",
+          "replacement" => ".num",
         }
       ]
     end
@@ -31,7 +45,8 @@ describe Transmogrifier::Engine do
       input_hash = {
         "top" => {
           "key1" => "value1",
-          "key3" => "value2"
+          "key3" => "value2",
+          "key4" => "value4",
         }
       }
   
@@ -39,6 +54,8 @@ describe Transmogrifier::Engine do
         "top" => {
           "some" => "attributes",
           "key2" => "value1",
+          "key4" => "value4",
+          "key5" => "value.num",
         }
       })
     end

data/spec/nodes/array_node_spec.rb

@@ -24,14 +24,78 @@ module Transmogrifier
         ])
       end
 
-      it "filters by attributes" do
-        array = [
-          {"type" => "object", "key1" => "value1"},
-          {"type" => "object", "key2" => "value2"},
-        ]
-        node = ArrayNode.new(array)
+      context "when attribute filters are specified" do
+        it "filters using '='" do
+          array = [
+            {"type" => "object", "key1" => "value1"},
+            {"type" => "object", "key2" => "value2"},
+          ]
+          node = ArrayNode.new(array)
+
+          expect(node.find_all([[["=", "type", "object"]]]).map(&:raw)).to eq(array)
+        end
+
+        it "filters using '!='" do
+          array = [
+            {"type" => "object1", "key1" => "value1"},
+            {"type" => "object2", "key2" => "value2"},
+          ]
+          node = ArrayNode.new(array)
+
+          expect(node.find_all([[["!=", "type", "object1"]]]).map(&:raw)).to eq(array.slice(1,1))
+        end
+
+        it "filters using '!=' and '='" do
+          array = [
+            {"type" => "object1", "key1" => "value1"},
+            {"type" => "object2", "key2" => "value2"},
+            {"type" => "object3", "key3" => "value3"},
+          ]
+          node = ArrayNode.new(array)
+
+          expect(node.find_all([[["!=", "type", "object1"],["=", "type", "object2"]]]).map(&:raw)).to eq(array.slice(1,1))
+        end
+
+        it "raises ArgumentError for unknown operators" do
+          array = [
+            {"type" => "object1", "key1" => "value1"},
+            {"type" => "object2", "key2" => "value2"},
+          ]
+          node = ArrayNode.new(array)
+
+          expect{node.find_all([[["~", "type", "object1"]]])}.to raise_error
+          expect{node.find_all([[["value"]]])}.to raise_error
+        end
+      end
+    end
+
+    describe "#clone" do
+      context "when one node matches the criteria" do
+        it "returns the node from the array" do
+          value = {"name" => "object1"}
+          array = [value, {"name" => "object2"}]
+          node = ArrayNode.new(array)
+          expect(node.clone([["=", "name", "object1"]])).to eq(value)
+          expect(node.clone([["=", "name", "object1"]])).to_not be(value)
+        end
+      end
+
+      context "when more than one node matches the criteria" do
+        it "raises an error" do
+          array = [{"name" => "object1", "common_key" => "common_value"}, {"name" => "object2", "common_key" => "common_value"}]
+          node = ArrayNode.new(array)
+          expect {
+            node.clone([["=", "common_key", "common_value"]])
+          }.to raise_error
+        end
+      end
 
-        expect(node.find_all([["type", "object"]]).map(&:raw)).to eq(array)
+      context "when no nodes match the criteria" do
+        it "returns nil" do
+          array = [{"name" => "object1"}, {"name" => "object2"}]
+          node = ArrayNode.new(array)
+          expect(node.clone([["=", "name", "not_present"]])).to be_nil
+        end
       end
     end
 
@@ -43,20 +107,18 @@ module Transmogrifier
             {"name" => "object2"},
           ]
           node = ArrayNode.new(array)
-          expect(node.delete([["name", "object1"]])).to eq({"name" => "object1"})
+          expect(node.delete([["=", "name", "object1"]])).to eq({"name" => "object1"})
         end
       end
 
       context "when more than one node matches the criteria" do
-        it "raises an error" do
+        it "deletes all matching nodes from array" do
           array = [
             {"name" => "object1", "common_key" => "common_value"},
             {"name" => "object2", "common_key" => "common_value"},
           ]
-          node = ArrayNode.new(array)
-          expect {
-            node.delete([["common_key", "common_value"]])
-          }.to raise_error
+          node = ArrayNode.new(array.clone)
+          expect(node.delete([["=", "common_key", "common_value"]])).to eq(array)
         end
       end
 
@@ -67,7 +129,7 @@ module Transmogrifier
             {"name" => "object2"},
           ]
           node = ArrayNode.new(array)
-          expect(node.delete([["name", "not_present"]])).to be_nil
+          expect(node.delete([["=", "name", "not_present"]])).to be_nil
         end
       end
     end
@@ -80,5 +142,13 @@ module Transmogrifier
         expect(node.raw).to eq([{"name" => "object1"}, {"name" => "object2"}])
       end
     end
+
+    describe "#modify" do
+      it "raises a NotImplementedError" do
+        expect {
+          ArrayNode.new("hello").modify("value", "value2")
+        }.to raise_error(NotImplementedError)
+      end
+    end
   end
 end

data/spec/nodes/hash_node_spec.rb

@@ -26,7 +26,7 @@ module Transmogrifier
       end
 
       context "given multiple selector keys" do
-        let(:keys) { ["key", [["k", "v"]]] }
+        let(:keys) { ["key", [["=", "k", "v"]]] }
 
         context "when the first key is a key in the hash" do
           let(:hash) {{
@@ -65,6 +65,18 @@ module Transmogrifier
       end
     end
 
+    describe "#clone" do
+      it "returns a copy of value" do
+        value = "other_value"
+        hash = {"key" => "value", "extra_key" => value}
+        node = HashNode.new(hash)
+
+        expect(node.clone("extra_key")).to eql(value)
+        expect(node.clone("extra_key")).to_not be(value)
+        expect(node.raw).to eq("key" => "value", "extra_key" => "other_value")
+      end
+    end
+
     describe "#delete" do
       it "deletes the given key" do
         hash = {"key" => "value", "extra_key" => "other_value"}
@@ -90,5 +102,13 @@ module Transmogrifier
         expect(node.raw).to eq({"key" => "value", "extra_key" => "extra_value"})
       end
     end
+
+    describe "#modify" do
+      it "raises a NotImplementedError" do
+        expect {
+          HashNode.new("hello").modify("value", "value2")
+        }.to raise_error(NotImplementedError)
+      end
+    end
   end
 end

data/spec/nodes/value_node_spec.rb

@@ -27,6 +27,14 @@ module Transmogrifier
       end
     end
 
+    describe "#clone" do
+      it "raises a NotImplementedError" do
+        expect {
+          ValueNode.new("hello").clone("key")
+        }.to raise_error(NotImplementedError)
+      end
+    end
+
     describe "#delete" do
       it "raises a NotImplementedError" do
         expect {
@@ -42,5 +50,29 @@ module Transmogrifier
         }.to raise_error(NotImplementedError)
       end
     end
+
+    describe "#modify" do
+      context "when pattern matches" do
+        it "modifies the value based on simple pattern" do
+          node = ValueNode.new("value")
+          expect(node.modify("al", "og")).to eq("vogue")
+          expect(node.raw).to eq("vogue")
+        end
+
+        it "modifies the value based on complex pattern" do
+          node = ValueNode.new("valllue")
+          expect(node.modify("a.*u", "ogu")).to eq("vogue")
+          expect(node.raw).to eq("vogue")
+        end
+      end
+
+      context "when pattern does not match" do
+        it "does not modify value"  do
+          node = ValueNode.new("value")
+          expect(node.modify("ss", "og")).to be_nil
+          expect(node.raw).to eq("value")
+        end
+      end
+    end
   end
 end

data/spec/rules/copy_spec.rb

@@ -0,0 +1,99 @@
+require "transmogrifier"
+
+describe Transmogrifier::Rules::Copy do
+  let(:input_hash) do
+    {
+      "key" => "value",
+      "array" => [{"inside" => "value"}],
+      "nested" => {
+        "key" => "value",
+      },
+    }
+  end
+
+  context "when the selector finds a HashNode" do
+    context "when the target key exists" do
+      subject(:copy) { described_class.new("", "array.[inside=value]", "nested") }
+      
+      it "moves the hash to the to selector" do
+        expect(copy.apply!(input_hash)).to eq({
+          "key" => "value",
+          "array" => [{"inside" => "value"}],
+          "nested" => {
+            "key" => "value",
+            "inside" => "value",
+          },
+        })
+      end
+    end
+
+    context "when the target key doesn't exist" do
+      subject(:copy) { described_class.new("", "array.[inside=value]", "nested.nested_again") }
+
+      it "moves the hash to a new child" do
+        expect(copy.apply!(input_hash)).to eq({
+          "key" => "value",
+          "array" => [{"inside" => "value"}],
+          "nested" => {
+            "key" => "value",
+            "nested_again" => {
+              "inside" => "value",
+            }
+          },
+        })
+      end
+    end
+
+    context "when the from selector has a wildcard" do
+      let(:input_hash) do
+        {
+          "list_of_things" => [
+            {
+              "name" => "object1",
+              "property" => "property1",
+              "nested" => {}
+            },
+            {
+              "name" => "object2",
+              "property" => "property2",
+              "nested" => {}
+            },
+          ]
+        }
+      end
+      subject(:copy) { described_class.new("list_of_things.[]", "property", "nested.property") }
+
+      it "moves the matched hash to the correct place" do
+        expect(copy.apply!(input_hash)).to eq({
+          "list_of_things" => [
+            {
+              "name" => "object1",
+              "property" => "property1",
+              "nested" => { "property" => "property1" }
+            },
+            {
+              "name" => "object2",
+              "property" => "property2",
+              "nested" => { "property" => "property2" }
+            },
+          ]
+        })
+      end
+    end
+  end
+
+  context "when the selector finds an ArrayNode" do
+    subject(:copy) { described_class.new("", "array", "nested.array") }
+
+    it "moves the array to the to selector" do
+      expect(copy.apply!(input_hash)).to eq({
+        "key" => "value",
+        "array" => [{"inside" => "value"}],
+        "nested" => {
+          "key" => "value",
+          "array" => [{"inside" => "value"}],
+        },
+      })
+    end
+  end
+end

data/spec/rules/delete_spec.rb

@@ -35,4 +35,20 @@ describe Transmogrifier::Rules::Delete do
       })
     end
   end
+
+  context "when the selector matches multiple nodes" do
+    subject(:delete) { described_class.new("array", "[inside=value]") }
+
+    before { input_hash["array"] << {"inside" => "value"} }
+
+    it "deletes all entries from the array" do
+      expect(delete.apply!(input_hash)).to eq({
+        "key" => "value",
+        "array" => [],
+        "nested" => {
+          "key" => "value"
+        },
+      })
+    end
+  end
 end

data/spec/rules/modify_spec.rb

@@ -0,0 +1,34 @@
+require "transmogrifier"
+
+describe Transmogrifier::Rules::Modify do
+  let(:input_hash) do
+    { "key" => "value", "array" => [] }
+  end
+
+  context "when the selector finds a ValueNode" do
+    subject(:modify) { described_class.new("key", "al", "og") }
+
+    it "modifies the value" do
+      expect(modify.apply!(input_hash)).to eq({
+        "key" => "vogue",
+        "array" => [],
+      })
+    end
+  end
+
+  context "when the selector finds a HashNode" do
+    subject(:modify) { described_class.new("", "al", "og") }
+
+    it "appends to the hash" do
+      expect{modify.apply!(input_hash)}.to raise_error(NotImplementedError)
+    end
+  end
+
+  context "when the selector finds an ArrayNode" do
+    subject(:modify) { described_class.new("array", "al", "og") }
+
+    it "appends to the array" do
+      expect{modify.apply!(input_hash)}.to raise_error(NotImplementedError)
+    end
+  end
+end

data/spec/rules/move_spec.rb

@@ -94,6 +94,23 @@ describe Transmogrifier::Rules::Move do
     end
   end
 
+  context "when the selector finds multiple nodes" do
+    subject(:move) { described_class.new("", "array.[inside=value]", "nested.array") }
+
+    before { input_hash["array"] << {"inside" => "value"} }
+
+    it "moves them all as an array to the selector" do
+      expect(move.apply!(input_hash)).to eq({
+        "key" => "value",
+        "array" => [],
+        "nested" => {
+          "key" => "value",
+          "array" => [{"inside" => "value"},{"inside" => "value"}],
+        },
+      })
+    end
+  end
+
   context "using move as a rename" do
     subject(:rename) { described_class.new("", "array", "new_array") }
 

data/spec/selector_spec.rb

@@ -11,15 +11,24 @@ describe Transmogrifier::Selector do
     end
 
     it "allows filtering by attribute" do
-      described_class.from_string("[attr1=val1,attr2=val2]").keys.should == [[["attr1", "val1"],["attr2", "val2"]]]
+      described_class.from_string("[attr1=val1,attr2=val2]").keys.should == [[["=", "attr1", "val1"],["=", "attr2", "val2"]]]
+    end
+
+    it "allows filtering by attribute with not-equal comparison" do
+      described_class.from_string("[attr1!=val1,attr2!=val2]").keys.should == [[["!=", "attr1", "val1"],["!=", "attr2", "val2"]]]
     end
 
     it "allows empty array" do
       described_class.from_string("[]").keys.should == [[]]
     end
 
+    it "returns array filter with the whole value for unknown or missing operator" do
+      described_class.from_string("[attr1,attr2~val2]").keys.should == [[["attr1"], ["attr2~val2"]]]
+    end
+
     it "combines hash and array filtering" do
-      described_class.from_string("foo.[attr=val]").keys.should == ["foo", [["attr", "val"]]]
+      described_class.from_string("foo.[attr=val,attr!=val1]").keys.should == ["foo", [["=", "attr", "val"],["!=", "attr", "val1"]]]
+      described_class.from_string("[attr=val,attr!=val1].bar").keys.should == [[["=", "attr", "val"],["!=", "attr", "val1"]], "bar"]
     end
   end
 end

data/spec/transmogrifier_spec.rb

@@ -12,19 +12,29 @@ describe Transmogrifier::Engine do
             "selector" => "top",
             "object" => {"some" => "attributes"}
           },
-
           {
             "type" => "move",
             "selector" => "top",
             "from" => "key1",
             "to" => "key2",
           },
-
           {
             "type" => "delete",
             "selector" => "top",
             "name" => "key3"
-          }
+          },
+          {
+            "type" => "copy",
+            "selector" => "top",
+            "from" => "key4",
+            "to" => "key5",
+          },
+          {
+            "type" => "modify",
+            "selector" => "top.key5",
+            "pattern" => "\\d+",
+            "replacement" => ".num",
+          },
         ]
       end
 
@@ -32,7 +42,8 @@ describe Transmogrifier::Engine do
         input_hash = {
           "top" => {
             "key1" => "value1",
-            "key3" => "value2"
+            "key3" => "value2",
+            "key4" => "value4",
           }
         }
     
@@ -40,6 +51,8 @@ describe Transmogrifier::Engine do
           "top" => {
             "some" => "attributes",
             "key2" => "value1",
+            "key4" => "value4",
+            "key5" => "value.num",
           }
         })
       end
@@ -75,6 +88,100 @@ describe Transmogrifier::Engine do
         end
       end
 
+      describe "copying keys" do
+        let(:input_hash) do
+          {
+            "key" => "value",
+            "array" => [{"inside" => "value"}],
+            "nested" => { "key" => "value" },
+          }
+        end
+
+        context "when the selector finds a HashNode" do
+          context "when the target key exists" do
+            let(:rules) { [ {"type" => "copy", "selector" => "", "from" => "array.[inside=value]", "to" => "nested"}]}
+
+            it "copies the hash to the to selector" do
+              expect(engine.run(input_hash)).to eq({
+                "key" => "value",
+                "array" => [{"inside" => "value"}],
+                "nested" => {
+                  "key" => "value",
+                  "inside" => "value",
+                },
+              })
+            end
+          end
+
+          context "when the target key doesn't exist" do
+            let(:rules) { [ {"type" => "copy", "selector" => "", "from" => "array.[inside=value]", "to" => "nested.nested_again"} ] }
+
+            it "copies the hash to a new child" do
+              expect(engine.run(input_hash)).to eq({
+                 "key" => "value",
+                 "array" => [{"inside" => "value"}],
+                 "nested" => {
+                   "key" => "value",
+                   "nested_again" => { "inside" => "value" }
+                 },
+              })
+            end
+          end
+
+          context "when the from selector has a wildcard" do
+            let(:input_hash) do
+              {
+                "list_of_things" => [
+                  {
+                    "name" => "object1",
+                    "property" => "property1",
+                    "nested" => {}
+                  },
+                  {
+                    "name" => "object2",
+                    "property" => "property2",
+                    "nested" => {}
+                  },
+                ]
+              }
+            end
+            let(:rules) { [ {"type" => "copy", "selector" => "list_of_things.[]", "from" => "property", "to" => "nested.property"} ] }
+
+            it "copies the matched hash to the correct place" do
+              expect(engine.run(input_hash)).to eq({
+                "list_of_things" => [
+                  {
+                   "name" => "object1",
+                   "property" => "property1",
+                   "nested" => { "property" => "property1" },
+                  },
+                  {
+                   "name" => "object2",
+                   "property" => "property2",
+                   "nested" => { "property" => "property2" },
+                  },
+                ]
+              })
+            end
+          end
+        end
+
+        context "when the selector finds an ArrayNode" do
+          let(:rules) { [ {"type" => "copy", "selector" => "", "from" => "array", "to" => "nested.array"} ] }
+
+          it "copies the array to the to selector" do
+            expect(engine.run(input_hash)).to eq({
+              "key" => "value",
+              "array" => [{"inside" => "value"}],
+              "nested" => {
+                "key" => "value",
+                "array" => [{"inside" => "value"}],
+              },
+            })
+          end
+        end
+      end
+
       describe "deleting keys" do
         let(:input_hash) do
           {
@@ -100,7 +207,23 @@ describe Transmogrifier::Engine do
         context "when the selector finds an ArrayNode" do
           let(:rules) { [ {"type" => "delete", "selector" => "array", "name" => "[inside=value]"} ] }
           
-          it "deletes the array from the parent" do
+          it "deletes matching item from array" do
+            expect(engine.run(input_hash)).to eq({
+              "key" => "value",
+              "array" => [],
+              "nested" => {
+                "key" => "value"
+              },
+            })
+          end
+        end
+
+        context "when the selector matches multiple nodes" do
+          let(:rules) { [ {"type" => "delete", "selector" => "array", "name" => "[inside=value]"} ] }
+
+          before { input_hash["array"] << {"inside" => "value"} }
+
+          it "deletes all entries from the array" do
             expect(engine.run(input_hash)).to eq({
               "key" => "value",
               "array" => [],
@@ -206,6 +329,23 @@ describe Transmogrifier::Engine do
           end
         end
 
+        context "when the selector finds multiple nodes" do
+          let(:rules) { [ {"type" => "move", "selector" => "", "from" => "array.[inside=value]", "to" => "nested.array"} ] }
+
+          before { input_hash["array"] << {"inside" => "value"} }
+
+          it "moves them all as an array to the selector" do
+            expect(engine.run(input_hash)).to eq({
+              "key" => "value",
+              "array" => [],
+              "nested" => {
+                "key" => "value",
+                "array" => [{"inside" => "value"},{"inside" => "value"}],
+              },
+            })
+          end
+        end
+
         context "using move as a rename" do
           let(:rules) { [ {"type" => "move", "selector" => "", "from" => "array", "to" => "new_array"} ] }
           
@@ -220,6 +360,39 @@ describe Transmogrifier::Engine do
           end
         end
       end
+
+      describe "modifying value" do
+        let(:input_hash) do
+          { "key" => "value", "array" => [] }
+        end
+
+        context "when the selector finds a ValueNode" do
+          let(:rules) { [ {"type" => "modify", "selector" => "key", "pattern" => "al", "replacement" => "og"} ] }
+
+          it "modifies the value" do
+            expect(engine.run(input_hash)).to eq({
+              "key" => "vogue",
+              "array" => [],
+            })
+          end
+        end
+
+        context "when the selector finds a HashNode" do
+          let(:rules) { [ {"type" => "modify", "selector" => "", "pattern" => "al", "replacement" => "og"} ] }
+
+          it "appends to the hash" do
+            expect{engine.run(input_hash)}.to raise_error(NotImplementedError)
+          end
+        end
+
+        context "when the selector finds an ArrayNode" do
+          let(:rules) { [ {"type" => "modify", "selector" => "array", "pattern" => "al", "replacement" => "og"} ] }
+
+          it "appends to the array" do
+            expect{engine.run(input_hash)}.to raise_error(NotImplementedError)
+          end
+        end
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: transmogrifier
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.3
 platform: ruby
 authors:
 - John Foley
@@ -59,11 +59,6 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- Gemfile
-- LICENSE.txt
-- README.md
-- Rakefile
 - lib/transmogrifier.rb
 - lib/transmogrifier/engine.rb
 - lib/transmogrifier/nodes.rb
@@ -73,21 +68,27 @@ files:
 - lib/transmogrifier/nodes/value_node.rb
 - lib/transmogrifier/rules.rb
 - lib/transmogrifier/rules/append.rb
+- lib/transmogrifier/rules/copy.rb
 - lib/transmogrifier/rules/delete.rb
+- lib/transmogrifier/rules/modify.rb
 - lib/transmogrifier/rules/move.rb
 - lib/transmogrifier/selector.rb
 - lib/transmogrifier/version.rb
+- README.md
+- LICENSE.txt
+- Gemfile
 - spec/engine_spec.rb
 - spec/nodes/array_node_spec.rb
 - spec/nodes/hash_node_spec.rb
 - spec/nodes/node_spec.rb
 - spec/nodes/value_node_spec.rb
 - spec/rules/append_spec.rb
+- spec/rules/copy_spec.rb
 - spec/rules/delete_spec.rb
+- spec/rules/modify_spec.rb
 - spec/rules/move_spec.rb
 - spec/selector_spec.rb
 - spec/transmogrifier_spec.rb
-- transmogrifier.gemspec
 homepage: http://github.com/jfoley/transmogrifier
 licenses:
 - MIT
@@ -108,7 +109,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.7
+rubygems_version: 2.1.11
 signing_key: 
 specification_version: 4
 summary: A tool for manipulating schemas
@@ -119,7 +120,10 @@ test_files:
 - spec/nodes/node_spec.rb
 - spec/nodes/value_node_spec.rb
 - spec/rules/append_spec.rb
+- spec/rules/copy_spec.rb
 - spec/rules/delete_spec.rb
+- spec/rules/modify_spec.rb
 - spec/rules/move_spec.rb
 - spec/selector_spec.rb
 - spec/transmogrifier_spec.rb
+has_rdoc: 

data/.gitignore

@@ -1,18 +0,0 @@
-*.gem
-*.rbc
-.bundle
-.config
-.yardoc
-Gemfile.lock
-InstalledFiles
-_yardoc
-coverage
-doc/
-lib/bundler/man
-pkg
-rdoc
-spec/reports
-test/tmp
-test/version_tmp
-tmp
-.idea/

data/Rakefile

@@ -1 +0,0 @@
-require "bundler/gem_tasks"

data/transmogrifier.gemspec

@@ -1,37 +0,0 @@
-# -*- encoding: utf-8 -*-
-
-Gem::Specification.new do |s|
-  s.name = "transmogrifier"
-  s.version = "0.0.1"
-
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["John Foley"]
-  s.date = "2013-11-06"
-  s.description = "A tool for manipulating schemas"
-  s.email = ["john@hisfoleyness.com"]
-  s.files = [".gitignore", "Gemfile", "LICENSE.txt", "README.md", "Rakefile", "lib/transmogrifier.rb", "lib/transmogrifier/engine.rb", "lib/transmogrifier/nodes.rb", "lib/transmogrifier/nodes/array_node.rb", "lib/transmogrifier/nodes/hash_node.rb", "lib/transmogrifier/nodes/node.rb", "lib/transmogrifier/nodes/value_node.rb", "lib/transmogrifier/rules.rb", "lib/transmogrifier/rules/append.rb", "lib/transmogrifier/rules/delete.rb", "lib/transmogrifier/rules/move.rb", "lib/transmogrifier/selector.rb", "lib/transmogrifier/version.rb", "spec/engine_spec.rb", "spec/nodes/array_node_spec.rb", "spec/nodes/hash_node_spec.rb", "spec/nodes/node_spec.rb", "spec/nodes/value_node_spec.rb", "spec/rules/append_spec.rb", "spec/rules/delete_spec.rb", "spec/rules/move_spec.rb", "spec/selector_spec.rb", "spec/transmogrifier_spec.rb", "transmogrifier.gemspec"]
-  s.homepage = "http://github.com/jfoley/transmogrifier"
-  s.licenses = ["MIT"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = "2.0.7"
-  s.summary = "A tool for manipulating schemas"
-  s.test_files = ["spec/engine_spec.rb", "spec/nodes/array_node_spec.rb", "spec/nodes/hash_node_spec.rb", "spec/nodes/node_spec.rb", "spec/nodes/value_node_spec.rb", "spec/rules/append_spec.rb", "spec/rules/delete_spec.rb", "spec/rules/move_spec.rb", "spec/selector_spec.rb", "spec/transmogrifier_spec.rb"]
-
-  if s.respond_to? :specification_version then
-    s.specification_version = 4
-
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<bundler>, ["~> 1.3"])
-      s.add_development_dependency(%q<rake>, [">= 0"])
-      s.add_development_dependency(%q<rspec>, [">= 0"])
-    else
-      s.add_dependency(%q<bundler>, ["~> 1.3"])
-      s.add_dependency(%q<rake>, [">= 0"])
-      s.add_dependency(%q<rspec>, [">= 0"])
-    end
-  else
-    s.add_dependency(%q<bundler>, ["~> 1.3"])
-    s.add_dependency(%q<rake>, [">= 0"])
-    s.add_dependency(%q<rspec>, [">= 0"])
-  end
-end