From 5f30ea36580acd0a2c4478cc1438b9d1a812d486 Mon Sep 17 00:00:00 2001 From: Nicola Murino Date: Sun, 17 Dec 2023 10:46:21 +0100 Subject: [PATCH] tests: add some logs to debug some sporadic test failures Signed-off-by: Nicola Murino --- internal/common/transferschecker_test.go | 2 +- internal/sftpd/sftpd_test.go | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/internal/common/transferschecker_test.go b/internal/common/transferschecker_test.go index a2f197192..e070fb812 100644 --- a/internal/common/transferschecker_test.go +++ b/internal/common/transferschecker_test.go @@ -314,7 +314,7 @@ func TestTransferCheckerTransferQuota(t *testing.T) { transfer1.BytesReceived.Store(1024*1024 + 1) transfer2.BytesReceived.Store(0) Connections.checkTransfers() - assert.True(t, conn1.IsQuotaExceededError(transfer1.errAbort)) + assert.True(t, conn1.IsQuotaExceededError(transfer1.errAbort), transfer1.errAbort) assert.Nil(t, transfer2.errAbort) transfer1.errAbort = nil transfer1.BytesReceived.Store(1024*1024 + 1) diff --git a/internal/sftpd/sftpd_test.go b/internal/sftpd/sftpd_test.go index eb462335b..23dd2ef52 100644 --- a/internal/sftpd/sftpd_test.go +++ b/internal/sftpd/sftpd_test.go @@ -10378,7 +10378,9 @@ func TestSCPTransferQuotaLimits(t *testing.T) { user, _, err = httpdtest.GetUserByUsername(user.Username, http.StatusOK) assert.NoError(t, err) assert.Greater(t, user.UsedDownloadDataTransfer, int64(1024*1024)) - assert.Greater(t, user.UsedUploadDataTransfer, int64(1024*1024)) + if !assert.Greater(t, user.UsedUploadDataTransfer, int64(1024*1024)) { + printLatestLogs(30) + } err = os.Remove(testFilePath) assert.NoError(t, err)