Skip to content
This repository has been archived by the owner on Mar 23, 2024. It is now read-only.

Commit

Permalink
requireSpacesInsideObjectBrackets: remove skipped test
Browse files Browse the repository at this point in the history
This is now covered by `requireSpacesInsideImportedObjectBraces`.

Ref f8fe573
Fixes #2151
  • Loading branch information
hzoo committed Mar 16, 2016
1 parent f0e8a21 commit 97dd2c4
Showing 1 changed file with 0 additions and 23 deletions.
23 changes: 0 additions & 23 deletions test/specs/rules/require-spaces-inside-object-brackets.js
Expand Up @@ -70,29 +70,6 @@ describe('rules/require-spaces-inside-object-brackets', function() {
it('should report missing spaces for destructive assignment', function() {
expect(checker.checkString('let {x} = { x: 1 };')).to.have.error.count.equal(2);
});

describe.skip('import (#1524)', function() {
beforeEach(function() {
checker.configure({ requireSpacesInsideObjectBrackets: 'all' });
});

it('should not report with absent brackets', function() {
expect(checker.checkString('import myModule from "test";')).to.have.no.errors();
expect(checker.checkString('import "test";')).to.have.no.errors();
});

it('should report for import statements', function() {
expect(checker.checkString('import {myMember} from "test";')).to.have.error.count.equal(2);

expect(checker.checkString('import {foo, bar} from "test";')).to.have.error.count.equal(2);

expect(checker.checkString(
'import MyModule, {foo, bar} from "test";'
)).to.have.error.count.equal(2);

expect(checker.checkString('import {a as b} from "test";')).to.have.error.count.equal(2);
});
});
});

describe('"allButNested"', function() {
Expand Down

0 comments on commit 97dd2c4

Please sign in to comment.