Ensure websocket handler types are only applied to websocket handlers

This corrects an issue introduced in #64 where upon adding `fastify-websocket` to a project all route handlers were (accidentally) assumed to be websocket handlers getting the different (and decidedly less useful) types. My bad! This corrects the issue by using a type-land overload of the RouteShorthand function definition to change the type of the handler only if the handler is in fact `{ websocket: true }`.

Also adds tests, `tsd` is handy!

Author: airhorns

index.d.ts

@@ -23,7 +23,7 @@ declare module 'fastify' {
   > {
     <RequestGeneric extends RequestGenericInterface = RequestGenericInterface, ContextConfig = ContextConfigDefault>(
       path: string,
-      opts: RouteShorthandOptions<RawServer, RawRequest, RawReply, RequestGeneric, ContextConfig>,
+      opts: RouteShorthandOptions<RawServer, RawRequest, RawReply, RequestGeneric, ContextConfig> & { websocket: true }, // this creates an overload that only applies these different types if the handler is for websockets
       handler?: WebsocketHandler
     ): FastifyInstance<RawServer, RawRequest, RawReply>;
   }

test/types/index.test-d.ts

@@ -1,5 +1,5 @@
 import wsPlugin, { SocketStream } from '../..';
-import fastify, { WebsocketHandler, FastifyRequest, FastifyInstance, RequestGenericInterface } from 'fastify';
+import fastify, { WebsocketHandler, FastifyRequest, FastifyInstance, RequestGenericInterface, FastifyReply } from 'fastify';
 import { expectType } from 'tsd';
 import { Server as HttpServer, IncomingMessage } from 'http'
 import { Server } from 'ws';
@@ -17,12 +17,27 @@ const handler: WebsocketHandler = (
 
 const handle = (connection: SocketStream): void => {
   expectType<SocketStream>(connection)
-} 
+}
 
 const app: FastifyInstance = fastify();
 app.register(wsPlugin);
 app.register(wsPlugin, {});
 app.register(wsPlugin, { handle } );
 app.register(wsPlugin, { options: { perMessageDeflate: true } });
 
-app.get('/', { websocket: true }, handler);
+app.get('/websockets', { websocket: true }, async (connection, req, params) => {
+  expectType<SocketStream>(connection);
+  expectType<Server>(app.websocketServer);
+  expectType<FastifyRequest<HttpServer, IncomingMessage, RequestGenericInterface>>(req)
+  expectType<{ [key: string]: any } | undefined>(params);
+});
+
+app.get('/not-specifed', async (request, reply) => {
+  expectType<FastifyRequest>(request);
+  expectType<FastifyReply>(reply)
+});
+
+app.get('/not-websockets', { websocket: false }, async (request, reply) => {
+  expectType<FastifyRequest>(request);
+  expectType<FastifyReply>(reply);
+});