Fix some types and tests

This commit is contained in:
Magdalena Markusik 2021-01-21 14:21:58 +01:00
parent 93238f8f85
commit 02c2fdf927
3 changed files with 23 additions and 23 deletions

View file

@ -47,8 +47,8 @@ export interface OrderReturnHandlers {
export interface OrderReturnFormData extends OrderReturnData { export interface OrderReturnFormData extends OrderReturnData {
itemsToBeReplaced: FormsetReplacementData; itemsToBeReplaced: FormsetReplacementData;
fulfiledItemsQuantities: FormsetQuantityData; fulfilledItemsQuantities: FormsetQuantityData;
unfulfiledItemsQuantities: FormsetQuantityData; unfulfilledItemsQuantities: FormsetQuantityData;
} }
export type OrderRefundSubmitData = OrderReturnFormData; export type OrderRefundSubmitData = OrderReturnFormData;
@ -184,9 +184,9 @@ function useOrderReturnForm(
}; };
const data: OrderReturnFormData = { const data: OrderReturnFormData = {
fulfiledItemsQuantities: fulfiledItemsQuatities.data, fulfilledItemsQuantities: fulfiledItemsQuatities.data,
itemsToBeReplaced: itemsToBeReplaced.data, itemsToBeReplaced: itemsToBeReplaced.data,
unfulfiledItemsQuantities: unfulfiledItemsQuantites.data, unfulfilledItemsQuantities: unfulfiledItemsQuantites.data,
...form.data ...form.data
}; };

View file

@ -738,7 +738,7 @@ describe("Get the total value of all replaced products", () => {
} }
]; ];
const unfulfiledItemsQuantities: FormsetData<LineItemData, number> = [ const unfulfilledItemsQuantities: FormsetData<LineItemData, number> = [
{ {
data: { isFulfillment: false, isRefunded: false }, data: { isFulfillment: false, isRefunded: false },
id: "1", id: "1",
@ -759,7 +759,7 @@ describe("Get the total value of all replaced products", () => {
} }
]; ];
const fulfiledItemsQuantities: FormsetData<LineItemData, number> = [ const fulfilledItemsQuantities: FormsetData<LineItemData, number> = [
{ {
data: { isFulfillment: true, isRefunded: false }, data: { isFulfillment: true, isRefunded: false },
id: "4", id: "4",
@ -861,8 +861,8 @@ describe("Get the total value of all replaced products", () => {
}, },
{ {
itemsToBeReplaced, itemsToBeReplaced,
unfulfiledItemsQuantities, unfulfilledItemsQuantities,
fulfiledItemsQuantities fulfilledItemsQuantities
} }
); );
@ -1085,42 +1085,42 @@ describe("Get the total value of all selected products", () => {
} }
]; ];
const unfulfiledItemsQuantities: FormsetData<null, number> = [ const unfulfiledItemsQuantities: FormsetData<LineItemData, number> = [
{ {
data: null, data: { isFulfillment: false, isRefunded: false },
id: "1", id: "1",
label: null, label: null,
value: 0 value: 0
}, },
{ {
data: null, data: { isFulfillment: false, isRefunded: false },
id: "2", id: "2",
label: null, label: null,
value: 2 value: 2
}, },
{ {
data: null, data: { isFulfillment: false, isRefunded: false },
id: "3", id: "3",
label: null, label: null,
value: 1 value: 1
} }
]; ];
const fulfiledItemsQuantities: FormsetData<null, number> = [ const fulfiledItemsQuantities: FormsetData<LineItemData, number> = [
{ {
data: null, data: { isFulfillment: true, isRefunded: false },
id: "4", id: "4",
label: null, label: null,
value: 4 value: 4
}, },
{ {
data: null, data: { isFulfillment: true, isRefunded: false },
id: "5", id: "5",
label: null, label: null,
value: 0 value: 0
}, },
{ {
data: null, data: { isFulfillment: true, isRefunded: false },
id: "6", id: "6",
label: null, label: null,
value: 3 value: 3

View file

@ -53,8 +53,8 @@ const getItemPriceAndQuantity = ({
const selectItemPriceAndQuantity = ( const selectItemPriceAndQuantity = (
order: OrderDetails_order, order: OrderDetails_order,
{ {
fulfiledItemsQuantities, fulfilledItemsQuantities,
unfulfiledItemsQuantities unfulfilledItemsQuantities
}: Partial<OrderReturnFormData>, }: Partial<OrderReturnFormData>,
id: string, id: string,
isFulfillment: boolean isFulfillment: boolean
@ -62,12 +62,12 @@ const selectItemPriceAndQuantity = (
isFulfillment isFulfillment
? getItemPriceAndQuantity({ ? getItemPriceAndQuantity({
id, id,
itemsQuantities: fulfiledItemsQuantities, itemsQuantities: fulfilledItemsQuantities,
orderLines: getAllOrderFulfilledLines(order) orderLines: getAllOrderFulfilledLines(order)
}) })
: getItemPriceAndQuantity({ : getItemPriceAndQuantity({
id, id,
itemsQuantities: unfulfiledItemsQuantities, itemsQuantities: unfulfilledItemsQuantities,
orderLines: order.lines orderLines: order.lines
}); });
@ -75,8 +75,8 @@ export const getReplacedProductsAmount = (
order: OrderDetails_order, order: OrderDetails_order,
{ {
itemsToBeReplaced, itemsToBeReplaced,
unfulfiledItemsQuantities, unfulfilledItemsQuantities,
fulfiledItemsQuantities fulfilledItemsQuantities
}: Partial<OrderReturnFormData> }: Partial<OrderReturnFormData>
) => { ) => {
if (!order || !itemsToBeReplaced.length) { if (!order || !itemsToBeReplaced.length) {
@ -94,7 +94,7 @@ export const getReplacedProductsAmount = (
const { unitPrice, selectedQuantity } = selectItemPriceAndQuantity( const { unitPrice, selectedQuantity } = selectItemPriceAndQuantity(
order, order,
{ fulfiledItemsQuantities, unfulfiledItemsQuantities }, { fulfilledItemsQuantities, unfulfilledItemsQuantities },
id, id,
isFulfillment isFulfillment
); );