From 620cb7be19c862c12d83456fe31ff714dc80465f Mon Sep 17 00:00:00 2001
From: Christopher Chedeau <vjeuxx@gmail.com>
Date: Wed, 25 Jan 2017 13:36:55 -0800
Subject: [PATCH] Add break points for class declaration (#466)

```js
class A extends B implements C {
  method() {}
}
```

now outputs

```js
class A
  extends B
  implements C {
  method() {}
}
```

Fixes #147
---
 src/printer.js                                | 20 ++++++++++++-------
 .../classes/__snapshots__/jsfmt.spec.js.snap  | 13 ++++++++++++
 tests/classes/break.js                        |  3 +++
 3 files changed, 29 insertions(+), 7 deletions(-)
 create mode 100644 tests/classes/break.js

diff --git a/src/printer.js b/src/printer.js
index b2ce78ee1..9681b45aa 100644
--- a/src/printer.js
+++ b/src/printer.js
@@ -1202,7 +1202,7 @@ function genericPrintNoParens(path, options, print) {
       return concat(parts);
     case "ClassDeclaration":
     case "ClassExpression":
-      return concat(printClass(path, print));
+      return concat(printClass(path, options, print));
     case "TemplateElement":
       return join(literalline, n.value.raw.split("\n"));
     case "TemplateLiteral":
@@ -1291,7 +1291,7 @@ function genericPrintNoParens(path, options, print) {
     case "BooleanLiteralTypeAnnotation":
       return "" + n.value;
     case "DeclareClass":
-      return printFlowDeclaration(path, printClass(path, print));
+      return printFlowDeclaration(path, printClass(path, options, print));
     case "DeclareFunction":
       return printFlowDeclaration(path, [
         "function ",
@@ -1960,7 +1960,7 @@ function printFlowDeclaration(path, parts) {
   return concat(parts);
 }
 
-function printClass(path, print) {
+function printClass(path, options, print) {
   const n = path.getValue();
   const parts = [ "class" ];
 
@@ -1968,18 +1968,24 @@ function printClass(path, print) {
     parts.push(" ", path.call(print, "id"), path.call(print, "typeParameters"));
   }
 
+  const partsGroup = [];
   if (n.superClass) {
-    parts.push(
-      " extends ",
+    partsGroup.push(
+      line,
+      "extends ",
       path.call(print, "superClass"),
       path.call(print, "superTypeParameters")
     );
   } else if (n.extends && n.extends.length > 0) {
-    parts.push(" extends ", join(", ", path.map(print, "extends")));
+    partsGroup.push(line, "extends ", join(", ", path.map(print, "extends")));
   }
 
   if (n["implements"] && n["implements"].length > 0) {
-    parts.push(" implements ", join(", ", path.map(print, "implements")));
+    partsGroup.push(line, "implements ", join(", ", path.map(print, "implements")));
+  }
+
+  if (partsGroup.length > 0) {
+    parts.push(group(indent(options.tabWidth, concat(partsGroup))));
   }
 
   parts.push(" ", path.call(print, "body"));
diff --git a/tests/classes/__snapshots__/jsfmt.spec.js.snap b/tests/classes/__snapshots__/jsfmt.spec.js.snap
index b2bd35aad..d5a3c2580 100644
--- a/tests/classes/__snapshots__/jsfmt.spec.js.snap
+++ b/tests/classes/__snapshots__/jsfmt.spec.js.snap
@@ -11,6 +11,19 @@ exports[`test binary.js 1`] = `
 "
 `;
 
+exports[`test break.js 1`] = `
+"class MyContractSelectionWidget extends React.Component<void,  MyContractSelectionWidgetPropsType, void> implements SomethingLarge {
+  method() {}
+}
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+class MyContractSelectionWidget
+  extends React.Component<void, MyContractSelectionWidgetPropsType, void>
+  implements SomethingLarge {
+  method() {}
+}
+"
+`;
+
 exports[`test call.js 1`] = `
 "(class {})(class {});
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
diff --git a/tests/classes/break.js b/tests/classes/break.js
new file mode 100644
index 000000000..5d2c5c507
--- /dev/null
+++ b/tests/classes/break.js
@@ -0,0 +1,3 @@
+class MyContractSelectionWidget extends React.Component<void,  MyContractSelectionWidgetPropsType, void> implements SomethingLarge {
+  method() {}
+}