Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/NbDataAccessTestCase.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/Attic/NbDataAccessTestCase.java,v
diff -u -r1.6 -r1.7
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/NbDataAccessTestCase.java	29 Jul 2005 04:31:47 -0000	1.6
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/NbDataAccessTestCase.java	11 Aug 2005 05:41:40 -0000	1.7
@@ -139,7 +139,11 @@
 	 /** Define the context files. Overrides method in AbstractLamsTestCase */
     protected String[] getContextConfigLocation() {
     	return new String[] {
-    	       "org/lamsfoundation/lams/tool/noticeboard/testApplicationContext.xml"};
+    	        //"org/lamsfoundation/lams/applicationContext.xml",
+    			//"org/lamsfoundation/lams/lesson/lessonApplicationContext.xml",
+    			//"org/lamsfoundation/lams/tool/toolApplicationContext.xml",
+    			//"org/lamsfoundation/lams/learning/learningApplicationContext.xml",
+    			"org/lamsfoundation/lams/tool/noticeboard/testApplicationContext.xml"};
     }
     
     /** Define the sessionFactory bean name located in testApplication.xml. */
@@ -293,8 +297,7 @@
     		assertEquals(content.getOfflineInstructions(), DEFAULT_OFFLINE_INSTRUCTIONS);
     		assertEquals(content.isDefineLater(), DEFAULT_DEFINE_LATER);
     		assertEquals(content.isForceOffline(), DEFAULT_FORCE_OFFLINE);
-    		assertEquals(content.getCreatorUserId(), DEFAULT_CREATOR_USER_ID);
-     } 
+    		     } 
     
     protected void assertEqualsForSessionContent(NoticeboardSession ns)
     {
Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/testApplicationContext.xml
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/Attic/testApplicationContext.xml,v
diff -u -r1.6 -r1.7
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/testApplicationContext.xml	29 Jul 2005 04:32:11 -0000	1.6
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/testApplicationContext.xml	11 Aug 2005 05:41:40 -0000	1.7
@@ -14,14 +14,14 @@
 		<property name="password"><value>lamsdemo</value></property>
 	</bean>
 
-	<!-- Hibernate SessionFactory -->
-	<bean id="nbSessionFactory" class="org.springframework.orm.hibernate.LocalSessionFactoryBean">
+<!-- Hibernate SessionFactory -->
+	  <bean id="nbSessionFactory" class="org.springframework.orm.hibernate.LocalSessionFactoryBean">
 		<property name="dataSource"><ref local="localDataSource"/></property>
 		<property name="mappingResources">
 			<list>	
 				<value>org/lamsfoundation/lams/tool/noticeboard/NoticeboardContent.hbm.xml</value>
-				<value>org/lamsfoundation/lams/tool/noticeboard/NoticeboardSession.hbm.xml</value> 
-				<value>org/lamsfoundation/lams/tool/noticeboard/NoticeboardUser.hbm.xml</value> 
+				<value>org/lamsfoundation/lams/tool/noticeboard/NoticeboardSession.hbm.xml</value>
+				<value>org/lamsfoundation/lams/tool/noticeboard/NoticeboardUser.hbm.xml</value>
 				<value>org/lamsfoundation/lams/tool/noticeboard/NoticeboardAttachment.hbm.xml</value> 
 			</list>
 		</property>
@@ -54,51 +54,41 @@
 				<property name="nbSessionDAO"><ref bean="nbSessionDAO"/></property>
 				<property name="nbUserDAO"><ref bean="nbUserDAO"/></property>
 				<property name="nbAttachmentDAO"><ref bean="nbAttachmentDAO"/></property>
-			</bean>
+			<!-- 	<property name="toolService"><ref bean="lamsToolService"/></property>
+				<property name="learnerService"><ref bean="learnerService"/></property> -->
+				</bean>
 		</property>
 		<property name="transactionAttributes">
 			<props>
 				<prop key="create*">PROPAGATION_REQUIRED</prop>
 				<prop key="copy*">PROPAGATION_REQUIRED</prop>
 				<prop key="update*">PROPAGATION_REQUIRED</prop>
 				<prop key="get*">PROPAGATION_REQUIRED</prop>
+				<prop key="save*">PROPAGATION_REQUIRED</prop>
 				<prop key="retrieve*">PROPAGATION_REQUIRED</prop>
 				<prop key="remove*">PROPAGATION_REQUIRED</prop>
-				<prop key="save*">PROPAGATION_REQUIRED</prop>
+				<prop key="add*">PROPAGATION_REQUIRED</prop>
 			</props>
 		</property>
 	</bean> 
 
-	<!--
-	  - Lams survey primary business object (default implementation), as an inner bean wrapped
-		- by an outer transactional proxy. The two bean definitions could have been separate,
-		- but this is cleaner as there is no need to ever access the unwrapped object.
-		-->
 
-	<!--  <bean id="nbService" parent="baseTransactionProxy">
-		<property name="target">
-			<bean class="org.lamsfoundation.lams.tool.noticeboard.service.NoticeboardServicePOJO">
-				<property name="nbContentDAO"><ref bean="nbContentDAO"/></property>
-				<property name="nbSessionDAO"><ref bean="nbSessionDAO"/></property>
-			</bean>
-		</property>
-	</bean>  -->
-	
+
 	<!--  DAO classes -->
 	<bean id="nbContentDAO" class="org.lamsfoundation.lams.tool.noticeboard.dao.hibernate.NoticeboardContentDAO">
 		<property name="sessionFactory"><ref local="nbSessionFactory"/></property>
 	</bean>  
-	
+
 	<bean id="nbSessionDAO" class="org.lamsfoundation.lams.tool.noticeboard.dao.hibernate.NoticeboardSessionDAO">
 		<property name="sessionFactory"><ref local="nbSessionFactory"/></property>
-	</bean> 
-
+	</bean>
+	
 	<bean id="nbUserDAO" class="org.lamsfoundation.lams.tool.noticeboard.dao.hibernate.NoticeboardUserDAO">
 		<property name="sessionFactory"><ref local="nbSessionFactory"/></property>
 	</bean>
 	
 	<bean id="nbAttachmentDAO" class="org.lamsfoundation.lams.tool.noticeboard.dao.hibernate.NoticeboardAttachmentDAO">
 		<property name="sessionFactory"><ref local="nbSessionFactory"/></property>
 	</bean>
-
+	
 </beans>
Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/TestNoticeboardAttachmentDAO.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/Attic/TestNoticeboardAttachmentDAO.java,v
diff -u -r1.2 -r1.3
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/TestNoticeboardAttachmentDAO.java	2 Aug 2005 06:36:04 -0000	1.2
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/TestNoticeboardAttachmentDAO.java	11 Aug 2005 05:41:40 -0000	1.3
@@ -53,6 +53,8 @@
 	     super.cleanNbContentData(TEST_NB_ID);
 	 }
 	 
