quic: fixup broken test when quic unavailable

PR-URL: nodejs#359
Reviewed-By: Anna Henningsen <anna@addaleax.net>

Author: jasnell

doc/api/errors.md

@@ -1695,6 +1695,13 @@ TBD
 
 TBD
 
+<a id="ERR_QUIC_UNAVAILABLE"></a>
+### `ERR_QUIC_UNAVAILABLE`
+
+> Stabililty: 1 - Experimental
+
+TBD
+
 <a id="ERR_QUICCLIENTSESSION_FAILED"></a>
 ### `ERR_QUICCLIENTSESSION_FAILED`
 

lib/internal/errors.js

@@ -1335,6 +1335,7 @@ E('ERR_QUIC_ERROR', function(code, family) {
   return `QUIC session closed with ${familyType} error code ${code}`;
 }, Error);
 E('ERR_QUIC_TLS13_REQUIRED', 'QUIC requires TLS version 1.3', Error);
+E('ERR_QUIC_UNAVAILABLE', 'QUIC is unavailable', Error);
 E('ERR_REQUIRE_ESM',
   (filename, parentPath = null, packageJsonPath = null) => {
     let msg = `Must use import to load ES Module: ${filename}`;

lib/internal/quic/core.js

@@ -245,6 +245,8 @@ const kSocketDestroyed = 4;
 let diagnosticPacketLossWarned = false;
 let warnedVerifyHostnameIdentity = false;
 
+assert(process.versions.ngtcp2 !== undefined);
+
 // Called by the C++ internals when the socket is closed.
 // When this is called, the only thing left to do is destroy
 // the QuicSocket instance.

lib/internal/quic/util.js

@@ -9,6 +9,9 @@ const {
   },
 } = require('internal/errors');
 
+const assert = require('internal/assert');
+assert(process.versions.ngtcp2 !== undefined);
+
 const {
   isIP,
 } = require('internal/net');

lib/quic.js

@@ -1,5 +1,16 @@
 'use strict';
 
+const {
+  codes: {
+    ERR_QUIC_UNAVAILABLE,
+  },
+} = require('internal/errors');
+const { assertCrypto } = require('internal/util');
+
+assertCrypto();
+if (process.versions.ngtcp2 === undefined)
+  throw new ERR_QUIC_UNAVAILABLE();
+
 const {
   createSocket
 } = require('internal/quic/core');

src/node_native_module.cc

@@ -87,23 +87,20 @@ void NativeModuleLoader::InitializeModuleCategories() {
       "crypto",
       "https",
       "http2",
-#if !defined(NODE_EXPERIMENTAL_QUIC)
-      "quic",
-#endif
       "tls",
       "_tls_common",
       "_tls_wrap",
       "internal/http2/core",
       "internal/http2/compat",
-#if !defined(NODE_EXPERIMENTAL_QUIC)
-      "internal/quic/core",
-      "internal/quic/util",
-#endif
       "internal/policy/manifest",
       "internal/process/policy",
       "internal/streams/lazy_transform",
 #endif  // !HAVE_OPENSSL
-
+#if !defined(NODE_EXPERIMENTAL_QUIC)
+      "quic",
+      "internal/quic/core",
+      "internal/quic/util",
+#endif
       "sys",  // Deprecated.
       "wasi",  // Experimental.
       "internal/test/binding",