oj 3.11.0 → 3.11.5

data/lib/oj/bag.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 
 module Oj
 

data/lib/oj/easy_hash.rb

@@ -12,13 +12,14 @@ module Oj
 
     # Replaces the Object.respond_to?() method.
     # @param [Symbol] m method symbol
+    # @param [Boolean] include_all whether to include private and protected methods in the search
     # @return [Boolean] true for any method that matches an instance
     #                   variable reader, otherwise false.
-    def respond_to?(m)
+    def respond_to?(m, include_all = false)
       return true if super
-      return true if has_key?(key)
-      return true if has_key?(key.to_s)
-      has_key?(key.to_sym)
+      return true if has_key?(m)
+      return true if has_key?(m.to_s)
+      has_key?(m.to_sym)
     end
 
     def [](key)

data/lib/oj/mimic.rb

@@ -133,18 +133,6 @@ module Oj
       end
     end
 
-    Date.class_eval do
-      # Both the JSON gem and Rails monkey patch as_json. Let them battle it out.
-      unless defined?(self.as_json)
-        def as_json(*)
-          { JSON.create_id => 'Date', 'y' => year, 'm' => month, 'd' => day, 'sg' => start }
-        end
-      end
-      def self.json_create(h)
-        civil(h['y'], h['m'], h['d'], h['sg'])
-      end
-    end
-
     DateTime.class_eval do
       # Both the JSON gem and Rails monkey patch as_json. Let them battle it out.
       unless defined?(self.as_json)

data/lib/oj/version.rb

@@ -1,5 +1,5 @@
 
 module Oj
   # Current version of the module.
-  VERSION = '3.11.0'
+  VERSION = '3.11.5'
 end

data/test/activerecord/result_test.rb

@@ -21,7 +21,12 @@ class ActiveRecordResultTest < Minitest::Test
 					["row 3 col 1", "row 3 col 2"],
 				      ])
     #puts "*** result: #{Oj.dump(result, indent: 2)}"
-   
-    assert_equal Oj.dump(result, mode: :rails), Oj.dump(result.to_hash)
+    json_result = if ActiveRecord.version >= Gem::Version.new("6")
+                    result.to_a
+                  else
+                    result.to_hash
+                  end
+
+    assert_equal Oj.dump(result, mode: :rails), Oj.dump(json_result)
   end
 end

data/test/foo.rb

@@ -6,47 +6,50 @@ $oj_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
   $: << File.join($oj_dir, dir)
 end
 
-require 'json'
-
-t = [Time.now.utc]
-
-puts "t.to_json - #{t.to_json}"
-
-puts "--- active support"
-
-require 'active_support'
-require "active_support/json"
-
-ActiveSupport::JSON::Encoding.use_standard_json_time_format = false
-
-puts "t.as_json - #{t.as_json}"
-puts "t.to_json - #{t.to_json}"
-
 require 'oj'
 
-t = [Time.now.utc]
+class Foo
+  def initialize
+    @x = 123
+  end
+
+  def xto_json(opt=nil, options=nil)
+    "---to_json---"
+  end
+end
 
-puts "-----------------------"
+class Bar < Foo
+  def initialize
+    @x = 321
+  end
+end
 
-#puts "t.as_json - #{t.as_json}"
-puts "t.to_json - #{t.to_json}"
+foo = Foo.new
+bar = Bar.new
 
-#Oj.mimic_JSON
+require 'json'
 
-#puts "Oj - t.as_json - #{t.as_json}"
+puts "JSON: #{JSON.generate(foo)}"
+puts "to_json: #{foo.to_json}"
+puts "bar JSON: #{JSON.generate(bar)}"
+puts "bar to_json: #{bar.to_json}"
 
-puts "--- active support"
+m = bar.method('to_json')
+puts "*** method: #{m} owner: #{m.owner.name}"
 
-require 'active_support'
-require "active_support/json"
+puts "---- rails"
+require 'rails'
 
-ActiveSupport::JSON::Encoding.use_standard_json_time_format = false
+m = bar.method('to_json')
+puts "*** method: #{m} owner: #{m.owner} params: #{m.parameters}"
 
-puts "t.as_json - #{t.as_json}"
-puts "t.to_json - #{t.to_json}"
+puts "JSON: #{JSON.generate(foo)}"
+puts "to_json: #{foo.to_json}"
 
-puts "--- optimize"
-Oj.optimize_rails
+puts "---- Oj.mimic_JSON"
+Oj.mimic_JSON()
+puts "Oj JSON: #{JSON.generate(foo)}"
+puts "Oj to_json: #{foo.to_json}"
 
-puts "t.as_json - #{t.as_json}"
-puts "t.to_json - #{t.to_json}"
+m = bar.method('to_json')
+puts "*** method: #{m} owner: #{m.owner} params: #{m.parameters}"

data/test/helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+#
 # Ubuntu does not accept arguments to ruby when called using env. To get warnings to show up the -w options is
 # required. That can be set in the RUBYOPT environment variable.
 # export RUBYOPT=-w
@@ -16,6 +18,14 @@ require 'bigdecimal'
 require 'pp'
 require 'oj'
 
+
+if defined?(GC.verify_compaction_references) == 'method'
+  # This method was added in Ruby 3.0.0. Calling it this way asks the GC to
+  # move objects around, helping to find object movement bugs.
+  GC.verify_compaction_references(double_heap: true, toward: :empty)
+end
+
+
 $ruby = RUBY_DESCRIPTION.split(' ')[0]
 $ruby = 'ree' if 'ruby' == $ruby && RUBY_DESCRIPTION.include?('Ruby Enterprise Edition')
 

data/test/json_gem/json_generator_test.rb

@@ -136,6 +136,10 @@ EOT
 
   def test_pretty_state
     state = JSON::PRETTY_STATE_PROTOTYPE.dup
+    # In come cases in Ruby 3.0 an :escape_slash is included in the state. It
+    # seems to occur on travis but not locally.
+    actual = state.to_h
+    actual.delete(:escape_slash)
     assert_equal({
       :allow_nan             => false,
       :array_nl              => "\n",
@@ -147,11 +151,15 @@ EOT
       :object_nl             => "\n",
       :space                 => " ",
       :space_before          => "",
-    }.sort_by { |n,| n.to_s }, state.to_h.sort_by { |n,| n.to_s })
+    }.sort_by { |n,| n.to_s }, actual.sort_by { |n,| n.to_s })
   end
 
   def test_safe_state
     state = JSON::SAFE_STATE_PROTOTYPE.dup
+    # In come cases in Ruby 3.0 an :escape_slash is included in the state. It
+    # seems to occur on travis but not locally.
+    actual = state.to_h
+    actual.delete(:escape_slash)
     assert_equal({
       :allow_nan             => false,
       :array_nl              => "",
@@ -163,11 +171,15 @@ EOT
       :object_nl             => "",
       :space                 => "",
       :space_before          => "",
-    }.sort_by { |n,| n.to_s }, state.to_h.sort_by { |n,| n.to_s })
+    }.sort_by { |n,| n.to_s }, actual.sort_by { |n,| n.to_s })
   end
 
   def test_fast_state
     state = JSON::FAST_STATE_PROTOTYPE.dup
+    # In come cases in Ruby 3.0 an :escape_slash is included in the state. It
+    # seems to occur on travis but not locally.
+    actual = state.to_h
+    actual.delete(:escape_slash)
     assert_equal({
       :allow_nan             => false,
       :array_nl              => "",
@@ -179,7 +191,7 @@ EOT
       :object_nl             => "",
       :space                 => "",
       :space_before          => "",
-    }.sort_by { |n,| n.to_s }, state.to_h.sort_by { |n,| n.to_s })
+    }.sort_by { |n,| n.to_s }, actual.sort_by { |n,| n.to_s })
   end
 
   def test_allow_nan

data/test/json_gem/test_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $: << File.dirname(__FILE__)
 $oj_dir = File.dirname(File.dirname(File.expand_path(File.dirname(__FILE__))))
 %w(lib ext).each do |dir|
@@ -12,6 +14,12 @@ if ENV['REAL_JSON_GEM']
 else
   require 'oj'
   Oj.mimic_JSON
+
+  if defined?(GC.verify_compaction_references) == 'method'
+    # This method was added in Ruby 3.0.0. Calling it this way asks the GC to
+    # move objects around, helping to find object movement bugs.
+    GC.verify_compaction_references(double_heap: true, toward: :empty)
+  end
 end
 
 NaN = JSON::NaN if defined?(JSON::NaN)

data/test/test_compat.rb

@@ -178,7 +178,7 @@ class CompatJuice < Minitest::Test
     assert_equal('"abc"', json)
   end
 
-  def test_time
+  def test_time_xml_schema
     t = Time.xmlschema("2012-01-05T23:58:07.123456000+09:00")
     #t = Time.local(2012, 1, 5, 23, 58, 7, 123456)
     json = Oj.dump(t, :mode => :compat)
@@ -297,7 +297,7 @@ class CompatJuice < Minitest::Test
   end
 
   # Time
-  def test_time
+  def test_time_from_time_object
     t = Time.new(2015, 1, 5, 21, 37, 7.123456, -8 * 3600)
     expect = '"' + t.to_s + '"'
     json = Oj.dump(t)

data/test/test_generate.rb

@@ -0,0 +1,21 @@
+#!/usr/bin/env ruby
+# encoding: utf-8
+
+$: << File.dirname(__FILE__)
+$oj_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
+%w(lib ext).each do |dir|
+  $: << File.join($oj_dir, dir)
+end
+
+require 'minitest'
+require 'minitest/autorun'
+require 'oj'
+
+class Generator < Minitest::Test
+
+  def test_before
+    json = Oj.generate({})
+    assert_equal("{}", json)
+  end
+
+end

data/test/test_hash.rb

@@ -25,5 +25,15 @@ class Hashi < Minitest::Test
     assert_equal(3, obj[:abc])
     assert_equal(3, obj.abc())
   end
+
+  def test_marshal
+    h = Oj::EasyHash.new()
+    h['abc'] = 3
+    out = Marshal.dump(h)
+
+    obj = Marshal.load(out)
+    assert_equal(Oj::EasyHash, obj.class)
+    assert_equal(3, obj[:abc])
+  end
   
 end # HashTest

data/test/test_scp.rb

@@ -328,7 +328,7 @@ class ScpTest < Minitest::Test
     IO.pipe do |read_io, write_io|
       if fork
         write_io.close
-        Oj.sc_parse(handler, read_io) {|v| p v}
+        Oj.sc_parse(handler, read_io)
         read_io.close
         assert_equal([[:hash_start],
                       [:hash_key, 'one'],

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: oj
 version: !ruby/object:Gem::Version
-  version: 3.11.0
+  version: 3.11.5
 platform: ruby
 authors:
 - Peter Ohler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-12 00:00:00.000000000 Z
+date: 2021-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -240,6 +240,7 @@ files:
 - test/test_fast.rb
 - test/test_file.rb
 - test/test_gc.rb
+- test/test_generate.rb
 - test/test_hash.rb
 - test/test_integer_range.rb
 - test/test_null.rb
@@ -364,6 +365,7 @@ test_files:
 - test/test_fast.rb
 - test/test_file.rb
 - test/test_gc.rb
+- test/test_generate.rb
 - test/test_hash.rb
 - test/test_integer_range.rb
 - test/test_null.rb