Fix #4150: Correctly outdent ternary followed by method call (#4535)

src/rewriter.coffee

@@ -396,7 +396,8 @@ exports.Rewriter = class Rewriter
  not (token[0] is 'TERMINATOR' and @tag(i + 1) in EXPRESSION_CLOSE) and
  not (token[0] is 'ELSE' and starter isnt 'THEN') and
  not (token[0] in ['CATCH', 'FINALLY'] and starter in ['->', '=>']) or
- token[0] in CALL_CLOSERS and @tokens[i - 1].newLine
+ token[0] in CALL_CLOSERS and
+ (@tokens[i - 1].newLine or @tokens[i - 1][0] is 'OUTDENT')
 
  action = (token, i) ->
  @tokens.splice (if @tag(i - 1) is ',' then i - 1 else i), 0, outdent

test/formatting.coffee

@@ -198,6 +198,29 @@ test "#1495, method call chaining", ->
  ).join ', '
  eq 'a, b, c', result
 
+test "chaining should not wrap spilling ternary", ->
+ throws -> CoffeeScript.compile """
+ if 0 then 1 else g
+ a: 42
+ .h()
+ """
+
+test "chaining should wrap calls containing spilling ternary", ->
+ f = (x) -> h: x
+ id = (x) -> x
+ result = f if true then 42 else id
+ a: 2
+ .h
+ eq 42, result
+
+test "chaining should work within spilling ternary", ->
+ f = (x) -> h: x
+ id = (x) -> x
+ result = f if false then 1 else id
+ a: 3
+ .a
+ eq 3, result.h
+
 # Nested blocks caused by paren unwrapping
 test "#1492: Nested blocks don't cause double semicolons", ->
  js = CoffeeScript.compile '(0;0)'