crimp 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: f951279d5308025d1f3184b1a823a1577d13ebad
-  data.tar.gz: 9444429cbc53f722f053815ced9db784e29a5374
+SHA256:
+  metadata.gz: 933033b434c42dd1f1bddcae7dab3a28ca3cd59ea6f3c1c9099f7b7b94f16eb4
+  data.tar.gz: a48ac5e269d70ac652422d4959372acd2937df77ae3a970887642b67ee5ad4af
 SHA512:
-  metadata.gz: f91307d9094be916d32b3e95ca5c620db68979ec5d3eb465efaa2e0876df96543d51a43fad76331d20f04baa34ed254ab355587886ad66db7453badff1afcf3b
-  data.tar.gz: 092f330b087472fd4ebda5653ccb6ba86857597d205e71d15636562fd2ec2b2626cce2598b733d74acd7a7a637796148526f43d91e0bb569a67aee494ea3c98b
+  metadata.gz: b270bad70530524c0110fa9e6e80e29f95751802d7732cb5c683aa3e610d4674485ed4196384f8b6312629dc2cd64d2a60de0cce301a2dd8520f38be9735f966
+  data.tar.gz: 269d0964c13056187ded2be18fc748bbbe9fc9e85b8b5e67d165aa27e9a07c7ff9d8681ccfddd6bd70bbc172514e6976e881d14b9df3da2783a7e04d6d2d0e99

data/.ruby-version

@@ -1 +1 @@
-jruby-1.7.17
+jruby-1.7.23

data/crimp.gemspec

@@ -7,7 +7,7 @@ Gem::Specification.new do |spec|
   spec.name          = "crimp"
   spec.version       = Crimp::VERSION
   spec.authors       = ["BBC News"]
-  spec.email         = ["FutureMediaNewsRubyGems@bbc.co.uk"]
+  spec.email         = ["D&ENewsFrameworksTeam@bbc.co.uk"]
   spec.summary       = %q{Creating an md5 hash of a number, string, array, or hash in Ruby}
   spec.description   = <<-EOS.gsub /^\s+/, ""
                          Shamelessly lifted from http://stackoverflow.com/questions/6461812/creating-an-md5-hash-of-a-number-string-array-or-hash-in-ruby

data/lib/crimp.rb

@@ -1,6 +1,13 @@
 require 'crimp/version'
 require 'digest'
 
+class Numeric
+  # see http://patshaughnessy.net/2014/1/9/how-big-is-a-bignum
+  def bignum?
+    self >= 4611686018427387904
+  end
+end
+
 module Crimp
   def self.signature(obj)
     Digest::MD5.hexdigest stringify(obj)
@@ -44,6 +51,24 @@ module Crimp
   end
 
   def self.to_string(obj)
-    "#{obj}#{obj.class}"
+    "#{obj}#{legacy_class(obj)}"
+  end
+
+  # This is for legacy/compatibilty reason:
+  #
+  # Ruby 2.1
+  # 2.class => Fixnum
+  # Ruby >= 2.4
+  # 2.class => Integer
+  #
+  # Say you have a huge number of stored keys and you migrate your app from 2.1 to >= 2.4
+  # this would cause a change of the signature for a subset of the keys which would be hard
+  # to debug especially for nested data structures.
+  #
+  def self.legacy_class(obj)
+    return obj.class unless obj.is_a?(Numeric)
+    return 'Float'   if obj.is_a?(Float)
+    return 'Bignum'  if obj.bignum?
+    'Fixnum'
   end
 end

data/lib/crimp/version.rb

@@ -1,3 +1,3 @@
 module Crimp
-  VERSION = '0.1.2'.freeze
+  VERSION = '0.2.0'.freeze
 end

data/spec/crimp_spec.rb

@@ -1,17 +1,16 @@
 require 'spec_helper'
 
 describe Crimp do
