Skip x509 tests when modules is not installed.

Author: RubenVerborgh

lib/api/authn/webid-tls.js

@@ -72,7 +72,13 @@ function getCertificateViaHeader (req) {
   }
 
   // Parse and convert the certificate to the format the webid library expects
-  if (!x509) x509 = require('x509')
+  if (!x509) {
+    try {
+      x509 = require('x509')
+    } catch (e) {
+      x509 = { parseCert: () => { throw new Error() } }
+    }
+  }
   try {
     const { publicKey, extensions } = x509.parseCert(rawCertificate)
     return {

test/integration/acl-tls-test.js

@@ -1073,6 +1073,14 @@ describe('ACL with WebID+TLS', function () {
 })
 
 describe('ACL with WebID through X-SSL-Cert', function () {
+  let hasX509
+  try {
+    require('x509')
+    hasX509 = true
+  } catch (error) {
+    hasX509 = false
+  }
+
   var ldpHttpsServer
   before(function (done) {
     const ldp = ldnode.createServer({
@@ -1122,10 +1130,12 @@ describe('ACL with WebID through X-SSL-Cert', function () {
     before(prepareRequest(escapedCert, res => { response = res }))
 
     it('should return 200', function () {
+      hasX509 || this.skip()
       assert.propertyVal(response, 'statusCode', 200)
     })
 
     it('should set the User header', function () {
+      hasX509 || this.skip()
       assert.propertyVal(response.headers, 'user', 'https://user1.databox.me/profile/card#me')
     })
   })