RubyGems - bullet - Versions diffs - 5.5.0 → 6.1.5 - Mend

bullet 5.5.0 → 6.1.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (123) hide show

checksums.yaml +5 -5
data/.github/workflows/main.yml +66 -0
data/CHANGELOG.md +83 -1
data/Gemfile +6 -1
data/Gemfile.mongoid-7.0 +15 -0
data/Gemfile.rails-4.0 +1 -1
data/Gemfile.rails-4.1 +1 -1
data/Gemfile.rails-4.2 +1 -1
data/Gemfile.rails-5.0 +2 -2
data/Gemfile.rails-5.1 +15 -0
data/Gemfile.rails-5.2 +15 -0
data/Gemfile.rails-6.0 +15 -0
data/Gemfile.rails-6.1 +15 -0
data/Guardfile +2 -2
data/Hacking.md +1 -1
data/README.md +58 -37
data/Rakefile +20 -21
data/bullet.gemspec +21 -15
data/lib/bullet/active_job.rb +13 -0
data/lib/bullet/active_record4.rb +15 -34
data/lib/bullet/active_record41.rb +14 -31
data/lib/bullet/active_record42.rb +23 -36
data/lib/bullet/active_record5.rb +178 -158
data/lib/bullet/active_record52.rb +251 -0
data/lib/bullet/active_record60.rb +278 -0
data/lib/bullet/active_record61.rb +278 -0
data/lib/bullet/bullet_xhr.js +64 -0
data/lib/bullet/dependency.rb +60 -32
data/lib/bullet/detector/association.rb +41 -32
data/lib/bullet/detector/base.rb +2 -0
data/lib/bullet/detector/counter_cache.rb +25 -17
data/lib/bullet/detector/n_plus_one_query.rb +38 -24
data/lib/bullet/detector/unused_eager_loading.rb +35 -26
data/lib/bullet/detector.rb +2 -0
data/lib/bullet/ext/object.rb +9 -5
data/lib/bullet/ext/string.rb +3 -1
data/lib/bullet/mongoid4x.rb +9 -8
data/lib/bullet/mongoid5x.rb +9 -8
data/lib/bullet/mongoid6x.rb +13 -12
data/lib/bullet/mongoid7x.rb +57 -0
data/lib/bullet/notification/base.rb +28 -25
data/lib/bullet/notification/counter_cache.rb +3 -1
data/lib/bullet/notification/n_plus_one_query.rb +8 -7
data/lib/bullet/notification/unused_eager_loading.rb +8 -7
data/lib/bullet/notification.rb +4 -1
data/lib/bullet/notification_collector.rb +2 -1
data/lib/bullet/rack.rb +66 -35
data/lib/bullet/registry/association.rb +2 -0
data/lib/bullet/registry/base.rb +2 -0
data/lib/bullet/registry/object.rb +2 -0
data/lib/bullet/registry.rb +2 -0
data/lib/bullet/stack_trace_filter.rb +50 -20
data/lib/bullet/version.rb +3 -2
data/lib/bullet.rb +136 -55
data/lib/generators/bullet/install_generator.rb +48 -0
data/perf/benchmark.rb +19 -22
data/rails/init.rb +2 -0
data/spec/bullet/detector/association_spec.rb +6 -4
data/spec/bullet/detector/base_spec.rb +2 -0
data/spec/bullet/detector/counter_cache_spec.rb +17 -15
data/spec/bullet/detector/n_plus_one_query_spec.rb +75 -46
data/spec/bullet/detector/unused_eager_loading_spec.rb +45 -26
data/spec/bullet/ext/object_spec.rb +18 -11
data/spec/bullet/ext/string_spec.rb +7 -5
data/spec/bullet/notification/base_spec.rb +38 -39
data/spec/bullet/notification/counter_cache_spec.rb +5 -3
data/spec/bullet/notification/n_plus_one_query_spec.rb +20 -5
data/spec/bullet/notification/unused_eager_loading_spec.rb +9 -3
data/spec/bullet/notification_collector_spec.rb +12 -10
data/spec/bullet/rack_spec.rb +200 -48
data/spec/bullet/registry/association_spec.rb +12 -10
data/spec/bullet/registry/base_spec.rb +22 -20
data/spec/bullet/registry/object_spec.rb +6 -4
data/spec/bullet_spec.rb +64 -33
data/spec/integration/{active_record4 → active_record}/association_spec.rb +238 -223
data/spec/integration/counter_cache_spec.rb +26 -44
data/spec/integration/mongoid/association_spec.rb +53 -65
data/spec/models/address.rb +2 -0
data/spec/models/attachment.rb +5 -0
data/spec/models/author.rb +2 -0
data/spec/models/base_user.rb +2 -0
data/spec/models/category.rb +2 -0
data/spec/models/city.rb +2 -0
data/spec/models/client.rb +4 -0
data/spec/models/comment.rb +3 -1
data/spec/models/company.rb +2 -0
data/spec/models/country.rb +2 -0
data/spec/models/deal.rb +5 -0
data/spec/models/document.rb +4 -2
data/spec/models/entry.rb +2 -0
data/spec/models/firm.rb +3 -0
data/spec/models/folder.rb +2 -0
data/spec/models/group.rb +4 -0
data/spec/models/mongoid/address.rb +3 -1
data/spec/models/mongoid/category.rb +4 -2
data/spec/models/mongoid/comment.rb +3 -1
data/spec/models/mongoid/company.rb +3 -1
data/spec/models/mongoid/entry.rb +3 -1
data/spec/models/mongoid/post.rb +6 -4
data/spec/models/mongoid/user.rb +2 -0
data/spec/models/newspaper.rb +3 -1
data/spec/models/page.rb +2 -0
data/spec/models/person.rb +2 -0
data/spec/models/pet.rb +2 -0
data/spec/models/post.rb +19 -0
data/spec/models/relationship.rb +2 -0
data/spec/models/reply.rb +2 -0
data/spec/models/student.rb +2 -0
data/spec/models/submission.rb +3 -0
data/spec/models/teacher.rb +2 -0
data/spec/models/user.rb +3 -0
data/spec/models/writer.rb +2 -0
data/spec/spec_helper.rb +19 -23
data/spec/support/bullet_ext.rb +10 -9
data/spec/support/mongo_seed.rb +38 -57
data/spec/support/rack_double.rb +13 -19
data/spec/support/sqlite_seed.rb +104 -76
data/tasks/bullet_tasks.rake +4 -2
data/test.sh +5 -0
data/update.sh +3 -0
metadata +34 -17
data/.travis.yml +0 -16
data/spec/integration/active_record5/association_spec.rb +0 -768

