sheldon 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 88cb9db2a8a8e0a2529f64d4c12a350ef2903447
-  data.tar.gz: 57864a0dc3aa58a04e8ee3e01652734716293e9d
+  metadata.gz: 73892f48dbd187ec12c86c61e4e4a7db3ba3fd28
+  data.tar.gz: 9b7c3048b7cd07c67fdcfc4d705ed559f0e30df3
 SHA512:
-  metadata.gz: f5de4d60afd7ef3b1a8ba270f6f0f141d97ed9d12c3626802f584fd7f24a5f89043ce571bbe521b7473a37ec08dc04469c67284d2c0703287edbca2b23961af7
-  data.tar.gz: 04ee09898119ee61ab41425767e15f50dba39622844e13c4cec3bec00bfcfcc6736f54ea60b510283cfc7e53d64a75472b5e6bcdbda79b53f7bce6045b1c69b2
+  metadata.gz: 6e90b362ed860b5de2a01629144c827628af88d70817864d6fe8b47106e6acd640547c627e3279ab79ba0e1862b080b5571ef8cabc3c33c405018f10716c3d89
+  data.tar.gz: 7c94643012c59f402686f4d2c2bddf0ae81361d2dafa916a3f0f7cdaf4814c9a22f25edf1d94c04f3b55082fee582d75ec5ea4ac9dc2e904cafb63aca99bf6bc

data/lib/sheldon/brain.rb

@@ -9,6 +9,10 @@ class Brain
 
 
   def learn(recall_cue, abs_learn_path)
+    raise "recall cue cannot be empty." if recall_cue.strip.empty?
+    raise "This cue has already been used." if has_cue?(recall_cue)
+    raise "Unable to find a file or folder at #{abs_learn_path}" unless File.exists?(abs_learn_path)
+
     brain_path = brain_path_for_cue(recall_cue)
     FileUtils.mkdir_p(brain_path)
     FileUtils.mv(abs_learn_path, brain_path)
@@ -29,8 +33,8 @@ class Brain
     entry = memory.recall(recall_cue)
     brain_path = brain_path_for_cue(recall_cue)
     destination_path = add_home(entry[:filepath])
+    FileUtils.rm_r(destination_path) if recalled?(recall_cue)
     FileUtils.rm_r(brain_path)
-    FileUtils.rm_r(destination_path)
 
     memory.forget(recall_cue)
   end

data/lib/sheldon/memory.rb

@@ -15,7 +15,7 @@ class Memory
   end
 
   def recall(recall_cue)
-    raise "no entry for cue" unless has_cue?(recall_cue)
+    raise "no entry for cue '#{recall_cue}'" unless has_cue?(recall_cue)
     @database.transaction { @database[recall_cue] }
   end
 

data/lib/sheldon/sheldon.rb

@@ -1,7 +1,7 @@
 require "fileutils"
 
 class Sheldon
-  VERSION = "0.4.0".freeze
+  VERSION = "0.4.1".freeze
   attr_reader :brain, :builder
 
   def initialize(sheldon_data_dir, opts = {})
@@ -18,21 +18,14 @@ class Sheldon
   end
 
   def learn(recall_cue, abs_learn_path)
-    raise "recall cue cannot be empty." if recall_cue.strip.empty?
-    if brain.has_cue?(recall_cue)
-      raise "This cue has already been used."
-    else
-      brain.learn(recall_cue, abs_learn_path)
-    end
+    brain.learn(recall_cue, abs_learn_path)
   end
 
   def recall(recall_cue)
-    raise "Cue '#{recall_cue}' could not be found." unless brain.has_cue?(recall_cue)
     brain.recall(recall_cue)
   end
 
   def forget(recall_cue)
-    raise "Cue '#{recall_cue}' could not be found." unless brain.has_cue?(recall_cue)
     brain.forget(recall_cue)
   end
 
@@ -41,7 +34,6 @@ class Sheldon
   end
 
   def recalled?(recall_cue)
-    raise "Cue '#{recall_cue}' could not be found." unless brain.has_cue?(recall_cue)
     brain.recalled?(recall_cue)
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sheldon
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - David Jones
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-27 00:00:00.000000000 Z
+date: 2017-06-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor