Unified props passing to PKPass.from between template and PKPass

spec/PKPass.ts

@@ -1052,18 +1052,20 @@ describe("PKPass", () => {
 					),
 				);
 
-				const newPass = await PKPass.from({
+				const newPass = await PKPass.from(
-					model: path.resolve(
+					{
-						__dirname,
+						model: path.resolve(
-						"../examples/models/exampleBooking.pass",
+							__dirname,
-					),
+							"../examples/models/exampleBooking.pass",
-					certificates: {
+						),
-						...baseCerts,
+						certificates: {
+							...baseCerts,
+						},
 					},
-					props: {
+					{
 						voided: true,
 					},
-				});
+				);
 
 				expect(Object.keys(newPass[filesSymbol]).length).toBe(7);
 

src/PKPass.ts

@@ -39,13 +39,10 @@ export default class PKPass extends Bundle {
 
 	static async from<S extends PKPass | Schemas.Template>(
 		source: S,
-		additionalProps?: S extends PKPass
+		props?: Schemas.OverridablePassProps,
-			? Schemas.OverridablePassProps
-			: never,
 	): Promise<PKPass> {
 		let certificates: Schemas.CertificatesSchema = undefined;
 		let buffers: Schemas.FileBuffers = undefined;
-		let props: Schemas.OverridablePassProps = {};
 
 		if (!source) {
 			throw new TypeError(
@@ -86,20 +83,13 @@ export default class PKPass extends Bundle {
 
 			buffers = await getModelFolderContents(source.model);
 			certificates = source.certificates;
-			props = Schemas.validate(
-				Schemas.OverridablePassProps,
-				source.props,
-			);
 		}
 
-		if (additionalProps && Object.keys(additionalProps).length) {
+		return new PKPass(
-			Object.assign(
+			buffers,
-				props,
+			certificates,
-				Schemas.validate(Schemas.OverridablePassProps, additionalProps),
+			Schemas.validate(Schemas.OverridablePassProps, props),
-			);
+		);
-		}
-
-		return new PKPass(buffers, certificates, props);
 	}
 
 	/**

src/schemas/index.ts

@@ -154,13 +154,11 @@ export const PassProps = Joi.object<
 export interface Template {
 	model: string;
 	certificates: CertificatesSchema;
-	props?: OverridablePassProps;
 }
 
 export const Template = Joi.object<Template>({
 	model: Joi.string().required(),
 	certificates: Joi.object().required(),
-	props: OverridablePassProps,
 });
 
 // --------- UTILITIES ---------- //