json_minify 0.1.0 → 0.2.0

data/bin/json_minify

@@ -32,8 +32,8 @@ end
 opt_parser.parse!
 if (input = ARGF.read)
   if options[:pretty]
-    puts JsonMinify.minify_pretty(input)
+    puts MultiJson.dump(MultiJson.load(input, minify: true), pretty: true)
   else
-    puts JsonMinify.minify(input)
+    puts MultiJson.minify(input)
   end
 end

data/lib/json_minify/version.rb

@@ -1,3 +1,3 @@
 module JsonMinify
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/lib/json_minify.rb

@@ -3,6 +3,7 @@ require 'multi_json'
 require "json_minify/version"
 
 module JsonMinify
+
   def minify(str)
     ss, buf = StringScanner.new(str), ''
 
@@ -41,20 +42,13 @@ module JsonMinify
     end
     buf
   end
-
-  def minify_parse(buf)
-    MultiJson.load(minify(buf))
-  end
-
-  def minify_pretty(buf)
-    MultiJson.dump(minify_parse(buf), pretty: true)
-  end
 end
 
-MultiJson.class_eval do
+MultiJson.module_eval do
   extend JsonMinify
-end
 
-JsonMinify.class_eval do
-  extend self
+  def self.load(string, options={})
+    options.fetch(:minify, false) && string = minify(string)
+    super(string, options)
+  end
 end

data/spec/minify_spec.rb

@@ -1,23 +1,51 @@
 require 'json_minify'
 require 'rspec/autorun'
 
-describe JsonMinify do
-  it "should remove whitespace from MultiJson" do
-    expect(MultiJson.minify("{ }")).to eql("{}")
-    expect(MultiJson.minify(%Q<{"foo": "bar"\n}\n>)).to eql(%Q<{"foo":"bar"}>)
-  end
+describe MultiJson do
+  describe "::minify" do
+    it "removes whitespace from JSON" do
+      expect(MultiJson.minify("{ }")).to eql("{}")
+      expect(MultiJson.minify(%Q<{"foo": "bar"\n}\n>)).to eql(%Q<{"foo":"bar"}>)
+    end
 
-  it "should remove comments from MultiJson" do
-    expect(MultiJson.minify("{ /* foo */ } /* bar */")).to eql("{}")
-    expect(MultiJson.minify(%Q<{"foo": "bar"\n}\n // this is a comment>)).to eql(%Q<{"foo":"bar"}>)
-  end
+    it "removes comments from JSON" do
+      expect(MultiJson.minify("{ /* foo */ } /* bar */")).to eql("{}")
+      expect(MultiJson.minify(%Q<{"foo": "bar"\n}\n // this is a comment>)).to eql(%Q<{"foo":"bar"}>)
+    end
 
-  it "should throw a SyntaxError on invalid MultiJson" do
-    expect { MultiJson.minify("{ /* foo */ } /* bar ") }.to raise_error(SyntaxError)
-    expect { MultiJson.minify(%Q<{ "foo": new Date(1023) }>) }.to raise_error(SyntaxError)
+    it "throws a SyntaxError on invalid JSON" do
+      expect { MultiJson.minify("{ /* foo */ } /* bar ") }.to raise_error(SyntaxError)
+      expect { MultiJson.minify(%Q<{ "foo": new Date(1023) }>) }.to raise_error(SyntaxError)
+    end
+
+    it "copes with the example from https://github.com/getify/JsonMinify.minify" do
+      expect(MultiJson.minify( %Q'{ /* comment */ "foo": 42 \n }' )).to eql('{"foo":42}')
+    end
   end
 
-  it "should cope with the example from https://github.com/getify/MultiJson.minify" do
-    expect(MultiJson.minify( %Q'{ /* comment */ "foo": 42 \n }' )).to eql('{"foo":42}')
+
+  describe "::load" do
+    context "with :minify option" do
+      let(:opts) { {minify: true, symbolize_keys: true} }
+
+      it "removes whitespace from JSON" do
+        expect(MultiJson.load("{ }", opts)).to eq({})
+        expect(MultiJson.load(%Q<{"foo": "bar"\n}\n>, opts)).to eq({foo:"bar"})
+      end
+
+      it "removes comments from JSON" do
+        expect(MultiJson.load("{ /* foo */ } /* bar */", opts)).to eq({})
+        expect(MultiJson.load(%Q<{"foo": "bar"\n}\n // this is a comment>, opts)).to eq({foo:"bar"})
+      end
+
+      it "throws a SyntaxError on invalid JSON" do
+        expect { MultiJson.load("{ /* foo */ } /* bar ", opts) }.to raise_error(SyntaxError)
+        expect { MultiJson.load(%Q<{ "foo": new Date(1023) }>, opts) }.to raise_error(SyntaxError)
+      end
+
+      it "copes with the example from https://github.com/getify/JsonMinify.load" do
+        expect(MultiJson.load( %Q'{ /* comment */ "foo": 42 \n }' , opts)).to eq({foo:42})
+      end
+    end
   end
 end

metadata

@@ -1,7 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: json_minify
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
+  prerelease: 
 platform: ruby
 authors:
 - Geoff Youngs, Alan Cohen
@@ -12,64 +13,50 @@ date: 2014-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &70357755509400 !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1'
+  version_requirements: *70357755509400
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &70357755508520 !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.1'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.1'
+  version_requirements: *70357755508520
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &70357755507620 !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.14'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.14'
+  version_requirements: *70357755507620
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &70357755506860 !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
-description: |-
-  Pre-parser for JSON that removes C/C++ style comments
-                            and whitespace from formatted JSON, similar to
-                            https://github.com/getify/JSON.minify.
+  version_requirements: *70357755506860
+description: ! "Pre-parser for JSON that removes C/C++ style comments\n                          and
+  whitespace from formatted JSON, similar to\n                          https://github.com/getify/JSON.minify."
 email:
 - git@intersect-uk.co.uk
 - acohen@climate.com
@@ -78,33 +65,35 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- bin/json_minify
-- lib/json_minify.rb
 - lib/json_minify/version.rb
+- lib/json_minify.rb
 - spec/minify_spec.rb
+- !binary |-
+  YmluL2pzb25fbWluaWZ5
 homepage: http://github.com/lumberj/json-minify-rb
 licenses:
 - MIT
-metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.1
+rubygems_version: 1.8.11
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: JSON.minify implementation
 test_files:
 - spec/minify_spec.rb

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: 4b1a521047dc7e5c7f8ca2db29a83790e894c16c
-  data.tar.gz: 6faea50280d8e213df26861dab2dbbb7c393a7a0
-SHA512:
-  metadata.gz: 48eb3797b93ae45ce392356c35b3b0580e4f29d19d19ee751d47635aa5cce236ac99bcc9565bea32e7332f5d2c824360eb94050e1953c625583f5f87768a1a5d
-  data.tar.gz: c34a2c0c33b72a25864082f5b618298666ba1d5560817ebe3cf799c4c20933d8df577090409f227d6565bcc0961dd667412ec17a6e0824743c1f338cb9a7f9f7