spec/unit/browser_spec.rb in browse-everything-0.10.5 vs spec/unit/browser_spec.rb in browse-everything-0.11.0

@@ -1,73 +1,73 @@
 include BrowserConfigHelper
 
 describe BrowseEverything::Browser do
-  let(:file_config) {
+  let(:file_config) do
     {
       file_system: { home: '/file/config/home' },
       dropbox:    { app_key: 'FileConfigKey', app_secret: 'FileConfigSecret' }
     }.to_yaml
-  }
+  end
 
-  let(:global_config) {
+  let(:global_config) do
     {
       file_system: { home: '/global/config/home' },
       dropbox:    { app_key: 'GlobalConfigKey', app_secret: 'GlobalConfigSecret' }
     }
-  }
+  end
 
-  let(:local_config) {
+  let(:local_config) do
     {
       file_system: { home: '/local/config/home' },
       dropbox:    { app_key: 'LocalConfigKey', app_secret: 'LocalConfigSecret' },
       url_options: url_options
     }
-  }
+  end
 
-  describe "file config" do
+  describe 'file config' do
     before(:each) { allow(File).to receive(:read).and_return(file_config) }
     subject { BrowseEverything::Browser.new(url_options) }
 
-    it "should have 2 providers" do
+    it 'should have 2 providers' do
       expect(subject.providers.keys).to eq([:file_system, :dropbox])
     end
 
-    it "should use the file configuration" do
+    it 'should use the file configuration' do
       expect(subject.providers[:dropbox].config[:app_key]).to eq('FileConfigKey')
     end
   end
 
-  describe "global config" do
+  describe 'global config' do
     before(:each) { BrowseEverything.configure(global_config) }
     subject { BrowseEverything::Browser.new(url_options) }
 
-    it "should have 2 providers" do
+    it 'should have 2 providers' do
       expect(subject.providers.keys).to eq([:file_system, :dropbox])
     end
 
-    it "should use the global configuration" do
+    it 'should use the global configuration' do
       expect(subject.providers[:dropbox].config[:app_key]).to eq('GlobalConfigKey')
     end
   end
 
-  describe "local config" do
+  describe 'local config' do
     subject { BrowseEverything::Browser.new(local_config) }
 
-    it "should have 2 providers" do
+    it 'should have 2 providers' do
       expect(subject.providers.keys).to eq([:file_system, :dropbox])
     end
 
-    it "should use the local configuration" do
+    it 'should use the local configuration' do
       expect(subject.providers[:dropbox].config[:app_key]).to eq('LocalConfigKey')
     end
   end
 
-  describe "unknown provider" do
-    subject {
+  describe 'unknown provider' do
+    subject do
       BrowseEverything::Browser.new(local_config.merge(foo: { key: 'bar', secret: 'baz' }))
-    }
+    end
 
-    it "should complain but continue" do
+    it 'should complain but continue' do
       allow(Rails.logger).to receive(:warn).with('Unknown provider: foo')
       expect(subject.providers.keys).to eq([:file_system, :dropbox])
     end
   end
 end