lita-wizard 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0713263b1b47aac699bf0284c07c6fae25ec0c64
-  data.tar.gz: 3f292a189e77cdb39200411c52506cd93518d973
+  metadata.gz: 88dc590dd6d02e8573709bc1036263435ae94e3e
+  data.tar.gz: 1d67f3027b98a6306684bf240a18a61bc2bbc674
 SHA512:
-  metadata.gz: 735eb9a13455a661717c83cc7f45abd4a086c7f970ff7399882d3dbf89717c9e3d197d7a30ca28e5f417f88652c73bacf9a386fc69b0eb1c53166cf299b7ccb9
-  data.tar.gz: 68b19a18b16e2b836882060236b60cf8c06c0f1eee5bd60ab93ba2c4fed758139646194145717b4a7773baa69e8ba76f8e6c64c2499389494418dc72d12444c3
+  metadata.gz: 3ebb360390d5e265a5ad3cab2c0e31e56c79360fc46571b1da14138d96f33270e50268e66c2b5cad30a13d9845c0370203e817223e9754dd2d7855cf9923a6fd
+  data.tar.gz: 12b092bb719569d7a7df2f63c4def931f549e4e41e77561a002053abae2dc3a12d61b471a4b3f6e0b1b9b36c8bf835e061138face5cc2dfb742d789c5aa2476a

data/.travis.yml

@@ -6,3 +6,5 @@ before_install:
   - gem update --system
 services:
   - redis-server
+notifications:
+  email: false

data/README.md

@@ -1,7 +1,7 @@
 # lita-wizard
 
 [![Build Status](https://travis-ci.org/cristianbica/lita-wizard.png?branch=master)](https://travis-ci.org/cristianbica/lita-wizard)
-[![Coverage Status](https://coveralls.io/repos/cristianbica/lita-wizard/badge.png)](https://coveralls.io/r/cristianbica/lita-wizard)
+[![Coverage Status](https://coveralls.io/repos/github/cristianbica/lita-wizard/badge.svg?branch=master)](https://coveralls.io/github/cristianbica/lita-wizard?branch=master)
 
 A lita extension to build wizards (surveys, standups, etc). You can instruct your chat bot to ask several questions, validate responses.
 

data/lib/lita/wizard.rb

@@ -182,6 +182,10 @@ class Lita::Wizard
       Lita.redis["pending-wizard-#{user_id.downcase}"]
     end
 
+    def cancel_wizard(user_id)
+      Lita.redis.del "pending-wizard-#{user_id.downcase}"
+    end
+
     def step(name, options = {})
       steps << OpenStruct.new(options.merge(name: name))
     end

data/lita-wizard.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-wizard"
-  spec.version       = "1.0.0"
+  spec.version       = "1.0.1"
   spec.authors       = ["Cristian Bica"]
   spec.email         = ["cristian.bica@gmail.com"]
   spec.description   = "A lita extension to build wizards"

data/spec/lita/integration_spec.rb

@@ -19,7 +19,7 @@ describe Lita::Handlers::Wizard, lita: true, lita_handler: true do
   it "should reply with the initial message if it's the first step" do
     message = Lita::Message.new(robot, "test", source)
     TestWizard.start(robot, message)
-    expect(replies.first).to eq("initial message")
+    expect(replies.first).to match(/^initial message/)
   end
 
   it "should call the start_wizard method if it's the first step" do
@@ -101,7 +101,6 @@ describe Lita::Handlers::Wizard, lita: true, lita_handler: true do
   it "should abort the wizard if requested by the user" do
     store_wizard(current_step_index: 0)
     send_message("abort", privately: true)
-    puts replies.inspect
     expect(Lita::Wizard.pending_wizard?("1")).to be_falsey
   end
 
@@ -117,6 +116,11 @@ describe Lita::Handlers::Wizard, lita: true, lita_handler: true do
     send_message("abort", privately: true)
   end
 
-
+  it "should abort the wizard if it cannot identify the current step" do
+    expect_any_instance_of(TestWizard).to receive(:destroy)
+    store_wizard(current_step_index: 42)
+    send_message("one", privately: true)
+    expect(replies.last).to match /Some error occured. Aborting/
+  end
 
 end

data/spec/lita/wizard_spec.rb

@@ -7,15 +7,15 @@ describe Lita::Wizard, lita: true do
   let(:message) { Lita::Message.new(robot, "test", Lita::Source.new(user: user, room: room, private_message: false)) }
 
   context "starting a wizard" do
-    after { Lita::Wizard.start(robot, message) }
     it "should check if a pending wizard exists for that user" do
       expect(Lita::Wizard).to receive(:pending_wizard?).with("42").and_return(true)
-
+      Lita::Wizard.start(robot, message)
     end
 
     it "should try to start a new wizard if a pending one exists" do
       allow(Lita::Wizard).to receive(:pending_wizard?).and_return(true)
       expect(Lita::Wizard).to receive(:new).never
+      Lita::Wizard.start(robot, message)
     end
 
     it "should initialize a new wizard class an advance" do
@@ -23,6 +23,12 @@ describe Lita::Wizard, lita: true do
       expect(mocked_wizard).to receive(:advance)
       allow(Lita::Wizard).to receive(:pending_wizard?).and_return(false)
       expect(Lita::Wizard).to receive(:new).and_return(mocked_wizard)
+      Lita::Wizard.start(robot, message)
+    end
+
+    it "should be able to start a wizard from a handler" do
+      expect(TestWizard).to receive(:start).with(robot, "message", "meta")
+      Lita::Handler.new(robot).start_wizard(TestWizard, "message", "meta")
     end
   end
 
@@ -53,13 +59,14 @@ describe Lita::Wizard, lita: true do
     end
 
     it "should return nil if data is malformed" do
+      Lita.redis["pending-wizard-42"] = "x"
       expect(Lita::Wizard.restore(robot, message)).to be_nil
     end
 
-    it "should restore the wizard", :focus do
+    it "should restore the wizard" do
       data = { class: "TestWizard", arg1: "43" }
       Lita.redis["pending-wizard-42"] = data.to_json
-      expect(TestWizard).to receive(:new)#.with(robot, message, data)
+      expect(TestWizard).to receive(:new)
       Lita::Wizard.restore(robot, message)
     end
   end
@@ -74,4 +81,22 @@ describe Lita::Wizard, lita: true do
       expect(Lita::Wizard.pending_wizard?("42")).to be_falsey
     end
   end
+
+  context "deleting a wizard" do
+    it "should delete a wizard" do
+      Lita.redis["pending-wizard-42"] = "1"
+      expect(Lita::Wizard.pending_wizard?("42")).to be_truthy
+      Lita::Wizard.cancel_wizard("42")
+      expect(Lita::Wizard.pending_wizard?("42")).to be_falsey
+    end
+  end
+
+  context "accessing answers" do
+    it "should be able to get a specific answer" do
+      data = { class: "TestWizard", values: %w(a b c d) }
+      Lita.redis["pending-wizard-42"] = data.to_json
+      wizard = Lita::Wizard.restore(robot, message)
+      expect(wizard.value_for(:two)).to eq("b")
+    end
+  end
 end

data/spec/support/test_wizard.rb

@@ -24,15 +24,15 @@ class TestWizard < Lita::Wizard
        options: %w(one two)
 
   def initial_message
-    "initial message"
+    "initial message" << super
   end
 
   def abort_message
-    "abort message"
+    "abort message" << super
   end
 
   def final_message
-    "final message"
+    "final message" << super
   end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-wizard
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Cristian Bica