test: remove SASL authentication tests

Author: happy-game

packages/pg-protocol/src/inbound-parser.test.ts

@@ -194,9 +194,10 @@ const testForMessage = function (buffer: Buffer, expectedMessage: any) {
 
 const plainPasswordBuffer = buffers.authenticationCleartextPassword()
 const md5PasswordBuffer = buffers.authenticationMD5Password()
-const SASLBuffer = buffers.authenticationSASL()
-const SASLContinueBuffer = buffers.authenticationSASLContinue()
-const SASLFinalBuffer = buffers.authenticationSASLFinal()
+// SASL authentication is no longer supported
+// const SASLBuffer = buffers.authenticationSASL()
+// const SASLContinueBuffer = buffers.authenticationSASLContinue()
+// const SASLFinalBuffer = buffers.authenticationSASLFinal()
 
 const expectedPlainPasswordMessage = {
   name: 'authenticationCleartextPassword',
@@ -207,6 +208,8 @@ const expectedMD5PasswordMessage = {
   salt: Buffer.from([1, 2, 3, 4]),
 }
 
+// SASL authentication is no longer supported
+/*
 const expectedSASLMessage = {
   name: 'authenticationSASL',
   mechanisms: ['SCRAM-SHA-256'],
@@ -221,6 +224,7 @@ const expectedSASLFinalMessage = {
   name: 'authenticationSASLFinal',
   data: 'data',
 }
+*/
 
 const notificationResponseBuffer = buffers.notification(4, 'hi', 'boom')
 const expectedNotificationResponseMessage = {
@@ -245,6 +249,9 @@ describe('PgPacketStream', function () {
   testForMessage(authOkBuffer, expectedAuthenticationOkayMessage)
   testForMessage(plainPasswordBuffer, expectedPlainPasswordMessage)
   testForMessage(md5PasswordBuffer, expectedMD5PasswordMessage)
+
+  // SASL authentication tests are commented out as SASL is no longer supported
+  /*
   testForMessage(SASLBuffer, expectedSASLMessage)
   testForMessage(SASLContinueBuffer, expectedSASLContinueMessage)
 
@@ -261,6 +268,7 @@ describe('PgPacketStream', function () {
   // and adds a test which is deterministic, rather than relying on network packet chunking
   const extendedSASLFinalBuffer = Buffer.concat([SASLFinalBuffer, Buffer.from([1, 2, 4, 5])])
   testForMessage(extendedSASLFinalBuffer, expectedSASLFinalMessage)
+  */
 
   testForMessage(paramStatusBuffer, expectedParameterStatusMessage)
   testForMessage(backendKeyDataBuffer, expectedBackendKeyDataMessage)

packages/pg-protocol/src/outbound-serializer.test.ts

@@ -12,7 +12,7 @@ describe('serializer', () => {
       actual,
       new BufferList()
         .addInt16(3)
-        .addInt16(0)
+        .addInt16(0x33) // this is for gaussdb
         .addCString('user')
         .addCString('brian')
         .addCString('database')

packages/pg-protocol/src/testing/test-buffers.ts

@@ -21,6 +21,8 @@ const buffers = {
       .join(true, 'R')
   },
 
+  // SASL authentication is no longer supported - commented out
+  /*
   authenticationSASL: function () {
     return new BufferList().addInt32(10).addCString('SCRAM-SHA-256').addCString('').join(true, 'R')
   },
@@ -32,6 +34,7 @@ const buffers = {
   authenticationSASLFinal: function () {
     return new BufferList().addInt32(12).addString('data').join(true, 'R')
   },
+  */
 
   parameterStatus: function (name: string, value: string) {
     return new BufferList().addCString(name).addCString(value).join(true, 'S')

packages/pg/test/test-buffers.js

@@ -23,6 +23,8 @@ buffers.authenticationMD5Password = function () {
     .join(true, 'R')
 }
 
+// SASL authentication is no longer supported - commented out
+/*
 buffers.authenticationSASL = function () {
   return new BufferList().addInt32(10).addCString('SCRAM-SHA-256').addCString('').join(true, 'R')
 }
@@ -34,6 +36,7 @@ buffers.authenticationSASLContinue = function () {
 buffers.authenticationSASLFinal = function () {
   return new BufferList().addInt32(12).addString('data').join(true, 'R')
 }
+*/
 
 buffers.parameterStatus = function (name, value) {
   return new BufferList().addCString(name).addCString(value).join(true, 'S')

packages/pg/test/unit/client/sasl-scram-tests.js

@@ -1,3 +1,7 @@
+// SASL authentication is no longer supported in this repository
+// All SASL tests have been commented out
+
+/*
 'use strict'
 const helper = require('./test-helper')
 const assert = require('assert')
@@ -312,3 +316,4 @@ suite.test('sasl/scram', function () {
     })
   })
 })
+*/