From 400ac1303c361e743e60826085135f311b976831 Mon Sep 17 00:00:00 2001 From: Todd Kloots Date: Mon, 23 Oct 2017 17:40:28 -0700 Subject: [PATCH] [chore] make sure all tests meet line-length requirement. --- specs/Modal.spec.js | 8 ++++---- specs/Modal.style.spec.js | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/specs/Modal.spec.js b/specs/Modal.spec.js index 5da1371a..6aed4586 100644 --- a/specs/Modal.spec.js +++ b/specs/Modal.spec.js @@ -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 @@ -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" }); @@ -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(, node); @@ -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(); }); diff --git a/specs/Modal.style.spec.js b/specs/Modal.style.spec.js index 2b17355f..c74d8eea 100644 --- a/specs/Modal.style.spec.js +++ b/specs/Modal.style.spec.js @@ -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"