data/spec/bullet/notification_collector_spec.rb CHANGED Viewed

@@ -1,29 +1,31 @@
+# frozen_string_literal: true
 require 'spec_helper'
 module Bullet
   describe NotificationCollector do
-    subject { NotificationCollector.new.tap { |collector| collector.add("value") } }
+    subject { NotificationCollector.new.tap { |collector| collector.add('value') } }
-    context "#add" do
-      it "should add a value" do
-        subject.add("value1")
-        expect(subject.collection).to be_include("value1")
+    context '#add' do
+      it 'should add a value' do
+        subject.add('value1')
+        expect(subject.collection).to be_include('value1')
       end
     end
-    context "#reset" do
-      it "should reset collector" do
+    context '#reset' do
+      it 'should reset collector' do
         subject.reset
         expect(subject.collection).to be_empty
       end
     end
-    context "#notifications_present?" do
-      it "should be true if collection is not empty" do
+    context '#notifications_present?' do
+      it 'should be true if collection is not empty' do
         expect(subject).to be_notifications_present
       end
-      it "should be false if collection is empty" do
+      it 'should be false if collection is empty' do
         subject.reset
         expect(subject).not_to be_notifications_present
       end

data/spec/bullet/rack_spec.rb CHANGED Viewed

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
 require 'spec_helper'
 module Bullet
@@ -6,123 +7,274 @@ module Bullet
     let(:middleware) { Bullet::Rack.new app }
     let(:app) { Support::AppDouble.new }
-    context "#html_request?" do
-      it "should be true if Content-Type is text/html and http body contains html tag" do
-        headers = {"Content-Type" => "text/html"}
-        response = double(:body => "<html><head></head><body></body></html>")
+    context '#html_request?' do
+      it 'should be true if Content-Type is text/html and http body contains html tag' do
+        headers = { 'Content-Type' => 'text/html' }
+        response = double(body: '<html><head></head><body></body></html>')
         expect(middleware).to be_html_request(headers, response)
       end
