Don't show Best Performance selector if results include nonbinary cat…

…egory

app/models/results_template.rb

@@ -19,10 +19,12 @@ class ResultsTemplate < ApplicationRecord
 
   validates_presence_of :name, :aggregation_method
 
+  # @return [ResultsTemplate]
   def self.default
     find_by(slug: "simple")
   end
 
+  # @return [ResultsTemplate]
   def dup_with_categories
     # This must be done first or relations will be lost
     set_category_positions
@@ -32,6 +34,11 @@ def dup_with_categories
     template
   end
 
+  # @return [Boolean]
+  def includes_nonbinary?
+    results_categories.where(nonbinary: true).exists?
+  end
+
   private
 
   def set_category_positions

app/presenters/podium_presenter.rb

@@ -4,7 +4,7 @@ class PodiumPresenter < BasePresenter
   attr_reader :event
 
   delegate :name, :course, :course_name, :organization, :organization_name, :to_param, :multiple_laps?,
-           :event_group, :ordered_events_within_group, :scheduled_start_time_local, to: :event
+           :event_group, :ordered_events_within_group, :results_template, :scheduled_start_time_local, to: :event
   delegate :available_live, :multiple_events?, to: :event_group
   delegate :course_groups, to: :course
 
@@ -19,6 +19,11 @@ def categories
     template&.results_categories || []
   end
 
+  # @return [Array<Symbol>]
+  def sort_methods
+    results_template.includes_nonbinary? ? [:category] : [:category, :best_performance]
+  end
+
   # @return [Array<Results::Category>]
   def sorted_categories
     if performance_sort?

app/views/events/podium.html.erb

@@ -7,7 +7,7 @@
     <div class="ost-heading row">
       <div class="col">
         <div class="ost-title">
-        <h1><strong><%= [@presenter.name, nil].compact.join(": ") %></strong></h1>
+          <h1><strong><%= [@presenter.name, nil].compact.join(": ") %></strong></h1>
           <ul class="breadcrumb breadcrumb-ost">
             <li class="breadcrumb-item"><%= link_to "Organizations", organizations_path %></li>
             <li class="breadcrumb-item"><%= link_to @presenter.organization.name, organization_path(@presenter.organization) %></li>
@@ -35,10 +35,12 @@
       </div>
       <div class="col text-end">
         <div class="btn-group btn-group-ost">
-          <% [:category, :best_performance].each do |sort_method| %>
-            <%= link_to sort_method.to_s.titleize,
-                        request.params.merge(sort: sort_method),
-                        class: "btn #{ @presenter.sort_method == sort_method ? 'btn-primary' : 'btn-outline-secondary' }" %>
+          <% if @presenter.sort_methods.many? %>
+            <% @presenter.sort_methods.each do |sort_method| %>
+              <%= link_to sort_method.to_s.titleize,
+                          request.params.merge(sort: sort_method),
+                          class: "btn #{ @presenter.sort_method == sort_method ? 'btn-primary' : 'btn-outline-secondary' }" %>
+            <% end %>
           <% end %>
         </div>
       </div>

spec/models/results_template_spec.rb

@@ -11,4 +11,20 @@
       expect { results_template.save }.to change { ResultsTemplate.count }.by(1)
     end
   end
+
+  describe "#includes_nonbinary?" do
+    let(:result) { results_template.includes_nonbinary? }
+
+    context "when the template includes a nonbinary category" do
+      let(:results_template) { results_templates(:masters_and_grandmasters_with_nonbinary) }
+
+      it { expect(result).to eq(true)}
+    end
+
+    context "when the template does not include a nonbinary category" do
+      let(:results_template) { results_templates(:masters_and_grandmasters) }
+
+      it { expect(result).to eq(false)}
+    end
+  end
 end