Merge pull request #153 from cdk/patch/rendering-regressions

Resolve regressions from recent patches - one is a behaviour change w…
cdk · Sep 13, 2015 · 04c820b · 04c820b
2 parents d30f869 + eeb4be0
commit 04c820b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/display/render/src/test/java/org/openscience/cdk/renderer/SymbolVisibilityTest.java b/display/render/src/test/java/org/openscience/cdk/renderer/SymbolVisibilityTest.java
@@ -24,6 +24,7 @@
 
 package org.openscience.cdk.renderer;
 
+import org.junit.Ignore;
 import org.junit.Test;
 import org.openscience.cdk.Atom;
 import org.openscience.cdk.Bond;
@@ -148,7 +149,7 @@ public void iupacCarbonParallel() {
     }
 
     // produces an NaN internally
-    @Test
+    @Ignore("Multiple Group Sgroup rendering can have zero length C-C bonds (e.g. overlaid coords), we don't want to show the symbols")
     public void iupacCarbonCornerCase() {
         IAtom a1 = new Atom("C");
         IAtom a2 = new Atom("C");

diff --git a/storage/io/src/main/java/org/openscience/cdk/io/MDLV2000Writer.java b/storage/io/src/main/java/org/openscience/cdk/io/MDLV2000Writer.java
@@ -848,6 +848,7 @@ private void writeSgroups(IAtomContainer container, BufferedWriter writer) throw
                             }
                             writer.newLine();
                         }
+                        break;
                     case CtabComponentNumber:
                         Integer compNumber = sgroup.getValue(key);
                         writer.write("M  SNC");