Fix test 'change item total' from controler_Item.t

Author: kuro610

lib/Coocook/Controller/Recipe.pm

@@ -100,7 +100,8 @@ sub edit : GET HEAD Chained('base') PathPart('') Args(0) RequiresCapability('vie
     );
 
     $recipe->project->is_public
-      and $c->stash( public_url => $c->uri_for_action( '/browse/recipe/show', [ $recipe->id, $recipe->url_name ] ) );
+      and $c->stash(
+        public_url => $c->uri_for_action( '/browse/recipe/show', [ $recipe->id, $recipe->url_name ] ) );
 
     for my $ingredient ( @{ $c->stash->{ingredients} } ) {
         $ingredient->{reposition_url} = $c->project_uri( '/recipe/reposition', $ingredient->{id} );

root/templates/purchase_list/edit.tt

@@ -44,24 +44,22 @@ html_title = html_title _ ' for ' _ display_date(list.date, {html=>1});
 							<option value="[% item.next_higher_value %]">
 							<option value="[% item.next_lower_value%]">
 						</datalist>
-                        <form id="total-[% loop.count %]" name="total" method="post" action="[% item.update_offset_url %]"></form>
-						<form id="units-[% loop.count %]" method="post" action="[% item.convert_url %]"></form>
-                        <div class="d-flex justify-content-between">
+                        <form id="total-[% loop.count %]" class="d-flex justify-content-between" name="total" method="post" action="[% item.update_offset_url %]">
 							<div class="input-group flex-nowrap">
 								[% IF item.offset %]<span class="input-group-text px-1">ca.</span>[% END %]
 								<input id="total" class="form-control" form="total-[% loop.count %]" type="number" min="0" step="any" name="total" list="values-[% loop.count %]" value="[% item.value + item.offset %]">
 								<span class="input-group-text">[% display_unit( item.unit, {html=>1} ) %]</span>
 							</div>
 							<button class="btn btn-outline-primary ms-1 material-icons" form="total-[% loop.count %]" type="submit" title="Update amount">sync</button>
-						</div>
+						</form>
 						[% IF item.convert_url %]
-						<div class="d-flex flex-wrap border rounded my-1 w-100">
+						<form id="units-[% loop.count %]" class="d-flex flex-wrap border rounded my-1 w-100" name="convert" method="post" action="[% item.convert_url %]">
 						[% idx = loop.count;
 							FOREACH unit IN item.convertible_into %]
 							<button class="btn btn-outline-secondary btn-sm m-1" form="units-[% idx %]" type="submit" name="unit" value="[% unit.id %]">[% unit.short_name | html %]</button>
-							[% END;
-						END %]
-						</div>
+							[% END %]
+						</form>
+						[% END %]
 					</td>
 					<td rowspan="[% rowspan %]">[% item.article.name | html %]</td>
 

root/templates/purchase_list/snippet_dish.tt

@@ -1,5 +1,3 @@
-
-
 <td style="text-align:right;">[% display_value_unit( ingredient.value, ingredient.unit, {html=>1} ) %]</td>
 <td style="text-align:center;">[% ingredient.dish.meal.date.strftime('%a, %F') %]</td>
 <td>[% ingredient.dish.meal.name | html %]</td>

t/controller_Item.t

@@ -69,7 +69,8 @@ subtest "change item total" => sub {
     $t->submit_form_ok(
         {
             form_name   => 'remove-offset',
-            form_number => 10,
+            form_number => 11,
+            button      => 'offset',
         },
         "Remove offset"
     );
@@ -86,7 +87,7 @@ subtest "change item total" => sub {
 
     $t->submit_form_ok(
         {
-            form_number => 8,
+            form_number => 9,
         },
         "Remove ingredient"
     );
@@ -101,7 +102,8 @@ subtest "change item total" => sub {
 
     $t->submit_form_ok(
         {
-            form_number => 5,
+            form_number => 3,
+            button      => 'unit',
         },
         "Convert item to kg"
     );