Skip to content

Commit

Permalink
feat(rules): add ion-menu-toggle-is-now-an-element rule
Browse files Browse the repository at this point in the history
  • Loading branch information
imhoffd committed Jul 6, 2018
1 parent 2a480dd commit 591b10a
Show file tree
Hide file tree
Showing 3 changed files with 88 additions and 2 deletions.
6 changes: 4 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -365,11 +365,13 @@ We are looking for contributors to help build these rules out! See [`CONTRIBUTIN
<a href="https://github.com/ionic-team/ionic/blob/master/angular/BREAKING.md#menu-toggle">Menu Toggle</a>
</th>
<td></td>
<td>:white_large_square:</td>
<td>:white_check_mark:</td>
<td>
<code>ion-menu-toggle-is-now-an-element</code>
</td>
<td></td>
<td>
<a href="https://github.com/dwieeb">@dwieeb</a>
</td>
</tr>
<tr>
<th>
Expand Down
6 changes: 6 additions & 0 deletions src/ionMenuToggleIsNowAnElementRule.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
import { createDirectiveToElementRuleClass } from './helpers/directiveToElement';

const directive = 'menuToggle';

export const ruleName = 'ion-menu-toggle-is-now-an-element';
export const Rule = createDirectiveToElementRuleClass(ruleName, directive, 'ion-menu-toggle');
78 changes: 78 additions & 0 deletions test/ionMenuToggleIsNowAnElement.spec.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,78 @@
import { ruleName } from '../src/ionMenuToggleIsNowAnElementRule';
import { assertAnnotated, assertSuccess } from './testHelper';

describe(ruleName, () => {
describe('success', () => {
it('should work with proper style', () => {
let source = `
@Component({
template: \`
<ion-menu-toggle>
<ion-button>
Toggle Menu
</ion-button>
</ion-menu-toggle>
\`
})
class Bar{}
`;
assertSuccess(ruleName, source);
});
});

describe('failure', () => {
it('should fail when menuToggle attribute is used on button', () => {
let source = `
@Component({
template: \`
<button ion-button menuToggle>Toggle Menu</button>\`
~~~~~~~~~~
})
class Bar{}
`;

assertAnnotated({
ruleName,
message: 'menuToggle is now an ion-menu-toggle element instead of an Angular directive.',
source
});
});

it('should fail when menuToggle attribute is used on anchor', () => {
let source = `
@Component({
template: \`
<a ion-button menuToggle (click)="doSomething()">Toggle Menu</a>\`
~~~~~~~~~~
})
class Bar{}
`;

assertAnnotated({
ruleName,
message: 'menuToggle is now an ion-menu-toggle element instead of an Angular directive.',
source
});
});

it('should fail when ion-button attribute is used with multiline', () => {
let source = `
@Component({
template: \`
<a
ion-button
menuToggle
~~~~~~~~~~
(click)="doSomething()">Toggle Menu</a>\`
})
class Bar{}
`;

assertAnnotated({
ruleName,
message: 'menuToggle is now an ion-menu-toggle element instead of an Angular directive.',
source
});
});
});
});

0 comments on commit 591b10a

Please sign in to comment.