From 947615d38f343660405f3155c7cfcde0d9f38ef5 Mon Sep 17 00:00:00 2001
From: Demian Katz <demian.katz@villanova.edu>
Date: Wed, 28 Feb 2018 16:02:27 -0500
Subject: [PATCH] Fix risky test by adding assertions.
---
.../src/VuFindTest/Auth/DatabaseTest.php | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/module/VuFind/tests/integration-tests/src/VuFindTest/Auth/DatabaseTest.php b/module/VuFind/tests/integration-tests/src/VuFindTest/Auth/DatabaseTest.php
index 3add1a20c53..171ec5dc803 100644
--- a/module/VuFind/tests/integration-tests/src/VuFindTest/Auth/DatabaseTest.php
+++ b/module/VuFind/tests/integration-tests/src/VuFindTest/Auth/DatabaseTest.php
@@ -185,14 +185,20 @@ class DatabaseTest extends \VuFindTest\Unit\DbTestCase
}
/**
- * Test password mismatch.
+ * Test successful account creation.
*
* @return void
*/
public function testSuccessfulCreation()
{
$request = $this->getAccountCreationRequest();
- $this->auth->create($request);
+ $newUser = $this->auth->create($request)->toArray();
+ foreach ($request->getPost() as $key => $value) {
+ // Skip the password confirmation value!
+ if ($key !== 'password2') {
+ $this->assertEquals($value, $newUser[$key]);
+ }
+ }
}
/**
--
GitLab