spec

apps/web/views/vacancies/index.rb

@@ -40,7 +40,7 @@ def subscribe_form # rubocop:disable Metrics/MethodLength
 
         def remote_filter_button(text:, remote_value:)
           link_params =
-            if remote_value == params[:remote]
+            if remote_value == current_remote_query
               { class: 'btn btn-light active' }
             else
               {
@@ -54,6 +54,12 @@ def remote_filter_button(text:, remote_value:)
             html.span { text }
           end
         end
+
+        private
+
+        def current_remote_query
+          params[:remote] if ['true', 'false'].include?(params[:remote])
+        end
       end
     end
   end

spec/core/queries/vacancy_spec.rb

@@ -4,9 +4,11 @@
   let(:repo) { described_class.new }
 
   describe '#all_with_contact' do
-    subject { repo.all_with_contact(limit: 10, page: 1) }
+    subject { repo.all_with_contact(limit: 10, page: 1, remote_available: remote_available) }
 
-    before { Fabricate.create(:vacancy, published: published, archived: archived, deleted_at: deleted_at) }
+    let(:remote_available) { nil }
+
+    before { Fabricate.create(:vacancy, published: published, archived: archived, deleted_at: deleted_at, remote_available: false) }
 
     context 'when vacancy published and not archived or deleted' do
       let(:published) { true }
@@ -16,6 +18,11 @@
       it { expect(subject.count).to eq(1) }
       it { expect(subject).to all(be_a(Vacancy)) }
       it { expect(subject.first.contact).to be_a(Contact) }
+
+      context 'when remote_available is true' do
+        let(:remote_available) { true }
+        it { expect(subject).to eq([]) }
+      end
     end
 
     context 'when vacancy published and archived' do

spec/vacancies/operations/list_spec.rb

@@ -25,6 +25,56 @@
     it { expect(subject.value!).to eq(result: [], pager: pager) }
   end
 
+  context 'with remote query' do
+    let(:vacancies) { [] }
+
+    subject { operation.call(remote_query: remote_query) }
+
+    context 'when remote_query nil' do
+      let(:remote_query) { nil }
+
+      it do
+        expect(vacancy_query)
+          .to receive(:all_with_contact)
+          .with(limit: 10, page: 1, remote_available: nil)
+        subject
+      end
+    end
+
+    context 'when remote_query not true or false string' do
+      let(:remote_query) { 'not true or false string' }
+
+      it do
+        expect(vacancy_query)
+          .to receive(:all_with_contact)
+          .with(limit: 10, page: 1, remote_available: nil)
+        subject
+      end
+    end
+
+    context 'when remote_query true' do
+      let(:remote_query) { 'true' }
+
+      it do
+        expect(vacancy_query)
+          .to receive(:all_with_contact)
+          .with(limit: 10, page: 1, remote_available: true)
+        subject
+      end
+    end
+
+    context 'when remote_query false' do
+      let(:remote_query) { 'false' }
+
+      it do
+        expect(vacancy_query)
+          .to receive(:all_with_contact)
+          .with(limit: 10, page: 1, remote_available: false)
+        subject
+      end
+    end
+  end
+
   context 'with real dependencies' do
     subject { operation.call }
 

spec/web/controllers/vacancies/index_spec.rb

@@ -18,13 +18,17 @@
       expect(action.pager).to eq(pager)
     end
 
-    context 'when params inlcludes query param' do
-      let(:params) { { query: 'remote:true search text' } }
+    context 'when params includes query param' do
+      let(:params) { { query: 'remote:true search text', remote: 'remote_query' } }
 
       it { expect(subject).to be_success }
 
       it do
-        expect(operation).to receive(:call).with(page: nil, search_query: { remote: 'true', text: 'search text' })
+        expect(operation).to receive(:call).with(
+          page: nil,
+          search_query: { remote: 'true', text: 'search text' },
+          remote_query: 'remote_query'
+        )
         subject
       end
     end

spec/web/views/vacancies/index_spec.rb

@@ -1,12 +1,75 @@
 # frozen_string_literal: true
 
 RSpec.describe Web::Views::Vacancies::Index, type: :view do
-  let(:exposures) { Hash[format: :html] }
+  let(:params) { Hash[] }
+  let(:exposures) { Hash[format: :html, flash: {}, vacancies: [], params: params] }
   let(:template)  { Hanami::View::Template.new('apps/web/templates/vacancies/index.html.slim') }
   let(:view)      { described_class.new(template, exposures) }
   let(:rendered)  { view.render }
 
   it 'exposes #format' do
     expect(view.format).to eq exposures.fetch(:format)
   end
+
+  describe 'remote radiobuttons' do
+    let(:rendered) { without_indentation(view.render) }
+
+    before { allow(view).to receive(:pagination) }
+
+    it do
+      expect(rendered).to include without_indentation <<~HTML
+        <div class="btn-group btn-group-toggle">
+        <a class="btn btn-light active"><span>Любые</span></a>
+        <a class="btn btn-light" href="/?remote=true"><span>Удаленные</span></a>
+        <a class="btn btn-light" href="/?remote=false"><span>В офисе</span></a>
+        </div>
+      HTML
+    end
+
+    context 'when remote is true' do
+      let(:params) { Hash[remote: 'true'] }
+
+      it do
+        expect(rendered).to include without_indentation <<~HTML
+          <div class="btn-group btn-group-toggle">
+          <a class="btn btn-light" href="/"><span>Любые</span></a>
+          <a class="btn btn-light active"><span>Удаленные</span></a>
+          <a class="btn btn-light" href="/?remote=false"><span>В офисе</span></a>
+          </div>
+        HTML
+      end
+    end
+
+    context 'when remote is false' do
+      let(:params) { Hash[remote: 'false'] }
+
+      it do
+        expect(rendered).to include without_indentation <<~HTML
+          <div class="btn-group btn-group-toggle">
+          <a class="btn btn-light" href="/"><span>Любые</span></a>
+          <a class="btn btn-light" href="/?remote=true"><span>Удаленные</span></a>
+          <a class="btn btn-light active"><span>В офисе</span></a>
+          </div>
+        HTML
+      end
+    end
+
+    context 'when remote is not valid' do
+      let(:params) { Hash[remote: 'foo'] }
+
+      it do
+        expect(rendered).to include without_indentation <<~HTML
+          <div class="btn-group btn-group-toggle">
+          <a class="btn btn-light active"><span>Любые</span></a>
+          <a class="btn btn-light" href="/?remote=true"><span>Удаленные</span></a>
+          <a class="btn btn-light" href="/?remote=false"><span>В офисе</span></a>
+          </div>
+        HTML
+      end
+    end
+  end
+
+  def without_indentation(string)
+    string.strip.gsub("\n", '')
+  end
 end