supported empty else. fixed and add test.

Author: takaokouji

src/lib/ruby-to-blocks-converter/index.js

@@ -1001,7 +1001,7 @@ class RubyToBlocksConverter {
         const cond = this._processCondition(node.children[0]);
         const statement = this._processStatement(node.children[1]);
         let elseStatement;
-        if (node.children[2] !== Opal.nil) {
+        if (node.children[2] !== Opal.nil || node.$loc().$else() !== Opal.nil) {
             elseStatement = this._processStatement(node.children[2]);
         }
 

test/unit/lib/ruby-to-blocks-converter/control.test.js

@@ -649,6 +649,122 @@ describe('RubyToBlocksConverter/Control', () => {
             convertAndExpectToEqualBlocks(converter, target, code, expected);
         });
 
+        test('unless', () => {
+            code = `
+                unless touching?("_edge_")
+                  bounce_if_on_edge
+                else
+                  move(10)
+                end
+            `;
+            expected = [
+                {
+                    opcode: 'control_if_else',
+                    inputs: [
+                        {
+                            name: 'CONDITION',
+                            block: rubyToExpected(converter, target, 'touching?("_edge_")')[0]
+                        }
+                    ],
+                    branches: [
+                        rubyToExpected(converter, target, 'move(10)')[0],
+                        rubyToExpected(converter, target, 'bounce_if_on_edge')[0]
+                    ]
+                }
+            ];
+            convertAndExpectToEqualBlocks(converter, target, code, expected);
+
+            code = `
+                unless touching?("_edge_")
+                  bounce_if_on_edge
+                  bounce_if_on_edge
+                else
+                  move(10)
+                  move(10)
+                end
+            `;
+            expected = [
+                {
+                    opcode: 'control_if_else',
+                    inputs: [
+                        {
+                            name: 'CONDITION',
+                            block: rubyToExpected(converter, target, 'touching?("_edge_")')[0]
+                        }
+                    ],
+                    branches: [
+                        rubyToExpected(converter, target, 'move(10); move(10)')[0],
+                        rubyToExpected(converter, target, 'bounce_if_on_edge; bounce_if_on_edge')[0]
+                    ]
+                }
+            ];
+            convertAndExpectToEqualBlocks(converter, target, code, expected);
+
+            code = `
+                unless touching?("_edge_")
+                else
+                  move(10)
+                end
+            `;
+            expected = [
+                {
+                    opcode: 'control_if_else',
+                    inputs: [
+                        {
+                            name: 'CONDITION',
+                            block: rubyToExpected(converter, target, 'touching?("_edge_")')[0]
+                        }
+                    ],
+                    branches: [
+                        rubyToExpected(converter, target, 'move(10)')[0]
+                    ]
+                }
+            ];
+            convertAndExpectToEqualBlocks(converter, target, code, expected);
+
+            code = `
+                unless touching?("_edge_")
+                  bounce_if_on_edge
+                else
+                end
+            `;
+            expected = [
+                {
+                    opcode: 'control_if_else',
+                    inputs: [
+                        {
+                            name: 'CONDITION',
+                            block: rubyToExpected(converter, target, 'touching?("_edge_")')[0]
+                        }
+                    ],
+                    branches: [
+                        null,
+                        rubyToExpected(converter, target, 'bounce_if_on_edge')[0]
+                    ]
+                }
+            ];
+            convertAndExpectToEqualBlocks(converter, target, code, expected);
+
+            code = `
+                unless touching?("_edge_")
+                else
+                end
+            `;
+            expected = [
+                {
+                    opcode: 'control_if_else',
+                    inputs: [
+                        {
+                            name: 'CONDITION',
+                            block: rubyToExpected(converter, target, 'touching?("_edge_")')[0]
+                        }
+                    ],
+                    branches: []
+                }
+            ];
+            convertAndExpectToEqualBlocks(converter, target, code, expected);
+        });
+
         test('error', () => {
             code = `
                 if move(10)