friends 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 018ee9e9043561fd60353d6db09a37ca42822a8d
-  data.tar.gz: 0591a729fb07a42bcd4ea872e280d849f6eb1bea
+  metadata.gz: fb75dfa2c1651767156d0c51f6a48f17cdcd6014
+  data.tar.gz: 36e29e9c4bb81409a7ed39bc6d9e5d2f7c396b41
 SHA512:
-  metadata.gz: fa12a4235f2aa2b5d24f6a507e872d8509caa06c356de44cf564c528e73767f181ea8f7d5ec459e48b95c221992d767f5a00844e87eeab9894bd05e82231859b
-  data.tar.gz: a8543253d63c267ac33a7dbb9535f6465f97df093528ac181d84325bc3b14728a98bd88366adf6191c2a3db075e683d0752d76909c401fe326b7dd14f1d06e83
+  metadata.gz: 2c8d25accc7fe8beacd6dda24dce6b0c0c4f8d515c2c7ca0f6462cf3e90ec05136b4111b3791086cf7bc4a54b8ad04df4a6f059efb25642005d3a2c2cb40447c
+  data.tar.gz: 8d54d8b6a133e620442d5176e18d9c496ad4669cb8d2c140b947a1d76664e46eedf552ea50f53e2dc7adae79f029881732c4725d3053dba3c091b3bfc16ef0bc

data/.gitignore

@@ -14,3 +14,4 @@
 mkmf.log
 *.gem
 ideas.txt
+TODO.txt

data/lib/friends/activity.rb

@@ -51,33 +51,39 @@ module Friends
     # Modify the description to turn inputted friend names
     # (e.g. "Jacob" or "Jacob Evelyn") into full asterisk'd names
     # (e.g. "**Jacob Evelyn**")
+    # @param introvert [Introvert] for use in aggregate computations
     # @param friends [Array] list of friends to highlight in the description
     # @raise [FriendsError] if more than one friend matches a part of the
     #   description
-    def highlight_friends(friends:)
+    def highlight_friends(introvert:, friends:)
       # Map each friend to a list of all possible regexes for that friend.
       friend_regexes = {}
-      friends.each { |f| friend_regexes[f.name] = regexes_for_name(f.name) }
+      friends.each { |f| friend_regexes[f] = regexes_for_name(f.name) }
 
-      # Create hash mapping regex to friend name. Note that because two friends
-      # may have the same regex (e.g. /John/), we need to store the names in an
+      # Create hash mapping regex to friend. Note that because two friends may
+      # have the same regex (e.g. /John/), we need to store the names in an
       # array since there may be more than one. We also iterate through the
       # regexes to add the most important regexes to the hash first, so
       # "Jacob Evelyn" takes precedence over all instances of "Jacob" (since
       # Ruby hashes are ordered).
       regex_map = Hash.new { |h, k| h[k] = [] }
       while !friend_regexes.empty?
-        friend_regexes.each do |friend_name, regex_list|
-          regex_map[regex_list.shift] << friend_name
-          friend_regexes.delete(friend_name) if regex_list.empty?
+        friend_regexes.each do |friend, regex_list|
+          regex_map[regex_list.shift] << friend
+          friend_regexes.delete(friend) if regex_list.empty?
         end
       end
 
-      # Go through the description and substitute in full, asterisk'd names for
+      # Go through the description and substitute full, asterisk'd names for
       # anything that matches a friend's name.
       new_description = description.clone
-      regex_map.each do |regex, names|
-        new_description.gsub!(regex, "**#{names.first}**") if names.size == 1
+      regex_map.each do |regex, friends|
+        if friends.size > 1 # If there are multiple matches, find best friend.
+          introvert.set_n_activities!
+          friends.sort_by! { |friend| -friend.n_activities }
+        end
+
+        new_description.gsub!(regex, "**#{friends.first.name}**")
       end
 
       @description = new_description

data/lib/friends/friend.rb

@@ -30,6 +30,13 @@ module Friends
       "#{SERIALIZATION_PREFIX}#{name}"
     end
 
+    # The number of activities this friend is in. This is for internal use only
+    # and is set by the Introvert as needed.
+    attr_writer :n_activities
+    def n_activities
+      @n_activities || 0
+    end
+
     private
 
     # Default sorting for an array of friends is alphabetical.

data/lib/friends/introvert.rb

@@ -79,7 +79,7 @@ module Friends
         raise FriendsError, e
       end
 
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: self, friends: friends)
       activities << activity
       clean # Write a cleaned file.
 
@@ -102,24 +102,15 @@ module Friends
         raise FriendsError, "Favorites limit must be positive or unlimited"
       end
 
-      # Construct a hash of friend name to frequency of appearance.
-      freq_table = Hash.new { |h, k| h[k] = 0 }
-      activities.each do |activity|
-        activity.friend_names.each do |friend_name|
-          freq_table[friend_name] += 1
-        end
-      end
-
-      # Remove names that are not in the friends list.
-      freq_table.select! { |name, _| friend_with_exact_name(name) }
+      set_n_activities! # Set n_activities for all friends.
 
       # Sort the results, with the most favorite friend first.
