fix(ion-button): Add a shape="round" attribute rule to ion-button (#101)

Author: anagstef

src/ionButtonAttributesRenamedRule.ts

@@ -17,7 +17,8 @@ const replacementMap = new Map([
   ['outline', 'fill="outline"'],
   ['solid', 'fill="solid"'],
   ['full', 'expand="full"'],
-  ['block', 'expand="block"']
+  ['block', 'expand="block"'],
+  ['round', 'shape="round"']
 ]);
 
 const IonButtonAttributesAreRenamedTemplateVisitor = createAttributesRenamedTemplateVisitorClass(['ion-button'], replacementMap);

test/ionButtonAttributesRenamed.spec.ts

@@ -203,6 +203,23 @@ describe(ruleName, () => {
         source
       });
     });
+
+    it('should fail when round is used', () => {
+      let source = `
+      @Component({
+        template: \`
+          <ion-button round></ion-button>\`
+                      ~~~~~
+      })
+      class Bar{}
+          `;
+
+      assertAnnotated({
+        ruleName,
+        message: 'The round attribute of ion-button has been renamed. Use shape="round" instead.',
+        source
+      });
+    });
   });
 
   describe('replacements', () => {
@@ -601,5 +618,41 @@ describe(ruleName, () => {
 
       expect(res).to.eq(expected);
     });
+
+    it('should replace round with shape="round"', () => {
+      let source = `
+        @Component({
+          template: \`<ion-button round></ion-button>
+          \`
+        })
+        class Bar {}
+      `;
+
+      const fail = {
+        message: 'The round attribute of ion-button has been renamed. Use shape="round" instead.',
+        startPosition: {
+          line: 2,
+          character: 33
+        },
+        endPosition: {
+          line: 2,
+          character: 38
+        }
+      };
+
+      const failures = assertFailure(ruleName, source, fail);
+      const fixes = failures.map(f => f.getFix());
+      const res = Replacement.applyAll(source, Utils.flatMap(fixes, Utils.arrayify));
+
+      let expected = `
+        @Component({
+          template: \`<ion-button shape="round"></ion-button>
+          \`
+        })
+        class Bar {}
+      `;
+
+      expect(res).to.eq(expected);
+    });
   });
 });