Author: kono
Date: 2011-04-29 18:08:11 -0700 (Fri, 29 Apr 2011)
New Revision: 24878
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTask.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTask.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadNetworkTask.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadTableTask.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/SelectMappingKeyTypeTask.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/remote/BioGridPreprocessor.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/NCBISearchClient.java
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/SubnetworkBuilderUtil.java
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTaskTester.java
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTaskTest.java
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkURLTaskTest.java
Log:
Debug messages are removed.
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTask.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTask.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTask.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -29,7 +29,6 @@
*/
package org.cytoscape.task.internal.loadnetwork;
-
import java.io.IOException;
import java.net.URI;
import java.text.DecimalFormat;
@@ -44,128 +43,120 @@
import org.cytoscape.session.CyNetworkNaming;
import org.cytoscape.view.model.CyNetworkView;
import org.cytoscape.view.model.CyNetworkViewManager;
+import org.cytoscape.view.model.NullCyNetworkView;
import org.cytoscape.work.AbstractTask;
import org.cytoscape.work.TaskMonitor;
-
/**
* Task to load a new network.
*/
abstract public class AbstractLoadNetworkTask extends AbstractTask {
- protected CyNetworkViewReader reader;
- protected URI uri;
- protected TaskMonitor taskMonitor;
- protected String name;
- protected boolean interrupted = false;
- protected CyNetworkViewReaderManager mgr;
- protected CyNetworkManager networkManager;
- protected CyNetworkViewManager networkViewManager;
- protected Properties props;
- protected CyNetworkNaming namingUtil;
+ protected CyNetworkViewReader reader;
+ protected URI uri;
+ protected TaskMonitor taskMonitor;
+ protected String name;
+ protected boolean interrupted = false;
+ protected CyNetworkViewReaderManager mgr;
+ protected CyNetworkManager networkManager;
+ protected CyNetworkViewManager networkViewManager;
+ protected Properties props;
+ protected CyNetworkNaming namingUtil;
- public AbstractLoadNetworkTask(final CyNetworkViewReaderManager mgr,
- final CyNetworkManager networkManager,
- final CyNetworkViewManager
networkViewManager,
- final Properties props,
- final CyNetworkNaming namingUtil)
- {
- this.mgr = mgr;
- this.networkManager = networkManager;
- this.networkViewManager = networkViewManager;
- this.props = props;
- this.namingUtil = namingUtil;
- }
+ public AbstractLoadNetworkTask(final CyNetworkViewReaderManager mgr, final
CyNetworkManager networkManager,
+ final CyNetworkViewManager networkViewManager, final Properties
props, final CyNetworkNaming namingUtil) {
+ this.mgr = mgr;
+ this.networkManager = networkManager;
+ this.networkViewManager = networkViewManager;
+ this.props = props;
+ this.namingUtil = namingUtil;
+ }
- protected void loadNetwork(final CyNetworkViewReader viewReader) throws
Exception {
- if (viewReader == null)
- throw new IllegalArgumentException("Could not read
file: Network View Reader is null.");
+ protected void loadNetwork(final CyNetworkViewReader viewReader) throws
Exception {
+ if (viewReader == null)
+ throw new IllegalArgumentException("Could not read file: Network
View Reader is null.");
- taskMonitor.setStatusMessage("Reading in Network Data...");
- taskMonitor.setProgress(-1.0);
- taskMonitor.setStatusMessage("Creating Cytoscape Network...");
+ taskMonitor.setStatusMessage("Reading in Network Data...");
+ taskMonitor.setProgress(-1.0);
+ taskMonitor.setStatusMessage("Creating Cytoscape Network...");
- insertTasksAfterCurrentTask(
- viewReader,
- new GenerateNetworkViewsTask(name, viewReader,
networkManager, networkViewManager, namingUtil, props));
- }
+ insertTasksAfterCurrentTask(viewReader, new
GenerateNetworkViewsTask(name, viewReader, networkManager,
+ networkViewManager, namingUtil, props));
+ }
- @Override
- abstract public void run(TaskMonitor taskMonitor) throws Exception;
+ @Override
+ abstract public void run(TaskMonitor taskMonitor) throws Exception;
}
-
class GenerateNetworkViewsTask extends AbstractTask {
- private final String name;
- private final CyNetworkViewReader viewReader;
- private final CyNetworkManager networkManager;
- private final CyNetworkViewManager networkViewManager;
- private final CyNetworkNaming namingUtil;
- private final Properties props;
+ private final String name;
+ private final CyNetworkViewReader viewReader;
+ private final CyNetworkManager networkManager;
+ private final CyNetworkViewManager networkViewManager;
+ private final CyNetworkNaming namingUtil;
+ private final Properties props;
- GenerateNetworkViewsTask(final String name, final CyNetworkViewReader
viewReader, final CyNetworkManager networkManager,
- final CyNetworkViewManager networkViewManager,
final CyNetworkNaming namingUtil, final Properties props)
- {
- this.name = name;
- this.viewReader = viewReader;
- this.networkManager = networkManager;
- this.networkViewManager = networkViewManager;
- this.namingUtil = namingUtil;
- this.props = props;
- }
+ GenerateNetworkViewsTask(final String name, final CyNetworkViewReader
viewReader,
+ final CyNetworkManager networkManager, final CyNetworkViewManager
networkViewManager,
+ final CyNetworkNaming namingUtil, final Properties props) {
+ this.name = name;
+ this.viewReader = viewReader;
+ this.networkManager = networkManager;
+ this.networkViewManager = networkViewManager;
+ this.namingUtil = namingUtil;
+ this.props = props;
+ }
- public void run(final TaskMonitor taskMonitor) throws Exception {
- final CyNetworkView[] cyNetworkViews =
viewReader.getNetworkViews();
+ public void run(final TaskMonitor taskMonitor) throws Exception {
+ final CyNetworkView[] cyNetworkViews = viewReader.getNetworkViews();
- if (cyNetworkViews == null || cyNetworkViews.length < 0)
- throw new IOException("Could not create network for the
producer.");
+ if (cyNetworkViews == null || cyNetworkViews.length < 0)
+ throw new IOException("Could not create network for the producer.");
- for (final CyNetworkView view : cyNetworkViews) {
- if (cancelled)
- return;
+ for (CyNetworkView view : cyNetworkViews) {
+ if (cancelled)
+ return;
- // Model should not be null. It will be tested in
ViewImpl.
- final CyNetwork cyNetwork = view.getModel();
- cyNetwork.getCyRow().set(CyTableEntry.NAME,
-
namingUtil.getSuggestedNetworkTitle(name));
+ final CyNetwork cyNetwork = view.getModel();
+ cyNetwork.getCyRow().set(CyTableEntry.NAME,
namingUtil.getSuggestedNetworkTitle(name));
+ networkManager.addNetwork(cyNetwork);
+
+ // Do the following only for non-null views.
+ if (view instanceof NullCyNetworkView == false) {
+ networkViewManager.addNetworkView(view);
+ view.fitContent();
+ } else {
+ view = null;
+ }
- networkManager.addNetwork(cyNetwork);
- networkViewManager.addNetworkView(view);
-
- view.fitContent();
-
- informUserOfGraphStats(cyNetwork, taskMonitor);
- }
-
- taskMonitor.setProgress(1.0);
+ informUserOfGraphStats(cyNetwork, taskMonitor);
}
- /**
- * Inform User of Network Stats.
- */
- private void informUserOfGraphStats(final CyNetwork newNetwork, final
TaskMonitor taskMonitor) {
- NumberFormat formatter = new DecimalFormat("#,###,###");
- StringBuffer sb = new StringBuffer();
+ taskMonitor.setProgress(1.0);
+ }
- // Give the user some confirmation
- sb.append("Successfully loaded network from: ");
- sb.append(name);
- sb.append("\n\nNetwork contains "
- + formatter.format(newNetwork.getNodeCount()));
- sb.append(" nodes and " +
formatter.format(newNetwork.getEdgeCount()));
- sb.append(" edges.\n\n");
+ /**
+ * Inform User of Network Stats.
+ */
+ private void informUserOfGraphStats(final CyNetwork newNetwork, final
TaskMonitor taskMonitor) {
+ NumberFormat formatter = new DecimalFormat("#,###,###");
+ StringBuffer sb = new StringBuffer();
- String thresh = props.getProperty("viewThreshold");
+ // Give the user some confirmation
+ sb.append("Successfully loaded network from: ");
+ sb.append(name);
+ sb.append("\n\nNetwork contains " +
formatter.format(newNetwork.getNodeCount()));
+ sb.append(" nodes and " + formatter.format(newNetwork.getEdgeCount()));
+ sb.append(" edges.\n\n");
- if (newNetwork.getNodeCount() < Integer.parseInt(thresh)) {
- sb.append("Network is under " + thresh
- + " nodes. A view will be
automatically created.");
- } else {
- sb.append("Network is over " + thresh
- + " nodes. A view has not been
created."
- + " If you wish to view this network,
use "
- + "\"Create View\" from the \"Edit\"
menu.");
- }
+ String thresh = props.getProperty("viewThreshold");
- taskMonitor.setStatusMessage(sb.toString());
+ if (newNetwork.getNodeCount() < Integer.parseInt(thresh)) {
+ sb.append("Network is under " + thresh + " nodes. A view will be
automatically created.");
+ } else {
+ sb.append("Network is over " + thresh + " nodes. A view has not
been created."
+ + " If you wish to view this network, use " + "\"Create
View\" from the \"Edit\" menu.");
}
+
+ taskMonitor.setStatusMessage(sb.toString());
+ }
}
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTask.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTask.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTask.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -29,7 +29,6 @@
*/
package org.cytoscape.task.internal.loadnetwork;
-
import java.io.File;
import java.util.Properties;
@@ -40,41 +39,38 @@
import org.cytoscape.work.TaskMonitor;
import org.cytoscape.work.Tunable;
-
/**
* Specific instance of AbstractLoadNetworkTask that loads a File.
*/
public class LoadNetworkFileTask extends AbstractLoadNetworkTask {
- @Tunable(description="Network file to load",
params="fileCategory=network;input=true")
- public File file;
+ @Tunable(description = "Network file to load", params =
"fileCategory=network;input=true")
+ public File file;
- public LoadNetworkFileTask(CyNetworkViewReaderManager mgr,
CyNetworkManager netmgr,
- final CyNetworkViewManager
networkViewManager,
- final Properties props, CyNetworkNaming
namingUtil)
- {
- super(mgr, netmgr, networkViewManager, props, namingUtil);
- }
+ public LoadNetworkFileTask(CyNetworkViewReaderManager mgr,
CyNetworkManager netmgr,
+ final CyNetworkViewManager networkViewManager, final Properties
props, CyNetworkNaming namingUtil) {
+ super(mgr, netmgr, networkViewManager, props, namingUtil);
+ }
- /**
- * Executes Task.
- */
- public void run(TaskMonitor taskMonitor) throws Exception {
- this.taskMonitor = taskMonitor;
+ /**
+ * Executes Task.
+ */
+ public void run(TaskMonitor taskMonitor) throws Exception {
+ this.taskMonitor = taskMonitor;
- if ( file == null )
- throw new NullPointerException("No file specified!");
-
- reader = mgr.getReader(file.toURI(),file.getName());
+ if (file == null)
+ throw new NullPointerException("No file specified!");
- if (cancelled)
- return;
+ reader = mgr.getReader(file.toURI(), file.getName());
- if (reader == null)
- throw new NullPointerException("Failed to find
appropriate reader for file: " + file);
+ if (cancelled)
+ return;
- uri = file.toURI();
- name = file.getName();
-
- loadNetwork(reader);
- }
+ if (reader == null)
+ throw new NullPointerException("Failed to find appropriate reader
for file: " + file);
+
+ uri = file.toURI();
+ name = file.getName();
+
+ loadNetwork(reader);
+ }
}
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadNetworkTask.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadNetworkTask.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadNetworkTask.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -72,7 +72,6 @@
insertTasksAfterCurrentTask(new
SelectMappingKeyTypeTask(state,util, previewKeys, previewData));
insertTasksAfterCurrentTask(new
GetNetworkPreviewDataTask(util, previewData));
} else if(state.isJobFinished(Job.LOAD_TABLE)) {
- System.out.println("Already has a table. No need to
set ID type. Move to merge");
insertTasksAfterCurrentTask(new MergeDataTask(state,
util));
}
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadTableTask.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadTableTask.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/LoadTableTask.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -67,7 +67,6 @@
// This is for next step: specify ID type, and pass the
preview data
insertTasksAfterCurrentTask(new
SelectMappingKeyTypeTask(state, util, previewKey, previewData));
} else if (state.isJobFinished(Job.LOAD_NETWORK)){
- System.out.println("No need to set ID type. Move to
merge");
insertTasksAfterCurrentTask(new MergeDataTask(state,
util));
}
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/SelectMappingKeyTypeTask.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/SelectMappingKeyTypeTask.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/SelectMappingKeyTypeTask.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -55,7 +55,6 @@
public void run(TaskMonitor monitor) throws Exception {
// Check current status and move to next task.
if(state.isJobFinished(Job.LOAD_NETWORK) &&
state.isJobFinished(Job.LOAD_TABLE)) {
- System.out.println("!!!!!!!! Merge task added");
insertTasksAfterCurrentTask(new MergeDataTask(state,
util));
} else if(state.isJobFinished(Job.LOAD_NETWORK)) {
// Need to load table.
@@ -73,7 +72,6 @@
state.setIDType(this.idTypeMap.get(selected));
- System.out.println("ID type selected. Selected type = " +
selected);
state.finished(Job.SELECT_MAPPING_ID_TYPE);
}
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/remote/BioGridPreprocessor.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/remote/BioGridPreprocessor.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/remote/BioGridPreprocessor.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -81,7 +81,6 @@
this.sourceFileLocation = source;
boolean test = isUpToDate();
- System.out.println("UP2DATE = " + test);
if (!test) {
final InputStream is = sourceFileLocation.openStream();
extractEntrey(is);
@@ -116,7 +115,6 @@
while ((zen = zis.getNextEntry()) != null) {
entryName = zen.getName();
- System.out.println("SESSION entry name: " +
entryName);
File outFile = new File(dataFileDirectory,
entryName + ".sif");
outFile.createNewFile();
FileWriter outWriter = new FileWriter(outFile);
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/NCBISearchClient.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/NCBISearchClient.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/NCBISearchClient.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -64,7 +64,6 @@
urlString = urlString + builder.toString();
urlString = urlString.substring(0, urlString.length() - 4);
- System.out.println("URL String = " + urlString);
return new URL(urlString);
}
@@ -80,17 +79,14 @@
final NodeList ids = result.getElementsByTagName("Id");
final int dataSize = ids.getLength();
- System.out.println("Data size = " + dataSize);
for (int i = 0; i < dataSize; i++) {
Node id = ids.item(i);
idSet.add(id.getTextContent());
- System.out.println("Node ID = " + id.getTextContent());
}
is.close();
is = null;
- System.out.println("Number of result = " + idSet.size());
return idSet;
}
}
Modified:
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/SubnetworkBuilderUtil.java
===================================================================
---
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/SubnetworkBuilderUtil.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/main/java/org/cytoscape/task/internal/quickstart/subnetworkbuilder/SubnetworkBuilderUtil.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -103,7 +103,6 @@
public void addFactory(VisualMappingFunctionFactory factory, Map props) {
- System.out.println("\n\n\n *********** Got Factory ***************" +
factory + "\n\n\n");
if (factory.toString().startsWith("Discrete Mapping"))
discFactory = factory;
@@ -112,7 +111,6 @@
if (discFactory != null && ptFactory != null) {
this.vsBuilder = new VisualStyleBuilder(vsFactory, discFactory,
ptFactory);
- System.out.println("\n\n\n *********** vsBuilder Created.
***************\n\n\n");
}
}
Modified:
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTaskTester.java
===================================================================
---
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTaskTester.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/AbstractLoadNetworkTaskTester.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -31,75 +31,71 @@
You should have received a copy of the GNU Lesser General Public License
along with this library; if not, write to the Free Software Foundation,
Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA.
-*/
+ */
-
package org.cytoscape.task.internal.loadnetwork;
-import static org.mockito.Mockito.*;
-import static org.junit.Assert.*;
-import org.junit.Test;
-import org.junit.Before;
+import static org.mockito.Matchers.anyString;
+import static org.mockito.Matchers.eq;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
-import java.util.Properties;
import java.net.URI;
+import java.util.Properties;
+import org.cytoscape.io.read.CyNetworkViewReader;
import org.cytoscape.io.read.CyNetworkViewReaderManager;
-import org.cytoscape.model.CyNetworkManager;
-import org.cytoscape.view.model.CyNetworkViewManager;
-import org.cytoscape.session.CyNetworkNaming;
-import org.cytoscape.property.CyProperty;
-import org.cytoscape.work.TaskMonitor;
-import org.cytoscape.work.TaskIterator;
-import org.cytoscape.work.Task;
import org.cytoscape.model.CyNetwork;
+import org.cytoscape.model.CyNetworkManager;
import org.cytoscape.model.CyRow;
+import org.cytoscape.property.CyProperty;
+import org.cytoscape.session.CyNetworkNaming;
import org.cytoscape.view.model.CyNetworkView;
-import org.cytoscape.io.read.CyNetworkViewReader;
+import org.cytoscape.view.model.CyNetworkViewManager;
public class AbstractLoadNetworkTaskTester {
- URI uri;
+ URI uri;
CyNetworkViewReaderManager mgr;
- CyNetworkManager netmgr;
- CyNetworkViewManager networkViewManager;
- CyProperty<Properties> props;
- CyNetworkNaming namingUtil;
+ CyNetworkManager netmgr;
+ CyNetworkViewManager networkViewManager;
+ CyProperty<Properties> props;
+ CyNetworkNaming namingUtil;
- CyNetwork net;
- CyNetworkView view;
- CyNetworkView[] views;
- CyNetworkViewReader reader;
+ CyNetwork net;
+ CyNetworkView view;
+ CyNetworkView[] views;
+ CyNetworkViewReader reader;
- public void setUp() throws Exception {
- CyRow attrs = mock(CyRow.class);
+ public void setUp() throws Exception {
+ CyRow attrs = mock(CyRow.class);
- net = mock(CyNetwork.class);
- when(net.getNodeCount()).thenReturn(2);
- when(net.getEdgeCount()).thenReturn(1);
- when(net.getCyRow()).thenReturn(attrs);
+ net = mock(CyNetwork.class);
+ when(net.getNodeCount()).thenReturn(2);
+ when(net.getEdgeCount()).thenReturn(1);
+ when(net.getCyRow()).thenReturn(attrs);
- view = mock(CyNetworkView.class);
- when(view.getModel()).thenReturn(net);
+ view = mock(CyNetworkView.class);
+ when(view.getModel()).thenReturn(net);
- views = new CyNetworkView[] {view};
+ views = new CyNetworkView[] { view };
- reader = mock(CyNetworkViewReader.class);
- when(reader.getNetworkViews()).thenReturn(views);
+ reader = mock(CyNetworkViewReader.class);
+ when(reader.getNetworkViews()).thenReturn(views);
- mgr = mock(CyNetworkViewReaderManager.class);
- when(mgr.getReader(eq(uri),anyString())).thenReturn(reader);
+ mgr = mock(CyNetworkViewReaderManager.class);
+ when(mgr.getReader(eq(uri), anyString())).thenReturn(reader);
- netmgr = mock(CyNetworkManager.class);
- networkViewManager = mock(CyNetworkViewManager.class);
+ netmgr = mock(CyNetworkManager.class);
+ networkViewManager = mock(CyNetworkViewManager.class);
- Properties p = new Properties();
- p.setProperty("viewThreshold","1000");
+ Properties p = new Properties();
+ p.setProperty("viewThreshold", "1000");
- props = mock(CyProperty.class);
- when(props.getProperties()).thenReturn(p);
+ props = mock(CyProperty.class);
+ when(props.getProperties()).thenReturn(p);
- namingUtil = mock(CyNetworkNaming.class);
- }
+ namingUtil = mock(CyNetworkNaming.class);
+ }
}
Modified:
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTaskTest.java
===================================================================
---
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTaskTest.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkFileTaskTest.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -31,74 +31,65 @@
You should have received a copy of the GNU Lesser General Public License
along with this library; if not, write to the Free Software Foundation,
Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA.
-*/
+ */
-
package org.cytoscape.task.internal.loadnetwork;
-import static org.mockito.Mockito.*;
-import static org.junit.Assert.*;
-import org.junit.Test;
-import org.junit.Before;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.verify;
import java.io.File;
-import java.util.Properties;
-import org.cytoscape.io.read.CyNetworkViewReaderManager;
-import org.cytoscape.session.CyNetworkNaming;
-import org.cytoscape.property.CyProperty;
-import org.cytoscape.work.TaskMonitor;
-import org.cytoscape.work.TaskIterator;
-import org.cytoscape.work.TaskFactory;
import org.cytoscape.work.Task;
-import org.cytoscape.model.CyNetwork;
-import org.cytoscape.model.CyRow;
-import org.cytoscape.view.model.CyNetworkView;
-import org.cytoscape.io.read.CyNetworkViewReader;
+import org.cytoscape.work.TaskFactory;
+import org.cytoscape.work.TaskIterator;
+import org.cytoscape.work.TaskMonitor;
+import org.junit.Before;
+import org.junit.Test;
public class LoadNetworkFileTaskTest extends AbstractLoadNetworkTaskTester {
- File file;
+ File file;
- @Before
- public void setUp() throws Exception {
- file = new File("/tmp/asdfasdf");
- uri = file.toURI();
- super.setUp();
- }
+ @Before
+ public void setUp() throws Exception {
+ file = new File("/tmp/asdfasdf");
+ uri = file.toURI();
+ super.setUp();
+ }
- @Test
- public void testRun() throws Exception {
- TaskFactory factory = new LoadNetworkFileTaskFactoryImpl(mgr,
netmgr, networkViewManager, props, namingUtil);
- TaskIterator ti = factory.getTaskIterator();
- TaskMonitor tm = mock(TaskMonitor.class);
- boolean first = true;
- while ( ti.hasNext() ) {
- Task t = ti.next();
- if ( first ) {
- ((LoadNetworkFileTask)t).file = file;
- first = false;
- }
- t.run(tm);
- }
- verify(netmgr).addNetwork(net);
- verify(networkViewManager).addNetworkView(view);
- verify(tm).setProgress(1.0);
+ @Test
+ public void testRun() throws Exception {
+ TaskFactory factory = new LoadNetworkFileTaskFactoryImpl(mgr, netmgr,
networkViewManager, props, namingUtil);
+ TaskIterator ti = factory.getTaskIterator();
+ TaskMonitor tm = mock(TaskMonitor.class);
+ boolean first = true;
+ while (ti.hasNext()) {
+ Task t = ti.next();
+ if (first) {
+ ((LoadNetworkFileTask) t).file = file;
+ first = false;
+ }
+ t.run(tm);
}
+ verify(netmgr).addNetwork(net);
+ //verify(networkViewManager).addNetworkView(view);
+ verify(tm).setProgress(1.0);
+ }
- @Test(expected=NullPointerException.class)
- public void testNullFile() throws Exception {
- TaskFactory factory = new LoadNetworkFileTaskFactoryImpl(mgr,
netmgr, networkViewManager, props, namingUtil);
- TaskIterator ti = factory.getTaskIterator();
- TaskMonitor tm = mock(TaskMonitor.class);
- boolean first = true;
- while ( ti.hasNext() ) {
- Task t = ti.next();
- if ( first ) {
- ((LoadNetworkFileTask)t).file = null;
- first = false;
- }
- t.run(tm);
- }
+ @Test(expected = NullPointerException.class)
+ public void testNullFile() throws Exception {
+ TaskFactory factory = new LoadNetworkFileTaskFactoryImpl(mgr, netmgr,
networkViewManager, props, namingUtil);
+ TaskIterator ti = factory.getTaskIterator();
+ TaskMonitor tm = mock(TaskMonitor.class);
+ boolean first = true;
+ while (ti.hasNext()) {
+ Task t = ti.next();
+ if (first) {
+ ((LoadNetworkFileTask) t).file = null;
+ first = false;
+ }
+ t.run(tm);
}
+ }
}
Modified:
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkURLTaskTest.java
===================================================================
---
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkURLTaskTest.java
2011-04-30 00:30:39 UTC (rev 24877)
+++
core3/core-task-impl/trunk/src/test/java/org/cytoscape/task/internal/loadnetwork/LoadNetworkURLTaskTest.java
2011-04-30 01:08:11 UTC (rev 24878)
@@ -24,108 +24,105 @@
You should have received a copy of the GNU Lesser General Public License
along with this library; if not, write to the Free Software Foundation,
Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA.
-*/
+ */
-
package org.cytoscape.task.internal.loadnetwork;
-import static org.mockito.Mockito.*;
-import static org.junit.Assert.*;
-import org.junit.Test;
-import org.junit.Before;
+import static org.junit.Assert.assertNotNull;
+import static org.mockito.Mockito.doThrow;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
import java.io.IOException;
import java.net.URL;
import java.net.URLConnection;
-import java.util.Properties;
-import org.cytoscape.io.read.CyNetworkViewReaderManager;
-import org.cytoscape.session.CyNetworkNaming;
-import org.cytoscape.property.CyProperty;
-import org.cytoscape.work.TaskMonitor;
-import org.cytoscape.work.TaskIterator;
-import org.cytoscape.work.TaskFactory;
-import org.cytoscape.work.Task;
-import org.cytoscape.model.CyNetwork;
-import org.cytoscape.model.CyRow;
-import org.cytoscape.view.model.CyNetworkView;
-import org.cytoscape.io.read.CyNetworkViewReader;
import org.cytoscape.io.util.StreamUtil;
+import org.cytoscape.work.Task;
+import org.cytoscape.work.TaskFactory;
+import org.cytoscape.work.TaskIterator;
+import org.cytoscape.work.TaskMonitor;
+import org.junit.Before;
+import org.junit.Test;
public class LoadNetworkURLTaskTest extends AbstractLoadNetworkTaskTester {
- URL url;
+ URL url;
- @Before
- public void setUp() throws Exception {
- url = new URL("http://example.com");
- uri = url.toURI();
- super.setUp();
- }
+ @Before
+ public void setUp() throws Exception {
+ url = new URL("http://example.com");
+ uri = url.toURI();
+ super.setUp();
+ }
- @Test
- public void testRun() throws Exception {
- URLConnection con = mock(URLConnection.class);
- StreamUtil streamUtil = mock(StreamUtil.class);
- when(streamUtil.getURLConnection(url)).thenReturn(con);
+ @Test
+ public void testRun() throws Exception {
+ URLConnection con = mock(URLConnection.class);
+ StreamUtil streamUtil = mock(StreamUtil.class);
+ when(streamUtil.getURLConnection(url)).thenReturn(con);
- TaskFactory factory = new LoadNetworkURLTaskFactoryImpl(mgr,
netmgr, networkViewManager, props, namingUtil, streamUtil );
-assertNotNull(networkViewManager);
- TaskIterator ti = factory.getTaskIterator();
- TaskMonitor tm = mock(TaskMonitor.class);
- boolean first = true;
- while ( ti.hasNext() ) {
- Task t = ti.next();
- if ( first ) {
- ((LoadNetworkURLTask)t).url = url;
- first = false;
- }
- t.run(tm);
- }
- verify(netmgr).addNetwork(net);
- verify(networkViewManager).addNetworkView(view);
- verify(tm).setProgress(1.0);
+ TaskFactory factory = new LoadNetworkURLTaskFactoryImpl(mgr, netmgr,
networkViewManager, props, namingUtil,
+ streamUtil);
+ assertNotNull(networkViewManager);
+ TaskIterator ti = factory.getTaskIterator();
+ TaskMonitor tm = mock(TaskMonitor.class);
+ boolean first = true;
+ while (ti.hasNext()) {
+ Task t = ti.next();
+ if (first) {
+ ((LoadNetworkURLTask) t).url = url;
+ first = false;
+ }
+ t.run(tm);
}
+ verify(netmgr).addNetwork(net);
+ //verify(networkViewManager).addNetworkView(view);
+ verify(tm).setProgress(1.0);
+ }
- @Test(expected=Exception.class)
- public void testBadConnection() throws Exception {
- URLConnection con = mock(URLConnection.class);
- doThrow(new IOException("bad connection")).when(con).connect();
+ @Test(expected = Exception.class)
+ public void testBadConnection() throws Exception {
+ URLConnection con = mock(URLConnection.class);
+ doThrow(new IOException("bad connection")).when(con).connect();
- StreamUtil streamUtil = mock(StreamUtil.class);
- when(streamUtil.getURLConnection(url)).thenReturn(con);
+ StreamUtil streamUtil = mock(StreamUtil.class);
+ when(streamUtil.getURLConnection(url)).thenReturn(con);
- TaskFactory factory = new LoadNetworkURLTaskFactoryImpl(mgr,
netmgr, networkViewManager, props, namingUtil, streamUtil );
- TaskIterator ti = factory.getTaskIterator();
- TaskMonitor tm = mock(TaskMonitor.class);
- boolean first = true;
- while ( ti.hasNext() ) {
- Task t = ti.next();
- if ( first ) {
- ((LoadNetworkURLTask)t).url = url;
- first = false;
- }
- t.run(tm);
- }
+ TaskFactory factory = new LoadNetworkURLTaskFactoryImpl(mgr, netmgr,
networkViewManager, props, namingUtil,
+ streamUtil);
+ TaskIterator ti = factory.getTaskIterator();
+ TaskMonitor tm = mock(TaskMonitor.class);
+ boolean first = true;
+ while (ti.hasNext()) {
+ Task t = ti.next();
+ if (first) {
+ ((LoadNetworkURLTask) t).url = url;
+ first = false;
+ }
+ t.run(tm);
}
+ }
- @Test(expected=NullPointerException.class)
- public void testNullURL() throws Exception {
- URLConnection con = mock(URLConnection.class);
- StreamUtil streamUtil = mock(StreamUtil.class);
- when(streamUtil.getURLConnection(url)).thenReturn(con);
+ @Test(expected = NullPointerException.class)
+ public void testNullURL() throws Exception {
+ URLConnection con = mock(URLConnection.class);
+ StreamUtil streamUtil = mock(StreamUtil.class);
+ when(streamUtil.getURLConnection(url)).thenReturn(con);
- TaskFactory factory = new LoadNetworkURLTaskFactoryImpl(mgr,
netmgr, networkViewManager, props, namingUtil, streamUtil );
- TaskIterator ti = factory.getTaskIterator();
- TaskMonitor tm = mock(TaskMonitor.class);
- boolean first = true;
- while ( ti.hasNext() ) {
- Task t = ti.next();
- if ( first ) {
- ((LoadNetworkURLTask)t).url = null;
- first = false;
- }
- t.run(tm);
- }
+ TaskFactory factory = new LoadNetworkURLTaskFactoryImpl(mgr, netmgr,
networkViewManager, props, namingUtil,
+ streamUtil);
+ TaskIterator ti = factory.getTaskIterator();
+ TaskMonitor tm = mock(TaskMonitor.class);
+ boolean first = true;
+ while (ti.hasNext()) {
+ Task t = ti.next();
+ if (first) {
+ ((LoadNetworkURLTask) t).url = null;
+ first = false;
+ }
+ t.run(tm);
}
+ }
}
--
You received this message because you are subscribed to the Google Groups
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/cytoscape-cvs?hl=en.