@duffel/components 3.0.7-canary → 3.1.2

package/src/styles/components/Seat.css

@@ -21,16 +21,17 @@
 }
 
 .map-element--available {
-  border: 2px solid rgba(var(--ACCENT), var(--ACCENT-LIGHT-200));
+  border: 2px solid
+    var(--TERTIARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-200)));
   background-color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-100));
-  color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-300));
+  color: var(--TERTIARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-300)));
 }
 
 .map-element--fee-payable {
   position: absolute;
   bottom: -2px;
   right: -2px;
-  color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-300));
+  color: var(--TERTIARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-300)));
 }
 
 @media (hover: hover) and (pointer: fine) {
@@ -40,18 +41,19 @@
 
   .map-element--actionable:not(.map-element--selected):hover {
     background-color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-200));
-    color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000));
+    color: var(--SECONDARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000)));
     transition: var(--TRANSITIONS-CUBIC-BEZIER);
   }
 
   .map-element--fee-payable {
-    color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000));
+    color: var(--TERTIARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000)));
     transition: var(--TRANSITIONS-CUBIC-BEZIER);
   }
 }
 
 .map-element--selected {
   background-color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000));
-  border: 2px solid rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000));
-  color: rgba(var(--WHITE), 1);
+  border: 2px solid
+    var(--SECONDARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000)));
+  color: var(--SECONDARY, rgba(var(--WHITE), 1));
 }

package/src/styles/components/SeatInfo.css

@@ -1,5 +1,5 @@
 .seat-info {
-  --INFO-CARD-MARGIN: 10px;
+  --INFO-CARD-MARGIN: 20px;
   background-color: rgb(var(--WHITE));
   padding: 24px;
   padding-bottom: 24px;

package/src/styles/components/Tabs.css

@@ -28,8 +28,11 @@
 .seat-map__tab-select-option:hover,
 .seat-map__tab-select-option:focus,
 .seat-map__tab-select-option:active {
-  color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000));
-  background-color: rgba(var(--ACCENT), var(--ACCENT-LIGHT-200));
+  color: var(--SECONDARY, rgba(var(--ACCENT), var(--ACCENT-LIGHT-1000)));
+  background-color: var(
+    --TERTIARY,
+    rgba(var(--ACCENT), var(--ACCENT-LIGHT-200))
+  );
   transition: var(--TRANSITIONS-CUBIC-BEZIER);
 }
 

package/src/styles/global.css

@@ -12,6 +12,7 @@
 @import "./components/CfarSelectionModal.css";
 @import "./components/Card.css";
 @import "./components/Counter.css";
+@import "./components/DuffelPayments.css";
 @import "./components/IconButton.css";
 @import "./components/Legend.css";
 @import "./components/Loader.css";
@@ -19,6 +20,7 @@
 @import "./components/Modal.css";
 @import "./components/PassengerSelect.css";
 @import "./components/PassengersLayout.css";
+@import "./components/PlacesLookup.css";
 @import "./components/Row.css";
 @import "./components/Seat.css";
 @import "./components/SeatInfo.css";

package/src/tests/components/DuffelAncillaries.test.tsx

@@ -1,6 +1,6 @@
 import { fireEvent, render } from "@testing-library/react";
 import React from "react";
-import { DuffelAncillaries } from "../../components/DuffelAncillaries";
+import { DuffelAncillaries } from "../../components/DuffelAncillaries/DuffelAncillaries";
 import mockPassengers from "../../fixtures/passengers/mock_passengers";
 import {
   DuffelAncillariesPropsWithOffersAndSeatMaps,

package/src/tests/lib/createPriceFormatters.test.tsx

@@ -1,4 +1,4 @@
-import { createPriceFormatters } from "@lib/createPriceFormatters";
+import { createPriceFormatters } from "../../lib/createPriceFormatters";
 
 describe("createPriceFormatters", () => {
   it("should return an empty object if neither markup nor priceFormatters are supplied", () => {

package/src/tests/lib/formatAvailableServices.test.tsx

@@ -1,4 +1,4 @@
-import { formatAvailableServices } from "@lib/formatAvailableServices";
+import { formatAvailableServices } from "../../lib/formatAvailableServices";
 import { OfferAvailableServiceBaggage } from "../../types/Offer";
 
 /* eslint-disable @typescript-eslint/no-var-requires */

package/src/tests/lib/formatSeatMaps.test.tsx

@@ -1,5 +1,5 @@
-import { formatSeatMaps } from "@lib/formatSeatMaps";
-import { getFirstSeatElementMatchingCriteria } from "@lib/getFirstSeatElementMatchingCriteria";
+import { formatSeatMaps } from "../../lib/formatSeatMaps";
+import { getFirstSeatElementMatchingCriteria } from "../../lib/getFirstSeatElementMatchingCriteria";
 import { SeatMapCabinRowSectionElementSeat } from "../../types/SeatMap";
 
 /* eslint-disable @typescript-eslint/no-var-requires */

package/src/tests/lib/getCurrencyForServices.test.tsx

@@ -1,4 +1,4 @@
-import { getCurrencyForServices } from "@lib/getCurrencyForServices";
+import { getCurrencyForServices } from "../../lib/getCurrencyForServices";
 import { Offer } from "../../types/Offer";
 
 describe("getCurrencyForServices", () => {

package/src/tests/lib/hasServiceOfSameMetadataTypeAlreadyBeenSelected.test.ts

@@ -1,4 +1,4 @@
-import { hasServiceOfSameMetadataTypeAlreadyBeenSelected } from "@lib/hasServiceOfSameMetadataTypeAlreadyBeenSelected";
+import { hasServiceOfSameMetadataTypeAlreadyBeenSelected } from "../../lib/hasServiceOfSameMetadataTypeAlreadyBeenSelected";
 import { CreateOrderPayloadService } from "../../types/CreateOrderPayload";
 import { OfferAvailableBaggageService } from "../../types/Offer";
 

package/src/tests/lib/logging.test.tsx

@@ -1,32 +1,32 @@
-import { initializeLogger } from "@lib/logging";
+import { initializeLogger, log, logGroup } from "../../lib/logging";
 
 const consoleSpies = {
-  log: jest.spyOn(console, "log"),
+  log: jest.spyOn(console, "info"),
   groupCollapsed: jest.spyOn(console, "groupCollapsed"),
   groupEnd: jest.spyOn(console, "groupEnd"),
 };
 
-describe("initializeLogger", () => {
+describe("logging", () => {
   it("should return functions that do nothing when debugMode is false", () => {
-    const logger = initializeLogger(false);
-    logger.log("This should not be logged");
+    initializeLogger(false);
+    log("This should not be logged");
     expect(consoleSpies.log).not.toHaveBeenCalled();
   });
 
   it("should return functions that log a message when debugMode is true", () => {
-    const logger = initializeLogger(true);
-    logger.log("This should be logged");
+    initializeLogger(true);
+    log("This should be logged");
 
-    // Twice because initializeLogger logs a message when it's called,
-    // and then the message we're testing.
-    expect(consoleSpies.log).toHaveBeenCalledTimes(2);
+    // Once because initializeLogger was initialised on the test above
+    // And so, the initialisation message would not show up twice.
+    expect(consoleSpies.log).toHaveBeenCalledTimes(1);
 
-    logger.logGroup("This should be logged", ["one", "two"]);
+    logGroup("This should be logged", ["one", "two"]);
     expect(consoleSpies.groupCollapsed).toHaveBeenCalledTimes(1);
     expect(consoleSpies.groupEnd).toHaveBeenCalledTimes(1);
 
-    // Four times: once for the initializeLogger message, once for the
-    // earlier message, and twice for the array passed to logGroup.
-    expect(consoleSpies.log).toHaveBeenCalledTimes(4);
+    // once for the call to `log` above and then
+    // twice for the array passed to logGroup
+    expect(consoleSpies.log).toHaveBeenCalledTimes(3);
   });
 });

package/src/tests/lib/moneyStringFormatter.test.tsx

@@ -1,4 +1,4 @@
-import { moneyStringFormatter } from "@lib/moneyStringFormatter";
+import { moneyStringFormatter } from "../../lib/moneyStringFormatter";
 
 describe("moneyStringFormatter", () => {
   it("should format known currencies correctly", () => {

package/src/tests/lib/validateProps.test.tsx

@@ -4,7 +4,7 @@
  * that we can test functions with invalid props.
  */
 
-import { hasCommonRequiredProps } from "@lib/validateProps";
+import { hasCommonRequiredProps } from "../../lib/validateProps";
 import { Ancillaries } from "../../types/DuffelAncillariesProps";
 
 describe("hasCommonRequiredProps", () => {

package/src/types/DuffelAncillariesProps.ts

@@ -75,7 +75,7 @@ export type DuffelAncillariesPriceFormatterForCancelForAnyReason = (
   amount: number,
   currency: string,
   service: OfferAvailableServiceCancelForAnyReason
-) => { amount: number; currency: string };
+) => { amount: number; currency?: string };
 
 export interface DuffelAncillariesPriceFormatters {
   bags?: DuffelAncillariesPriceFormatterForBags;