-      it "should be true if Content-Type is text/html and http body contains html tag with attributes" do
-        headers = {"Content-Type" => "text/html"}
-        response = double(:body => "<html attr='hello'><head></head><body></body></html>")
+      it 'should be true if Content-Type is text/html and http body contains html tag with attributes' do
+        headers = { 'Content-Type' => 'text/html' }
+        response = double(body: "<html attr='hello'><head></head><body></body></html>")
         expect(middleware).to be_html_request(headers, response)
       end
-      it "should be false if there is no Content-Type header" do
+      it 'should be false if there is no Content-Type header' do
         headers = {}
-        response = double(:body => "<html><head></head><body></body></html>")
+        response = double(body: '<html><head></head><body></body></html>')
         expect(middleware).not_to be_html_request(headers, response)
       end
-      it "should be false if Content-Type is javascript" do
-        headers = {"Content-Type" => "text/javascript"}
-        response = double(:body => "<html><head></head><body></body></html>")
+      it 'should be false if Content-Type is javascript' do
+        headers = { 'Content-Type' => 'text/javascript' }
+        response = double(body: '<html><head></head><body></body></html>')
         expect(middleware).not_to be_html_request(headers, response)
       end
       it "should be false if response body doesn't contain html tag" do
-        headers = {"Content-Type" => "text/html"}
-        response = double(:body => "<div>Partial</div>")
+        headers = { 'Content-Type' => 'text/html' }
+        response = double(body: '<div>Partial</div>')
         expect(middleware).not_to be_html_request(headers, response)
       end
     end
-    context "empty?" do
-      it "should be false if response is a string and not empty" do
-        response = double(:body => "<html><head></head><body></body></html>")
+    context 'empty?' do
+      it 'should be false if response is a string and not empty' do
+        response = double(body: '<html><head></head><body></body></html>')
         expect(middleware).not_to be_empty(response)
       end
-      it "should be true if response is not found" do
-        response = ["Not Found"]
-        expect(middleware).to be_empty(response)
+      it 'should be false if response is not found' do
+        response = ['Not Found']
+        expect(middleware).not_to be_empty(response)
       end
-      it "should be true if response body is empty" do
-        response = double(:body => "")
+      it 'should be true if response body is empty' do
+        response = double(body: '')
         expect(middleware).to be_empty(response)
       end
     end
-    context "#call" do
-      context "when Bullet is enabled" do
-        it "should return original response body" do
-          expected_response = Support::ResponseDouble.new "Actual body"
+    context '#call' do
+      context 'when Bullet is enabled' do
+        it 'should return original response body' do
+          expected_response = Support::ResponseDouble.new 'Actual body'
           app.response = expected_response
           _, _, response = middleware.call({})
           expect(response).to eq(expected_response)
         end
-        it "should change response body if notification is active" do
+        it 'should change response body if notification is active' do
           expect(Bullet).to receive(:notification?).and_return(true)
-          expect(Bullet).to receive(:gather_inline_notifications).and_return("<bullet></bullet>")
+          expect(Bullet).to receive(:console_enabled?).and_return(true)
+          expect(Bullet).to receive(:gather_inline_notifications).and_return('<bullet></bullet>')
           expect(Bullet).to receive(:perform_out_of_channel_notifications)
-          status, headers, response = middleware.call({"Content-Type" => "text/html"})
-          expect(headers["Content-Length"]).to eq("56")
-          expect(response).to eq(["<html><head></head><body><bullet></bullet></body></html>"])
+          _, headers, response = middleware.call('Content-Type' => 'text/html')
+          expect(headers['Content-Length']).to eq('56')
+          expect(response).to eq(%w[<html><head></head><body><bullet></bullet></body></html>])
         end
-        it "should set the right Content-Length if response body contains accents" do
+        it 'should set the right Content-Length if response body contains accents' do
           response = Support::ResponseDouble.new
-          response.body = "<html><head></head><body>é</body></html>"
+          response.body = '<html><head></head><body>é</body></html>'
           app.response = response
           expect(Bullet).to receive(:notification?).and_return(true)
