fix(pacmak): use rubyName for method and property generation to escape reserved words

Author: omarqureshi

packages/jsii-pacmak/lib/targets/ruby.ts

@@ -291,7 +291,7 @@ class RubyGenerator extends Generator {
         } else {
           // Behavioral interfaces in JSII can have properties and methods
           for (const prop of typeSpec.allProperties) {
-            const propRubyName = toSnakeCase(prop.name);
+            const propRubyName = this.rubyName(prop.name);
             this.code.line(`def ${propRubyName}()`);
             this.code.line(`  jsii_get_property("${prop.name}")`);
             this.code.line(`end`);
@@ -329,7 +329,7 @@ class RubyGenerator extends Generator {
                 return rubyParam;
               })
               .join(', ');
-            this.code.line(`def ${toSnakeCase(method.name)}(${sigParams})`);
+            this.code.line(`def ${this.rubyName(method.name)}(${sigParams})`);
             this.code.open('  ');
             for (const p of method.parameters) {
               const rubyParam = this.rubyName(p.name);
@@ -371,12 +371,12 @@ class RubyGenerator extends Generator {
           for (const prop of typeSpec.allProperties) {
             const isOptional = prop.optional ? 'true' : 'false';
             this.code.line(
-              `:${toSnakeCase(prop.name)} => { kind: :property, name: "${prop.name}", is_optional: ${isOptional} },`,
+              `:${this.rubyName(prop.name)} => { kind: :property, name: "${prop.name}", is_optional: ${isOptional} },`,
             );
           }
           for (const method of typeSpec.allMethods) {
             this.code.line(
-              `:${toSnakeCase(method.name)} => { kind: :method, name: "${method.name}", is_optional: false },`,
+              `:${this.rubyName(method.name)} => { kind: :method, name: "${method.name}", is_optional: false },`,
             );
           }
           this.code.close('}');
@@ -495,7 +495,7 @@ class RubyGenerator extends Generator {
         this.code.open('  ');
         const emittedOverridables = new Set<string>();
         for (const prop of overridableProps) {
-          const rubyName = toSnakeCase(prop.name);
+          const rubyName = this.rubyName(prop.name);
           if (emittedOverridables.has(rubyName)) continue;
           emittedOverridables.add(rubyName);
           const isOptional = prop.optional ? 'true' : 'false';
@@ -504,7 +504,7 @@ class RubyGenerator extends Generator {
           );
         }
         for (const method of overridableMethods) {
-          const rubyName = toSnakeCase(method.name);
+          const rubyName = this.rubyName(method.name);
           if (emittedOverridables.has(rubyName)) continue;
           emittedOverridables.add(rubyName);
           this.code.line(
@@ -535,7 +535,9 @@ class RubyGenerator extends Generator {
             })
             .join(', ');
 
-          this.code.line(`def self.${toSnakeCase(method.name)}(${sigParams})`);
+          this.code.line(
+            `def self.${this.rubyName(method.name)}(${sigParams})`,
+          );
           this.code.open('  ');
           for (const p of method.parameters) {
             const rubyParam = this.rubyName(p.name);
@@ -572,7 +574,7 @@ class RubyGenerator extends Generator {
 
         // 4. Properties
         for (const prop of typeSpec.allProperties) {
-          const rubyName = toSnakeCase(prop.name);
+          const rubyName = this.rubyName(prop.name);
 
           if (prop.static) {
             // Static Getter
@@ -649,7 +651,7 @@ class RubyGenerator extends Generator {
             })
             .join(', ');
 
-          this.code.line(`def ${toSnakeCase(method.name)}(${sigParams})`);
+          this.code.line(`def ${this.rubyName(method.name)}(${sigParams})`);
           this.code.open('  ');
           for (const p of method.parameters) {
             const rubyParam = this.rubyName(p.name);