Merge pull request #1254 from edx/christina/enable-overview-tests
Re-enable tests that were failing on Jenkins.
This commit is contained in:
@@ -97,22 +97,20 @@ define ["js/views/overview", "js/views/feedback_notification", "sinon", "js/base
|
||||
$('a.save-button').click()
|
||||
expect(@notificationSpy).toHaveBeenCalled()
|
||||
|
||||
# Failing in Jenkins (passes locally).
|
||||
# it "should delete model when delete is clicked", ->
|
||||
# $('a.delete-section-button').click()
|
||||
# $('a.action-primary').click()
|
||||
# expect(@requests[0].url).toEqual('/delete_item')
|
||||
it "should delete model when delete is clicked", ->
|
||||
$('a.delete-section-button').click()
|
||||
$('a.action-primary').click()
|
||||
expect(@requests[0].url).toEqual('/delete_item')
|
||||
|
||||
it "should not delete model when cancel is clicked", ->
|
||||
$('a.delete-section-button').click()
|
||||
$('a.action-secondary').click()
|
||||
expect(@requests.length).toEqual(0)
|
||||
|
||||
# Failing in Jenkins (passes locally).
|
||||
# it "should show a confirmation on delete", ->
|
||||
# $('a.delete-section-button').click()
|
||||
# $('a.action-primary').click()
|
||||
# expect(@notificationSpy).toHaveBeenCalled()
|
||||
it "should show a confirmation on delete", ->
|
||||
$('a.delete-section-button').click()
|
||||
$('a.action-primary').click()
|
||||
expect(@notificationSpy).toHaveBeenCalled()
|
||||
|
||||
describe "findDestination", ->
|
||||
it "correctly finds the drop target of a drag", ->
|
||||
|
||||
Reference in New Issue
Block a user