Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 1 | //===- unittest/Format/FormatTestJS.cpp - Formatting unit tests for JS ----===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 10 | #include "FormatTestUtils.h" |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 11 | #include "clang/Format/Format.h" |
| 12 | #include "llvm/Support/Debug.h" |
| 13 | #include "gtest/gtest.h" |
| 14 | |
Chandler Carruth | 1034666 | 2014-04-22 03:17:02 +0000 | [diff] [blame] | 15 | #define DEBUG_TYPE "format-test" |
| 16 | |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 17 | namespace clang { |
| 18 | namespace format { |
| 19 | |
| 20 | class FormatTestJS : public ::testing::Test { |
| 21 | protected: |
| 22 | static std::string format(llvm::StringRef Code, unsigned Offset, |
| 23 | unsigned Length, const FormatStyle &Style) { |
| 24 | DEBUG(llvm::errs() << "---\n"); |
| 25 | DEBUG(llvm::errs() << Code << "\n\n"); |
| 26 | std::vector<tooling::Range> Ranges(1, tooling::Range(Offset, Length)); |
Daniel Jasper | 56691b8 | 2015-06-11 13:29:20 +0000 | [diff] [blame] | 27 | bool IncompleteFormat = false; |
| 28 | tooling::Replacements Replaces = |
| 29 | reformat(Style, Code, Ranges, "<stdin>", &IncompleteFormat); |
| 30 | EXPECT_FALSE(IncompleteFormat); |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 31 | std::string Result = applyAllReplacements(Code, Replaces); |
| 32 | EXPECT_NE("", Result); |
| 33 | DEBUG(llvm::errs() << "\n" << Result << "\n\n"); |
| 34 | return Result; |
| 35 | } |
| 36 | |
Daniel Jasper | 069e5f4 | 2014-05-20 11:14:57 +0000 | [diff] [blame] | 37 | static std::string format( |
| 38 | llvm::StringRef Code, |
| 39 | const FormatStyle &Style = getGoogleStyle(FormatStyle::LK_JavaScript)) { |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 40 | return format(Code, 0, Code.size(), Style); |
| 41 | } |
| 42 | |
Alexander Kornienko | c1637f1 | 2013-12-10 11:28:13 +0000 | [diff] [blame] | 43 | static FormatStyle getGoogleJSStyleWithColumns(unsigned ColumnLimit) { |
Nico Weber | 514ecc8 | 2014-02-02 20:50:45 +0000 | [diff] [blame] | 44 | FormatStyle Style = getGoogleStyle(FormatStyle::LK_JavaScript); |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 45 | Style.ColumnLimit = ColumnLimit; |
| 46 | return Style; |
| 47 | } |
| 48 | |
Nico Weber | 514ecc8 | 2014-02-02 20:50:45 +0000 | [diff] [blame] | 49 | static void verifyFormat( |
| 50 | llvm::StringRef Code, |
| 51 | const FormatStyle &Style = getGoogleStyle(FormatStyle::LK_JavaScript)) { |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 52 | EXPECT_EQ(Code.str(), format(test::messUp(Code), Style)); |
| 53 | } |
| 54 | }; |
| 55 | |
| 56 | TEST_F(FormatTestJS, UnderstandsJavaScriptOperators) { |
| 57 | verifyFormat("a == = b;"); |
| 58 | verifyFormat("a != = b;"); |
| 59 | |
| 60 | verifyFormat("a === b;"); |
Daniel Jasper | e551bb7 | 2014-11-05 17:22:31 +0000 | [diff] [blame] | 61 | verifyFormat("aaaaaaa ===\n b;", getGoogleJSStyleWithColumns(10)); |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 62 | verifyFormat("a !== b;"); |
Daniel Jasper | e551bb7 | 2014-11-05 17:22:31 +0000 | [diff] [blame] | 63 | verifyFormat("aaaaaaa !==\n b;", getGoogleJSStyleWithColumns(10)); |
| 64 | verifyFormat("if (a + b + c +\n" |
| 65 | " d !==\n" |
| 66 | " e + f + g)\n" |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 67 | " q();", |
Alexander Kornienko | c1637f1 | 2013-12-10 11:28:13 +0000 | [diff] [blame] | 68 | getGoogleJSStyleWithColumns(20)); |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 69 | |
| 70 | verifyFormat("a >> >= b;"); |
| 71 | |
| 72 | verifyFormat("a >>> b;"); |
Daniel Jasper | e551bb7 | 2014-11-05 17:22:31 +0000 | [diff] [blame] | 73 | verifyFormat("aaaaaaa >>>\n b;", getGoogleJSStyleWithColumns(10)); |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 74 | verifyFormat("a >>>= b;"); |
Daniel Jasper | e551bb7 | 2014-11-05 17:22:31 +0000 | [diff] [blame] | 75 | verifyFormat("aaaaaaa >>>=\n b;", getGoogleJSStyleWithColumns(10)); |
| 76 | verifyFormat("if (a + b + c +\n" |
| 77 | " d >>>\n" |
| 78 | " e + f + g)\n" |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 79 | " q();", |
Alexander Kornienko | c1637f1 | 2013-12-10 11:28:13 +0000 | [diff] [blame] | 80 | getGoogleJSStyleWithColumns(20)); |
Daniel Jasper | e551bb7 | 2014-11-05 17:22:31 +0000 | [diff] [blame] | 81 | verifyFormat("var x = aaaaaaaaaa ?\n" |
| 82 | " bbbbbb :\n" |
| 83 | " ccc;", |
Alexander Kornienko | c1637f1 | 2013-12-10 11:28:13 +0000 | [diff] [blame] | 84 | getGoogleJSStyleWithColumns(20)); |
Daniel Jasper | 7821439 | 2014-05-19 07:27:02 +0000 | [diff] [blame] | 85 | |
| 86 | verifyFormat("var b = a.map((x) => x + 1);"); |
Daniel Jasper | 3549ea1 | 2014-09-19 10:48:15 +0000 | [diff] [blame] | 87 | verifyFormat("return ('aaa') in bbbb;"); |
Daniel Jasper | 0805199 | 2015-05-26 07:18:56 +0000 | [diff] [blame] | 88 | |
| 89 | // ES6 spread operator. |
| 90 | verifyFormat("someFunction(...a);"); |
| 91 | verifyFormat("var x = [1, ...a, 2];"); |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 92 | } |
| 93 | |
Daniel Jasper | 3a038de | 2014-09-05 08:53:45 +0000 | [diff] [blame] | 94 | TEST_F(FormatTestJS, UnderstandsAmpAmp) { |
| 95 | verifyFormat("e && e.SomeFunction();"); |
| 96 | } |
| 97 | |
Daniel Jasper | 4db69bd | 2014-09-04 18:23:42 +0000 | [diff] [blame] | 98 | TEST_F(FormatTestJS, LiteralOperatorsCanBeKeywords) { |
| 99 | verifyFormat("not.and.or.not_eq = 1;"); |
| 100 | } |
| 101 | |
Daniel Jasper | 0dd5291 | 2014-05-19 07:37:07 +0000 | [diff] [blame] | 102 | TEST_F(FormatTestJS, ES6DestructuringAssignment) { |
| 103 | verifyFormat("var [a, b, c] = [1, 2, 3];"); |
Daniel Jasper | 60948b1 | 2015-03-15 13:55:54 +0000 | [diff] [blame] | 104 | verifyFormat("var {a, b} = {a: 1, b: 2};"); |
Daniel Jasper | 0dd5291 | 2014-05-19 07:37:07 +0000 | [diff] [blame] | 105 | } |
| 106 | |
Daniel Jasper | 17062ff | 2014-06-10 14:44:02 +0000 | [diff] [blame] | 107 | TEST_F(FormatTestJS, ContainerLiterals) { |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 108 | verifyFormat("var x = {y: function(a) { return a; }};"); |
Daniel Jasper | 17062ff | 2014-06-10 14:44:02 +0000 | [diff] [blame] | 109 | verifyFormat("return {\n" |
| 110 | " link: function() {\n" |
| 111 | " f(); //\n" |
| 112 | " }\n" |
| 113 | "};"); |
| 114 | verifyFormat("return {\n" |
| 115 | " a: a,\n" |
| 116 | " link: function() {\n" |
| 117 | " f(); //\n" |
| 118 | " }\n" |
| 119 | "};"); |
| 120 | verifyFormat("return {\n" |
| 121 | " a: a,\n" |
Daniel Jasper | 90ebc98 | 2014-09-05 09:27:38 +0000 | [diff] [blame] | 122 | " link: function() {\n" |
| 123 | " f(); //\n" |
| 124 | " },\n" |
| 125 | " link: function() {\n" |
| 126 | " f(); //\n" |
| 127 | " }\n" |
Daniel Jasper | 17062ff | 2014-06-10 14:44:02 +0000 | [diff] [blame] | 128 | "};"); |
Daniel Jasper | 94e11d0 | 2014-09-04 14:58:30 +0000 | [diff] [blame] | 129 | verifyFormat("var stuff = {\n" |
| 130 | " // comment for update\n" |
| 131 | " update: false,\n" |
| 132 | " // comment for modules\n" |
| 133 | " modules: false,\n" |
| 134 | " // comment for tasks\n" |
| 135 | " tasks: false\n" |
| 136 | "};"); |
Daniel Jasper | 97bfb7b | 2014-09-05 08:29:31 +0000 | [diff] [blame] | 137 | verifyFormat("return {\n" |
| 138 | " 'finish':\n" |
| 139 | " //\n" |
| 140 | " a\n" |
| 141 | "};"); |
Daniel Jasper | 4087432 | 2014-11-27 15:24:48 +0000 | [diff] [blame] | 142 | verifyFormat("var obj = {\n" |
| 143 | " fooooooooo: function(x) {\n" |
| 144 | " return x.zIsTooLongForOneLineWithTheDeclarationLine();\n" |
| 145 | " }\n" |
| 146 | "};"); |
Daniel Jasper | 60948b1 | 2015-03-15 13:55:54 +0000 | [diff] [blame] | 147 | // Simple object literal, as opposed to enum style below. |
| 148 | verifyFormat("var obj = {a: 123};"); |
| 149 | // Enum style top level assignment. |
| 150 | verifyFormat("X = {\n a: 123\n};"); |
| 151 | verifyFormat("X.Y = {\n a: 123\n};"); |
Daniel Jasper | 11ca263 | 2015-06-10 09:21:09 +0000 | [diff] [blame] | 152 | // But only on the top level, otherwise its a plain object literal assignment. |
| 153 | verifyFormat("function x() {\n" |
| 154 | " y = {z: 1};\n" |
| 155 | "}"); |
Daniel Jasper | 60948b1 | 2015-03-15 13:55:54 +0000 | [diff] [blame] | 156 | verifyFormat("x = foo && {a: 123};"); |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 157 | |
| 158 | // Arrow functions in object literals. |
| 159 | verifyFormat("var x = {y: (a) => { return a; }};"); |
| 160 | verifyFormat("var x = {y: (a) => a};"); |
Daniel Jasper | 8c42d44 | 2015-05-29 06:19:49 +0000 | [diff] [blame] | 161 | |
| 162 | // Computed keys. |
Daniel Jasper | cd8d4ff | 2015-05-31 08:40:37 +0000 | [diff] [blame] | 163 | verifyFormat("var x = {[a]: 1, b: 2, [c]: 3};"); |
Daniel Jasper | 8c42d44 | 2015-05-29 06:19:49 +0000 | [diff] [blame] | 164 | verifyFormat("var x = {\n" |
| 165 | " [a]: 1,\n" |
Daniel Jasper | cd8d4ff | 2015-05-31 08:40:37 +0000 | [diff] [blame] | 166 | " b: 2,\n" |
| 167 | " [c]: 3,\n" |
Daniel Jasper | 8c42d44 | 2015-05-29 06:19:49 +0000 | [diff] [blame] | 168 | "};"); |
Daniel Jasper | 17062ff | 2014-06-10 14:44:02 +0000 | [diff] [blame] | 169 | } |
| 170 | |
Daniel Jasper | f46dec8 | 2015-03-31 14:34:15 +0000 | [diff] [blame] | 171 | TEST_F(FormatTestJS, MethodsInObjectLiterals) { |
| 172 | verifyFormat("var o = {\n" |
| 173 | " value: 'test',\n" |
| 174 | " get value() { // getter\n" |
| 175 | " return this.value;\n" |
| 176 | " }\n" |
| 177 | "};"); |
| 178 | verifyFormat("var o = {\n" |
| 179 | " value: 'test',\n" |
| 180 | " set value(val) { // setter\n" |
| 181 | " this.value = val;\n" |
| 182 | " }\n" |
| 183 | "};"); |
| 184 | verifyFormat("var o = {\n" |
| 185 | " value: 'test',\n" |
| 186 | " someMethod(val) { // method\n" |
| 187 | " doSomething(this.value + val);\n" |
| 188 | " }\n" |
| 189 | "};"); |
Daniel Jasper | 739ec53 | 2015-04-04 07:56:55 +0000 | [diff] [blame] | 190 | verifyFormat("var o = {\n" |
| 191 | " someMethod(val) { // method\n" |
| 192 | " doSomething(this.value + val);\n" |
| 193 | " },\n" |
| 194 | " someOtherMethod(val) { // method\n" |
| 195 | " doSomething(this.value + val);\n" |
| 196 | " }\n" |
| 197 | "};"); |
Daniel Jasper | f46dec8 | 2015-03-31 14:34:15 +0000 | [diff] [blame] | 198 | } |
| 199 | |
Daniel Jasper | b2e10a5 | 2014-01-15 15:09:08 +0000 | [diff] [blame] | 200 | TEST_F(FormatTestJS, SpacesInContainerLiterals) { |
| 201 | verifyFormat("var arr = [1, 2, 3];"); |
Daniel Jasper | a4e55f4 | 2014-12-04 16:07:17 +0000 | [diff] [blame] | 202 | verifyFormat("f({a: 1, b: 2, c: 3});"); |
Nico Weber | 514ecc8 | 2014-02-02 20:50:45 +0000 | [diff] [blame] | 203 | |
Daniel Jasper | 2a95832 | 2014-05-21 13:26:58 +0000 | [diff] [blame] | 204 | verifyFormat("var object_literal_with_long_name = {\n" |
| 205 | " a: 'aaaaaaaaaaaaaaaaaa',\n" |
| 206 | " b: 'bbbbbbbbbbbbbbbbbb'\n" |
| 207 | "};"); |
| 208 | |
Daniel Jasper | a4e55f4 | 2014-12-04 16:07:17 +0000 | [diff] [blame] | 209 | verifyFormat("f({a: 1, b: 2, c: 3});", |
Nico Weber | 514ecc8 | 2014-02-02 20:50:45 +0000 | [diff] [blame] | 210 | getChromiumStyle(FormatStyle::LK_JavaScript)); |
Daniel Jasper | a4e55f4 | 2014-12-04 16:07:17 +0000 | [diff] [blame] | 211 | verifyFormat("f({'a': [{}]});"); |
Daniel Jasper | b2e10a5 | 2014-01-15 15:09:08 +0000 | [diff] [blame] | 212 | } |
| 213 | |
Daniel Jasper | 86fee2f | 2014-01-31 12:49:42 +0000 | [diff] [blame] | 214 | TEST_F(FormatTestJS, SingleQuoteStrings) { |
| 215 | verifyFormat("this.function('', true);"); |
| 216 | } |
| 217 | |
Daniel Jasper | 4a39c84 | 2014-05-06 13:54:10 +0000 | [diff] [blame] | 218 | TEST_F(FormatTestJS, GoogScopes) { |
| 219 | verifyFormat("goog.scope(function() {\n" |
| 220 | "var x = a.b;\n" |
| 221 | "var y = c.d;\n" |
| 222 | "}); // goog.scope"); |
Daniel Jasper | 02c7bca | 2015-03-30 09:56:50 +0000 | [diff] [blame] | 223 | verifyFormat("goog.scope(function() {\n" |
| 224 | "// test\n" |
| 225 | "var x = 0;\n" |
| 226 | "// test\n" |
| 227 | "});"); |
Daniel Jasper | 4a39c84 | 2014-05-06 13:54:10 +0000 | [diff] [blame] | 228 | } |
| 229 | |
Daniel Jasper | 616de864 | 2014-11-23 16:46:28 +0000 | [diff] [blame] | 230 | TEST_F(FormatTestJS, GoogModules) { |
| 231 | verifyFormat("goog.module('this.is.really.absurdly.long');", |
| 232 | getGoogleJSStyleWithColumns(40)); |
| 233 | verifyFormat("goog.require('this.is.really.absurdly.long');", |
| 234 | getGoogleJSStyleWithColumns(40)); |
| 235 | verifyFormat("goog.provide('this.is.really.absurdly.long');", |
| 236 | getGoogleJSStyleWithColumns(40)); |
| 237 | verifyFormat("var long = goog.require('this.is.really.absurdly.long');", |
| 238 | getGoogleJSStyleWithColumns(40)); |
Daniel Jasper | 53c38f4 | 2014-11-27 14:46:03 +0000 | [diff] [blame] | 239 | |
| 240 | // These should be wrapped normally. |
| 241 | verifyFormat( |
| 242 | "var MyLongClassName =\n" |
| 243 | " goog.module.get('my.long.module.name.followedBy.MyLongClassName');"); |
Daniel Jasper | 616de864 | 2014-11-23 16:46:28 +0000 | [diff] [blame] | 244 | } |
| 245 | |
Daniel Jasper | ad9eb0d | 2014-06-30 13:24:54 +0000 | [diff] [blame] | 246 | TEST_F(FormatTestJS, FormatsFreestandingFunctions) { |
| 247 | verifyFormat("function outer1(a, b) {\n" |
| 248 | " function inner1(a, b) { return a; }\n" |
| 249 | " inner1(a, b);\n" |
| 250 | "}\n" |
| 251 | "function outer2(a, b) {\n" |
| 252 | " function inner2(a, b) { return a; }\n" |
| 253 | " inner2(a, b);\n" |
| 254 | "}"); |
Daniel Jasper | 20580fd | 2015-06-11 13:31:45 +0000 | [diff] [blame] | 255 | verifyFormat("function f() {}"); |
Daniel Jasper | ad9eb0d | 2014-06-30 13:24:54 +0000 | [diff] [blame] | 256 | } |
| 257 | |
Daniel Jasper | f841d3a | 2015-05-28 07:21:50 +0000 | [diff] [blame] | 258 | TEST_F(FormatTestJS, ArrayLiterals) { |
Daniel Jasper | 9970df1 | 2015-06-03 08:57:36 +0000 | [diff] [blame] | 259 | verifyFormat( |
| 260 | "var aaaaa: List<SomeThing> =\n" |
| 261 | " [new SomeThingAAAAAAAAAAAA(), new SomeThingBBBBBBBBB()];"); |
Daniel Jasper | d492b5e | 2015-06-02 21:57:51 +0000 | [diff] [blame] | 262 | verifyFormat("return [\n" |
| 263 | " aaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 264 | " bbbbbbbbbbbbbbbbbbbbbbbbbbb,\n" |
| 265 | " ccccccccccccccccccccccccccc\n" |
| 266 | "];"); |
Daniel Jasper | 5ce80de | 2015-06-02 13:56:43 +0000 | [diff] [blame] | 267 | verifyFormat("var someVariable = SomeFuntion([\n" |
| 268 | " aaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 269 | " bbbbbbbbbbbbbbbbbbbbbbbbbbb,\n" |
| 270 | " ccccccccccccccccccccccccccc\n" |
| 271 | "]);"); |
Daniel Jasper | 40432ce | 2015-06-02 15:04:29 +0000 | [diff] [blame] | 272 | verifyFormat("var someVariable = SomeFuntion([\n" |
| 273 | " [aaaaaaaaaaaaaaaaaaaaaa, bbbbbbbbbbbbbbbbbbbbbb],\n" |
Daniel Jasper | 199d0c9 | 2015-06-02 15:14:21 +0000 | [diff] [blame] | 274 | "]);", |
| 275 | getGoogleJSStyleWithColumns(51)); |
Daniel Jasper | 4284e36 | 2015-06-02 14:20:08 +0000 | [diff] [blame] | 276 | verifyFormat("var someVariable = SomeFuntion(aaaa, [\n" |
| 277 | " aaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 278 | " bbbbbbbbbbbbbbbbbbbbbbbbbbb,\n" |
| 279 | " ccccccccccccccccccccccccccc\n" |
| 280 | "]);"); |
Daniel Jasper | b2ad4d4 | 2015-06-15 09:23:17 +0000 | [diff] [blame] | 281 | verifyFormat("var someVariable = SomeFuntion(aaaa,\n" |
| 282 | " [\n" |
| 283 | " aaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 284 | " bbbbbbbbbbbbbbbbbbbbbbbbbbb,\n" |
| 285 | " ccccccccccccccccccccccccccc\n" |
| 286 | " ],\n" |
| 287 | " aaaa);"); |
Daniel Jasper | 1feab0f | 2015-06-02 15:31:37 +0000 | [diff] [blame] | 288 | |
| 289 | verifyFormat("someFunction([], {a: a});"); |
Daniel Jasper | f841d3a | 2015-05-28 07:21:50 +0000 | [diff] [blame] | 290 | } |
| 291 | |
Daniel Jasper | 5217a8b | 2014-06-13 07:02:04 +0000 | [diff] [blame] | 292 | TEST_F(FormatTestJS, FunctionLiterals) { |
Daniel Jasper | 3f69ba1 | 2014-09-05 08:42:27 +0000 | [diff] [blame] | 293 | verifyFormat("doFoo(function() {});"); |
Daniel Jasper | 79dffb4 | 2014-05-07 09:48:30 +0000 | [diff] [blame] | 294 | verifyFormat("doFoo(function() { return 1; });"); |
Daniel Jasper | 67f8ad2 | 2014-09-30 17:57:06 +0000 | [diff] [blame] | 295 | verifyFormat("var func = function() {\n" |
| 296 | " return 1;\n" |
| 297 | "};"); |
Daniel Jasper | c03e16a | 2014-05-08 09:25:39 +0000 | [diff] [blame] | 298 | verifyFormat("return {\n" |
| 299 | " body: {\n" |
| 300 | " setAttribute: function(key, val) { this[key] = val; },\n" |
| 301 | " getAttribute: function(key) { return this[key]; },\n" |
| 302 | " style: {direction: ''}\n" |
| 303 | " }\n" |
| 304 | "};"); |
Daniel Jasper | e551bb7 | 2014-11-05 17:22:31 +0000 | [diff] [blame] | 305 | EXPECT_EQ("abc = xyz ?\n" |
| 306 | " function() {\n" |
| 307 | " return 1;\n" |
| 308 | " } :\n" |
| 309 | " function() {\n" |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 310 | " return -1;\n" |
| 311 | " };", |
Daniel Jasper | 069e5f4 | 2014-05-20 11:14:57 +0000 | [diff] [blame] | 312 | format("abc=xyz?function(){return 1;}:function(){return -1;};")); |
Daniel Jasper | b16b969 | 2014-05-21 12:51:23 +0000 | [diff] [blame] | 313 | |
| 314 | verifyFormat("var closure = goog.bind(\n" |
| 315 | " function() { // comment\n" |
| 316 | " foo();\n" |
| 317 | " bar();\n" |
| 318 | " },\n" |
| 319 | " this, arg1IsReallyLongAndNeeedsLineBreaks,\n" |
| 320 | " arg3IsReallyLongAndNeeedsLineBreaks);"); |
| 321 | verifyFormat("var closure = goog.bind(function() { // comment\n" |
| 322 | " foo();\n" |
| 323 | " bar();\n" |
| 324 | "}, this);"); |
Daniel Jasper | 58cb2ed | 2014-06-06 13:49:04 +0000 | [diff] [blame] | 325 | verifyFormat("return {\n" |
| 326 | " a: 'E',\n" |
| 327 | " b: function() {\n" |
| 328 | " return function() {\n" |
| 329 | " f(); //\n" |
| 330 | " };\n" |
| 331 | " }\n" |
| 332 | "};"); |
Daniel Jasper | 41368e9 | 2014-11-27 15:37:42 +0000 | [diff] [blame] | 333 | verifyFormat("{\n" |
| 334 | " var someVariable = function(x) {\n" |
| 335 | " return x.zIsTooLongForOneLineWithTheDeclarationLine();\n" |
| 336 | " };\n" |
| 337 | "}"); |
Daniel Jasper | 7325aee | 2015-05-08 08:38:52 +0000 | [diff] [blame] | 338 | verifyFormat("someLooooooooongFunction(\n" |
| 339 | " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 340 | " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 341 | " function(aaaaaaaaaaaaaaaaaaaaaaaaaaaaa) {\n" |
| 342 | " // code\n" |
| 343 | " });"); |
Daniel Jasper | 5f3ea47 | 2014-05-22 08:36:53 +0000 | [diff] [blame] | 344 | |
Daniel Jasper | a4e55f4 | 2014-12-04 16:07:17 +0000 | [diff] [blame] | 345 | verifyFormat("f({a: function() { return 1; }});", |
| 346 | getGoogleJSStyleWithColumns(33)); |
| 347 | verifyFormat("f({\n" |
Daniel Jasper | 5f3ea47 | 2014-05-22 08:36:53 +0000 | [diff] [blame] | 348 | " a: function() { return 1; }\n" |
Daniel Jasper | a4e55f4 | 2014-12-04 16:07:17 +0000 | [diff] [blame] | 349 | "});", |
| 350 | getGoogleJSStyleWithColumns(32)); |
Daniel Jasper | 5217a8b | 2014-06-13 07:02:04 +0000 | [diff] [blame] | 351 | |
| 352 | verifyFormat("return {\n" |
| 353 | " a: function SomeFunction() {\n" |
| 354 | " // ...\n" |
| 355 | " return 1;\n" |
| 356 | " }\n" |
| 357 | "};"); |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 358 | verifyFormat("this.someObject.doSomething(aaaaaaaaaaaaaaaaaaaaaaaaaa)\n" |
| 359 | " .then(goog.bind(function(aaaaaaaaaaa) {\n" |
| 360 | " someFunction();\n" |
| 361 | " someFunction();\n" |
| 362 | " }, this), aaaaaaaaaaaaaaaaa);"); |
| 363 | |
| 364 | // FIXME: This is not ideal yet. |
| 365 | verifyFormat("someFunction(goog.bind(\n" |
| 366 | " function() {\n" |
| 367 | " doSomething();\n" |
| 368 | " doSomething();\n" |
| 369 | " },\n" |
| 370 | " this),\n" |
| 371 | " goog.bind(function() {\n" |
| 372 | " doSomething();\n" |
| 373 | " doSomething();\n" |
| 374 | " }, this));"); |
Daniel Jasper | 1699eca | 2015-06-01 09:56:32 +0000 | [diff] [blame] | 375 | |
| 376 | // FIXME: This is bad, we should be wrapping before "function() {". |
| 377 | verifyFormat("someFunction(function() {\n" |
| 378 | " doSomething(); // break\n" |
| 379 | "})\n" |
| 380 | " .doSomethingElse(\n" |
| 381 | " // break\n" |
| 382 | " );"); |
Daniel Jasper | 79dffb4 | 2014-05-07 09:48:30 +0000 | [diff] [blame] | 383 | } |
| 384 | |
Daniel Jasper | 67f8ad2 | 2014-09-30 17:57:06 +0000 | [diff] [blame] | 385 | TEST_F(FormatTestJS, InliningFunctionLiterals) { |
| 386 | FormatStyle Style = getGoogleStyle(FormatStyle::LK_JavaScript); |
| 387 | Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_Inline; |
| 388 | verifyFormat("var func = function() {\n" |
| 389 | " return 1;\n" |
| 390 | "};", |
| 391 | Style); |
| 392 | verifyFormat("var func = doSomething(function() { return 1; });", Style); |
| 393 | verifyFormat("var outer = function() {\n" |
| 394 | " var inner = function() { return 1; }\n" |
| 395 | "};", |
| 396 | Style); |
| 397 | verifyFormat("function outer1(a, b) {\n" |
| 398 | " function inner1(a, b) { return a; }\n" |
| 399 | "}", |
| 400 | Style); |
| 401 | |
| 402 | Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_All; |
| 403 | verifyFormat("var func = function() { return 1; };", Style); |
| 404 | verifyFormat("var func = doSomething(function() { return 1; });", Style); |
| 405 | verifyFormat( |
| 406 | "var outer = function() { var inner = function() { return 1; } };", |
| 407 | Style); |
| 408 | verifyFormat("function outer1(a, b) {\n" |
| 409 | " function inner1(a, b) { return a; }\n" |
| 410 | "}", |
| 411 | Style); |
| 412 | |
| 413 | Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_None; |
| 414 | verifyFormat("var func = function() {\n" |
| 415 | " return 1;\n" |
| 416 | "};", |
| 417 | Style); |
| 418 | verifyFormat("var func = doSomething(function() {\n" |
| 419 | " return 1;\n" |
| 420 | "});", |
| 421 | Style); |
| 422 | verifyFormat("var outer = function() {\n" |
| 423 | " var inner = function() {\n" |
| 424 | " return 1;\n" |
| 425 | " }\n" |
| 426 | "};", |
| 427 | Style); |
| 428 | verifyFormat("function outer1(a, b) {\n" |
| 429 | " function inner1(a, b) {\n" |
| 430 | " return a;\n" |
| 431 | " }\n" |
| 432 | "}", |
| 433 | Style); |
| 434 | } |
| 435 | |
Daniel Jasper | 114a2bc | 2014-06-03 12:02:45 +0000 | [diff] [blame] | 436 | TEST_F(FormatTestJS, MultipleFunctionLiterals) { |
| 437 | verifyFormat("promise.then(\n" |
| 438 | " function success() {\n" |
| 439 | " doFoo();\n" |
| 440 | " doBar();\n" |
| 441 | " },\n" |
| 442 | " function error() {\n" |
| 443 | " doFoo();\n" |
| 444 | " doBaz();\n" |
| 445 | " },\n" |
| 446 | " []);\n"); |
| 447 | verifyFormat("promise.then(\n" |
| 448 | " function success() {\n" |
| 449 | " doFoo();\n" |
| 450 | " doBar();\n" |
| 451 | " },\n" |
| 452 | " [],\n" |
| 453 | " function error() {\n" |
| 454 | " doFoo();\n" |
| 455 | " doBaz();\n" |
| 456 | " });\n"); |
| 457 | // FIXME: Here, we should probably break right after the "(" for consistency. |
| 458 | verifyFormat("promise.then([],\n" |
| 459 | " function success() {\n" |
| 460 | " doFoo();\n" |
| 461 | " doBar();\n" |
| 462 | " },\n" |
| 463 | " function error() {\n" |
| 464 | " doFoo();\n" |
| 465 | " doBaz();\n" |
| 466 | " });\n"); |
Daniel Jasper | 1779d43 | 2014-09-29 07:54:54 +0000 | [diff] [blame] | 467 | |
| 468 | verifyFormat("getSomeLongPromise()\n" |
| 469 | " .then(function(value) { body(); })\n" |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 470 | " .thenCatch(function(error) {\n" |
| 471 | " body();\n" |
| 472 | " body();\n" |
| 473 | " });"); |
Daniel Jasper | 1779d43 | 2014-09-29 07:54:54 +0000 | [diff] [blame] | 474 | verifyFormat("getSomeLongPromise()\n" |
| 475 | " .then(function(value) {\n" |
| 476 | " body();\n" |
| 477 | " body();\n" |
| 478 | " })\n" |
| 479 | " .thenCatch(function(error) {\n" |
| 480 | " body();\n" |
| 481 | " body();\n" |
| 482 | " });"); |
Daniel Jasper | 11a0ac6 | 2014-12-12 09:40:58 +0000 | [diff] [blame] | 483 | |
Daniel Jasper | acf67e3 | 2015-04-07 08:20:35 +0000 | [diff] [blame] | 484 | verifyFormat("getSomeLongPromise()\n" |
| 485 | " .then(function(value) { body(); })\n" |
| 486 | " .thenCatch(function(error) { body(); });"); |
Daniel Jasper | 114a2bc | 2014-06-03 12:02:45 +0000 | [diff] [blame] | 487 | } |
| 488 | |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 489 | TEST_F(FormatTestJS, ArrowFunctions) { |
| 490 | verifyFormat("var x = (a) => {\n" |
| 491 | " return a;\n" |
| 492 | "};"); |
| 493 | verifyFormat("var x = (a) => {\n" |
| 494 | " function y() { return 42; }\n" |
| 495 | " return a;\n" |
| 496 | "};"); |
| 497 | verifyFormat("var x = (a: type): {some: type} => {\n" |
| 498 | " return a;\n" |
| 499 | "};"); |
| 500 | verifyFormat("var x = (a) => a;"); |
Daniel Jasper | e497bed | 2015-06-02 22:06:07 +0000 | [diff] [blame] | 501 | verifyFormat("return () => [];"); |
Daniel Jasper | 3c306e8 | 2015-06-03 17:08:40 +0000 | [diff] [blame] | 502 | verifyFormat("var aaaaaaaaaaaaaaaaaaaa = {\n" |
| 503 | " aaaaaaaaaaaaaaaaaaaaaaaaaaaa:\n" |
| 504 | " (aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
| 505 | " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa) =>\n" |
Daniel Jasper | 3b0f304 | 2015-06-05 08:25:37 +0000 | [diff] [blame] | 506 | " aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa,\n" |
Daniel Jasper | 3c306e8 | 2015-06-03 17:08:40 +0000 | [diff] [blame] | 507 | "};"); |
Daniel Jasper | 3b0f304 | 2015-06-05 08:25:37 +0000 | [diff] [blame] | 508 | verifyFormat( |
| 509 | "var a = a.aaaaaaa((a: a) => aaaaaaaaaaaaaaaaaaaaa(bbbbbbbbb) &&\n" |
| 510 | " aaaaaaaaaaaaaaaaaaaaa(bbbbbbb));"); |
| 511 | verifyFormat( |
| 512 | "var a = a.aaaaaaa((a: a) => aaaaaaaaaaaaaaaaaaaaa(bbbbbbbbb) ?\n" |
| 513 | " aaaaaaaaaaaaaaaaaaaaa(bbbbbbb) :\n" |
| 514 | " aaaaaaaaaaaaaaaaaaaaa(bbbbbbb));"); |
Daniel Jasper | 1699eca | 2015-06-01 09:56:32 +0000 | [diff] [blame] | 515 | |
| 516 | // FIXME: This is bad, we should be wrapping before "() => {". |
| 517 | verifyFormat("someFunction(() => {\n" |
| 518 | " doSomething(); // break\n" |
| 519 | "})\n" |
| 520 | " .doSomethingElse(\n" |
| 521 | " // break\n" |
| 522 | " );"); |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 523 | } |
| 524 | |
Daniel Jasper | 166c19b | 2014-05-06 14:12:21 +0000 | [diff] [blame] | 525 | TEST_F(FormatTestJS, ReturnStatements) { |
Daniel Jasper | 67f8ad2 | 2014-09-30 17:57:06 +0000 | [diff] [blame] | 526 | verifyFormat("function() {\n" |
| 527 | " return [hello, world];\n" |
| 528 | "}"); |
Daniel Jasper | 166c19b | 2014-05-06 14:12:21 +0000 | [diff] [blame] | 529 | } |
| 530 | |
Daniel Jasper | 4c0bf70 | 2015-06-12 04:58:27 +0000 | [diff] [blame] | 531 | TEST_F(FormatTestJS, AutomaticSemicolonInsertion) { |
| 532 | // The following statements must not wrap, as otherwise the program meaning |
| 533 | // would change due to automatic semicolon insertion. |
| 534 | // See http://www.ecma-international.org/ecma-262/5.1/#sec-7.9.1. |
| 535 | verifyFormat("return aaaaa;", getGoogleJSStyleWithColumns(10)); |
| 536 | verifyFormat("continue aaaaa;", getGoogleJSStyleWithColumns(10)); |
| 537 | verifyFormat("break aaaaa;", getGoogleJSStyleWithColumns(10)); |
| 538 | verifyFormat("throw aaaaa;", getGoogleJSStyleWithColumns(10)); |
| 539 | verifyFormat("aaaaaaaaa++;", getGoogleJSStyleWithColumns(10)); |
| 540 | verifyFormat("aaaaaaaaa--;", getGoogleJSStyleWithColumns(10)); |
| 541 | } |
| 542 | |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 543 | TEST_F(FormatTestJS, ClosureStyleCasts) { |
Daniel Jasper | 484033b | 2014-05-06 14:41:29 +0000 | [diff] [blame] | 544 | verifyFormat("var x = /** @type {foo} */ (bar);"); |
| 545 | } |
| 546 | |
Daniel Jasper | 04a71a4 | 2014-05-08 11:58:24 +0000 | [diff] [blame] | 547 | TEST_F(FormatTestJS, TryCatch) { |
| 548 | verifyFormat("try {\n" |
| 549 | " f();\n" |
| 550 | "} catch (e) {\n" |
| 551 | " g();\n" |
| 552 | "} finally {\n" |
| 553 | " h();\n" |
| 554 | "}"); |
Daniel Jasper | 8f2e94c | 2014-09-04 15:03:34 +0000 | [diff] [blame] | 555 | |
| 556 | // But, of course, "catch" is a perfectly fine function name in JavaScript. |
| 557 | verifyFormat("someObject.catch();"); |
Daniel Jasper | 7912123 | 2014-11-27 14:55:17 +0000 | [diff] [blame] | 558 | verifyFormat("someObject.new();"); |
| 559 | verifyFormat("someObject.delete();"); |
Daniel Jasper | 04a71a4 | 2014-05-08 11:58:24 +0000 | [diff] [blame] | 560 | } |
| 561 | |
Daniel Jasper | 49802ef | 2014-05-22 09:10:04 +0000 | [diff] [blame] | 562 | TEST_F(FormatTestJS, StringLiteralConcatenation) { |
| 563 | verifyFormat("var literal = 'hello ' +\n" |
| 564 | " 'world';"); |
| 565 | } |
| 566 | |
Daniel Jasper | f9ae312 | 2014-05-08 07:01:45 +0000 | [diff] [blame] | 567 | TEST_F(FormatTestJS, RegexLiteralClassification) { |
| 568 | // Regex literals. |
| 569 | verifyFormat("var regex = /abc/;"); |
| 570 | verifyFormat("f(/abc/);"); |
| 571 | verifyFormat("f(abc, /abc/);"); |
| 572 | verifyFormat("some_map[/abc/];"); |
| 573 | verifyFormat("var x = a ? /abc/ : /abc/;"); |
| 574 | verifyFormat("for (var i = 0; /abc/.test(s[i]); i++) {\n}"); |
| 575 | verifyFormat("var x = !/abc/.test(y);"); |
| 576 | verifyFormat("var x = a && /abc/.test(y);"); |
| 577 | verifyFormat("var x = a || /abc/.test(y);"); |
| 578 | verifyFormat("var x = a + /abc/.search(y);"); |
Daniel Jasper | f7405c1 | 2014-05-08 07:45:18 +0000 | [diff] [blame] | 579 | verifyFormat("var regexs = {/abc/, /abc/};"); |
| 580 | verifyFormat("return /abc/;"); |
Daniel Jasper | f9ae312 | 2014-05-08 07:01:45 +0000 | [diff] [blame] | 581 | |
| 582 | // Not regex literals. |
| 583 | verifyFormat("var a = a / 2 + b / 3;"); |
| 584 | } |
| 585 | |
| 586 | TEST_F(FormatTestJS, RegexLiteralSpecialCharacters) { |
Daniel Jasper | 69694b0 | 2015-05-08 07:55:13 +0000 | [diff] [blame] | 587 | verifyFormat("var regex = /=/;"); |
Daniel Jasper | f9ae312 | 2014-05-08 07:01:45 +0000 | [diff] [blame] | 588 | verifyFormat("var regex = /a*/;"); |
| 589 | verifyFormat("var regex = /a+/;"); |
| 590 | verifyFormat("var regex = /a?/;"); |
| 591 | verifyFormat("var regex = /.a./;"); |
| 592 | verifyFormat("var regex = /a\\*/;"); |
| 593 | verifyFormat("var regex = /^a$/;"); |
| 594 | verifyFormat("var regex = /\\/a/;"); |
| 595 | verifyFormat("var regex = /(?:x)/;"); |
| 596 | verifyFormat("var regex = /x(?=y)/;"); |
| 597 | verifyFormat("var regex = /x(?!y)/;"); |
| 598 | verifyFormat("var regex = /x|y/;"); |
| 599 | verifyFormat("var regex = /a{2}/;"); |
| 600 | verifyFormat("var regex = /a{1,3}/;"); |
| 601 | verifyFormat("var regex = /[abc]/;"); |
| 602 | verifyFormat("var regex = /[^abc]/;"); |
| 603 | verifyFormat("var regex = /[\\b]/;"); |
| 604 | verifyFormat("var regex = /\\b/;"); |
| 605 | verifyFormat("var regex = /\\B/;"); |
| 606 | verifyFormat("var regex = /\\d/;"); |
| 607 | verifyFormat("var regex = /\\D/;"); |
| 608 | verifyFormat("var regex = /\\f/;"); |
| 609 | verifyFormat("var regex = /\\n/;"); |
| 610 | verifyFormat("var regex = /\\r/;"); |
| 611 | verifyFormat("var regex = /\\s/;"); |
| 612 | verifyFormat("var regex = /\\S/;"); |
| 613 | verifyFormat("var regex = /\\t/;"); |
| 614 | verifyFormat("var regex = /\\v/;"); |
| 615 | verifyFormat("var regex = /\\w/;"); |
| 616 | verifyFormat("var regex = /\\W/;"); |
| 617 | verifyFormat("var regex = /a(a)\\1/;"); |
| 618 | verifyFormat("var regex = /\\0/;"); |
Daniel Jasper | fb4333b | 2014-05-12 11:29:50 +0000 | [diff] [blame] | 619 | verifyFormat("var regex = /\\\\/g;"); |
| 620 | verifyFormat("var regex = /\\a\\\\/g;"); |
| 621 | verifyFormat("var regex = /\a\\//g;"); |
Daniel Jasper | 2337625 | 2014-09-09 14:37:39 +0000 | [diff] [blame] | 622 | verifyFormat("var regex = /a\\//;\n" |
| 623 | "var x = 0;"); |
Daniel Jasper | 49a9a28 | 2014-10-29 16:51:38 +0000 | [diff] [blame] | 624 | EXPECT_EQ("var regex = /\\/*/;\n" |
| 625 | "var x = 0;", |
| 626 | format("var regex = /\\/*/;\n" |
| 627 | "var x=0;")); |
Daniel Jasper | f9ae312 | 2014-05-08 07:01:45 +0000 | [diff] [blame] | 628 | } |
| 629 | |
| 630 | TEST_F(FormatTestJS, RegexLiteralModifiers) { |
| 631 | verifyFormat("var regex = /abc/g;"); |
| 632 | verifyFormat("var regex = /abc/i;"); |
| 633 | verifyFormat("var regex = /abc/m;"); |
| 634 | verifyFormat("var regex = /abc/y;"); |
| 635 | } |
| 636 | |
| 637 | TEST_F(FormatTestJS, RegexLiteralLength) { |
| 638 | verifyFormat("var regex = /aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa/;", |
| 639 | getGoogleJSStyleWithColumns(60)); |
| 640 | verifyFormat("var regex =\n" |
| 641 | " /aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa/;", |
| 642 | getGoogleJSStyleWithColumns(60)); |
Daniel Jasper | 0580ff0 | 2014-12-17 09:11:08 +0000 | [diff] [blame] | 643 | verifyFormat("var regex = /\\xaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa/;", |
| 644 | getGoogleJSStyleWithColumns(50)); |
Daniel Jasper | f9ae312 | 2014-05-08 07:01:45 +0000 | [diff] [blame] | 645 | } |
| 646 | |
| 647 | TEST_F(FormatTestJS, RegexLiteralExamples) { |
| 648 | verifyFormat("var regex = search.match(/(?:\?|&)times=([^?&]+)/i);"); |
| 649 | } |
| 650 | |
Daniel Jasper | b10bdff | 2015-02-18 17:09:53 +0000 | [diff] [blame] | 651 | TEST_F(FormatTestJS, TypeAnnotations) { |
| 652 | verifyFormat("var x: string;"); |
| 653 | verifyFormat("function x(): string {\n return 'x';\n}"); |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 654 | verifyFormat("function x(): {x: string} {\n return {x: 'x'};\n}"); |
Daniel Jasper | b10bdff | 2015-02-18 17:09:53 +0000 | [diff] [blame] | 655 | verifyFormat("function x(y: string): string {\n return 'x';\n}"); |
| 656 | verifyFormat("for (var y: string in x) {\n x();\n}"); |
| 657 | verifyFormat("((a: string, b: number): string => a + b);"); |
| 658 | verifyFormat("var x: (y: number) => string;"); |
| 659 | verifyFormat("var x: P<string, (a: number) => string>;"); |
Manuel Klimek | 79e0608 | 2015-05-21 12:23:34 +0000 | [diff] [blame] | 660 | verifyFormat("var x = {y: function(): z { return 1; }};"); |
| 661 | verifyFormat("var x = {y: function(): {a: number} { return 1; }};"); |
Daniel Jasper | b10bdff | 2015-02-18 17:09:53 +0000 | [diff] [blame] | 662 | } |
| 663 | |
Daniel Jasper | 8370908 | 2015-02-18 17:14:05 +0000 | [diff] [blame] | 664 | TEST_F(FormatTestJS, ClassDeclarations) { |
| 665 | verifyFormat("class C {\n x: string = 12;\n}"); |
| 666 | verifyFormat("class C {\n x(): string => 12;\n}"); |
| 667 | verifyFormat("class C {\n ['x' + 2]: string = 12;\n}"); |
| 668 | verifyFormat("class C {\n private x: string = 12;\n}"); |
| 669 | verifyFormat("class C {\n private static x: string = 12;\n}"); |
| 670 | verifyFormat("class C {\n static x(): string { return 'asd'; }\n}"); |
| 671 | verifyFormat("class C extends P implements I {}"); |
Daniel Jasper | 6a5d38d | 2015-04-13 14:56:54 +0000 | [diff] [blame] | 672 | verifyFormat("class C extends p.P implements i.I {}"); |
Daniel Jasper | 81dbb56 | 2015-06-03 08:43:18 +0000 | [diff] [blame] | 673 | |
| 674 | // ':' is not a type declaration here. |
| 675 | verifyFormat("class X {\n" |
| 676 | " subs = {\n" |
| 677 | " 'b': {\n" |
| 678 | " 'c': 1,\n" |
| 679 | " },\n" |
| 680 | " };\n" |
| 681 | "}"); |
Daniel Jasper | 8370908 | 2015-02-18 17:14:05 +0000 | [diff] [blame] | 682 | } |
| 683 | |
Daniel Jasper | 2964749 | 2015-05-05 08:12:50 +0000 | [diff] [blame] | 684 | TEST_F(FormatTestJS, InterfaceDeclarations) { |
| 685 | verifyFormat("interface I {\n" |
| 686 | " x: string;\n" |
Daniel Jasper | 259188b | 2015-06-12 04:56:34 +0000 | [diff] [blame] | 687 | "}\n" |
| 688 | "var y;"); |
Daniel Jasper | 2964749 | 2015-05-05 08:12:50 +0000 | [diff] [blame] | 689 | } |
| 690 | |
Daniel Jasper | 90cf380 | 2015-06-17 09:44:02 +0000 | [diff] [blame] | 691 | TEST_F(FormatTestJS, EnumDeclarations) { |
| 692 | verifyFormat("enum Foo {\n" |
Daniel Jasper | 9f4ec15 | 2015-06-17 09:44:07 +0000 | [diff] [blame^] | 693 | " A = 1,\n" |
| 694 | " B\n" |
| 695 | "}"); |
| 696 | verifyFormat("export /* somecomment*/ enum Foo {\n" |
| 697 | " A = 1,\n" |
| 698 | " B\n" |
| 699 | "}"); |
| 700 | verifyFormat("enum Foo {\n" |
Daniel Jasper | 90cf380 | 2015-06-17 09:44:02 +0000 | [diff] [blame] | 701 | " A = 1, // comment\n" |
| 702 | " B\n" |
| 703 | "}\n" |
| 704 | "var x = 1;"); |
| 705 | } |
| 706 | |
Daniel Jasper | 3c42dba | 2015-02-18 17:17:15 +0000 | [diff] [blame] | 707 | TEST_F(FormatTestJS, MetadataAnnotations) { |
| 708 | verifyFormat("@A\nclass C {\n}"); |
| 709 | verifyFormat("@A({arg: 'value'})\nclass C {\n}"); |
| 710 | verifyFormat("@A\n@B\nclass C {\n}"); |
| 711 | verifyFormat("class C {\n @A x: string;\n}"); |
| 712 | verifyFormat("class C {\n" |
| 713 | " @A\n" |
| 714 | " private x(): string {\n" |
| 715 | " return 'y';\n" |
| 716 | " }\n" |
| 717 | "}"); |
Daniel Jasper | 6fa9ec7 | 2015-02-19 16:03:16 +0000 | [diff] [blame] | 718 | verifyFormat("class X {}\n" |
| 719 | "class Y {}"); |
Daniel Jasper | 3c42dba | 2015-02-18 17:17:15 +0000 | [diff] [blame] | 720 | } |
| 721 | |
Daniel Jasper | 354aa51 | 2015-02-19 16:07:32 +0000 | [diff] [blame] | 722 | TEST_F(FormatTestJS, Modules) { |
| 723 | verifyFormat("import SomeThing from 'some/module.js';"); |
| 724 | verifyFormat("import {X, Y} from 'some/module.js';"); |
| 725 | verifyFormat("import {\n" |
| 726 | " VeryLongImportsAreAnnoying,\n" |
| 727 | " VeryLongImportsAreAnnoying,\n" |
| 728 | " VeryLongImportsAreAnnoying,\n" |
| 729 | " VeryLongImportsAreAnnoying\n" |
| 730 | "} from 'some/module.js';"); |
| 731 | verifyFormat("import {\n" |
| 732 | " X,\n" |
| 733 | " Y,\n" |
| 734 | "} from 'some/module.js';"); |
| 735 | verifyFormat("import {\n" |
| 736 | " X,\n" |
| 737 | " Y,\n" |
| 738 | "} from 'some/long/module.js';", |
| 739 | getGoogleJSStyleWithColumns(20)); |
| 740 | verifyFormat("import {X as myLocalX, Y as myLocalY} from 'some/module.js';"); |
| 741 | verifyFormat("import * as lib from 'some/module.js';"); |
Daniel Jasper | 60948b1 | 2015-03-15 13:55:54 +0000 | [diff] [blame] | 742 | verifyFormat("var x = {import: 1};\nx.import = 2;"); |
Daniel Jasper | fca735c | 2015-02-19 16:14:18 +0000 | [diff] [blame] | 743 | |
| 744 | verifyFormat("export function fn() {\n" |
| 745 | " return 'fn';\n" |
| 746 | "}"); |
Daniel Jasper | 20580fd | 2015-06-11 13:31:45 +0000 | [diff] [blame] | 747 | verifyFormat("export function A() {}\n" |
| 748 | "export default function B() {}\n" |
| 749 | "export function C() {}"); |
Daniel Jasper | 354aa51 | 2015-02-19 16:07:32 +0000 | [diff] [blame] | 750 | verifyFormat("export const x = 12;"); |
| 751 | verifyFormat("export default class X {}"); |
Daniel Jasper | fca735c | 2015-02-19 16:14:18 +0000 | [diff] [blame] | 752 | verifyFormat("export {X, Y} from 'some/module.js';"); |
| 753 | verifyFormat("export {\n" |
| 754 | " X,\n" |
| 755 | " Y,\n" |
| 756 | "} from 'some/module.js';"); |
| 757 | verifyFormat("export class C {\n" |
| 758 | " x: number;\n" |
| 759 | " y: string;\n" |
| 760 | "}"); |
| 761 | verifyFormat("export class X { y: number; }"); |
| 762 | verifyFormat("export default class X { y: number }"); |
| 763 | verifyFormat("export default function() {\n return 1;\n}"); |
| 764 | verifyFormat("export var x = 12;"); |
Daniel Jasper | 910807d | 2015-06-12 04:52:02 +0000 | [diff] [blame] | 765 | verifyFormat("class C {}\n" |
| 766 | "export function f() {}\n" |
| 767 | "var v;"); |
Daniel Jasper | fca735c | 2015-02-19 16:14:18 +0000 | [diff] [blame] | 768 | verifyFormat("export var x: number = 12;"); |
| 769 | verifyFormat("export const y = {\n" |
| 770 | " a: 1,\n" |
| 771 | " b: 2\n" |
| 772 | "};"); |
Daniel Jasper | 216c9cd | 2015-06-12 05:08:18 +0000 | [diff] [blame] | 773 | verifyFormat("export enum Foo {\n" |
| 774 | " BAR,\n" |
| 775 | " // adsdasd\n" |
| 776 | " BAZ\n" |
| 777 | "}"); |
Daniel Jasper | 354aa51 | 2015-02-19 16:07:32 +0000 | [diff] [blame] | 778 | } |
| 779 | |
Daniel Jasper | a0ef4f3 | 2015-02-20 13:47:38 +0000 | [diff] [blame] | 780 | TEST_F(FormatTestJS, TemplateStrings) { |
| 781 | // Keeps any whitespace/indentation within the template string. |
| 782 | EXPECT_EQ("var x = `hello\n" |
| 783 | " ${ name }\n" |
| 784 | " !`;", |
| 785 | format("var x = `hello\n" |
| 786 | " ${ name }\n" |
| 787 | " !`;")); |
| 788 | |
| 789 | // FIXME: +1 / -1 offsets are to work around clang-format miscalculating |
| 790 | // widths for unknown tokens that are not whitespace (e.g. '`'). Remove when |
| 791 | // the code is corrected. |
| 792 | |
| 793 | verifyFormat("var x =\n" |
| 794 | " `hello ${world}` >= some();", |
| 795 | getGoogleJSStyleWithColumns(34)); // Barely doesn't fit. |
| 796 | verifyFormat("var x = `hello ${world}` >= some();", |
| 797 | getGoogleJSStyleWithColumns(35 + 1)); // Barely fits. |
| 798 | EXPECT_EQ("var x = `hello\n" |
| 799 | " ${world}` >=\n" |
| 800 | " some();", |
| 801 | format("var x =\n" |
| 802 | " `hello\n" |
| 803 | " ${world}` >= some();", |
| 804 | getGoogleJSStyleWithColumns(21))); // Barely doesn't fit. |
| 805 | EXPECT_EQ("var x = `hello\n" |
| 806 | " ${world}` >= some();", |
| 807 | format("var x =\n" |
| 808 | " `hello\n" |
| 809 | " ${world}` >= some();", |
| 810 | getGoogleJSStyleWithColumns(22))); // Barely fits. |
| 811 | |
| 812 | verifyFormat("var x =\n `h`;", getGoogleJSStyleWithColumns(13 - 1)); |
| 813 | EXPECT_EQ( |
| 814 | "var x =\n `multi\n line`;", |
| 815 | format("var x = `multi\n line`;", getGoogleJSStyleWithColumns(14 - 1))); |
| 816 | |
Daniel Jasper | f69b922 | 2015-05-02 08:05:38 +0000 | [diff] [blame] | 817 | // Make sure template strings get a proper ColumnWidth assigned, even if they |
| 818 | // are first token in line. |
| 819 | verifyFormat( |
| 820 | "var a = aaaaaaaaaaaaaaaaaaaaaaaaaaaa ||\n" |
| 821 | " `aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa`;"); |
| 822 | |
Daniel Jasper | a0ef4f3 | 2015-02-20 13:47:38 +0000 | [diff] [blame] | 823 | // Two template strings. |
| 824 | verifyFormat("var x = `hello` == `hello`;"); |
Daniel Jasper | 0d6ac27 | 2015-04-16 08:20:51 +0000 | [diff] [blame] | 825 | |
| 826 | // Comments in template strings. |
| 827 | EXPECT_EQ("var x = `//a`;\n" |
| 828 | "var y;", |
| 829 | format("var x =\n `//a`;\n" |
| 830 | "var y ;")); |
| 831 | EXPECT_EQ("var x = `/*a`;\n" |
| 832 | "var y;", |
| 833 | format("var x =\n `/*a`;\n" |
| 834 | "var y;")); |
Daniel Jasper | 2ebb0c5 | 2015-06-14 07:16:57 +0000 | [diff] [blame] | 835 | // Unterminated string literals in a template string. |
| 836 | verifyFormat("var x = `'`; // comment with matching quote '\n" |
| 837 | "var y;"); |
| 838 | verifyFormat("var x = `\"`; // comment with matching quote \"\n" |
| 839 | "var y;"); |
Daniel Jasper | 0d6ac27 | 2015-04-16 08:20:51 +0000 | [diff] [blame] | 840 | // Backticks in a comment - not a template string. |
| 841 | EXPECT_EQ("var x = 1 // `/*a`;\n" |
| 842 | " ;", |
| 843 | format("var x =\n 1 // `/*a`;\n" |
| 844 | " ;")); |
| 845 | EXPECT_EQ("/* ` */ var x = 1; /* ` */", |
| 846 | format("/* ` */ var x\n= 1; /* ` */")); |
| 847 | // Comment spans multiple template strings. |
| 848 | EXPECT_EQ("var x = `/*a`;\n" |
| 849 | "var y = ` */ `;", |
| 850 | format("var x =\n `/*a`;\n" |
| 851 | "var y =\n ` */ `;")); |
| 852 | // Escaped backtick. |
| 853 | EXPECT_EQ("var x = ` \\` a`;\n" |
| 854 | "var y;", |
| 855 | format("var x = ` \\` a`;\n" |
| 856 | "var y;")); |
Daniel Jasper | a0ef4f3 | 2015-02-20 13:47:38 +0000 | [diff] [blame] | 857 | } |
| 858 | |
Daniel Jasper | bc46b93 | 2015-03-15 13:59:51 +0000 | [diff] [blame] | 859 | TEST_F(FormatTestJS, CastSyntax) { |
| 860 | verifyFormat("var x = <type>foo;"); |
| 861 | } |
| 862 | |
| 863 | TEST_F(FormatTestJS, TypeArguments) { |
| 864 | verifyFormat("class X<Y> {}"); |
| 865 | verifyFormat("new X<Y>();"); |
| 866 | verifyFormat("foo<Y>(a);"); |
| 867 | verifyFormat("var x: X<Y>[];"); |
| 868 | verifyFormat("class C extends D<E> implements F<G>, H<I> {}"); |
Daniel Jasper | 20580fd | 2015-06-11 13:31:45 +0000 | [diff] [blame] | 869 | verifyFormat("function f(a: List<any> = null) {}"); |
| 870 | verifyFormat("function f(): List<any> {}"); |
Daniel Jasper | bc46b93 | 2015-03-15 13:59:51 +0000 | [diff] [blame] | 871 | } |
| 872 | |
Daniel Jasper | a74f507 | 2015-04-13 15:01:40 +0000 | [diff] [blame] | 873 | TEST_F(FormatTestJS, OptionalTypes) { |
Daniel Jasper | 20580fd | 2015-06-11 13:31:45 +0000 | [diff] [blame] | 874 | verifyFormat("function x(a?: b, c?, d?) {}"); |
Daniel Jasper | a74f507 | 2015-04-13 15:01:40 +0000 | [diff] [blame] | 875 | verifyFormat("class X {\n" |
| 876 | " y?: z;\n" |
| 877 | " z?;\n" |
| 878 | "}"); |
Daniel Jasper | 9326f91 | 2015-05-05 08:40:32 +0000 | [diff] [blame] | 879 | verifyFormat("interface X {\n" |
| 880 | " y?(): z;\n" |
| 881 | "}"); |
| 882 | verifyFormat("x ? 1 : 2;"); |
Daniel Jasper | 3e0dcc2 | 2015-05-27 05:37:40 +0000 | [diff] [blame] | 883 | verifyFormat("constructor({aa}: {\n" |
| 884 | " aa?: string,\n" |
| 885 | " aaaaaaaa?: string,\n" |
| 886 | " aaaaaaaaaaaaaaa?: boolean,\n" |
| 887 | " aaaaaa?: List<string>\n" |
Daniel Jasper | 20580fd | 2015-06-11 13:31:45 +0000 | [diff] [blame] | 888 | "}) {}"); |
Daniel Jasper | a74f507 | 2015-04-13 15:01:40 +0000 | [diff] [blame] | 889 | } |
| 890 | |
Daniel Jasper | d930977 | 2015-04-13 15:03:30 +0000 | [diff] [blame] | 891 | TEST_F(FormatTestJS, IndexSignature) { |
| 892 | verifyFormat("var x: {[k: string]: v};"); |
| 893 | } |
| 894 | |
Alexander Kornienko | a48a12c | 2013-12-03 10:50:16 +0000 | [diff] [blame] | 895 | } // end namespace tooling |
| 896 | } // end namespace clang |