-          expect(Bullet).to receive(:gather_inline_notifications).and_return("<bullet></bullet>")
-          status, headers, response = middleware.call({"Content-Type" => "text/html"})
-          expect(headers["Content-Length"]).to eq("58")
+          allow(Bullet).to receive(:console_enabled?).and_return(true)
+          expect(Bullet).to receive(:gather_inline_notifications).and_return('<bullet></bullet>')
+          _, headers, response = middleware.call('Content-Type' => 'text/html')
+          expect(headers['Content-Length']).to eq('58')
+        end
+        context 'with injection notifiers' do
+          before do
+            expect(Bullet).to receive(:notification?).and_return(true)
+            allow(Bullet).to receive(:gather_inline_notifications).and_return('<bullet></bullet>')
+            allow(middleware).to receive(:xhr_script).and_return('<script></script>')
+            allow(middleware).to receive(:footer_note).and_return('footer')
+            expect(Bullet).to receive(:perform_out_of_channel_notifications)
+          end
+          it 'should change response body if add_footer is true' do
+            expect(Bullet).to receive(:add_footer).exactly(3).times.and_return(true)
+            _, headers, response = middleware.call('Content-Type' => 'text/html')
+            expect(headers['Content-Length']).to eq((73 + middleware.send(:footer_note).length).to_s)
+            expect(response).to eq(%w[<html><head></head><body>footer<bullet></bullet><script></script></body></html>])
+          end
+          it 'should change response body for html safe string if add_footer is true' do
+            expect(Bullet).to receive(:add_footer).exactly(3).times.and_return(true)
+            app.response =
+              Support::ResponseDouble.new.tap do |response|
+                response.body = ActiveSupport::SafeBuffer.new('<html><head></head><body></body></html>')
+              end
+            _, headers, response = middleware.call('Content-Type' => 'text/html')
+            expect(headers['Content-Length']).to eq((73 + middleware.send(:footer_note).length).to_s)
+            expect(response).to eq(%w[<html><head></head><body>footer<bullet></bullet><script></script></body></html>])
+          end
+          it 'should add the footer-text header for non-html requests when add_footer is true' do
+            allow(Bullet).to receive(:add_footer).at_least(:once).and_return(true)
+            allow(Bullet).to receive(:footer_info).and_return(['footer text'])
+            app.headers = { 'Content-Type' => 'application/json' }
+            _, headers, _response = middleware.call({})
+            expect(headers).to include('X-bullet-footer-text' => '["footer text"]')
+          end
+          it 'should change response body if console_enabled is true' do
+            expect(Bullet).to receive(:console_enabled?).and_return(true)
+            _, headers, response = middleware.call('Content-Type' => 'text/html')
+            expect(headers['Content-Length']).to eq('56')
+            expect(response).to eq(%w[<html><head></head><body><bullet></bullet></body></html>])
+          end
+          it 'should change response body for html safe string if console_enabled is true' do
+            expect(Bullet).to receive(:console_enabled?).and_return(true)
+            app.response =
+              Support::ResponseDouble.new.tap do |response|
+                response.body = ActiveSupport::SafeBuffer.new('<html><head></head><body></body></html>')
+              end
+            _, headers, response = middleware.call('Content-Type' => 'text/html')
+            expect(headers['Content-Length']).to eq('56')
+            expect(response).to eq(%w[<html><head></head><body><bullet></bullet></body></html>])
+          end
+          it 'should add headers for non-html requests when console_enabled is true' do
+            allow(Bullet).to receive(:console_enabled?).at_least(:once).and_return(true)
+            allow(Bullet).to receive(:text_notifications).and_return(['text notifications'])
+            app.headers = { 'Content-Type' => 'application/json' }
+            _, headers, _response = middleware.call({})
+            expect(headers).to include('X-bullet-console-text' => '["text notifications"]')
+          end
+          it "shouldn't change response body unnecessarily" do
+            expected_response = Support::ResponseDouble.new 'Actual body'
+            app.response = expected_response
+            _, _, response = middleware.call({})
+            expect(response).to eq(expected_response)
+          end
+          it "shouldn't add headers unnecessarily" do
+            app.headers = { 'Content-Type' => 'application/json' }
+            _, headers, _response = middleware.call({})
+            expect(headers).not_to include('X-bullet-footer-text')
+            expect(headers).not_to include('X-bullet-console-text')
+          end
+          context 'when skip_http_headers is enabled' do
+            before do
+              allow(Bullet).to receive(:skip_http_headers).and_return(true)
+            end
+            it 'should include the footer but not the xhr script tag if add_footer is true' do
+              expect(Bullet).to receive(:add_footer).at_least(:once).and_return(true)
+              _, headers, response = middleware.call({})
+              expect(headers['Content-Length']).to eq((56 + middleware.send(:footer_note).length).to_s)
+              expect(response).to eq(%w[<html><head></head><body>footer<bullet></bullet></body></html>])
+            end
+            it 'should not include the xhr script tag if console_enabled is true' do
+              expect(Bullet).to receive(:console_enabled?).and_return(true)
+              _, headers, response = middleware.call({})
+              expect(headers['Content-Length']).to eq('56')
+              expect(response).to eq(%w[<html><head></head><body><bullet></bullet></body></html>])
+            end
+            it 'should not add the footer-text header for non-html requests when add_footer is true' do
+              allow(Bullet).to receive(:add_footer).at_least(:once).and_return(true)
+              app.headers = { 'Content-Type' => 'application/json' }
+              _, headers, _response = middleware.call({})
+              expect(headers).not_to include('X-bullet-footer-text')
+            end
+            it 'should not add headers for non-html requests when console_enabled is true' do
+              allow(Bullet).to receive(:console_enabled?).at_least(:once).and_return(true)
+              app.headers = { 'Content-Type' => 'application/json' }
+              _, headers, _response = middleware.call({})
+              expect(headers).not_to include('X-bullet-console-text')
+            end
+          end
+        end
+        context 'when skip_html_injection is enabled' do
+          it 'should not try to inject html' do
+            expected_response = Support::ResponseDouble.new 'Actual body'
+            app.response = expected_response
+            allow(Bullet).to receive(:notification?).and_return(true)
+            allow(Bullet).to receive(:skip_html_injection?).and_return(true)
+            expect(Bullet).to receive(:gather_inline_notifications).never
+            expect(middleware).to receive(:xhr_script).never
+            expect(Bullet).to receive(:perform_out_of_channel_notifications)
+            _, _, response = middleware.call('Content-Type' => 'text/html')
+            expect(response).to eq(expected_response)
+          end
         end
       end
