Move tests

Author: n-rodriguez

spec/ajax-datatables-rails/base_spec.rb

@@ -192,40 +192,6 @@ def paginate_records(records)
         end
       end
     end
-
-    describe 'helper methods' do
-      describe '#offset' do
-        it 'defaults to 0' do
-          datatable = described_class.new({})
-          expect(datatable.datatable.send(:offset)).to eq(0)
-        end
-
-        it 'matches the value on view params[:start]' do
-          datatable = described_class.new({ start: '11' })
-          expect(datatable.datatable.send(:offset)).to eq(11)
-        end
-      end
-
-      describe '#page' do
-        it 'calculates page number from params[:start] and #per_page' do
-          datatable = described_class.new({ start: '11' })
-          expect(datatable.datatable.send(:page)).to eq(2)
-        end
-      end
-
-      describe '#per_page' do
-        it 'defaults to 10' do
-          datatable = described_class.new(sample_params)
-          expect(datatable.datatable.send(:per_page)).to eq(10)
-        end
-
-        it 'matches the value on view params[:length]' do
-          other_view = { length: 20 }
-          datatable = described_class.new(other_view)
-          expect(datatable.datatable.send(:per_page)).to eq(20)
-        end
-      end
-    end
   end
 
   describe 'User helper methods' do

spec/ajax-datatables-rails/datatable/datatable_spec.rb

@@ -3,9 +3,10 @@
 describe AjaxDatatablesRails::Datatable::Datatable do
 
   let(:datatable) { ComplexDatatable.new(sample_params).datatable }
-  let(:order_option) { {'0'=>{'column'=>'0', 'dir'=>'asc'}, '1'=>{'column'=>'1', 'dir'=>'desc'}} }
 
   describe 'order methods' do
+    let(:order_option) { {'0'=>{'column'=>'0', 'dir'=>'asc'}, '1'=>{'column'=>'1', 'dir'=>'desc'}} }
+
     it 'should be orderable' do
       expect(datatable.orderable?).to eq(true)
     end
@@ -62,25 +63,50 @@
   end
 
   describe 'option methods' do
-    before :each do
-      datatable.options[:start] = '50'
-      datatable.options[:length] = '15'
+    describe '#paginate?' do
+      it {
+        expect(datatable.paginate?).to be(true)
+      }
     end
 
-    it 'paginate?' do
-      expect(datatable.paginate?).to be(true)
-    end
+    describe '#per_page' do
+      context 'when params[:length] is missing' do
+        it 'defaults to 10' do
+          expect(datatable.per_page).to eq(10)
+        end
+      end
+
+      context 'when params[:length] is passed' do
+        let(:datatable) { ComplexDatatable.new({ length: '20' }).datatable }
 
-    it 'offset' do
-      expect(datatable.offset).to eq(50)
+        it 'matches the value on view params[:length]' do
+          expect(datatable.per_page).to eq(20)
+        end
+      end
     end
 
-    it 'page' do
-      expect(datatable.page).to eq(4)
+    describe '#offset' do
+      context 'when params[:start] is missing' do
+        it 'defaults to 0' do
+          expect(datatable.offset).to eq(0)
+        end
+      end
+
+      context 'when params[:start] is passed' do
+        let(:datatable) { ComplexDatatable.new({ start: '11' }).datatable }
+
+        it 'matches the value on view params[:start]' do
+          expect(datatable.offset).to eq(11)
+        end
+      end
     end
 
-    it 'per_page' do
-      expect(datatable.per_page).to eq(15)
+    describe '#page' do
+      let(:datatable) { ComplexDatatable.new({ start: '11' }).datatable }
+
+      it 'calculates page number from params[:start] and #per_page' do
+        expect(datatable.page).to eq(2)
+      end
     end
   end
 end