From bf251d4bcc9a0ad455ef36e942298acb3887fb64 Mon Sep 17 00:00:00 2001
From: brinn <brinn>
Date: Fri, 8 Oct 2010 12:47:04 +0000
Subject: [PATCH] fix: unit test expectations (merged from S91.x)

SVN: 18224
---
 .../generic/server/CommonServerTest.java       | 18 +++++++++---------
 .../generic/shared/AbstractServerTestCase.java |  4 ++--
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/server/CommonServerTest.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/server/CommonServerTest.java
index f606d26c81a..8c7a4696425 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/server/CommonServerTest.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/server/CommonServerTest.java
@@ -134,7 +134,7 @@ public final class CommonServerTest extends AbstractServerTestCase
     @Test
     public void testLogout()
     {
-        final Session session = createSession();
+        final Session session = createSession(CommonTestUtils.USER_ID);
         context.checking(new Expectations()
             {
                 {
@@ -153,7 +153,7 @@ public final class CommonServerTest extends AbstractServerTestCase
     {
         final String user = "user";
         final String password = "password";
-        final Session session = createSession();
+        final Session session = createSession(user);
         context.checking(new Expectations()
             {
                 {
@@ -176,7 +176,7 @@ public final class CommonServerTest extends AbstractServerTestCase
     {
         final String user = "user";
         final String password = "password";
-        final Session session = createSession();
+        final Session session = createSession(user);
         final PersonPE systemPerson = createSystemUser();
         final PersonPE person = CommonTestUtils.createPersonFromPrincipal(PRINCIPAL);
         context.checking(new Expectations()
@@ -223,7 +223,7 @@ public final class CommonServerTest extends AbstractServerTestCase
     {
         final String user = "user";
         final String password = "password";
-        final Session session = createSession();
+        final Session session = createSession(user);
         final PersonPE systemPerson = createSystemUser();
         final PersonPE person = CommonTestUtils.createPersonFromPrincipal(PRINCIPAL);
         context.checking(new Expectations()
@@ -260,7 +260,7 @@ public final class CommonServerTest extends AbstractServerTestCase
     {
         final String user = "user";
         final String password = "password";
-        final Session session = createSession();
+        final Session session = createSession(user);
         final PersonPE systemPerson = createSystemUser();
         final PersonPE person = CommonTestUtils.createPersonFromPrincipal(PRINCIPAL);
         context.checking(new Expectations()
@@ -294,7 +294,7 @@ public final class CommonServerTest extends AbstractServerTestCase
         final DatabaseInstanceIdentifier identifier = DatabaseInstanceIdentifier.createHome();
         final GroupPE g1 = CommonTestUtils.createGroup("g1", homeDatabaseInstance);
         final GroupPE g2 = CommonTestUtils.createGroup("g2", homeDatabaseInstance);
-        final Session session = createSession();
+        final Session session = createSession(CommonTestUtils.USER_ID);
         session.setPerson(person);
         person.setHomeGroup(g1);
         g1.setId(42L);
@@ -1365,7 +1365,7 @@ public final class CommonServerTest extends AbstractServerTestCase
             {
                 {
                     allowing(sessionManager).getSession(SESSION_TOKEN);
-                    Session session = createSession();
+                    Session session = createSession(CommonTestUtils.USER_ID);
                     session.setPerson(person);
                     will(returnValue(session));
 
@@ -1391,7 +1391,7 @@ public final class CommonServerTest extends AbstractServerTestCase
             {
                 {
                     allowing(sessionManager).getSession(SESSION_TOKEN);
-                    Session session = createSession();
+                    Session session = createSession(CommonTestUtils.USER_ID);
                     session.setPerson(person);
                     will(returnValue(session));
 
@@ -1420,7 +1420,7 @@ public final class CommonServerTest extends AbstractServerTestCase
             {
                 {
                     allowing(sessionManager).getSession(SESSION_TOKEN);
-                    Session session = createSession();
+                    Session session = createSession(CommonTestUtils.USER_ID);
                     session.setPerson(person);
                     will(returnValue(session));
                 }
diff --git a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/AbstractServerTestCase.java b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/AbstractServerTestCase.java
index 2ce8d421f6f..a39db0efe50 100644
--- a/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/AbstractServerTestCase.java
+++ b/openbis/sourceTest/java/ch/systemsx/cisd/openbis/generic/shared/AbstractServerTestCase.java
@@ -261,9 +261,9 @@ public abstract class AbstractServerTestCase extends AssertJUnit
         context.assertIsSatisfied();
     }
 
-    protected Session createSession()
+    protected Session createSession(String user)
     {
-        return new Session(CommonTestUtils.USER_ID, SESSION_TOKEN, PRINCIPAL, "remote-host", 2);
+        return new Session(user, SESSION_TOKEN, PRINCIPAL, "remote-host", 2);
     }
 
     protected void prepareGetSession()
-- 
GitLab