+	 
+	
 	 public void testRetrieveAttachment()
 	 {
 	     //test retrieveAttachmentByUuid
@@ -96,7 +98,21 @@
 	     assertNull(nbAttachment);
 	     
 	 } 
+	 
+	 public void testRemoveAttachmentByUuid()
+	 {
+	     nbAttachment = attachmentDao.retrieveAttachmentByUuid(TEST_UUID);
+	     assertNotNull(nbAttachment);
+	     
+	     attachmentDao.removeAttachment(TEST_UUID);
+	     
+	     nbAttachment = attachmentDao.retrieveAttachmentByUuid(TEST_UUID);
+	     assertNull(nbAttachment);
+	     
+	 }
+	 
 	
+	
 	 
 	 
 }
Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/TestNoticeboardContentDAO.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/Attic/TestNoticeboardContentDAO.java,v
diff -u -r1.4 -r1.5
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/TestNoticeboardContentDAO.java	14 Jul 2005 02:24:25 -0000	1.4
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/dao/hibernate/TestNoticeboardContentDAO.java	11 Aug 2005 05:41:40 -0000	1.5
@@ -69,18 +69,7 @@
         	super.cleanNbContentData(TEST_NB_ID);
         }
     }
-    
    
-    public void testgetNbContentByUID()
-    {
-       nbContent = noticeboardDAO.getNbContentByUID(new Long(1));
-       
-       assertEquals(nbContent.getUid(), new Long(1));
-       assertEquals(nbContent.getNbContentId(), DEFAULT_CONTENT_ID);
-       assertContentEqualsDefaultData(nbContent);
-	  
-    } 
-
    public void testfindNbContentByID()
     {
         nbContent = noticeboardDAO.findNbContentById(TEST_NB_ID);
Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestNoticeboardServicePOJO.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/Attic/TestNoticeboardServicePOJO.java,v
diff -u -r1.8 -r1.9
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestNoticeboardServicePOJO.java	2 Aug 2005 06:36:24 -0000	1.8
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestNoticeboardServicePOJO.java	11 Aug 2005 05:41:40 -0000	1.9
@@ -33,6 +33,7 @@
 import org.lamsfoundation.lams.tool.noticeboard.NbApplicationException;
 import org.lamsfoundation.lams.tool.noticeboard.NbDataAccessTestCase;
 import org.lamsfoundation.lams.tool.noticeboard.NoticeboardContent;
+import org.lamsfoundation.lams.tool.noticeboard.NoticeboardConstants;
 import org.lamsfoundation.lams.tool.noticeboard.NoticeboardSession;
 import org.lamsfoundation.lams.tool.noticeboard.NoticeboardUser;
 import org.lamsfoundation.lams.tool.noticeboard.NoticeboardAttachment;
@@ -521,6 +522,24 @@
 	    assertNull(attachment);
 	    
 	}
+	
+	public void testRemoveAttachmentByUuid()
+	{
+	    initNbAttachmentData();
+	    	    
+	    nbService.removeAttachmentByUuid(TEST_UUID);
+	   
+	    attachment = nbService.retrieveAttachmentByUuid(TEST_UUID);
+	    
+	    assertNull(attachment);
+	    
+	}
+	
+	public void testGetToolDefaultContentIdBySignature()
+	{
+	    Long defaultToolContentId = nbService.getToolDefaultContentIdBySignature(NoticeboardConstants.TOOL_SIGNATURE);
+	    assertNotNull(defaultToolContentId);
+	}
 }
 	
 	
\ No newline at end of file
Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestToolContentManager.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/Attic/TestToolContentManager.java,v
diff -u -r1.4 -r1.5
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestToolContentManager.java	14 Jul 2005 02:25:15 -0000	1.4
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestToolContentManager.java	11 Aug 2005 05:41:40 -0000	1.5
@@ -37,6 +37,7 @@
 /**
  * @author mtruong
  *
+ *TODO: the default content id is now not hard coded and taken from the database, ensure the test cases stillw ork
  */
 public class TestToolContentManager extends NbDataAccessTestCase {
 	
@@ -53,7 +54,7 @@
 	{
 		super(name);
 	}
-
+	
 	protected void setUp() throws Exception
 	{
 		super.setUp();
@@ -168,7 +169,7 @@
 	   // cleanCopyContent = true;
 	    
 	    //remove the default content so it will trigger the exception
-	    nbService.removeNoticeboard(NoticeboardConstants.DEFAULT_CONTENT_ID);
+	    nbService.removeNoticeboard(nbService.getToolDefaultContentIdBySignature(NoticeboardConstants.TOOL_SIGNATURE));
 	    
 	    try
 	    {
Index: lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestToolSessionManager.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/Attic/TestToolSessionManager.java,v
diff -u -r1.1 -r1.2
--- lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestToolSessionManager.java	14 Jul 2005 02:25:15 -0000	1.1
+++ lams_tool_nb/test/java/org/lamsfoundation/lams/tool/noticeboard/service/TestToolSessionManager.java	11 Aug 2005 05:41:40 -0000	1.2
@@ -23,8 +23,6 @@
 /*
  * Created on Jul 8, 2005
  *
- * TODO To change the template for this generated file go to
- * Window - Preferences - Java - Code Style - Code Templates
  */
 package org.lamsfoundation.lams.tool.noticeboard.service;
 import org.lamsfoundation.lams.tool.noticeboard.NbDataAccessTestCase;
@@ -39,6 +37,7 @@
 /**
  * @author mtruong
  *
+ * *TODO: the default content id is now not hard coded and taken from the database, ensure the test cases stillw ork
  */
 public class TestToolSessionManager extends NbDataAccessTestCase {
     
@@ -51,8 +50,8 @@
 	{
 		super(name);
 	}
-
-	protected void setUp() throws Exception
+    
+   protected void setUp() throws Exception
 	{
 		super.setUp();
 		//setup some data
@@ -145,7 +144,7 @@
 	        nbSession = nbService.retrieveNoticeboardSession(NEW_SESSION_ID);
 		    
 		    assertEquals("validating session id:", nbSession.getNbSessionId(), NEW_SESSION_ID);
-		    assertEquals("Validating content Id:", nbSession.getNbContent().getNbContentId(), DEFAULT_CONTENT_ID);
+		    assertEquals("Validating content Id:", nbSession.getNbContent().getNbContentId(), nbService.getToolDefaultContentIdBySignature(NoticeboardConstants.TOOL_SIGNATURE));
 		    assertContentEqualsDefaultData(nbSession.getNbContent());
 	    }
 	    catch (ToolException e)
@@ -160,7 +159,7 @@
 	
 	public void testCreateToolSessionDefaultContentMissing() throws ToolException
 	{
-	    nbService.removeNoticeboard(DEFAULT_CONTENT_ID);
+	    nbService.removeNoticeboard(nbService.getToolDefaultContentIdBySignature(NoticeboardConstants.TOOL_SIGNATURE));
 	    
 	    try
 	    {