-      context "when Bullet is disabled" do
+      context 'when Bullet is disabled' do
         before(:each) { allow(Bullet).to receive(:enable?).and_return(false) }
-        it "should not call Bullet.start_request" do
+        it 'should not call Bullet.start_request' do
           expect(Bullet).not_to receive(:start_request)
           middleware.call({})
         end
       end
     end
-    describe "#response_body" do
+    context '#set_header' do
+      it 'should truncate headers to under 8kb' do
+        long_header = ['a' * 1_024] * 10
+        expected_res = (['a' * 1_024] * 7).to_json
+        expect(middleware.set_header({}, 'Dummy-Header', long_header)).to eq(expected_res)
+      end
+    end
+    describe '#response_body' do
       let(:response) { double }
-      let(:body_string) { "<html><body>My Body</body></html>" }
+      let(:body_string) { '<html><body>My Body</body></html>' }
-      context "when `response` responds to `body`" do
+      context 'when `response` responds to `body`' do
         before { allow(response).to receive(:body).and_return(body) }
-        context "when `body` returns an Array" do
+        context 'when `body` returns an Array' do
           let(:body) { [body_string, 'random string'] }
-          it "should return the plain body string" do
+          it 'should return the plain body string' do
             expect(middleware.response_body(response)).to eq body_string
           end
         end
-        context "when `body` does not return an Array" do
+        context 'when `body` does not return an Array' do
           let(:body) { body_string }
-          it "should return the plain body string" do
+          it 'should return the plain body string' do
             expect(middleware.response_body(response)).to eq body_string
           end
         end
       end
-      context "when `response` does not respond to `body`" do
+      context 'when `response` does not respond to `body`' do
         before { allow(response).to receive(:first).and_return(body_string) }
-        it "should return the plain body string" do
+        it 'should return the plain body string' do
           expect(middleware.response_body(response)).to eq body_string
         end
       end
+      begin
+        require 'rack/files'
+        context 'when `response` is a Rack::Files::Iterator' do
+          let(:response) { instance_double(::Rack::Files::Iterator) }
+          before { allow(response).to receive(:is_a?).with(::Rack::Files::Iterator) { true } }
+          it 'should return nil' do
+            expect(middleware.response_body(response)).to be_nil
+          end
+        end
+      rescue LoadError
+      end
     end
   end
 end

