Body type is not assignable to type string error

[iby]

The API Gateway example doesn't compile in TypeScript:

TS2322: Type '{ first_name: string; last_name: string; }' is not assignable to type 'string'.

The body is expected to be a string.

[mgarciadelojo]

Having the same issue over here:

Type '{ first_name: string; last_name: string; }' is not assignable to type 'string'.ts(2322)
index.d.ts(80, 5): The expected type comes from property 'body' which is declared here on type 'APIGatewayProxyEvent'

[martysweet]

Looking at the tests for this project, the call to createEvent is defined in a more lax way. This way worked for me over the examples shown in the README.

event-mocks/lib/index.spec.ts

Lines 39 to 44 in 074ef29

	const event = createEvent("aws:apiGateway", {
	body: JSON.stringify({
	first_name: "Sam",
	last_name: "Smith",
	}),
	} as any);

[james-gardner]

Cool but having to cast it as 'any' seems a bit awkward? I've hit a similar issue today and I'm left in a situation where I might as well just mock out the entire event myself and ditch this library.

[joshyork]

Casting to any will work but I believe this issue is because the type of the body param isn't taking into account that some of the required params are supplied by the template.

https://github.com/serverless/event-mocks/blob/master/lib/index.ts#L51

body: typeof dictionary[T],

could instead be defined as

body: Partial<typeof dictionary[T]>,

Even then that's not quite accurate since it'll make all of the properties optional. Even better would be to let the body parameter's type be the type for the corresponding event but with the required fields that are provided by the template marked as optional. I think that's pretty tricky to accomplish dynamically since there isn't a native "merge" type that does what I'm suggesting. Perhaps it'd be more realistic to maintain that type manually but that has its own host of problems.