Skip to content

E
edx-platform
Commit 070d24cf by David Baumgold
Options

Jasmine doesn't like asserting on the <body> element :(

parent 10ec6046
Showing with 10 additions and 1 deletions
cms/static/coffee/spec/views/feedback_spec.coffee
......@@ -40,5 +40,13 @@ describe "CMS.Views.SystemFeedback", ->
expect(view.$('.wrapper')).not.toHaveClass("is-shown")
expect(view.$('.wrapper')).toHaveClass("is-hiding")
# for some reason, expect($("body")) blows up the test runner, so this test
# just exercises the Prompt rather than asserting on anything. Best I can
# do for now. :(
it "prompt view changes class on body", ->
# expect($("body")).not.toHaveClass("prompt-is-shown")
view = new CMS.Views.Prompt({model: @model})
# expect($("body")).toHaveClass("prompt-is-shown")
@model.hide()
# expect($("body")).not.toHaveClass("prompt-is-shown")
cms/static/js/views/feedback.js
......@@ -55,6 +55,7 @@ CMS.Views.Notification = CMS.Views.SystemFeedback.extend({
CMS.Views.Prompt = CMS.Views.SystemFeedback.extend({
type: "prompt",
render: function() {
if(!window.$body) { window.$body = $(document.body); }
if(this.model.get('shown')) {
$body.addClass('prompt-is-shown');
} else {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment