LP#1673059: Update passwd storage test
authorKathy Lussier <klussier@masslnc.org>
Wed, 15 Mar 2017 12:37:25 +0000 (08:37 -0400)
committerBen Shum <ben@evergreener.net>
Wed, 15 Mar 2017 12:44:49 +0000 (08:44 -0400)
The passwd storage PgTAP test is failing because the user ids shifted when new
sample records were added to the actor.usr table in the Concerto dataset. This
branch changes the id in the test to point to the correct user.

Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <ben@evergreener.net>
Open-ILS/src/sql/Pg/live_t/lp1468422_passwd_storage.pg

index 30c8ad1..18b7cdd 100644 (file)
@@ -19,30 +19,30 @@ SELECT plan(6);
 
 SELECT ok(
     (SELECT TRUE AS verify_old_pw FROM actor.usr 
-        WHERE id = 187 AND passwd = MD5('montyc1234')),
+        WHERE id = 189 AND passwd = MD5('montyc1234')),
     'Legacy password should match'
 );
 
 SELECT isnt_empty(
-    'SELECT actor.get_salt(187, ''main'')',
+    'SELECT actor.get_salt(189, ''main'')',
     'get_salt() returns a new salt'
 );
 
 SELECT isnt_empty(
-    'SELECT * FROM actor.passwd WHERE usr = 187 AND passwd_type = ''main''',
+    'SELECT * FROM actor.passwd WHERE usr = 189 AND passwd_type = ''main''',
     'get_salt() should migrate the password'
 );
 
 SELECT ok(
-    (SELECT actor.verify_passwd(187, 'main', 
-        MD5(actor.get_salt(187, 'main') || MD5('montyc1234')))),
+    (SELECT actor.verify_passwd(189, 'main', 
+        MD5(actor.get_salt(189, 'main') || MD5('montyc1234')))),
     'verify_passwd should verify migrated password'
 );
 
 SELECT ok(
     (SELECT NOT (
-        SELECT actor.verify_passwd(187, 'main', 
-            MD5(actor.get_salt(187, 'main') || MD5('BADPASSWORD'))))
+        SELECT actor.verify_passwd(189, 'main', 
+            MD5(actor.get_salt(189, 'main') || MD5('BADPASSWORD'))))
     ),
     'verify_passwd should fail with wrong password'
 );
@@ -55,12 +55,12 @@ BEGIN
     -- calls will create a new one.
     SELECT INTO new_salt actor.create_salt('main');
     PERFORM actor.set_passwd(
-        187, 'main', MD5(new_salt || MD5('bobblehead')), new_salt);
+        189, 'main', MD5(new_salt || MD5('bobblehead')), new_salt);
 END $$;
 
 SELECT ok(
-    (SELECT actor.verify_passwd(187, 'main', 
-        MD5(actor.get_salt(187, 'main') || MD5('bobblehead')))),
+    (SELECT actor.verify_passwd(189, 'main', 
+        MD5(actor.get_salt(189, 'main') || MD5('bobblehead')))),
     'verify_passwd should verify new password'
 );