-      results = freq_table.sort_by { |_, count| -count }
+      results = friends.sort_by { |friend| -friend.n_activities }
 
       # If we need to, trim the list.
       results = results.take(limit) unless limit.nil?
 
-      results.map(&:first)
+      results.map(&:name)
     end
 
     # List all activity details.
@@ -153,6 +144,23 @@ module Friends
       acts.map(&:display_text)
     end
 
+    # Sets the n_activities field on each friend.
+    def set_n_activities!
+      # Construct a hash of friend name to frequency of appearance.
+      freq_table = Hash.new { |h, k| h[k] = 0 }
+      activities.each do |activity|
+        activity.friend_names.each do |friend_name|
+          freq_table[friend_name] += 1
+        end
+      end
+
+      # Remove names that are not in the friends list.
+      freq_table.each do |name, count|
+        friend = friend_with_exact_name(name)
+        friend.n_activities = count if friend # Do nothing if name not valid.
+      end
+    end
+
     private
 
     # Gets the list of friends as read from the file.

data/lib/friends/version.rb

@@ -1,3 +1,3 @@
 module Friends
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/test/activity_spec.rb

@@ -74,7 +74,10 @@ describe Friends::Activity do
     let(:friend2) { Friends::Friend.new(name: "John Cage") }
     let(:friends) { [friend1, friend2] }
     let(:description) { "Lunch with #{friend1.name} and #{friend2.name}." }
-    subject { activity.highlight_friends(friends: friends) }
+    let(:introvert) { Minitest::Mock.new }
+    subject do
+      activity.highlight_friends(introvert: introvert, friends: friends)
+    end
 
     it "finds all friends" do
       subject
@@ -87,7 +90,7 @@ describe Friends::Activity do
         date_s: Date.today.to_s,
         description: "Lunch with Elizabeth and John."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
       activity.description.
         must_equal "Lunch with **#{friend1.name}** and **#{friend2.name}**."
     end
@@ -97,27 +100,17 @@ describe Friends::Activity do
         date_s: Date.today.to_s,
         description: "Lunch with elizabeth cady stanton."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
       activity.description.
         must_equal "Lunch with **Elizabeth Cady Stanton**."
     end
 
-    it "ignores when there are multiple matches" do
-      friend2.name = "Elizabeth II"
-      activity = Friends::Activity.new(
-        date_s: Date.today.to_s,
-        description: "Dinner with Elizabeth."
-      )
-      activity.highlight_friends(friends: friends)
-      activity.description.must_equal "Dinner with Elizabeth." # No match found.
-    end
-
     it "ignores when at beginning of word" do
       activity = Friends::Activity.new(
         date_s: Date.today.to_s,
         description: "Field trip to the Johnson Co."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
 
       # No match found.
       activity.description.must_equal "Field trip to the Johnson Co."
@@ -128,7 +121,7 @@ describe Friends::Activity do
         date_s: Date.today.to_s,
         description: "Field trip to the JimJohnJames Co."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
 
       # No match found.
       activity.description.must_equal "Field trip to the JimJohnJames Co."
@@ -139,7 +132,7 @@ describe Friends::Activity do
         date_s: Date.today.to_s,
         description: "Field trip to the JimJohn Co."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
 
       # No match found.
       activity.description.must_equal "Field trip to the JimJohn Co."
@@ -150,7 +143,7 @@ describe Friends::Activity do
         date_s: Date.today.to_s,
         description: "Dinner with **Elizabeth Cady Stanton."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
 
       # No match found.
       activity.description.must_equal "Dinner with **Elizabeth Cady Stanton."
@@ -164,11 +157,31 @@ describe Friends::Activity do
         # match, because the Elizabeth isn't surrounded by asterisks.
         description: "Dinner with Elizabeth**."
       )
-      activity.highlight_friends(friends: friends)
+      activity.highlight_friends(introvert: introvert, friends: friends)
 
       # No match found.
       activity.description.must_equal "Dinner with Elizabeth**."
     end
+
+    it "chooses the better friend when there are multiple matches" do
+      friend2.name = "Elizabeth II"
+      activity = Friends::Activity.new(
+        date_s: Date.today.to_s,
+        description: "Dinner with Elizabeth."
+      )
+
+      # Pretend the introvert sets the friends' n_activities values.
+      introvert.expect(:set_n_activities!, nil)
+      friend1.n_activities = 5
+      friend2.n_activities = 7
+
+      activity.highlight_friends(introvert: introvert, friends: friends)
+
+      # Pick the friend with more activities.
+      activity.description.must_equal "Dinner with **Elizabeth II**."
+
+      # introvert.verify
+    end
   end
 
   describe "#friend_names" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: friends
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Jacob Evelyn
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-28 00:00:00.000000000 Z
+date: 2015-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gli