Skip to content

Commit

Permalink
[chore] make sure all tests meet line-length requirement.
Browse files Browse the repository at this point in the history
  • Loading branch information
kloots authored and diasbruno committed Oct 25, 2017
1 parent 18a15eb commit 400ac13
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions specs/Modal.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ export default () => {
document.activeElement.should.be.eql(mcontent(modal));
});

it("does not focus the modal content when shouldFocusAfterRender is false", () => {
it("does not focus modal content if shouldFocusAfterRender is false", () => {
const modal = renderModal(
{ isOpen: true, shouldFocusAfterRender: false },
null
Expand Down Expand Up @@ -317,7 +317,7 @@ export default () => {
isBodyWithReactModalOpenClass("testBodyClass").should.not.be.ok();
});

it("should not remove classes from document.body when rendering unopened modal", () => {
it("should not remove classes from document.body if modal is closed", () => {
renderModal({ isOpen: true });
isBodyWithReactModalOpenClass().should.be.ok();
renderModal({ isOpen: false, bodyOpenClassName: "testBodyClass" });
Expand All @@ -340,7 +340,7 @@ export default () => {
unmountModal();
});

it("adding/removing aria-hidden without an appElement will try to fallback to document.body", () => {
it("uses document.body for aria-hidden if no appElement", () => {
ariaAppHider.documentNotReadyOrSSRTesting();
const node = document.createElement("div");
ReactDOM.render(<Modal isOpen />, node);
Expand All @@ -349,7 +349,7 @@ export default () => {
should(document.body.getAttribute("aria-hidden")).not.be.ok();
});

it("raise an exception if appElement is a selector and no elements were found.", () => {
it("raises an exception if the appElement selector does not match", () => {
should(() => ariaAppHider.setElement(".test")).throw();
});

Expand Down
2 changes: 1 addition & 1 deletion specs/Modal.style.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ export default () => {
mcontent(modal).style.top.should.be.eql("");
});

it("overrides the default styles when a custom overlayClassName is used", () => {
it("overrides the default styles when using custom overlayClassName", () => {
const modal = renderModal({
isOpen: true,
overlayClassName: "myOverlayClass"
Expand Down

0 comments on commit 400ac13

Please sign in to comment.