All patches and comments are welcome. Please squash your changes to logical
commits before using git-format-patch and git-send-email to
patches@git.madduck.net.
If you'd read over the Git project's submission guidelines and adhered to them,
I'd be especially grateful.
1 x = "This is a really long string that can't possibly be expected to fit all together on one line. In fact it may even take up three or more lines... like four or five... but probably just three."
3 x += "This is a really long string that can't possibly be expected to fit all together on one line. In fact it may even take up three or more lines... like four or five... but probably just three."
9 print('This is a really long string inside of a print statement with extra arguments attached at the end of it.', x, y, z)
11 print("This is a really long string inside of a print statement with no extra arguments attached at the end of it.")
13 D1 = {"The First": "This is a really long string that can't possibly be expected to fit all together on one line. Also it is inside a dictionary, so formatting is more difficult.", "The Second": "This is another really really (not really) long string that also can't be expected to fit on one line and is, like the other string, inside a dictionary."}
15 D2 = {1.0: "This is a really long string that can't possibly be expected to fit all together on one line. Also it is inside a dictionary, so formatting is more difficult.", 2.0: "This is another really really (not really) long string that also can't be expected to fit on one line and is, like the other string, inside a dictionary."}
17 D3 = {x: "This is a really long string that can't possibly be expected to fit all together on one line. Also it is inside a dictionary, so formatting is more difficult.", y: "This is another really really (not really) long string that also can't be expected to fit on one line and is, like the other string, inside a dictionary."}
19 D4 = {"A long and ridiculous {}".format(string_key): "This is a really really really long string that has to go i,side of a dictionary. It is soooo bad.", some_func("calling", "some", "stuff"): "This is a really really really long string that has to go inside of a dictionary. It is {soooo} bad (#{x}).".format(sooo="soooo", x=2), "A %s %s" % ("formatted", "string"): "This is a really really really long string that has to go inside of a dictionary. It is %s bad (#%d)." % ("soooo", 2)}
21 D5 = { # Test for https://github.com/psf/black/issues/3261
22 ("This is a really long string that can't be expected to fit in one line and is used as a nested dict's key"): {"inner": "value"},
25 D6 = { # Test for https://github.com/psf/black/issues/3261
26 ("This is a really long string that can't be expected to fit in one line and is used as a dict's key"): ["value1", "value2"],
29 L1 = ["The is a short string", "This is a really long string that can't possibly be expected to fit all together on one line. Also it is inside a list literal, so it's expected to be wrapped in parens when spliting to avoid implicit str concatenation.", short_call("arg", {"key": "value"}), "This is another really really (not really) long string that also can't be expected to fit on one line and is, like the other string, inside a list literal.", ("parens should be stripped for short string in list")]
31 L2 = ["This is a really long string that can't be expected to fit in one line and is the only child of a list literal."]
33 S1 = {"The is a short string", "This is a really long string that can't possibly be expected to fit all together on one line. Also it is inside a set literal, so it's expected to be wrapped in parens when spliting to avoid implicit str concatenation.", short_call("arg", {"key": "value"}), "This is another really really (not really) long string that also can't be expected to fit on one line and is, like the other string, inside a set literal.", ("parens should be stripped for short string in set")}
35 S2 = {"This is a really long string that can't be expected to fit in one line and is the only child of a set literal."}
37 T1 = ("The is a short string", "This is a really long string that can't possibly be expected to fit all together on one line. Also it is inside a tuple literal, so it's expected to be wrapped in parens when spliting to avoid implicit str concatenation.", short_call("arg", {"key": "value"}), "This is another really really (not really) long string that also can't be expected to fit on one line and is, like the other string, inside a tuple literal.", ("parens should be stripped for short string in list"))
39 T2 = ("This is a really long string that can't be expected to fit in one line and is the only child of a tuple literal.",)
41 func_with_keywords(my_arg, my_kwarg="Long keyword strings also need to be wrapped, but they will probably need to be handled a little bit differently.")
44 'But what should happen when code has already been formatted but in the wrong way? Like'
45 " with a space at the end instead of the beginning. Or what about when it is split too soon?"
48 bad_split2 = "But what should happen when code has already " \
49 "been formatted but in the wrong way? Like " \
50 "with a space at the end instead of the " \
51 "beginning. Or what about when it is split too " \
52 "soon? In the case of a split that is too " \
53 "short, black will try to honer the custom " \
57 "What if we have inline comments on " # First Comment
58 "each line of a bad split? In that " # Second Comment
59 "case, we should just leave it alone." # Third Comment
63 "But what should happen when code has already "
64 "been formatted but in the wrong way? Like "
65 "with a space at the end instead of the "
66 "beginning. Or what about when it is split too "
67 "soon? In the case of a split that is too "
68 "short, black will try to honer the custom "
75 long_string_kwarg="But what should happen when code has already been formatted but in the wrong way? Like "
76 "with a space at the end instead of the beginning. Or what about when it is split too "
82 "But what should happen when code has already "
83 r"been formatted but in the wrong way? Like "
84 "with a space at the end instead of the "
85 r"beginning. Or what about when it is split too "
86 r"soon? In the case of a split that is too "
87 "short, black will try to honer the custom "
95 inline_comments_func1(
96 "if there are inline "
97 "comments in the middle "
98 # Here is the standard alone comment.
99 "of the implicitly concatenated "
100 "string, we should handle "
105 inline_comments_func2(
106 "what if the string is very very very very very very very very very very long and this part does "
107 "not fit into a single line? "
108 # Here is the standard alone comment.
109 "then the string should still be properly handled by merging and splitting "
110 "it into parts that fit in line length.",
114 raw_string = r"This is a long raw string. When re-formatting this string, black needs to make sure it prepends the 'r' onto the new string."
116 fmt_string1 = "We also need to be sure to preserve any and all {} which may or may not be attached to the string in question.".format("method calls")
118 fmt_string2 = "But what about when the string is {} but {}".format("short", "the method call is really really really really really really really really long?")
120 old_fmt_string1 = "While we are on the topic of %s, we should also note that old-style formatting must also be preserved, since some %s still uses it." % ("formatting", "code")
122 old_fmt_string2 = "This is a %s %s %s %s" % ("really really really really really", "old", "way to format strings!", "Use f-strings instead!")
124 old_fmt_string3 = "Whereas only the strings after the percent sign were long in the last example, this example uses a long initial string as well. This is another %s %s %s %s" % ("really really really really really", "old", "way to format strings!", "Use f-strings instead!")
126 fstring = f"f-strings definitely make things more {difficult} than they need to be for {{black}}. But boy they sure are handy. The problem is that some lines will need to have the 'f' whereas others do not. This {line}, for example, needs one."
128 fstring_with_no_fexprs = f"Some regular string that needs to get split certainly but is NOT an fstring by any means whatsoever."
130 comment_string = "Long lines with inline comments should have their comments appended to the reformatted string's enclosing right parentheses." # This comment gets thrown to the top.
132 arg_comment_string = print("Long lines with inline comments which are apart of (and not the only member of) an argument list should have their comments appended to the reformatted string's enclosing left parentheses.", # This comment gets thrown to the top.
133 "Arg #2", "Arg #3", "Arg #4", "Arg #5")
135 pragma_comment_string1 = "Lines which end with an inline pragma comment of the form `# <pragma>: <...>` should be left alone." # noqa: E501
137 pragma_comment_string2 = "Lines which end with an inline pragma comment of the form `# <pragma>: <...>` should be left alone." # noqa
139 """This is a really really really long triple quote string and it should not be touched."""
141 triple_quote_string = """This is a really really really long triple quote string assignment and it should not be touched."""
143 assert some_type_of_boolean_expression, "Followed by a really really really long string that is used to provide context to the AssertionError exception."
145 assert some_type_of_boolean_expression, "Followed by a really really really long string that is used to provide context to the AssertionError exception, which uses dynamic string {}.".format("formatting")
147 assert some_type_of_boolean_expression, "Followed by a really really really long string that is used to provide context to the AssertionError exception, which uses dynamic string %s." % "formatting"
149 assert some_type_of_boolean_expression, "Followed by a really really really long string that is used to provide context to the AssertionError exception, which uses dynamic %s %s." % ("string", "formatting")
151 some_function_call("With a reallly generic name and with a really really long string that is, at some point down the line, " + added + " to a variable and then added to another string.")
153 some_function_call("With a reallly generic name and with a really really long string that is, at some point down the line, " + added + " to a variable and then added to another string. But then what happens when the final string is also supppppperrrrr long?! Well then that second (realllllllly long) string should be split too.", "and a second argument", and_a_third)
155 return "A really really really really really really really really really really really really really long {} {}".format("return", "value")
158 "This is a really long string argument to a function that has a trailing comma which should NOT be there.",
162 "This is a really long string argument to a function that has a trailing comma which should NOT be there.", # comment after comma
167 "This is a really long string argument to a function that has a trailing comma"
168 " which should NOT be there."
174 "This is a really long string argument to a function that has a trailing comma"
175 " which should NOT be there."
176 ), # comment after comma
179 func_with_bad_parens_that_wont_fit_in_one_line(
180 ("short string that should have parens stripped"),
186 func_with_bad_parens_that_wont_fit_in_one_line(
189 ("short string that should have parens stripped"),
193 func_with_bad_parens(
194 ("short string that should have parens stripped"),
200 func_with_bad_parens(
203 ("short string that should have parens stripped"),
207 annotated_variable: Final = "This is a large " + STRING + " that has been " + CONCATENATED + "using the '+' operator."
208 annotated_variable: Final = "This is a large string that has a type annotation attached to it. A type annotation should NOT stop a long string from being wrapped."
209 annotated_variable: Literal["fakse_literal"] = "This is a large string that has a type annotation attached to it. A type annotation should NOT stop a long string from being wrapped."
211 backslashes = "This is a really long string with \"embedded\" double quotes and 'single' quotes that also handles checking for an even number of backslashes \\"
212 backslashes = "This is a really long string with \"embedded\" double quotes and 'single' quotes that also handles checking for an even number of backslashes \\\\"
213 backslashes = "This is a really 'long' string with \"embedded double quotes\" and 'single' quotes that also handles checking for an odd number of backslashes \\\", like this...\\\\\\"
227 raw_strings = r"Don't" " get" r" merged" " unless they are all raw."
230 yield "This is a really long string that can't possibly be expected to fit all together on one line. In fact it may even take up three or more lines... like four or five... but probably just three."
232 x = f"This is a {{really}} long string that needs to be split without a doubt (i.e. most definitely). In short, this {string} that can't possibly be {{expected}} to fit all together on one line. In {fact} it may even take up three or more lines... like four or five... but probably just four."
234 long_unmergable_string_with_pragma = (
235 "This is a really long string that can't be merged because it has a likely pragma at the end" # type: ignore
239 long_unmergable_string_with_pragma = (
240 "This is a really long string that can't be merged because it has a likely pragma at the end" # noqa
244 long_unmergable_string_with_pragma = (
245 "This is a really long string that can't be merged because it has a likely pragma at the end" # pylint: disable=some-pylint-check
249 string_with_nameescape = (
250 "........................................................................ \N{LAO KO LA}"
253 string_with_nameescape = (
254 "........................................................................... \N{LAO KO LA}"
257 string_with_nameescape = (
258 "............................................................................ \N{LAO KO LA}"
261 string_with_nameescape_and_escaped_backslash = (
262 "...................................................................... \\\N{LAO KO LA}"
265 string_with_nameescape_and_escaped_backslash = (
266 "......................................................................... \\\N{LAO KO LA}"
269 string_with_nameescape_and_escaped_backslash = (
270 ".......................................................................... \\\N{LAO KO LA}"
273 string_with_escaped_nameescape = (
274 "........................................................................ \\N{LAO KO LA}"
277 string_with_escaped_nameescape = (
278 "........................................................................... \\N{LAO KO LA}"
281 msg = lambda x: f"this is a very very very long lambda value {x} that doesn't fit on a single line"
283 dict_with_lambda_values = {
285 f"{j.__class__.__name__}({some_function_call(j.left)}, "
286 f"{some_function_call(j.right)})"
290 # Complex string concatenations with a method call in the middle.
295 " (%r, self.%s, visitor.%s)"
296 % (attrname, attrname, visit_name)
297 for attrname, visit_name in names
304 # Test case of an outer string' parens enclose an inner string's parens.
305 call(body=("%s %s" % ((",".join(items)), suffix)))
312 "This is a really long string that can't possibly be expected to fit all together"
313 " on one line. In fact it may even take up three or more lines... like four or"
314 " five... but probably just three."
318 "This is a really long string that can't possibly be expected to fit all together"
319 " on one line. In fact it may even take up three or more lines... like four or"
320 " five... but probably just three."
327 "This is a really long string inside of a print statement with extra arguments"
328 " attached at the end of it."
336 "This is a really long string inside of a print statement with no extra arguments"
337 " attached at the end of it."
342 "This is a really long string that can't possibly be expected to fit all"
343 " together on one line. Also it is inside a dictionary, so formatting is more"
347 "This is another really really (not really) long string that also can't be"
348 " expected to fit on one line and is, like the other string, inside a"
355 "This is a really long string that can't possibly be expected to fit all"
356 " together on one line. Also it is inside a dictionary, so formatting is more"
360 "This is another really really (not really) long string that also can't be"
361 " expected to fit on one line and is, like the other string, inside a"
368 "This is a really long string that can't possibly be expected to fit all"
369 " together on one line. Also it is inside a dictionary, so formatting is more"
373 "This is another really really (not really) long string that also can't be"
374 " expected to fit on one line and is, like the other string, inside a"
380 "A long and ridiculous {}".format(string_key): (
381 "This is a really really really long string that has to go i,side of a"
382 " dictionary. It is soooo bad."
384 some_func("calling", "some", "stuff"): (
385 "This is a really really really long string that has to go inside of a"
386 " dictionary. It is {soooo} bad (#{x}).".format(sooo="soooo", x=2)
389 % ("formatted", "string"): (
390 "This is a really really really long string that has to go inside of a"
391 " dictionary. It is %s bad (#%d)." % ("soooo", 2)
395 D5 = { # Test for https://github.com/psf/black/issues/3261
396 "This is a really long string that can't be expected to fit in one line and is used as a nested dict's key": {
401 D6 = { # Test for https://github.com/psf/black/issues/3261
402 "This is a really long string that can't be expected to fit in one line and is used as a dict's key": [
409 "The is a short string",
411 "This is a really long string that can't possibly be expected to fit all"
412 " together on one line. Also it is inside a list literal, so it's expected to"
413 " be wrapped in parens when spliting to avoid implicit str concatenation."
415 short_call("arg", {"key": "value"}),
417 "This is another really really (not really) long string that also can't be"
418 " expected to fit on one line and is, like the other string, inside a list"
421 "parens should be stripped for short string in list",
425 "This is a really long string that can't be expected to fit in one line and is the"
426 " only child of a list literal."
430 "The is a short string",
432 "This is a really long string that can't possibly be expected to fit all"
433 " together on one line. Also it is inside a set literal, so it's expected to be"
434 " wrapped in parens when spliting to avoid implicit str concatenation."
436 short_call("arg", {"key": "value"}),
438 "This is another really really (not really) long string that also can't be"
439 " expected to fit on one line and is, like the other string, inside a set"
442 "parens should be stripped for short string in set",
446 "This is a really long string that can't be expected to fit in one line and is the"
447 " only child of a set literal."
451 "The is a short string",
453 "This is a really long string that can't possibly be expected to fit all"
454 " together on one line. Also it is inside a tuple literal, so it's expected to"
455 " be wrapped in parens when spliting to avoid implicit str concatenation."
457 short_call("arg", {"key": "value"}),
459 "This is another really really (not really) long string that also can't be"
460 " expected to fit on one line and is, like the other string, inside a tuple"
463 "parens should be stripped for short string in list",
468 "This is a really long string that can't be expected to fit in one line and is"
469 " the only child of a tuple literal."
476 "Long keyword strings also need to be wrapped, but they will probably need to"
477 " be handled a little bit differently."
482 "But what should happen when code has already been formatted but in the wrong way?"
483 " Like with a space at the end instead of the beginning. Or what about when it is"
488 "But what should happen when code has already "
489 "been formatted but in the wrong way? Like "
490 "with a space at the end instead of the "
491 "beginning. Or what about when it is split too "
492 "soon? In the case of a split that is too "
493 "short, black will try to honer the custom "
498 "What if we have inline comments on " # First Comment
499 "each line of a bad split? In that " # Second Comment
500 "case, we should just leave it alone." # Third Comment
505 "But what should happen when code has already "
506 "been formatted but in the wrong way? Like "
507 "with a space at the end instead of the "
508 "beginning. Or what about when it is split too "
509 "soon? In the case of a split that is too "
510 "short, black will try to honer the custom "
523 "But what should happen when code has already been formatted but in the wrong"
524 " way? Like with a space at the end instead of the beginning. Or what about"
525 " when it is split too soon?"
531 "But what should happen when code has already "
532 r"been formatted but in the wrong way? Like "
533 "with a space at the end instead of the "
534 r"beginning. Or what about when it is split too "
535 r"soon? In the case of a split that is too "
536 "short, black will try to honer the custom "
544 inline_comments_func1(
545 "if there are inline comments in the middle "
546 # Here is the standard alone comment.
547 "of the implicitly concatenated string, we should handle them correctly",
551 inline_comments_func2(
552 "what if the string is very very very very very very very very very very long and"
553 " this part does not fit into a single line? "
554 # Here is the standard alone comment.
555 "then the string should still be properly handled by merging and splitting "
556 "it into parts that fit in line length.",
561 r"This is a long raw string. When re-formatting this string, black needs to make"
562 r" sure it prepends the 'r' onto the new string."
566 "We also need to be sure to preserve any and all {} which may or may not be"
567 " attached to the string in question.".format("method calls")
570 fmt_string2 = "But what about when the string is {} but {}".format(
572 "the method call is really really really really really really really really long?",
576 "While we are on the topic of %s, we should also note that old-style formatting"
577 " must also be preserved, since some %s still uses it." % ("formatting", "code")
580 old_fmt_string2 = "This is a %s %s %s %s" % (
581 "really really really really really",
583 "way to format strings!",
584 "Use f-strings instead!",
588 "Whereas only the strings after the percent sign were long in the last example,"
589 " this example uses a long initial string as well. This is another %s %s %s %s"
591 "really really really really really",
593 "way to format strings!",
594 "Use f-strings instead!",
599 f"f-strings definitely make things more {difficult} than they need to be for"
600 " {black}. But boy they sure are handy. The problem is that some lines will need"
601 f" to have the 'f' whereas others do not. This {line}, for example, needs one."
604 fstring_with_no_fexprs = (
605 f"Some regular string that needs to get split certainly but is NOT an fstring by"
606 f" any means whatsoever."
609 comment_string = ( # This comment gets thrown to the top.
610 "Long lines with inline comments should have their comments appended to the"
611 " reformatted string's enclosing right parentheses."
614 arg_comment_string = print(
615 ( # This comment gets thrown to the top.
616 "Long lines with inline comments which are apart of (and not the only member"
617 " of) an argument list should have their comments appended to the reformatted"
618 " string's enclosing left parentheses."
626 pragma_comment_string1 = "Lines which end with an inline pragma comment of the form `# <pragma>: <...>` should be left alone." # noqa: E501
628 pragma_comment_string2 = "Lines which end with an inline pragma comment of the form `# <pragma>: <...>` should be left alone." # noqa
630 """This is a really really really long triple quote string and it should not be touched."""
632 triple_quote_string = """This is a really really really long triple quote string assignment and it should not be touched."""
634 assert some_type_of_boolean_expression, (
635 "Followed by a really really really long string that is used to provide context to"
636 " the AssertionError exception."
639 assert some_type_of_boolean_expression, (
640 "Followed by a really really really long string that is used to provide context to"
641 " the AssertionError exception, which uses dynamic string {}.".format("formatting")
644 assert some_type_of_boolean_expression, (
645 "Followed by a really really really long string that is used to provide context to"
646 " the AssertionError exception, which uses dynamic string %s." % "formatting"
649 assert some_type_of_boolean_expression, (
650 "Followed by a really really really long string that is used to provide context to"
651 " the AssertionError exception, which uses dynamic %s %s."
652 % ("string", "formatting")
656 "With a reallly generic name and with a really really long string that is, at some"
657 " point down the line, "
659 + " to a variable and then added to another string."
663 "With a reallly generic name and with a really really long string that is, at some"
664 " point down the line, "
666 + " to a variable and then added to another string. But then what happens when the"
667 " final string is also supppppperrrrr long?! Well then that second (realllllllly"
668 " long) string should be split too.",
669 "and a second argument",
674 "A really really really really really really really really really really really"
675 " really really long {} {}".format("return", "value")
680 "This is a really long string argument to a function that has a trailing comma"
681 " which should NOT be there."
686 ( # comment after comma
687 "This is a really long string argument to a function that has a trailing comma"
688 " which should NOT be there."
694 "This is a really long string argument to a function that has a trailing comma"
695 " which should NOT be there."
700 ( # comment after comma
701 "This is a really long string argument to a function that has a trailing comma"
702 " which should NOT be there."
706 func_with_bad_parens_that_wont_fit_in_one_line(
707 "short string that should have parens stripped", x, y, z
710 func_with_bad_parens_that_wont_fit_in_one_line(
711 x, y, "short string that should have parens stripped", z
714 func_with_bad_parens(
715 "short string that should have parens stripped",
721 func_with_bad_parens(
724 "short string that should have parens stripped",
728 annotated_variable: Final = (
733 + "using the '+' operator."
735 annotated_variable: Final = (
736 "This is a large string that has a type annotation attached to it. A type"
737 " annotation should NOT stop a long string from being wrapped."
739 annotated_variable: Literal["fakse_literal"] = (
740 "This is a large string that has a type annotation attached to it. A type"
741 " annotation should NOT stop a long string from being wrapped."
745 "This is a really long string with \"embedded\" double quotes and 'single' quotes"
746 " that also handles checking for an even number of backslashes \\"
749 "This is a really long string with \"embedded\" double quotes and 'single' quotes"
750 " that also handles checking for an even number of backslashes \\\\"
753 "This is a really 'long' string with \"embedded double quotes\" and 'single' quotes"
754 ' that also handles checking for an odd number of backslashes \\", like'
758 short_string = "Hi there."
760 func_call(short_string="Hi there.")
762 raw_strings = r"Don't" " get" r" merged" " unless they are all raw."
767 "This is a really long string that can't possibly be expected to fit all"
768 " together on one line. In fact it may even take up three or more lines... like"
769 " four or five... but probably just three."
774 "This is a {really} long string that needs to be split without a doubt (i.e."
775 f" most definitely). In short, this {string} that can't possibly be {{expected}} to"
776 f" fit all together on one line. In {fact} it may even take up three or more"
777 " lines... like four or five... but probably just four."
780 long_unmergable_string_with_pragma = (
781 "This is a really long string that can't be merged because it has a likely pragma at the end" # type: ignore
785 long_unmergable_string_with_pragma = (
786 "This is a really long string that can't be merged because it has a likely pragma at the end" # noqa
790 long_unmergable_string_with_pragma = (
791 "This is a really long string that can't be merged because it has a likely pragma at the end" # pylint: disable=some-pylint-check
795 string_with_nameescape = (
796 "........................................................................"
800 string_with_nameescape = (
801 "..........................................................................."
805 string_with_nameescape = (
806 "............................................................................"
810 string_with_nameescape_and_escaped_backslash = (
811 "......................................................................"
815 string_with_nameescape_and_escaped_backslash = (
816 "........................................................................."
820 string_with_nameescape_and_escaped_backslash = (
821 ".........................................................................."
825 string_with_escaped_nameescape = (
826 "........................................................................ \\N{LAO"
830 string_with_escaped_nameescape = (
831 "..........................................................................."
837 f"this is a very very very long lambda value {x} that doesn't fit on a single"
842 dict_with_lambda_values = {
844 f"{j.__class__.__name__}({some_function_call(j.left)}, "
845 f"{some_function_call(j.right)})"
849 # Complex string concatenations with a method call in the middle.
853 " (%r, self.%s, visitor.%s)" % (attrname, attrname, visit_name)
854 for attrname, visit_name in names
860 # Test case of an outer string' parens enclose an inner string's parens.
861 call(body="%s %s" % (",".join(items), suffix))