getObject does not return Error objects for NoSuchKey

[gabegorelick]

When an object doesn't exist, getObject returns an object with the requisite properties, but it's not an Error object. This makes debugging more difficult (since there's no stack property), breaks Nodejs conventions (callers typically expect the first argument to a callback to be an Error object), and deviates from the official AWS SDK (which returns Error objects).

mock-aws-s3/lib/mock.js

Lines 331 to 340 in 6b5aa02

	if (err.code === 'ENOENT') {
	return callback({
	cfId: undefined,
	code: 'NoSuchKey',
	message: 'The specified key does not exist.',
	name: 'NoSuchKey',
	region: null,
	statusCode: 404
	}, search)
	}

Every other method seems to correctly return Error objects. So this appears to be an outlier.