Index: lams_tool_lamc/build.xml =================================================================== diff -u -r74e45f782571e1740f0a8cfa54b91d39e8df80b3 -rca5b61179fb4a48b6d02a869fdcd515f0335ea3c --- lams_tool_lamc/build.xml (.../build.xml) (revision 74e45f782571e1740f0a8cfa54b91d39e8df80b3) +++ lams_tool_lamc/build.xml (.../build.xml) (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -417,8 +417,8 @@ - - + + Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/McSession.java =================================================================== diff -u -r48c973dcec3754f35dce1f2468a5739067b47179 -rca5b61179fb4a48b6d02a869fdcd515f0335ea3c --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/McSession.java (.../McSession.java) (revision 48c973dcec3754f35dce1f2468a5739067b47179) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/McSession.java (.../McSession.java) (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -2,7 +2,9 @@ import java.io.Serializable; import java.util.Date; +import java.util.HashSet; import java.util.Set; + import org.apache.commons.lang.builder.ToStringBuilder; @@ -34,16 +36,16 @@ private Set mcQueUsers; /** full constructor */ - public McSession(Long mcSessionId, Date session_start_date, Date session_end_date, String session_status, Long mcContentId, org.lamsfoundation.lams.tool.mc.McContent mcContent, Set mcQueUsers) { + public McSession(Long mcSessionId, Date session_start_date, Date session_end_date, String session_status, org.lamsfoundation.lams.tool.mc.McContent mcContent, Set mcQueUsers) { this.mcSessionId = mcSessionId; this.session_start_date = session_start_date; this.session_end_date = session_end_date; this.session_status = session_status; - this.mcContentId = mcContentId; this.mcContent = mcContent; this.mcQueUsers = mcQueUsers; } - + + /** default constructor */ public McSession() { } @@ -111,6 +113,8 @@ } public Set getMcQueUsers() { + if (this.mcQueUsers == null) + setMcQueUsers(new HashSet()); return this.mcQueUsers; } Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/hibernate/McContentDAO.java =================================================================== diff -u -rd19e9cf6efc92b212a276892cdbbc718518037da -rca5b61179fb4a48b6d02a869fdcd515f0335ea3c --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/hibernate/McContentDAO.java (.../McContentDAO.java) (revision d19e9cf6efc92b212a276892cdbbc718518037da) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/dao/hibernate/McContentDAO.java (.../McContentDAO.java) (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -115,7 +115,7 @@ public void removeMcById(Long mcContentId) { String query = "from mc in class org.lamsfoundation.lams.tool.mc.McContent" - + " where qa.mcContentId = ?"; + + " where mc.mcContentId = ?"; this.getHibernateTemplate().delete(query,mcContentId,Hibernate.LONG); } Index: lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/McDataAccessTestCase.java =================================================================== diff -u -r899f826e2aff2a1a624a0c4a212a284660fe4a60 -rca5b61179fb4a48b6d02a869fdcd515f0335ea3c --- lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/McDataAccessTestCase.java (.../McDataAccessTestCase.java) (revision 899f826e2aff2a1a624a0c4a212a284660fe4a60) +++ lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/McDataAccessTestCase.java (.../McDataAccessTestCase.java) (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -13,20 +13,14 @@ public class McDataAccessTestCase extends AbstractLamsTestCase { //These both refer to the same entry in the db. - protected final long TEST_EXISTING_CONTENT_ID = 10; - protected final long DEFAULT_CONTENT_ID = 10; - - protected final long TEST_NEW_CONTENT_ID = 11; + protected final Long DEFAULT_CONTENT_ID = new Long(10); + protected final Long TEST_CONTENT_ID = new Long(2); + protected final Long TEST_CONTENT_ID_OTHER = new Long(3); - protected final long TEST_EXISTING_SESSION_ID = 101; - protected final long TEST_NEW_SESSION_ID = 102; + protected final Long TEST_SESSION_ID = new Long(20); + protected final Long TEST_SESSION_ID_OTHER = new Long(21); - protected final long TEST_NEW_USER_ID = 700; - protected final long TEST_EXISTING_QUE_CONTENT_ID = 20; - protected final long TEST_NEW_QUE_CONTENT_ID = 23; - protected final long TEST_NONEXISTING_CONTENT_ID=2475733396382404l; - protected final long ONE_DAY = 60 * 60 * 1000 * 24; public final String NOT_ATTEMPTED = "NOT_ATTEMPTED"; @@ -75,5 +69,9 @@ super.tearDown(); } + public void testDummy() + { + System.out.println("dummy McDataAccessTestCase"); + } } Index: lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/TestMcContent.java =================================================================== diff -u -r44ebdc79af2b77d9f5ae22774c552279633f866a -rca5b61179fb4a48b6d02a869fdcd515f0335ea3c --- lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/TestMcContent.java (.../TestMcContent.java) (revision 44ebdc79af2b77d9f5ae22774c552279633f866a) +++ lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/TestMcContent.java (.../TestMcContent.java) (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -49,7 +49,7 @@ { //create new mc content McContent mc = new McContent(); - mc.setMcContentId(new Long(4)); + mc.setMcContentId(TEST_CONTENT_ID); mc.setTitle("New - Put Title Here"); mc.setInstructions("New - Put instructions here."); mc.setQuestionsSequenced(false); @@ -72,6 +72,66 @@ mc.setMcSessions(new TreeSet()); mcContentDAO.saveMcContent(mc); + assertNotNull(mc); } + /* + public void testCreateNewMcContentOther() + { + //create new mc content + McContent mc = new McContent(); + mc.setMcContentId(TEST_CONTENT_ID_OTHER); + mc.setTitle("Other - Put Title Here"); + mc.setInstructions("Other - Put instructions here."); + mc.setQuestionsSequenced(false); + mc.setUsernameVisible(false); + mc.setCreatedBy(0); + mc.setMonitoringReportTitle("Other-Monitoring Report title"); + mc.setReportTitle("Other-Report title"); + mc.setRunOffline(false); + mc.setDefineLater(false); + mc.setSynchInMonitor(false); + mc.setOnlineInstructions("Other- online instructions"); + mc.setOfflineInstructions("Other- offline instructions"); + mc.setEndLearningMessage("Other- endLearningMessage"); + mc.setContentInUse(false); + mc.setRetries(false); + mc.setShowFeedback(false); + mc.setShowTopUsers(false); + + mc.setMcQueContents(new TreeSet()); + mc.setMcSessions(new TreeSet()); + + mcContentDAO.saveMcContent(mc); + assertNotNull(mc); + } + + + public void testRetrieveMcContent() + { + McContent mcContent = mcContentDAO.findMcContentById(TEST_CONTENT_ID); + assertNotNull(mcContent); + assertEquals(mcContent.getMcContentId(), new Long(2)); + } + + public void testRemoveMcContent() + { + McContent mcContent = mcContentDAO.findMcContentById(TEST_CONTENT_ID); + mcContentDAO.removeMc(mcContent); + assertNull(mcContent); + } + + + public void testRemoveMcContentById() + { + mcContentDAO.removeMc(TEST_CONTENT_ID_OTHER); + } + + + public void testRemoveMcContentByIdExplicit() + { + mcContentDAO.removeMcById(TEST_CONTENT_ID); + } + + */ } \ No newline at end of file Index: lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/TestMcSession.java =================================================================== diff -u --- lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/TestMcSession.java (revision 0) +++ lams_tool_lamc/test/java/org/lamsfoundation/lams/tool/mc/TestMcSession.java (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -0,0 +1,66 @@ +/* ******************************************************************************** + * Copyright Notice + * ================= + * This file contains propriety information of LAMS Foundation. + * Copying or reproduction with prior written permission is prohibited. + * Copyright (c) 2004 + * Created on 2004-12-23 + ******************************************************************************** */ + +package org.lamsfoundation.lams.tool.mc; + +import java.util.Date; +import java.util.HashSet; + +import org.lamsfoundation.lams.tool.service.ILamsToolService; + + + +/* + * + * @author ozgurd + * + * TODO To change the template for this generated type comment go to + * Window - Preferences - Java - Code Style - Code Templates + */ + + +public class TestMcSession extends McDataAccessTestCase +{ + public org.lamsfoundation.lams.tool.dao.IToolDAO toolDAO; + public ILamsToolService lamsToolService; + + protected void setUp() throws Exception + { + super.setUp(); + } + + protected void tearDown() throws Exception + { + super.tearDown(); + } + + public TestMcSession(String name) + { + super(name); + } + + + public void testCreateNewMcSession() + { + McContent mcContent = mcContentDAO.findMcContentById(TEST_CONTENT_ID); + + + McSession mcSession = new McSession(TEST_SESSION_ID_OTHER, + new Date(System.currentTimeMillis()), + new Date(System.currentTimeMillis()+ ONE_DAY), + this.NOT_ATTEMPTED, + mcContent, + new HashSet()); + + + mcSessionDAO.saveMcSession(mcSession); + assertEquals(mcSession.getMcSessionId(),new Long(21)); + } + +} \ No newline at end of file Index: lams_tool_lamc/test/web/WEB-INF/tiles/tiles-defs.xml =================================================================== diff -u -r653f0f0eff40af5e6891c22ef2b8efe9bf2f75af -rca5b61179fb4a48b6d02a869fdcd515f0335ea3c --- lams_tool_lamc/test/web/WEB-INF/tiles/tiles-defs.xml (.../tiles-defs.xml) (revision 653f0f0eff40af5e6891c22ef2b8efe9bf2f75af) +++ lams_tool_lamc/test/web/WEB-INF/tiles/tiles-defs.xml (.../tiles-defs.xml) (revision ca5b61179fb4a48b6d02a869fdcd515f0335ea3c) @@ -1,4 +1,9 @@ + + + + + + SystemSessionFilter + + org.lamsfoundation.lams.web.session.SystemSessionFilter + + + + + SystemSessionFilter + /* + + + + + exportPortfolio + Export Portfolio + Export Portfolio + org.lamsfoundation.lams.tool.qa.web.QaExportServlet + + + + exportPortfolio + /portfolioExport + +