From 1033b00b574f055b43a5a0187d31e76302da94ab Mon Sep 17 00:00:00 2001 From: Patrick Gotthard Date: Sun, 13 Apr 2014 20:39:21 +0200 Subject: [PATCH] Removed useless TODOs --- .../module/mediarss/io/MediaModuleParser.java | 15 ++------------- .../feed/module/content/ContentItemTest.java | 7 +++---- .../module/content/ContentModuleImplTest.java | 3 --- 3 files changed, 5 insertions(+), 20 deletions(-) diff --git a/src/main/java/org/rometools/feed/module/mediarss/io/MediaModuleParser.java b/src/main/java/org/rometools/feed/module/mediarss/io/MediaModuleParser.java index b7f6d03..c620c66 100644 --- a/src/main/java/org/rometools/feed/module/mediarss/io/MediaModuleParser.java +++ b/src/main/java/org/rometools/feed/module/mediarss/io/MediaModuleParser.java @@ -58,27 +58,15 @@ import com.sun.syndication.io.impl.NumberParser; * */ public class MediaModuleParser implements ModuleParser { - private static final Logger LOG = Logger.getLogger(MediaModuleParser.class.getName()); - /* - * Namespace instance for this URI. - */ + private static final Logger LOG = Logger.getLogger(MediaModuleParser.class.getName()); private static final Namespace NS = Namespace.getNamespace(MediaModule.URI); - /* - * (non-Javadoc) - * @see com.sun.syndication.io.ModuleParser#getNamespaceUri() - */ @Override public String getNamespaceUri() { - // TODO Auto-generated method stub return MediaModule.URI; } - /* - * (non-Javadoc) - * @see com.sun.syndication.io.ModuleParser#parse(org.jdom2.Element) - */ @Override public Module parse(final Element mmRoot, final Locale locale) { MediaModuleImpl mod = null; @@ -436,4 +424,5 @@ public class MediaModuleParser implements ModuleParser { public Namespace getNS() { return NS; } + } diff --git a/src/test/java/org/rometools/feed/module/content/ContentItemTest.java b/src/test/java/org/rometools/feed/module/content/ContentItemTest.java index fee37fb..a8dddcf 100644 --- a/src/test/java/org/rometools/feed/module/content/ContentItemTest.java +++ b/src/test/java/org/rometools/feed/module/content/ContentItemTest.java @@ -115,14 +115,15 @@ public class ContentItemTest extends TestCase { /** * this is a stupid test. * - * @todo make better Test of ContentResource method, of class com.totsp.xml.syndication.content.ContentItem. + * @todo make better Test of ContentResource method, of class + * com.totsp.xml.syndication.content.ContentItem. */ public void testEquals() { System.out.println("testEquals"); final String test = "http://www.w3.org/2000/svg"; assertTrue(test.equals(test)); - final ContentItem test2 = (ContentItem) ContentModuleImplTest.contentItems.get(0); + final ContentItem test2 = ContentModuleImplTest.contentItems.get(0); assertTrue(!test.equals(test2)); final ContentItem test3 = new ContentItem(); test3.setContentFormat(new String("http://www.w3.org/1999/xhtml")); @@ -132,7 +133,5 @@ public class ContentItemTest extends TestCase { test3.setContentValue(new String("This is very cool.")); // assertEquals( test2, test3 ); } - // TODO add test methods here. The name must begin with 'test'. For example: - // public void testHello() {} } diff --git a/src/test/java/org/rometools/feed/module/content/ContentModuleImplTest.java b/src/test/java/org/rometools/feed/module/content/ContentModuleImplTest.java index 257e77c..d685412 100644 --- a/src/test/java/org/rometools/feed/module/content/ContentModuleImplTest.java +++ b/src/test/java/org/rometools/feed/module/content/ContentModuleImplTest.java @@ -126,7 +126,4 @@ public class ContentModuleImplTest extends TestCase { & test.getEncodeds().equals(module.getEncodeds())); } - // TODO add test methods here. The name must begin with 'test'. For example: - // public void testHello() {} - }