From 55baa7b586398f9ee4b529d57ad38cbdca067ae6 Mon Sep 17 00:00:00 2001 From: Côme Chilliet Date: Thu, 12 Jan 2023 15:36:50 +0100 Subject: Fixing samba test and adding details to kerberos job MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Côme Chilliet --- .github/workflows/smb-kerberos.yml | 2 ++ apps/files_external/tests/Storage/SmbTest.php | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/smb-kerberos.yml b/.github/workflows/smb-kerberos.yml index 3783c0ecfc7..182e9787bfe 100644 --- a/.github/workflows/smb-kerberos.yml +++ b/.github/workflows/smb-kerberos.yml @@ -53,8 +53,10 @@ jobs: chmod 0777 cookies DC_IP=$(docker inspect dc --format '{{.NetworkSettings.IPAddress}}') + echo "SAML login" docker run --rm --name client -v $PWD/cookies:/cookies -v /tmp/shared:/shared --dns $DC_IP --hostname client.domain.test icewind1991/samba-krb-test-client \ curl -c /cookies/jar -s --negotiate -u testuser@DOMAIN.TEST: --delegation always http://httpd.domain.test/index.php/apps/user_saml/saml/login + echo "Check we are logged in" CONTENT=$(docker run --rm --name client -v $PWD/cookies:/cookies -v /tmp/shared:/shared --dns $DC_IP --hostname client.domain.test icewind1991/samba-krb-test-client \ curl -b /cookies/jar -s --negotiate -u testuser@DOMAIN.TEST: --delegation always http://httpd.domain.test/remote.php/webdav/smb/test.txt) echo $CONTENT diff --git a/apps/files_external/tests/Storage/SmbTest.php b/apps/files_external/tests/Storage/SmbTest.php index e1d7b82479d..30040e499c8 100644 --- a/apps/files_external/tests/Storage/SmbTest.php +++ b/apps/files_external/tests/Storage/SmbTest.php @@ -122,7 +122,7 @@ class SmbTest extends \Test\Files\Storage\Storage { ]; foreach ($expected as $expectedChange) { - $this->assertContains($expectedChange, $changes, 'Actual changes are:' . PHP_EOL . print_r($changes, true), false, false); // dont check object identity + $this->assertTrue(in_array($expectedChange, $changes), 'Actual changes are:' . PHP_EOL . print_r($changes, true) . PHP_EOL . 'Expected to find: ' . PHP_EOL . print_r($expectedChange, true)); } } -- cgit v1.2.3