-  let (:hash) { { a: { b: 'b', c: 'c' }, d: 'd' } }
-  let (:hash_unordered) { { d: 'd', a: { c: 'c', b: 'b' } } }
-  let (:array) { [1, 2, 3, [4, [5, 6]]] }
-  let (:array_unordered) { [3, 2, 1, [[5, 6], 4]] }
+  let(:hash) { { a: { b: 'b', c: 'c' }, d: 'd' } }
+  let(:hash_with_numbers) { { a: { b: 1, c: 3.14 }, d: 'd' } }
+  let(:hash_unordered) { { d: 'd', a: { c: 'c', b: 'b' } } }
+  let(:array) { [1, 2, 3, [4, [5, 6]]] }
+  let(:array_unordered) { [3, 2, 1, [[5, 6], 4]] }
 
   describe '.signature' do
     context 'given a Hash' do
-      specify { expect(subject.signature(hash)).to be_a String }
-
       it 'returns MD5 hash of stringified Hash' do
-        expect(subject.signature(hash)).to eq(Digest::MD5.hexdigest(subject.stringify(hash)))
+        expect(subject.signature(hash)).to eq('68d07febc4f47f56fa6ef5de063a77b1')
       end
 
       it 'does not modify original hash' do
@@ -24,11 +23,15 @@ describe Crimp do
       end
     end
 
-    context 'given an Array' do
-      specify { expect(subject.signature(array)).to be_a String }
+    context 'Given an hash with numbers' do
+      it 'returns MD5 hash of stringified hash' do
+        expect(subject.signature(hash_with_numbers)).to eq 'b1fec09904b6ff36c92e3bd48234def7'
+      end
+    end
 
+    context 'given an Array' do
       it 'returns MD5 hash of stringified Array' do
-        expect(subject.signature(array)).to eq(Digest::MD5.hexdigest(subject.stringify(array)))
+        expect(subject.signature(array)).to eq('4dc4e1161c9315db0bc43c0201b3ec05')
       end
 
       it 'does not modify original array' do
@@ -40,12 +43,28 @@ describe Crimp do
         expect(original_array).to eq(expected_array)
       end
     end
+
+    context 'Given an integer' do
+      it 'returns MD5 hash of an Integer' do
+        expect(subject.signature(123)).to eq '519d3381631851be66711f6d7dfbb4f8'
+      end
+    end
+
+    context 'Given an Bignum' do
+      it 'returns MD5 hash of a Bignum' do
+        expect(subject.signature(9999999999999999999)).to eq 'f00e75abca720e18fd4213e2a6de96c6'
+      end
+    end
+
+    context 'Given an float' do
+      it 'returns MD5 hash of a Float' do
+        expect(subject.signature(3.14)).to eq 'b07d506e3701fddd083ae9095df43218'
+      end
+    end
   end
 
   describe '.stringify' do
     context 'given a Hash' do
-      specify { expect(subject.stringify(hash)).to be_a String }
-
       it 'returns equal strings for differently ordered hashes' do
         expect(subject.stringify(hash)).to eq(subject.stringify(hash_unordered))
       end

metadata

@@ -1,167 +1,167 @@
 --- !ruby/object:Gem::Specification
 name: crimp
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - BBC News
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-25 00:00:00.000000000 Z
+date: 2018-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
-  name: bundler
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
 - !ruby/object:Gem::Dependency
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: rake
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: rspec
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: rake-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: rake-rspec
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: rspec-nc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: rspec-nc
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: guard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: guard
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: guard-rspec
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: pry
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: pry-remote
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: pry-remote
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
+  name: pry-nav
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  name: pry-nav
-  prerelease: false
   type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: |
   Shamelessly lifted from http://stackoverflow.com/questions/6461812/creating-an-md5-hash-of-a-number-string-array-or-hash-in-ruby
   All credit should go to http://stackoverflow.com/users/394282/luke
 email:
-- FutureMediaNewsRubyGems@bbc.co.uk
+- D&ENewsFrameworksTeam@bbc.co.uk
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .ruby-version
-- .travis.yml
+- ".gitignore"
+- ".ruby-version"
+- ".travis.yml"
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -177,24 +177,24 @@ homepage: ''
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project:
-rubygems_version: 2.1.9
-signing_key:
+rubyforge_project: 
+rubygems_version: 2.7.6
+signing_key: 
 specification_version: 4
 summary: Creating an md5 hash of a number, string, array, or hash in Ruby
 test_files: