cleaned up discourse_plugin

vendor/gems/discourse_plugin/spec/discourse_event_spec.rb

@@ -4,13 +4,35 @@ require 'ostruct'
 
 describe DiscourseEvent do
 
-  it "doesn't raise an error if we call an event that doesn't exist" do
-    DiscourseEvent.trigger(:missing_event)
+  describe "#events" do
+    it "defaults to {}" do
+      DiscourseEvent.instance_variable_set(:@events, nil)
+      DiscourseEvent.events.should == {}
+    end
+
+    describe "key value" do
+      it "defaults to an empty set" do
+        DiscourseEvent.events["event42"].should == Set.new
+      end
+    end
   end
 
-  context 'with an event to call' do
+  describe ".clear" do
+    it "clears out events" do
+      DiscourseEvent.events["event42"] << "test event"
+      DiscourseEvent.clear
+      DiscourseEvent.events.should be_empty
+    end
+  end
 
-    let(:harvey) { OpenStruct.new(name: 'Harvey Dent', job: 'District Attorney') }
+  context 'when calling events' do
+
+    let(:harvey) {
+      OpenStruct.new(
+        name: 'Harvey Dent',
+        job: 'District Attorney'
+      )
+    }
 
     before do
       DiscourseEvent.on(:acid_face) do |user|
@@ -18,30 +40,42 @@ describe DiscourseEvent do
       end
     end
 
-    it "doesn't raise an error" do
-      DiscourseEvent.trigger(:acid_face, harvey)      
+    context 'when event does not exist' do
+
+      it "does not raise an error" do
+        DiscourseEvent.trigger(:missing_event)
+      end
+
     end
 
-    it "chnages the name" do
-      DiscourseEvent.trigger(:acid_face, harvey)      
-      harvey.name.should == 'Two Face'
-    end
+    context 'when single event exists' do
 
-    context 'multiple events' do
-      before do
-        DiscourseEvent.on(:acid_face) do |user|
-          user.job =  'Supervillian'
-        end        
+      it "doesn't raise an error" do
         DiscourseEvent.trigger(:acid_face, harvey)
       end
 
-      it 'triggerred the email event' do
-        harvey.job.should == 'Supervillian'
-      end
-
-      it 'triggerred the username change' do
+      it "changes the name" do
+        DiscourseEvent.trigger(:acid_face, harvey)
         harvey.name.should == 'Two Face'
       end
+
+    end
+
+    context 'when multiple events exist' do
+
+      before do
+        DiscourseEvent.on(:acid_face) do |user|
+          user.job =  'Supervillian'
+        end
+
+        DiscourseEvent.trigger(:acid_face, harvey)
+      end
+
+      it 'triggers both events' do
+        harvey.job.should == 'Supervillian'
+        harvey.name.should == 'Two Face'
+      end
+
     end
 
   end