(SP: 1) [SHOP] fix review feedback for legal copy and test helpers

Author: liudmylasovetovs

frontend/lib/tests/shop/checkout-legal-consent-phase4.test.ts

@@ -13,6 +13,8 @@ import { IdempotencyConflictError } from '@/lib/services/errors';
 import { createOrderWithItems } from '@/lib/services/orders';
 import { toDbMoney } from '@/lib/shop/money';
 
+import { TEST_LEGAL_CONSENT } from './test-legal-consent';
+
 type SeedProduct = {
   productId: string;
 };
@@ -94,12 +96,7 @@ describe('checkout legal consent phase 4', () => {
         locale: 'en-US',
         country: 'US',
         items: [{ productId, quantity: 1 }],
-        legalConsent: {
-          termsAccepted: true,
-          privacyAccepted: true,
-          termsVersion: 'terms-2026-02-27',
-          privacyVersion: 'privacy-2026-02-27',
-        },
+        legalConsent: TEST_LEGAL_CONSENT,
       });
 
       orderId = result.order.id;
@@ -152,12 +149,7 @@ describe('checkout legal consent phase 4', () => {
         locale: 'en-US',
         country: 'US',
         items: [{ productId, quantity: 1 }],
-        legalConsent: {
-          termsAccepted: true,
-          privacyAccepted: true,
-          termsVersion: 'terms-2026-02-27',
-          privacyVersion: 'privacy-2026-02-27',
-        },
+        legalConsent: TEST_LEGAL_CONSENT,
       });
 
       orderId = first.order.id;
@@ -223,12 +215,7 @@ describe('checkout legal consent phase 4', () => {
         locale: 'en-US',
         country: 'US',
         items: [{ productId, quantity: 1 }],
-        legalConsent: {
-          termsAccepted: true,
-          privacyAccepted: true,
-          termsVersion: 'terms-2026-02-27',
-          privacyVersion: 'privacy-2026-02-27',
-        },
+        legalConsent: TEST_LEGAL_CONSENT,
       });
 
       orderId = first.order.id;
@@ -243,12 +230,7 @@ describe('checkout legal consent phase 4', () => {
         locale: 'en-US',
         country: 'US',
         items: [{ productId, quantity: 1 }],
-        legalConsent: {
-          termsAccepted: true,
-          privacyAccepted: true,
-          termsVersion: 'terms-2026-02-27',
-          privacyVersion: 'privacy-2026-02-27',
-        },
+        legalConsent: TEST_LEGAL_CONSENT,
       });
 
       expect(replay.isNew).toBe(false);
@@ -287,12 +269,7 @@ describe('checkout legal consent phase 4', () => {
         locale: 'en-US',
         country: 'US',
         items: [{ productId, quantity: 1 }],
-        legalConsent: {
-          termsAccepted: true,
-          privacyAccepted: true,
-          termsVersion: 'terms-2026-02-27',
-          privacyVersion: 'privacy-2026-02-27',
-        },
+        legalConsent: TEST_LEGAL_CONSENT,
       });
 
       orderId = first.order.id;
