Resolves mismatch here (balderdashy#2291 (comment))

pdipietro · Oct 12, 2014 · ff7b850 · ff7b850
1 parent d40a60b
commit ff7b850
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/test/integration/hook.cors_csrf.test.js b/test/integration/hook.cors_csrf.test.js
@@ -85,7 +85,17 @@ describe('CORS and CSRF ::', function() {
           }, function(err, response) {
             if (err) return done(new Error(err));
             assert.equal(response.statusCode, 200);
-            assert.equal(response.body, "GET,POST,PUT,HEAD,DELETE,TRACE,COPY,LOCK,MKCOL,MOVE,PURGE,PROPFIND,PROPPATCH,UNLOCK,REPORT,MKACTIVITY,CHECKOUT,MERGE,M-SEARCH,NOTIFY,SUBSCRIBE,UNSUBSCRIBE,PATCH,SEARCH,CONNECT");
+            //
+            // NOTE:
+            // The bump to Express 3.16.0 caused a change here.
+            // see https://github.com/balderdashy/sails/pull/2291#issuecomment-58809939
+            //
+            // Original test:
+            // assert.equal(response.body, "GET,POST,PUT,HEAD,DELETE,TRACE,COPY,LOCK,MKCOL,MOVE,PURGE,PROPFIND,PROPPATCH,UNLOCK,REPORT,MKACTIVITY,CHECKOUT,MERGE,M-SEARCH,NOTIFY,SUBSCRIBE,UNSUBSCRIBE,PATCH,SEARCH,CONNECT");
+
+            // Changed to match what comes back post-upgrade:
+            // (removed "PURGE", "SEARCH", and "CONNECT")
+            assert.equal(response.body, "GET,POST,PUT,HEAD,DELETE,TRACE,COPY,LOCK,MKCOL,MOVE,PROPFIND,PROPPATCH,UNLOCK,REPORT,MKACTIVITY,CHECKOUT,MERGE,M-SEARCH,NOTIFY,SUBSCRIBE,UNSUBSCRIBE,PATCH", require('util').format('Unexpected HTTP methods:  "%s"', response.body));
             done();
           });