Only reference css_paths config

Author: michael-misshore

lib/critical_path_css/configuration.rb

@@ -9,10 +9,6 @@ def base_url
       @config['base_url']
     end
 
-    def css_path
-      @config['css_path']
-    end
-
     def css_paths
       @config['css_paths']
     end

lib/critical_path_css/rails/config_loader.rb

@@ -19,15 +19,12 @@ def configuration_file_path
       end
 
       def format_css_paths
-        if config['css_path']
-          config['css_path']  = format_path(config['css_path'])
-          config['css_paths'] = []
-        elsif config['css_paths']
-          config['css_path']  = ''
-          config['css_paths'] = config['css_paths'].collect { |path| format_path(path) }
+        config['css_paths'] = [config['css_path']] if config['css_path']
+
+        if config['css_paths']
+          config['css_paths'].map! { |path| format_path(path) }
         else
-          config['css_path']  = ActionController::Base.helpers.stylesheet_path(config['manifest_name'], host: '')
-          config['css_paths'] = []
+          config['css_paths'] = [ActionController::Base.helpers.stylesheet_path(config['manifest_name'], host: '')]
         end
       end
 

spec/lib/critical_path_css/css_fetcher_spec.rb

@@ -18,8 +18,7 @@
 
   describe '#fetch_route' do
     context 'when a single css_path is configured' do
-      let(:css_path) { '/test.css' }
-      let(:css_paths) { [] }
+      let(:css_paths) { ['/test.css'] }
 
       it 'generates css for the single route' do
         expect(Open3).to receive(:capture3) do |arg1, arg2, arg3|
@@ -35,8 +34,7 @@
 
   describe '#fetch' do
     context 'when a single css_path is configured' do
-      let(:css_path) { '/test.css' }
-      let(:css_paths) { [] }
+      let(:css_paths) { ['/test.css'] }
 
       it 'generates css for each route from the same file' do
         expect(Open3).to receive(:capture3) do |arg1, arg2, arg3|
@@ -50,7 +48,6 @@
     end
 
     context 'when multiple css_paths are configured' do
-      let(:css_path) { '' }
       let(:css_paths) { ['/test.css', '/test2.css'] }
 
       it 'generates css for each route from the respective file' do
@@ -67,7 +64,6 @@
     end
 
     context 'when same css file applies to multiple routes' do
-      let(:css_path) { '' }
       let(:css_paths) { ['/test.css', '/test2.css', '/test.css'] }
       let(:routes) { ['/', '/new_route', '/newer_route'] }
 

spec/lib/critical_path_css/rails/config_loader_spec.rb

@@ -11,22 +11,14 @@
     context 'when single css_path is specified' do
       let(:config_file) { file_fixture('config/single-css-path.yml').read }
 
-      it 'sets css_path with the path' do
-        expect(subject.config['css_path']).to eq '/app/spec/internal/public/test.css'
-      end
-
-      it 'leaves css_paths empty' do
-        expect(subject.config['css_paths']).to eq []
+      it 'sets css_paths with the lone path' do
+        expect(subject.config['css_paths']).to eq ['/app/spec/internal/public/test.css']
       end
     end
 
     context 'when multiple css_paths are specified' do
       let(:config_file) { file_fixture('config/mutliple-css-paths.yml').read }
 
-      it 'sets css_path to empty string' do
-        expect(subject.config['css_path']).to eq ''
-      end
-
       it 'leaves css_paths to an array of paths' do
         expect(subject.config['css_paths']).to eq ['/app/spec/internal/public/test.css','/app/spec/internal/public/test2.css']
       end
@@ -35,12 +27,8 @@
     context 'when no paths are specified' do
       let(:config_file) { file_fixture('config/no-paths-specified.yml').read }
 
-      it 'sets css_path with the path' do
-        expect(subject.config['css_path']).to eq '/stylesheets/application.css'
-      end
-
-      it 'leaves css_paths empty' do
-        expect(subject.config['css_paths']).to eq []
+      it 'sets css_paths with the lone manifest path' do
+        expect(subject.config['css_paths']).to eq ['/stylesheets/application.css']
       end
     end