data/spec/bullet/registry/association_spec.rb CHANGED Viewed

@@ -1,24 +1,26 @@
+# frozen_string_literal: true
 require 'spec_helper'
 module Bullet
   module Registry
     describe Association do
-      subject { Association.new.tap { |association| association.add(["key1", "key2"], "value") } }
+      subject { Association.new.tap { |association| association.add(%w[key1 key2], 'value') } }
-      context "#merge" do
-        it "should merge key/value" do
-          subject.merge("key0", "value0")
-          expect(subject["key0"]).to be_include("value0")
+      context '#merge' do
+        it 'should merge key/value' do
+          subject.merge('key0', 'value0')
+          expect(subject['key0']).to be_include('value0')
         end
       end
-      context "#similarly_associated" do
-        it "should return similarly associated keys" do
-          expect(subject.similarly_associated("key1", Set.new(["value"]))).to eq(["key1", "key2"])
+      context '#similarly_associated' do
+        it 'should return similarly associated keys' do
+          expect(subject.similarly_associated('key1', Set.new(%w[value]))).to eq(%w[key1 key2])
         end
-        it "should return empty if key does not exist" do
-          expect(subject.similarly_associated("key3", Set.new(["value"]))).to be_empty
+        it 'should return empty if key does not exist' do
+          expect(subject.similarly_associated('key3', Set.new(%w[value]))).to be_empty
         end
       end
     end

data/spec/bullet/registry/base_spec.rb CHANGED Viewed

@@ -1,42 +1,44 @@
+# frozen_string_literal: true
 require 'spec_helper'
 module Bullet
   module Registry
     describe Base do
-      subject { Base.new.tap { |base| base.add("key", "value") } }
+      subject { Base.new.tap { |base| base.add('key', 'value') } }
-      context "#[]" do
-        it "should get value by key" do
-          expect(subject["key"]).to eq(Set.new(["value"]))
+      context '#[]' do
+        it 'should get value by key' do
+          expect(subject['key']).to eq(Set.new(%w[value]))
         end
       end
-      context "#delete" do
-        it "should delete key" do
-          subject.delete("key")
-          expect(subject["key"]).to be_nil
+      context '#delete' do
+        it 'should delete key' do
+          subject.delete('key')
+          expect(subject['key']).to be_nil
         end
       end
-      context "#add" do
-        it "should add value with string" do
-          subject.add("key", "new_value")
-          expect(subject["key"]).to eq(Set.new(["value", "new_value"]))
+      context '#add' do
+        it 'should add value with string' do
+          subject.add('key', 'new_value')
+          expect(subject['key']).to eq(Set.new(%w[value new_value]))
         end
-        it "should add value with array" do
-          subject.add("key", ["value1", "value2"])
-          expect(subject["key"]).to eq(Set.new(["value", "value1", "value2"]))
+        it 'should add value with array' do
+          subject.add('key', %w[value1 value2])
+          expect(subject['key']).to eq(Set.new(%w[value value1 value2]))
         end
       end
-      context "#include?" do
-        it "should include key/value" do
-          expect(subject.include?("key", "value")).to eq true
+      context '#include?' do
+        it 'should include key/value' do
+          expect(subject.include?('key', 'value')).to eq true
         end
-        it "should not include wrong key/value" do
-          expect(subject.include?("key", "val")).to eq false
+        it 'should not include wrong key/value' do
+          expect(subject.include?('key', 'val')).to eq false
         end
       end
     end

data/spec/bullet/registry/object_spec.rb CHANGED Viewed

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
 require 'spec_helper'
 module Bullet
@@ -7,14 +9,14 @@ module Bullet
       let(:another_post) { Post.last }
       subject { Object.new.tap { |object| object.add(post.bullet_key) } }
-      context "#include?" do
-        it "should include the object" do
+      context '#include?' do
+        it 'should include the object' do
           expect(subject).to be_include(post.bullet_key)
         end
       end
-      context "#add" do
-        it "should add an object" do
+      context '#add' do
+        it 'should add an object' do
           subject.add(another_post.bullet_key)
           expect(subject).to be_include(another_post.bullet_key)
         end