memoist 0.16.1 → 0.16.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af4a5344777f88d07223d20fae69de50867916ed579032325013f4c5160b37c7
-  data.tar.gz: 5997e635a721a2bfcf779592e61ea1bda7361208c9182cdbd8f6a288bf32e78c
+  metadata.gz: '058dae917e46d13306497f0bcd18df607cb1d5dca2d85ee6f9cbc18739172e27'
+  data.tar.gz: c0874b135f70db573a863e4b54bb141905c003d9e451d7c3637d8f67c96a0696
 SHA512:
-  metadata.gz: ff876e4b317f71b26b6c3505977fb707d839e97820852fc97d117cc58d0f00c3d1f2a952a000062ef63af2c1289ad6852c02a68e11f8342d73c95dc42976fe6c
-  data.tar.gz: 15f0e432b722826bff600c642be765dcc4f6e2a7ae8b32b2f907a945cbbcbf97b87f9589262141430646d8880570223ef592e0f2cde92b21cc0dbb8263a3b3b0
+  metadata.gz: 409f37c0861015cc81704bf4fbbb1002d7a011a5f3dccb0e1430baf8b5eac601f5caae99cf4e1634fe044ea01123c1a0319692fad6d64c848026329f9a4dfa7a
+  data.tar.gz: 7f3134101c02c37ce88fd6a814c6bce19f5ffc4085fcfbb08e22ce720f125b7f8c2c2644c6a694cf4bb5dc4feb40974785f1209d598c18ba098bc055bca33146

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [Unreleased](https://github.com/matthewrudy/memoist/tree/HEAD)
+
+[Full Changelog](https://github.com/matthewrudy/memoist/compare/v0.16.1...HEAD)
+
+**Merged pull requests:**
+
+- Fix regression introduced by frozen symbol fix [\#86](https://github.com/matthewrudy/memoist/pull/86) ([sebjacobs](https://github.com/sebjacobs))
+
 ## [v0.16.1](https://github.com/matthewrudy/memoist/tree/v0.16.1) (2019-11-08)
 
 [Full Changelog](https://github.com/matthewrudy/memoist/compare/v0.16.0...v0.16.1)

data/lib/memoist.rb

@@ -38,12 +38,10 @@ module Memoist
   end
 
   def self.escape_punctuation(string)
-    string = string.to_s
+    string = string.is_a?(String) ? string.dup : string.to_s.dup
 
     return string unless string.end_with?('?'.freeze, '!'.freeze)
 
-    string = string.dup if string.frozen?
-
     # A String can't end in both ? and !
     if string.sub!(/\?\Z/, '_query'.freeze)
     else

data/lib/memoist/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Memoist
-  VERSION = '0.16.1'.freeze
+  VERSION = '0.16.2'.freeze
 end

data/test/memoist_test.rb

@@ -67,6 +67,12 @@ class MemoistTest < Minitest::Test
 
     memoize :name, :age
 
+    def age?
+      @counter.call(:age?)
+      true
+    end
+    memoize 'age?'
+
     def sleep(hours = 8)
       @counter.call(:sleep)
       hours
@@ -272,6 +278,13 @@ class MemoistTest < Minitest::Test
     @person.unmemoize_all
   end
 
+  def test_memoization_when_memoize_is_called_with_punctuated_string
+    assert_equal true, @person.age?
+
+    @person.memoize_all
+    @person.unmemoize_all
+  end
+
   def test_memoization_flush_with_punctuation
     assert_equal true, @person.name?
     @person.flush_cache(:name?)
@@ -344,11 +357,11 @@ class MemoistTest < Minitest::Test
   end
 
   def test_all_memoized_structs
-    # Person             memoize :age, :is_developer?, :memoize_protected_test, :name, :name?, :sleep, :update, :update_attributes
+    # Person             memoize :age, :age?, :is_developer?, :memoize_protected_test, :name, :name?, :sleep, :update, :update_attributes
     # Student < Person   memoize :name, :identifier => :student
     # Teacher < Person   memoize :seniority
 
-    expected = %w[age is_developer? memoize_protected_test name name? sleep update update_attributes]
+    expected = %w[age age? is_developer? memoize_protected_test name name? sleep update update_attributes]
     structs = Person.all_memoized_structs
     assert_equal expected, structs.collect(&:memoized_method).collect(&:to_s).sort
     assert_equal '@_memoized_name', structs.detect { |s| s.memoized_method == :name }.ivar

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: memoist
 version: !ruby/object:Gem::Version
-  version: 0.16.1
+  version: 0.16.2
 platform: ruby
 authors:
 - Joshua Peek
@@ -19,7 +19,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-08 00:00:00.000000000 Z
+date: 2019-12-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips