oj 3.14.3 → 3.15.0

data/test/test_fast.rb

@@ -1,8 +1,7 @@
 #!/usr/bin/env ruby
-# coding: utf-8
 # frozen_string_literal: true
 
-$: << File.dirname(__FILE__)
+$LOAD_PATH << __dir__
 
 require 'helper'
 
@@ -52,7 +51,7 @@ class DocTest < Minitest::Test
     json = %{true}
     Oj::Doc.open(json) do |doc|
       assert_equal(TrueClass, doc.type)
-      assert_equal(true, doc.fetch())
+      assert(doc.fetch())
     end
   end
 
@@ -60,7 +59,7 @@ class DocTest < Minitest::Test
     json = %{false}
     Oj::Doc.open(json) do |doc|
       assert_equal(FalseClass, doc.type)
-      assert_equal(false, doc.fetch())
+      refute(doc.fetch())
     end
   end
 
@@ -76,7 +75,7 @@ class DocTest < Minitest::Test
     json = %{"ぴーたー"}
     Oj::Doc.open(json) do |doc|
       assert_equal(String, doc.type)
-      assert_equal("ぴーたー", doc.fetch())
+      assert_equal('ぴーたー', doc.fetch())
     end
   end
 
@@ -84,7 +83,7 @@ class DocTest < Minitest::Test
     json = %{"\\u3074\\u30fc\\u305f\\u30fc"}
     Oj::Doc.open(json) do |doc|
       assert_equal(String, doc.type)
-      assert_equal("ぴーたー", doc.fetch())
+      assert_equal('ぴーたー', doc.fetch())
     end
   end
 
@@ -92,7 +91,7 @@ class DocTest < Minitest::Test
     json = %{12345}
     Oj::Doc.open(json) do |doc|
       assert_equal(Integer, doc.type)
-      assert_equal(12345, doc.fetch())
+      assert_equal(12_345, doc.fetch())
     end
   end
 
@@ -100,7 +99,7 @@ class DocTest < Minitest::Test
     json = %{12345.6789}
     Oj::Doc.open(json) do |doc|
       assert_equal(Float, doc.type)
-      assert_equal(12345.6789, doc.fetch())
+      assert_in_delta(12_345.6789, doc.fetch())
     end
   end
 
@@ -108,8 +107,8 @@ class DocTest < Minitest::Test
     json = %{12345.6789e7}
     Oj::Doc.open(json) do |doc|
       assert_equal(Float, doc.type)
-      #assert_equal(12345.6789e7, doc.fetch())
-      assert_equal(12345.6789e7.to_i, doc.fetch().to_i)
+      # assert_equal(12345.6789e7, doc.fetch())
+      assert_equal(12_345.6789e7.to_i, doc.fetch().to_i)
     end
   end
 
@@ -117,7 +116,7 @@ class DocTest < Minitest::Test
     json = %{[]}
     Oj::Doc.open(json) do |doc|
       assert_equal(Array, doc.type)
-      assert_equal([], doc.fetch())
+      assert_empty(doc.fetch())
     end
   end
 
@@ -133,7 +132,7 @@ class DocTest < Minitest::Test
     json = %{{}}
     Oj::Doc.open(json) do |doc|
       assert_equal(Hash, doc.type)
-      assert_equal({}, doc.fetch())
+      assert_empty(doc.fetch())
     end
   end
 
@@ -168,7 +167,8 @@ class DocTest < Minitest::Test
 
   def test_move
     Oj::Doc.open(@json1) do |doc|
-      [ '/',
+      [
+        '/',
         '/array',
         '/boolean',
         '/array/1/hash/h2/a/3',
@@ -201,12 +201,13 @@ class DocTest < Minitest::Test
 
   def test_move_relative
     Oj::Doc.open(@json1) do |doc|
-      [['/', 'array', '/array'],
-       ['/array', '1/num', '/array/1/num'],
-       ['/array/1/hash', 'h2/a', '/array/1/hash/h2/a'],
-       ['/array/1', 'hash/h2/a/2', '/array/1/hash/h2/a/2'],
-       ['/array/1/hash', '../string', '/array/1/string'],
-       ['/array/1/hash', '..', '/array/1'],
+      [
+        ['/', 'array', '/array'],
+        ['/array', '1/num', '/array/1/num'],
+        ['/array/1/hash', 'h2/a', '/array/1/hash/h2/a'],
+        ['/array/1', 'hash/h2/a/2', '/array/1/hash/h2/a/2'],
+        ['/array/1/hash', '../string', '/array/1/string'],
+        ['/array/1/hash', '..', '/array/1'],
       ].each do |start, path, where|
         doc.move(start)
         doc.move(path)
@@ -217,14 +218,15 @@ class DocTest < Minitest::Test
 
   def test_type
     Oj::Doc.open(@json1) do |doc|
-      [['/', Hash],
-       ['/array', Array],
-       ['/array/1', Hash],
-       ['/array/1/num', Integer],
-       ['/array/1/string', String],
-       ['/array/1/hash/h2/a', Array],
-       ['/array/1/hash/../num', Integer],
-       ['/array/1/hash/../..', Array],
+      [
+        ['/', Hash],
+        ['/array', Array],
+        ['/array/1', Hash],
+        ['/array/1/num', Integer],
+        ['/array/1/string', String],
+        ['/array/1/hash/h2/a', Array],
+        ['/array/1/hash/../num', Integer],
+        ['/array/1/hash/../..', Array],
       ].each do |path, type|
         assert_equal(type, doc.type(path))
       end
@@ -233,15 +235,16 @@ class DocTest < Minitest::Test
 
   def test_local_key
     Oj::Doc.open(@json1) do |doc|
-      [['/', nil],
-       ['/array', 'array'],
-       ['/array/1', 1],
-       ['/array/1/num', 'num'],
-       ['/array/1/string', 'string'],
-       ['/array/1/hash/h2/a', 'a'],
-       ['/array/1/hash/../num', 'num'],
-       ['/array/1/hash/..', 1],
-       ['/array/1/hash/../..', 'array'],
+      [
+        ['/', nil],
+        ['/array', 'array'],
+        ['/array/1', 1],
+        ['/array/1/num', 'num'],
+        ['/array/1/string', 'string'],
+        ['/array/1/hash/h2/a', 'a'],
+        ['/array/1/hash/../num', 'num'],
+        ['/array/1/hash/..', 1],
+        ['/array/1/hash/../..', 'array'],
       ].each do |path, key|
         doc.move(path)
         if key.nil?
@@ -255,15 +258,16 @@ class DocTest < Minitest::Test
 
   def test_fetch_move
     Oj::Doc.open(@json1) do |doc|
-      [['/array/1/num', 3],
-       ['/array/1/string', 'message'],
-       ['/array/1/hash/h2/a', [1, 2, 3]],
-       ['/array/1/hash/../num', 3],
-       ['/array/1/hash/..', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
-       ['/array/1/hash/../..', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
-       ['/array/1', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
-       ['/array', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
-       ['/', {'array' => [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}], 'boolean' => true}],
+      [
+        ['/array/1/num', 3],
+        ['/array/1/string', 'message'],
+        ['/array/1/hash/h2/a', [1, 2, 3]],
+        ['/array/1/hash/../num', 3],
+        ['/array/1/hash/..', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
+        ['/array/1/hash/../..', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
+        ['/array/1', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
+        ['/array', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
+        ['/', {'array' => [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}], 'boolean' => true}],
       ].each do |path, val|
         doc.move(path)
         assert_equal(val, doc.fetch())
@@ -273,17 +277,18 @@ class DocTest < Minitest::Test
 
   def test_fetch_path
     Oj::Doc.open(@json1) do |doc|
-      [['/array/1/num', 3],
-       ['/array/1/string', 'message'],
-       ['/array/1/hash/h2/a', [1, 2, 3]],
-       ['/array/1/hash/../num', 3],
-       ['/array/1/hash/..', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
-       ['/array/1/hash/../..', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
-       ['/array/1', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
-       ['/array', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
-       ['/', {'array' => [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}], 'boolean' => true}],
-       ['/nothing', nil],
-       ['/array/10', nil],
+      [
+        ['/array/1/num', 3],
+        ['/array/1/string', 'message'],
+        ['/array/1/hash/h2/a', [1, 2, 3]],
+        ['/array/1/hash/../num', 3],
+        ['/array/1/hash/..', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
+        ['/array/1/hash/../..', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
+        ['/array/1', {'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}],
+        ['/array', [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}]],
+        ['/', {'array' => [{'num' => 3, 'string' => 'message', 'hash' => {'h2' => {'a' => [1, 2, 3]}}}], 'boolean' => true}],
+        ['/nothing', nil],
+        ['/array/10', nil],
       ].each do |path, val|
         if val.nil?
           assert_nil(doc.fetch(path))
@@ -305,9 +310,10 @@ class DocTest < Minitest::Test
 
   def test_move_fetch_path
     Oj::Doc.open(@json1) do |doc|
-      [['/array/1', 'num', 3],
-       ['/array/1', 'string', 'message'],
-       ['/array/1/hash', 'h2/a', [1, 2, 3]],
+      [
+        ['/array/1', 'num', 3],
+        ['/array/1', 'string', 'message'],
+        ['/array/1/hash', 'h2/a', [1, 2, 3]],
       ].each do |path, fetch_path, val|
         doc.move(path)
         assert_equal(val, doc.fetch(fetch_path))
@@ -317,12 +323,13 @@ class DocTest < Minitest::Test
 
   def test_exists
     Oj::Doc.open(@json1) do |doc|
-      [['/array/1', true],
-       ['/array/1', true],
-       ['/array/1/hash', true],
-       ['/array/1/dash', false],
-       ['/array/3', false],
-       ['/nothing', false],
+      [
+        ['/array/1', true],
+        ['/array/1', true],
+        ['/array/1/hash', true],
+        ['/array/1/dash', false],
+        ['/array/3', false],
+        ['/nothing', false],
       ].each do |path, val|
         assert_equal(val, doc.exists?(path), "failed for #{path.inspect}")
       end
@@ -396,7 +403,7 @@ class DocTest < Minitest::Test
         end
       end
     end
-    assert_equal({"/a"=>1, "/c"=>[2], "/c/1"=>2, "/d"=>3}, h)
+    assert_equal({'/a'=>1, '/c'=>[2], '/c/1'=>2, '/d'=>3}, h)
   end
 
   def test_size
@@ -409,8 +416,8 @@ class DocTest < Minitest::Test
   end
 
   def test_open_file
-    filename = File.join(File.dirname(__FILE__), 'open_file_test.json')
-    File.open(filename, 'w') { |f| f.write('{"a":[1,2,3]}') }
+    filename = File.join(__dir__, 'open_file_test.json')
+    File.write(filename, '{"a":[1,2,3]}')
     Oj::Doc.open_file(filename) do |doc|
       assert_equal(5, doc.size)
     end
@@ -435,8 +442,8 @@ class DocTest < Minitest::Test
   end
 
   def test_file_open_close
-    filename = File.join(File.dirname(__FILE__), 'open_file_test.json')
-    File.open(filename, 'w') { |f| f.write('{"a":[1,2,3]}') }
+    filename = File.join(__dir__, 'open_file_test.json')
+    File.write(filename, '{"a":[1,2,3]}')
     doc = Oj::Doc.open_file(filename)
     assert_equal(Oj::Doc, doc.class)
     assert_equal(5, doc.size)
@@ -463,7 +470,6 @@ class DocTest < Minitest::Test
     doc.home()
     assert_equal(2, doc.fetch('/a/2'))
     assert_equal(2, doc.fetch('a/2'))
-    doc = nil
     GC.start
     # a print statement confirms close is called
   end
@@ -496,7 +502,7 @@ class DocTest < Minitest::Test
   end
 
   def test_doc_empty
-    result = Oj::Doc.open("") { |doc| doc.each_child {} }
+    result = Oj::Doc.open('') { |doc| doc.each_child {} }
     assert_nil(result)
   end
 

data/test/test_file.rb

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << File.dirname(__FILE__)
+$LOAD_PATH << __dir__
 
 require 'helper'
 
@@ -22,32 +22,26 @@ class FileJuice < Minitest::Test
   end # Jam
 
   class Jeez < Jam
-    def initialize(x, y)
-      super
-    end
 
     def to_json(*_args)
       %{{"json_class":"#{self.class}","x":#{@x},"y":#{@y}}}
     end
 
     def self.json_create(h)
-      self.new(h['x'], h['y'])
+      new(h['x'], h['y'])
     end
   end # Jeez
 
   class Orange < Jam
-    def initialize(x, y)
-      super
-    end
 
-    def as_json()
+    def as_json
       { :json_class => self.class,
         :x => @x,
         :y => @y }
     end
 
     def self.json_create(h)
-      self.new(h['x'], h['y'])
+      new(h['x'], h['y'])
     end
   end
 
@@ -73,8 +67,8 @@ class FileJuice < Minitest::Test
 
   def test_fixnum
     dump_and_load(0, false)
-    dump_and_load(12345, false)
-    dump_and_load(-54321, false)
+    dump_and_load(12_345, false)
+    dump_and_load(-54_321, false)
     dump_and_load(1, false)
   end
 
@@ -82,9 +76,9 @@ class FileJuice < Minitest::Test
     mode = Oj.default_options()[:mode]
     Oj.default_options = {:mode => :object}
     dump_and_load(0.0, false)
-    dump_and_load(12345.6789, false)
+    dump_and_load(12_345.6789, false)
     dump_and_load(70.35, false)
-    dump_and_load(-54321.012, false)
+    dump_and_load(-54_321.012, false)
     dump_and_load(1.7775, false)
     dump_and_load(2.5024, false)
     dump_and_load(2.48e16, false)
@@ -118,9 +112,9 @@ class FileJuice < Minitest::Test
   # Symbol
   def test_symbol_object
     Oj.default_options = { :mode => :object }
-    #dump_and_load(''.to_sym, false)
+    # dump_and_load(''.to_sym, false)
     dump_and_load(:abc, false)
-    dump_and_load(':xyz'.to_sym, false)
+    dump_and_load(:":xyz", false)
   end
 
   # Time
@@ -129,10 +123,11 @@ class FileJuice < Minitest::Test
     Oj.default_options = { :mode => :object, :time_format => :unix_zone }
     dump_and_load(t, false)
   end
+
   def test_time_object_early
     skip 'Windows does not support dates before 1970.' if RbConfig::CONFIG['host_os'] =~ /(mingw|mswin)/
 
-    t = Time.xmlschema("1954-01-05T00:00:00.123456")
+    t = Time.xmlschema('1954-01-05T00:00:00.123456')
     Oj.default_options = { :mode => :object, :time_format => :unix_zone }
     dump_and_load(t, false)
   end
@@ -165,7 +160,7 @@ class FileJuice < Minitest::Test
     if $ruby == 'ruby'
       assert_equal(%{{"^u":["Range",1,7,false]}}, json)
     else
-      assert(%{{"^O":"Range","begin":1,"end":7,"exclude_end?":false}} == json)
+      assert_equal(%{{"^O":"Range","begin":1,"end":7,"exclude_end?":false}}, json)
     end
     dump_and_load(1..7, false)
     dump_and_load(1..1, false)
@@ -229,8 +224,8 @@ class FileJuice < Minitest::Test
   end
 
   def dump_and_load(obj, trace=false)
-    filename = File.join(File.dirname(__FILE__), 'file_test.json')
-    File.open(filename, "w") { |f|
+    filename = File.join(__dir__, 'file_test.json')
+    File.open(filename, 'w') { |f|
       Oj.to_stream(f, obj, :indent => 2)
     }
     puts "\n*** file: '#{File.read(filename)}'" if trace

data/test/test_gc.rb

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << File.dirname(__FILE__)
+$LOAD_PATH << __dir__
 
 require 'helper'
 
@@ -14,13 +14,13 @@ class GCTest < Minitest::Test
       @child = child
     end
 
-    def to_hash()
-      { 'json_class' => "#{self.class}", 'x' => x, 'child' => child }
+    def to_hash
+      { 'json_class' => self.class.to_s, 'x' => x, 'child' => child }
     end
 
     def self.json_create(h)
       GC.start
-      self.new(h['x'], h['child'])
+      new(h['x'], h['child'])
     end
   end # Goo
 

data/test/test_generate.rb

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

data/test/test_hash.rb

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << File.dirname(__FILE__)
+$LOAD_PATH << __dir__
 
 require 'helper'
 
@@ -19,7 +19,7 @@ class HashTest < Minitest::Test
 
   def test_load
     obj = Oj.load(%|{"abc":3}|, :mode => :compat, :hash_class => Oj::EasyHash)
-    
+
     assert_equal(Oj::EasyHash, obj.class)
     assert_equal(3, obj['abc'])
     assert_equal(3, obj[:abc])
@@ -35,5 +35,5 @@ class HashTest < Minitest::Test
     assert_equal(Oj::EasyHash, obj.class)
     assert_equal(3, obj[:abc])
   end
-  
+
 end # HashTest

data/test/test_integer_range.rb

@@ -1,10 +1,10 @@
 #!/usr/bin/env ruby
-# encoding: utf-8
+# frozen_string_literal: true
 
-$: << File.dirname(__FILE__)
-$oj_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
+$LOAD_PATH << __dir__
+@oj_dir = File.dirname(File.expand_path(__dir__))
 %w(lib ext).each do |dir|
-  $: << File.join($oj_dir, dir)
+  $LOAD_PATH << File.join(@oj_dir, dir)
 end
 
 require 'minitest'
@@ -23,19 +23,19 @@ class IntegerRangeTest < Minitest::Test
   end
 
   def test_range
-    test = {s: 0, s2: -1, s3: 1, u: -2, u2: 2, u3: 9007199254740993}
+    test = {s: 0, s2: -1, s3: 1, u: -2, u2: 2, u3: 9_007_199_254_740_993}
     exp = '{"s":0,"s2":-1,"s3":1,"u":"-2","u2":"2","u3":"9007199254740993"}'
     assert_equal(exp, Oj.dump(test, integer_range: (-1..1)))
   end
 
   def test_bignum
-    test = {u: -10000000000000000000, u2: 10000000000000000000}
+    test = {u: -10_000_000_000_000_000_000, u2: 10_000_000_000_000_000_000}
     exp = '{"u":"-10000000000000000000","u2":"10000000000000000000"}'
     assert_equal(exp, Oj.dump(test, integer_range: (-1..1)))
   end
 
   def test_valid_modes
-    test = {safe: 0, unsafe: 9007199254740993}
+    test = {safe: 0, unsafe: 9_007_199_254_740_993}
     exp  = '{"safe":0,"unsafe":"9007199254740993"}'
 
     [:strict, :null, :compat, :rails, :custom].each do |mode|
@@ -49,7 +49,7 @@ class IntegerRangeTest < Minitest::Test
   end
 
   def test_modes_without_opt
-    test = {safe: 0, unsafe: 10000000000000000000}
+    test = {safe: 0, unsafe: 10_000_000_000_000_000_000}
     exp = '{"safe":0,"unsafe":10000000000000000000}'
 
     [:strict, :null, :compat, :rails, :custom].each do |mode|
@@ -63,7 +63,7 @@ class IntegerRangeTest < Minitest::Test
   end
 
   def test_accept_nil_and_false
-    test = {safe: 0, unsafe: 10000000000000000000}
+    test = {safe: 0, unsafe: 10_000_000_000_000_000_000}
     exp = '{"safe":0,"unsafe":10000000000000000000}'
 
     assert_equal(exp, Oj.dump(test, integer_range: nil))

data/test/test_null.rb

@@ -1,10 +1,10 @@
 #!/usr/bin/env ruby
-# encoding: UTF-8
+# frozen_string_literal: true
 
-$: << File.dirname(__FILE__)
-$oj_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
+$LOAD_PATH << __dir__
+@oj_dir = File.dirname(File.expand_path(__dir__))
 %w(lib ext).each do |dir|
-  $: << File.join($oj_dir, dir)
+  $LOAD_PATH << File.join(@oj_dir, dir)
 end
 
 require 'minitest'
@@ -52,16 +52,16 @@ class NullJuice < Minitest::Test
 
   def test_fixnum
     dump_and_load(0, false)
-    dump_and_load(12345, false)
-    dump_and_load(-54321, false)
+    dump_and_load(12_345, false)
+    dump_and_load(-54_321, false)
     dump_and_load(1, false)
   end
 
   def test_float
     dump_and_load(0.0, false)
-    dump_and_load(12345.6789, false)
+    dump_and_load(12_345.6789, false)
     dump_and_load(70.35, false)
-    dump_and_load(-54321.012, false)
+    dump_and_load(-54_321.012, false)
     dump_and_load(1.7775, false)
     dump_and_load(2.5024, false)
     dump_and_load(2.48e16, false)
@@ -78,7 +78,7 @@ class NullJuice < Minitest::Test
       assert(true)
       return
     end
-    assert(false, "*** expected an exception")
+    assert(false, '*** expected an exception')
   end
 
   def test_infinity_dump
@@ -90,7 +90,7 @@ class NullJuice < Minitest::Test
       assert(true)
       return
     end
-    assert(false, "*** expected an exception")
+    assert(false, '*** expected an exception')
   end
 
   def test_neg_infinity_dump
@@ -102,7 +102,7 @@ class NullJuice < Minitest::Test
       assert(true)
       return
     end
-    assert(false, "*** expected an exception")
+    assert(false, '*** expected an exception')
   end
 
   def test_string
@@ -116,13 +116,13 @@ class NullJuice < Minitest::Test
     opts = Oj.default_options
     Oj.default_options = { :ascii_only => false }
     unless 'jruby' == $ruby
-      dump_and_load("ぴーたー", false)
+      dump_and_load('ぴーたー', false)
     end
     Oj.default_options = { :ascii_only => true }
-    json = Oj.dump("ぴーたー")
+    json = Oj.dump('ぴーたー')
     assert_equal(%{"\\u3074\\u30fc\\u305f\\u30fc"}, json)
     unless 'jruby' == $ruby
-      dump_and_load("ぴーたー", false)
+      dump_and_load('ぴーたー', false)
     end
     Oj.default_options = opts
   end
@@ -198,7 +198,7 @@ class NullJuice < Minitest::Test
       assert(true)
       return
     end
-    assert(false, "*** expected an exception")
+    assert(false, '*** expected an exception')
   end
 
   def test_bignum_object
@@ -244,15 +244,13 @@ class NullJuice < Minitest::Test
   end
 
   def test_io_file
-    filename = File.join(File.dirname(__FILE__), 'open_file_test.json')
-    File.open(filename, 'w') { |f|
-      f.write(%{{
+    filename = File.join(__dir__, 'open_file_test.json')
+    File.write(filename, %{{
   "x":true,
   "y":58,
   "z": [1,2,3]
 }
 })
-    }
     f = File.new(filename)
     obj = Oj.strict_load(f)
     f.close()
@@ -366,7 +364,7 @@ class NullJuice < Minitest::Test
   def dump_and_load(obj, trace=false)
     json = Oj.dump(obj, :indent => 2)
     puts json if trace
-    loaded = Oj.strict_load(json);
+    loaded = Oj.strict_load(json)
     if obj.nil?
       assert_nil(loaded)
     else