From 8b43f22e66515bacada090defde0450dabc0a8bb Mon Sep 17 00:00:00 2001
From: cryptogopher <git6453hub@michalczyk.pro>
Date: Sat, 22 Aug 2020 16:11:43 +0200
Subject: [PATCH] Added:
 test_create_binding_target_when_binding_goal_does_not_exist

---
 app/controllers/targets_controller.rb | 14 +++++++-------
 app/models/measurement_routine.rb     |  1 +
 test/system/targets_test.rb           | 18 ++++++++++++++++++
 3 files changed, 26 insertions(+), 7 deletions(-)

diff --git a/app/controllers/targets_controller.rb b/app/controllers/targets_controller.rb
index d019136..f83e0e2 100644
--- a/app/controllers/targets_controller.rb
+++ b/app/controllers/targets_controller.rb
@@ -22,19 +22,19 @@ class TargetsController < ApplicationController
   end
 
   def create
-    goal = @project.goals.find_by(id: params[:goal][:id]) || @project.goals.build(goal_params)
+    goal = @project.goals.binding if params[:goal][:id].blank?
+    goal ||= @project.goals.find_by(id: params[:goal][:id])
+    goal ||= @project.goals.build(goal_params)
+
     @targets = goal.targets.build(targets_params[:targets]) do |target|
       target.effective_from = params[:target][:effective_from]
     end
-    # FIXME: add goal exposures before save and require (in model) goal.target_exposures to
-    # be present (same for measurement/food?)
+    if goal.target_exposures.empty?
+      goal.quantities << @targets.map { |t| t.thresholds.first.quantity }.first(6)
+    end
 
     # :save only after build, to re-display values in case records are invalid
     if goal.save && Target.transaction { @targets.all?(&:save) }
-      if goal.target_exposures.empty?
-        goal.quantities << @targets.map { |t| t.thresholds.first.quantity }.first(6)
-      end
-
       flash[:notice] = 'Created new target(s)'
       prepare_targets
     else
diff --git a/app/models/measurement_routine.rb b/app/models/measurement_routine.rb
index 4cb6a04..e916741 100644
--- a/app/models/measurement_routine.rb
+++ b/app/models/measurement_routine.rb
@@ -7,5 +7,6 @@ class MeasurementRoutine < ActiveRecord::Base
     class_name: 'Exposure', extend: BodyTracking::TogglableExposures
   has_many :quantities, -> { order "lft" }, through: :readout_exposures
 
+  # TODO: require readout_exposures to be present
   validates :name, presence: true, uniqueness: {scope: :project_id}
 end
diff --git a/test/system/targets_test.rb b/test/system/targets_test.rb
index 37ea8e9..49e51f2 100644
--- a/test/system/targets_test.rb
+++ b/test/system/targets_test.rb
@@ -48,9 +48,27 @@ class TargetsTest < BodyTrackingSystemTestCase
         click_on t(:button_create)
       end
     end
+    assert_no_selector 'form#new-target-form'
     assert_selector 'table#targets tbody tr', count: @project1.targets.count
   end
 
   def test_create_binding_target_when_binding_goal_does_not_exist
+    @project1.goals.delete_all
+    assert_equal 0, @project1.goals.count
+    visit project_targets_path(@project1)
+    click_link t('targets.contextual.link_new_target')
+    within 'form#new-target-form' do
+      # Assume binding Goal is initialized (not saved) and selected by default
+      within 'p.target' do
+        select quantities(:quantities_energy).name
+        select '=='
+        fill_in with: '1750'
+        select units(:units_kcal).shortname
+      end
+      assert_difference ['Goal.count', '@project1.goals.reload.count(&:is_binding?)'], 1 do
+        click_on t(:button_create)
+      end
+    end
+    assert_selector 'table#targets tbody tr', count: 1
   end
 end