test: incomplete/skip で放置されたテストを棚卸しし、隠れていた実装の不具合を修正

src/Eccube/Controller/ContactController.php

@@ -80,9 +80,16 @@ public function index(Request $request): Response|RedirectResponse|array
         if ($form->isSubmitted() && $form->isValid()) {
             switch ($request->get('mode')) {
                 case 'confirm':
+                    $Page = $this->pageRepository->getPageByRoute('contact_confirm');
+
                     return $this->render('Contact/confirm.twig', [
                         'form' => $form->createView(),
-                        'Page' => $this->pageRepository->getPageByRoute('contact_confirm'),
+                        'Page' => $Page,
+                        // contact と contact_confirm は同一パス '/contact' のため, ルータは常に
+                        // contact にマッチする. TwigInitializeListener が _route から引く
+                        // twig グローバル title は「入力ページ」のままになるので,
+                        // title より優先される subtitle で確認ページ名を上書きする.
+                        'subtitle' => $Page->getName(),
                     ]);
 
                 case 'complete':

src/Eccube/Entity/Order.php

@@ -128,6 +128,10 @@ public function getTotalByTaxRate(): array
         $taxFreeDiscount = $this->getTaxFreeDiscount();
 
         foreach ($this->getTaxableTotalByTaxRate() as $rate => $totalPrice) {
+            if (!array_key_exists($rate, $roundingTypes) || null === $roundingTypes[$rate]) {
+                continue;
+            }
+
             if (bccomp($taxableTotal, '0', 2) !== 0) {
                 // 按分計算: totalPrice - (abs(taxFreeDiscount) * totalPrice / taxableTotal)
                 $absDiscount = ltrim($taxFreeDiscount, '-');

tests/Eccube/Tests/Entity/OrderTest.php

@@ -264,6 +264,47 @@ public function testGetTaxByTaxRate()
         $this->assertSame('65261', $Order->getTaxByTaxRate()['10'], '10%対象値引き後消費税額');
     }
 
+    /**
+     * RoundingType 未設定の課税明細があっても致命的エラーにならないこと.
+     *
+     * 通常は PurchaseFlow の TaxProcessor が必ず設定するが, 明細を直接組み立てた場合は
+     * null になり得る. getTaxByTaxRate() は元から null を読み飛ばすため, 両者で
+     * 集計対象の税率がずれると OrderPdfService が getTaxByTaxRate()[$rate] で
+     * 未定義キーを引く. 税率キーの集合が一致することを検証する.
+     */
+    #[Group(name: 'decimal')]
+    public function testGetTotalByTaxRateWithoutRoundingType()
+    {
+        $Order = $this->createTestOrder();
+
+        $Taxation = $this->entityManager->find(TaxType::class, TaxType::TAXATION);
+        $this->assertInstanceOf(TaxType::class, $Taxation);
+        $ProductItem = $this->entityManager->find(OrderItemType::class, OrderItemType::PRODUCT);
+        $this->assertInstanceOf(OrderItemType::class, $ProductItem);
+        $TaxExcluded = $this->entityManager->find(TaxDisplayType::class, TaxDisplayType::EXCLUDED);
+        $this->assertInstanceOf(TaxDisplayType::class, $TaxExcluded);
+
+        // RoundingType を設定しない課税明細を追加する
+        $OrderItem = new OrderItem();
+        $OrderItem->setTaxType($Taxation);
+        $OrderItem->setTaxRate('20');
+        $OrderItem->setPrice('1000');
+        $OrderItem->setTax('200');
+        $OrderItem->setQuantity('1');
+        $OrderItem->setOrderItemType($ProductItem);
+        $OrderItem->setTaxDisplayType($TaxExcluded);
+        $Order->addOrderItem($OrderItem);
+
+        $total = $Order->getTotalByTaxRate();
+
+        $this->assertArrayNotHasKey('20', $total, 'RoundingType 未設定の税率は集計対象外');
+        $this->assertSame(
+            array_keys($Order->getTaxByTaxRate()),
+            array_keys($total),
+            'getTotalByTaxRate と getTaxByTaxRate の税率が一致すること'
+        );
+    }
+
     protected function createTestOrder()
     {
         $Taxation = $this->entityManager->find(TaxType::class, TaxType::TAXATION);