[flow] Do not call this#optional for invalid void_test

Summary:
For invalid `void_test` (e.g. `foo === undefined` where undefined is defined locally), we shouldn't call optional_chaining because it will add refinement to `obj` in `obj.foo === undefined`, while in this case there shouldn't be any refinements.

Changelog: [internal]

Reviewed By: jbrown215

Differential Revision: D66019564

fbshipit-source-id: 67ea8f172e1f4de49ec340c2e1954dad9fad21bb

src/analysis/env_builder/name_resolver.ml

@@ -4951,7 +4951,7 @@ module Make (Context : C) (FlowAPIUtils : F with type cx = Context.t) :
           this#commit_refinement refis;
           if will_negate then this#negate_new_refinements ()
         end else begin
-          ignore @@ this#optional_chain expr;
+          ignore @@ this#expression expr;
           this#commit_refinement refis
         end