added test for self.x += <n>, self.y += <n>.

Author: takaokouji

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

@@ -987,21 +987,22 @@ class RubyToBlocksConverter {
             if (this._isBlock(lh)) {
                 switch (lh.opcode) {
                 case 'motion_xposition':
-                case 'motion_yposition': {
-                    delete this._context.blocks[lh.id];
+                case 'motion_yposition':
+                    if (this._isNumberOrBlock(rh)) {
+                        delete this._context.blocks[lh.id];
 
-                    let xy;
-                    if (lh.opcode === 'motion_xposition') {
-                        xy = 'x';
-                    } else {
-                        xy = 'y';
-                    }
+                        let xy;
+                        if (lh.opcode === 'motion_xposition') {
+                            xy = 'x';
+                        } else {
+                            xy = 'y';
+                        }
 
-                    block = this._createBlock(`motion_change${xy}by`, 'statement');
-                    this._addInput(block, `D${_.toUpper(xy)}`, this._createNumberBlock('math_number', rh, block.id));
+                        block = this._createBlock(`motion_change${xy}by`, 'statement');
+                        this._addInput(block, `D${_.toUpper(xy)}`, this._createNumberBlock('math_number', rh, block.id));
+                    }
                     break;
                 }
-                }
             } else if (_.isString(lh)) {
                 const variable = this._findOrCreateVariable(lh);
                 if (variable.scope !== 'local') {

test/helpers/expect-to-equal-blocks.js

@@ -152,6 +152,13 @@ const expectToEqualRubyStatement = function (converter, expectedStatement) {
     expectToEqualBlocks(converter, expected);
 };
 
+const convertAndExpectToEqualRubyStatement = function (converter, target, code, expectedStatement) {
+    const res = converter.targetCodeToBlocks(target, code);
+    expect(converter.errors).toHaveLength(0);
+    expectToEqualRubyStatement(converter, expectedStatement);
+    expect(res).toBeTruthy();
+};
+
 const fieldsToExpected = function (fields) {
     if (!fields) {
         return null;
@@ -261,13 +268,24 @@ const expectedInfo = {
             }
         ],
         shadow: true
+    }),
+    makeNumber: (num, opcode = 'math_number') => ({
+        opcode: opcode,
+        fields: [
+            {
+                name: 'NUM',
+                value: num
+            }
+        ],
+        shadow: true
     })
 };
 
 export {
     expectToEqualBlocks,
     convertAndExpectToEqualBlocks,
     expectToEqualRubyStatement,
+    convertAndExpectToEqualRubyStatement,
     rubyToExpected,
     expectedInfo
 };