@@ -317,12 +294,7 @@ describe('checkout legal consent phase 4', () => {
           locale: 'en-US',
           country: 'US',
           items: [{ productId, quantity: 1 }],
-          legalConsent: {
-            termsAccepted: true,
-            privacyAccepted: true,
-            termsVersion: 'terms-2026-02-27',
-            privacyVersion: 'privacy-2026-02-27',
-          },
+          legalConsent: TEST_LEGAL_CONSENT,
         })
       ).rejects.toMatchObject({
         code: 'IDEMPOTENCY_CONFLICT',

frontend/lib/tests/shop/checkout-shipping-phase3.test.ts

@@ -18,20 +18,15 @@ import {
 } from '@/lib/services/errors';
 import { createOrderWithItems } from '@/lib/services/orders';
 
+import { TEST_LEGAL_CONSENT } from './test-legal-consent';
+
 type SeedData = {
   productId: string;
   cityRef: string;
   warehouseRefA: string;
   warehouseRefB: string;
 };
 
-const TEST_LEGAL_CONSENT = {
-  termsAccepted: true,
-  privacyAccepted: true,
-  termsVersion: 'terms-2026-02-27',
-  privacyVersion: 'privacy-2026-02-27',
-} as const;
-
 async function seedCheckoutShippingData(): Promise<SeedData> {
   const productId = crypto.randomUUID();
   const cityRef = crypto.randomUUID();

frontend/lib/tests/shop/order-items-variants.test.ts

@@ -6,12 +6,7 @@ import { db } from '@/db';
 import { orderItems, orders, productPrices, products } from '@/db/schema/shop';
 import { createOrderWithItems } from '@/lib/services/orders';
 
-const TEST_LEGAL_CONSENT = {
-  termsAccepted: true,
-  privacyAccepted: true,
-  termsVersion: 'terms-2026-02-27',
-  privacyVersion: 'privacy-2026-02-27',
-} as const;
+import { TEST_LEGAL_CONSENT } from './test-legal-consent';
 
 describe('order_items variants (selected_size/selected_color)', () => {
   it('creates two distinct order_items rows for same product with different variants', async () => {

frontend/lib/tests/shop/test-legal-consent.ts

@@ -0,0 +1,6 @@
+export const TEST_LEGAL_CONSENT = {
+  termsAccepted: true,
+  privacyAccepted: true,
+  termsVersion: 'terms-2026-02-27',
+  privacyVersion: 'privacy-2026-02-27',
+} as const;

frontend/messages/en.json

@@ -400,8 +400,8 @@
         "shipping": "Shipping",
         "shippingCalc": "Calculated at checkout",
         "total": "Total",
-        "shippingInformationalOnly": "Informational only",
-        "shippingPayOnDeliveryNote": "Shipping is paid to the carrier on delivery; we currently charge only for goods."
+        "shippingInformationalOnly": "Shown before confirmation",
+        "shippingPayOnDeliveryNote": "When applicable, the shipping cost is included in the total shown before order confirmation. Additional carrier charges for extra services may still apply separately under the carrier's rules."
       },
       "delivery": {
         "legend": "Delivery",
@@ -1732,7 +1732,7 @@
       "title": "Payment Policy",
       "methods": {
         "title": "Available payment methods",
-        "body": "Available payment methods are shown during order checkout. Depending on the order, you may see online card payment via Stripe or a Monobank payment option.",
+        "body": "Available payment methods are shown before order confirmation. Depending on the order, you may see online card payment via Stripe or a Monobank payment option.",
         "items": {
           "stripe": "Online card payment via Stripe.",
           "monobank": "Available Monobank payment option."
@@ -1757,7 +1757,7 @@
       "title": "Delivery Policy",
       "methods": {
         "title": "Available delivery methods",
-        "body": "Available delivery methods are shown during order checkout for orders that can be delivered.",
+        "body": "Available delivery methods are shown before order confirmation for orders that can be delivered.",
         "items": {
           "warehouse": "Nova Poshta warehouse pickup",
           "locker": "Nova Poshta parcel locker pickup",
@@ -1774,7 +1774,7 @@
       },
       "charges": {
         "title": "Delivery charges",
-        "body": "The delivery cost for the selected method is shown during order checkout and is added to the order total before order confirmation. Additional carrier charges for extra services requested after order placement, storage beyond the free period, redirection, or other carrier services may be charged separately under the carrier’s own rules."
+        "body": "The delivery cost for the selected method is shown before order confirmation and is added to the order total before order confirmation. Additional carrier charges for extra services requested after order placement, storage beyond the free period, redirection, or other carrier services may be charged separately under the carrier’s own rules."
       },
       "support": {
         "title": "Delivery questions",

frontend/messages/pl.json

@@ -323,9 +323,9 @@
     "builtWith": "Zbudowane przez",
     "byCommunity": "przez społeczność.",
     "sellerInformation": "Informacje o sprzedawcy",
-    "payment": "Płatność",
-    "delivery": "Dostawa",
-    "returns": "Zwroty",
+    "payment": "Polityka płatności",
+    "delivery": "Polityka dostawy",
+    "returns": "Polityka zwrotów",
     "privacyPolicy": "Polityka Prywatności",
     "termsOfService": "Warunki Świadczenia Usług"
   },
@@ -482,13 +482,13 @@
         "message": "Zostaniesz przekierowany do bezpiecznej płatności lub zobaczysz potwierdzenie, jeśli płatność nie jest wymagana w tym środowisku",
         "notRedirected": "Jeśli nie zostałeś automatycznie przekierowany, otwórz swoje zamówienie",
         "goToOrder": "Przejdź do zamówienia",
-        "consent": {
-          "prefix": "Zapoznałem(-am) się i akceptuję",
-          "termsLink": "Warunki świadczenia usług",
-          "and": "oraz",
-          "privacyLink": "Politykę prywatności",
-          "suffix": " dla tego zamówienia.",
-          "required": "Potwierdź akceptację Warunków świadczenia usług i Polityki prywatności przed złożeniem zamówienia."
+      "consent": {
+        "prefix": "Potwierdzam akceptację",
+        "termsLink": "Warunki świadczenia usług",
+        "and": "oraz",
+        "privacyLink": "Politykę prywatności",
+        "suffix": " dla tego zamówienia.",
+        "required": "Potwierdź akceptację Warunków świadczenia usług i Polityki prywatności przed złożeniem zamówienia."
         },
         "errors": {
           "unexpectedResponse": "Nieoczekiwana odpowiedź checkout.",
@@ -1711,7 +1711,7 @@
     "lastUpdated": "Ostatnia aktualizacja",
     "seller": {
       "metaTitle": "Informacje o sprzedawcy | Sklep DevLovers",
-      "metaDescription": "Publiczne dane sprzedawcy i kontaktowe dla sklepu DevLovers.",
+      "metaDescription": "Publiczne dane sprzedawcy i dane kontaktowe sklepu DevLovers.",
       "title": "Informacje o sprzedawcy",
       "sellerDetailsTitle": "Dane sprzedawcy",
       "sellerDetailsBody": "Poniżej znajdują się dane sprzedawcy, które są obecnie opublikowane dla sklepu.",

frontend/messages/uk.json

@@ -400,8 +400,8 @@
         "shipping": "Доставка",
         "shippingCalc": "Розраховується при оформленні",
         "total": "Всього",
-        "shippingInformationalOnly": "Лише інформаційно",
-        "shippingPayOnDeliveryNote": "Доставка оплачується перевізнику при отриманні; зараз списуємо лише товари."
+        "shippingInformationalOnly": "Показується до підтвердження",
+        "shippingPayOnDeliveryNote": "Якщо доставка застосовується, її вартість додається під час оформлення замовлення і входить до суми, яку ви бачите до підтвердження замовлення. Додаткові платежі перевізнику за окремі послуги можуть стягуватися окремо за його правилами."
       },
       "delivery": {
         "legend": "Доставка",
@@ -430,7 +430,7 @@
           "placeholder": "Почніть вводити назву міста (мін. 2 символи)",
           "selected": "Обране місто: {city}",
           "searching": "Пошук міст...",
-          "noResults": "Міста не знайдено. Перевірте назву або локальні дані Nova Poshta."
+          "noResults": "Міста не знайдено. Перевірте назву або локальні дані Нової пошти."
         },
         "warehouse": {
           "label": "Відділення / поштомат",
@@ -464,15 +464,15 @@
         "methodCards": {
           "warehouse": {
             "title": "Відділення",
-            "description": "Самовивіз із відділення Nova Poshta"
+            "description": "Самовивіз із відділення Нової пошти"
           },
           "locker": {
             "title": "Поштомат",
-            "description": "Самовивіз із поштомату Nova Poshta"
+            "description": "Самовивіз із поштомата Нової пошти"
           },
           "courier": {
             "title": "Кур’єр",
-            "description": "Доставка Nova Poshta до дверей"
+            "description": "Кур’єрська доставка Новою поштою"
           }
         }
       },
@@ -1742,7 +1742,7 @@
       },
       "confirmation": {
         "title": "Коли оплата вважається підтвердженою",
-        "body": "Замовлення вважається оплаченим лише після того, як обраний платіжний провайдер підтвердить транзакцію, а магазин збереже цей результат. Сторінка повернення або редирект у браузері самі по собі не є авторитетним підтвердженням оплати."
+        "body": "Замовлення вважається оплаченим лише після того, як обраний платіжний провайдер підтвердить транзакцію, а магазин збереже цей результат. Сама сторінка повернення чи перенаправлення в браузері ще не означає, що оплату підтверджено."
       },
       "charges": {
         "title": "Що списується онлайн",
@@ -1761,9 +1761,9 @@
         "title": "Доступні способи доставки",
         "body": "Доступні способи доставки показуються під час оформлення замовлення для замовлень, які можна доставити.",
         "items": {
-          "warehouse": "Самовивіз із відділення Nova Poshta",
-          "locker": "Самовивіз із поштомата Nova Poshta",
-          "courier": "Кур’єрська доставка Nova Poshta"
+          "warehouse": "Самовивіз із відділення Нової пошти",
+          "locker": "Самовивіз із поштомата Нової пошти",
+          "courier": "Кур’єрська доставка Новою поштою"
         }
       },
       "availability": {