Refactor handling of Nodes.ForwardingSuperNode

Author: andrykonchin

src/main/java/org/truffleruby/parser/TranslatorEnvironment.java

@@ -349,6 +349,7 @@ public boolean getNeverAssignInParentScope() {
         return !isBlock();
     }
 
+    /** A way to check current scope is a module/class. If in a block, isModuleBody is always false. */
     public boolean isModuleBody() {
         return isModuleBody;
     }

src/main/java/org/truffleruby/parser/YARPTranslator.java

@@ -1531,15 +1531,13 @@ public RubyNode visitForwardingSuperNode(Nodes.ForwardingSuperNode node) {
         boolean insideDefineMethod = false;
         var environment = this.environment;
         while (environment.isBlock()) {
-            if (environment.isModuleBody()) { // TODO: when could we be in a block and in a module body at the same time?
-                return assignPositionAndFlags(node, new ZSuperOutsideMethodNode(insideDefineMethod));
-            } else if (Objects.equals(environment.literalBlockPassedToMethod, "define_method")) {
+            if (Objects.equals(environment.literalBlockPassedToMethod, "define_method")) {
                 insideDefineMethod = true;
             }
             environment = environment.getParent();
         }
 
-        if (environment.isModuleBody() || environment.isTopLevelScope()) {
+        if (environment.isModuleBody()) {
             return assignPositionAndFlags(node, new ZSuperOutsideMethodNode(insideDefineMethod));
         }