From ca99606f9f7cda3c543b6e12d0ef18276277c231 Mon Sep 17 00:00:00 2001
From: stv <stv@stanford.edu>
Date: Sun, 18 Oct 2015 04:39:24 -0400
Subject: [PATCH] Remove unused specs
These have been commented-out for more than 2 and a half years;
I can't think of a legitimate reason for keeping them around at this
point.
These were originally commented out here in
474096541389d67764d2a9b419710512c9a53114
---
.../xmodule/js/spec/capa/display_spec.coffee | 28 ---
.../js/spec/sequence/display_spec.coffee | 171 ------------------
2 files changed, 199 deletions(-)
delete mode 100644 common/lib/xmodule/xmodule/js/spec/sequence/display_spec.coffee
diff --git a/common/lib/xmodule/xmodule/js/spec/capa/display_spec.coffee b/common/lib/xmodule/xmodule/js/spec/capa/display_spec.coffee
index 170f0b38262..690c748cdec 100644
--- a/common/lib/xmodule/xmodule/js/spec/capa/display_spec.coffee
+++ b/common/lib/xmodule/xmodule/js/spec/capa/display_spec.coffee
@@ -71,13 +71,6 @@ describe 'Problem', ->
it 'bind the math input', ->
expect($('input.math')).toHandleWith 'keyup', @problem.refreshMath
- # TODO: figure out why failing
- xit 'replace math content on the page', ->
- expect(MathJax.Hub.Queue.mostRecentCall.args).toEqual [
- ['Text', @stubbedJax, ''],
- [@problem.updateMathML, @stubbedJax, $('#input_example_1').get(0)]
- ]
-
describe 'bind_with_custom_input_id', ->
beforeEach ->
spyOn window, 'update_schematics'
@@ -205,15 +198,6 @@ describe 'Problem', ->
expect(@problem.el.html()).toEqual 'Incorrect!'
expect(window.SR.readElts).toHaveBeenCalled()
- # TODO: figure out why failing
- xdescribe 'when the response is undetermined', ->
- it 'alert the response', ->
- spyOn window, 'alert'
- spyOn($, 'postWithPrefix').andCallFake (url, answers, callback) ->
- callback(success: 'Number Only!')
- @problem.check()
- expect(window.alert).toHaveBeenCalledWith 'Number Only!'
-
describe 'reset', ->
beforeEach ->
@problem = new Problem($('.xblock-student_view'))
@@ -554,13 +538,6 @@ describe 'Problem', ->
runs ->
expect(window.SR.readElts).toHaveBeenCalled()
- # TODO: figure out why failing
- xit 'alert to the user', ->
- spyOn window, 'alert'
- spyOn($, 'postWithPrefix').andCallFake (url, answers, callback) -> callback(success: 'OK')
- @problem.save()
- expect(window.alert).toHaveBeenCalledWith 'Saved'
-
describe 'refreshMath', ->
beforeEach ->
@problem = new Problem($('.xblock-student_view'))
@@ -613,11 +590,6 @@ describe 'Problem', ->
@problem.refreshAnswers()
expect(@stubCodeMirror.save).toHaveBeenCalled()
- # TODO: figure out why failing
- xit 'serialize all answers', ->
- @problem.refreshAnswers()
- expect(@problem.answers).toEqual "input_1_1=one&input_1_2=two"
-
describe 'multiple JsInput in single problem', ->
jsinput_html = readFixtures('jsinput_problem.html')
diff --git a/common/lib/xmodule/xmodule/js/spec/sequence/display_spec.coffee b/common/lib/xmodule/xmodule/js/spec/sequence/display_spec.coffee
deleted file mode 100644
index 0ca17368612..00000000000
--- a/common/lib/xmodule/xmodule/js/spec/sequence/display_spec.coffee
+++ /dev/null
@@ -1,171 +0,0 @@
-# TODO: figure out why failing
-xdescribe 'Sequence', ->
- beforeEach ->
- # Stub MathJax
- window.MathJax = { Hub: { Queue: -> } }
- spyOn Logger, 'log'
-
- loadFixtures 'sequence.html'
- @items = $.parseJSON readFixtures('items.json')
-
- describe 'constructor', ->
- beforeEach ->
- @sequence = new Sequence '1', 'sequence_1', @items, 'sequence', 1
-
- it 'set the element', ->
- expect(@sequence.el).toEqual $('#sequence_1')
-
- it 'build the navigation', ->
- classes = $('#sequence-list li>a').map(-> $(this).attr('class')).get()
- elements = $('#sequence-list li>a').map(-> $(this).attr('data-element')).get()
- titles = $('#sequence-list li>a>p').map(-> $(this).html()).get()
-
- # expect(classes).toEqual ['seq_video_active', 'seq_video_inactive', 'seq_problem_inactive']
- expect(classes).toEqual ['active', 'inactive', 'visited']
- expect(elements).toEqual ['1', '2', '3']
- expect(titles).toEqual ['Video 1', 'Video 2', 'Sample Problem']
-
- it 'bind the page events', ->
- expect($('#sequence-list a')).toHandleWith 'click', @sequence.goto
-
- it 'render the active sequence content', ->
- expect($('#seq_content').html()).toEqual 'Video 1'
-
- describe 'toggleArrows', ->
- beforeEach ->
- @sequence = new Sequence '1', 'sequence_1', @items, 'sequence', 1
-
- describe 'when the first tab is active', ->
- beforeEach ->
- @sequence.position = 1
- @sequence.toggleArrows()
-
- it 'disable the previous button', ->
- expect($('.sequence-nav-button.button-previous')).toHaveClass 'disabled'
-
- it 'enable the next button', ->
- expect($('.sequence-nav-button.button-next')).not.toHaveClass 'disabled'
- expect($('.sequence-nav-button.button-next')).toHandleWith 'click', @sequence.next
-
- describe 'when the middle tab is active', ->
- beforeEach ->
- @sequence.position = 2
- @sequence.toggleArrows()
-
- it 'enable the previous button', ->
- expect($('.sequence-nav-button.button-previous')).not.toHaveClass 'disabled'
- expect($('.sequence-nav-button.button-previous')).toHandleWith 'click', @sequence.previous
-
- it 'enable the next button', ->
- expect($('.sequence-nav-button.button-next')).not.toHaveClass 'disabled'
- expect($('.sequence-nav-button.button-next')).toHandleWith 'click', @sequence.next
-
- describe 'when the last tab is active', ->
- beforeEach ->
- @sequence.position = 3
- @sequence.toggleArrows()
-
- it 'enable the previous button', ->
- expect($('.sequence-nav-button.button-previous')).not.toHaveClass 'disabled'
- expect($('.sequence-nav-button.button-previous')).toHandleWith 'click', @sequence.previous
-
- it 'disable the next button', ->
- expect($('.sequence-nav-button.button-next')).toHaveClass 'disabled'
-
- describe 'render', ->
- beforeEach ->
- spyOn $, 'postWithPrefix'
- @sequence = new Sequence '1', 'sequence_1', @items, 'sequence'
- spyOnEvent @sequence.el, 'contentChanged'
- spyOn(@sequence, 'toggleArrows').andCallThrough()
-
- describe 'with a different position than the current one', ->
- beforeEach ->
- @sequence.render 1
-
- describe 'with no previous position', ->
- it 'does not save the new position', ->
- expect($.postWithPrefix).not.toHaveBeenCalled()
-
- describe 'with previous position', ->
- beforeEach ->
- @sequence.position = 2
- @sequence.render 1
-
- it 'mark the previous tab as visited', ->
- # expect($('[data-element="2"]')).toHaveClass 'seq_video_visited'
- expect($('[data-element="2"]')).toHaveClass 'visited'
-
- it 'save the new position', ->
- expect($.postWithPrefix).toHaveBeenCalledWith '/modx/1/goto_position', position: 1
-
- it 'mark new tab as active', ->
- # expect($('[data-element="1"]')).toHaveClass 'seq_video_active'
- expect($('[data-element="1"]')).toHaveClass 'active'
-
- it 'render the new content', ->
- expect($('#seq_content').html()).toEqual 'Video 1'
-
- it 'update the position', ->
- expect(@sequence.position).toEqual 1
-
- it 're-update the arrows', ->
- expect(@sequence.toggleArrows).toHaveBeenCalled()
-
- it 'trigger contentChanged event', ->
- expect('contentChanged').toHaveBeenTriggeredOn @sequence.el
-
- describe 'with the same position as the current one', ->
- it 'should not trigger contentChanged event', ->
- @sequence.position = 2
- @sequence.render 2
- expect('contentChanged').not.toHaveBeenTriggeredOn @sequence.el
-
- describe 'goto', ->
- beforeEach ->
- jasmine.stubRequests()
- @sequence = new Sequence '1', 'sequence_1', @items, 'sequence', 2
- $('[data-element="3"]').click()
-
- it 'log the sequence goto event', ->
- expect(Logger.log).toHaveBeenCalledWith 'seq_goto', old: 2, new: 3, id: '1'
-
- it 'call render on the right sequence', ->
- expect($('#seq_content').html()).toEqual 'Sample Problem'
-
- describe 'next', ->
- beforeEach ->
- jasmine.stubRequests()
- @sequence = new Sequence '1', 'sequence_1', @items, 'sequence', 2
- $.scrollTo 150
- $('.sequence-nav-button.button-next').click()
-
- it 'log the next sequence event', ->
- expect(Logger.log).toHaveBeenCalledWith 'seq_next', old: 2, new: 3, id: '1'
-
- it 'call render on the next sequence', ->
- expect($('#seq_content').html()).toEqual 'Sample Problem'
-
- it 'scrolls to the top of the page', ->
- expect($('body').scrollTop()).toBe 0
-
- describe 'previous', ->
- beforeEach ->
- jasmine.stubRequests()
- @sequence = new Sequence '1', 'sequence_1', @items, 'sequence', 2
- $.scrollTo 150
- $('.sequence-nav-button.button-previous').click()
-
- it 'log the previous sequence event', ->
- expect(Logger.log).toHaveBeenCalledWith 'seq_prev', old: 2, new: 1, id: '1'
-
- it 'call render on the previous sequence', ->
- expect($('#seq_content').html()).toEqual 'Video 1'
-
- it 'scrolls to the top of the page', ->
- expect($('body').scrollTop()).toBe 0
-
- describe 'link_for', ->
- it 'return a link for specific position', ->
- sequence = new Sequence '1', 'sequence_1', @items, 2
- expect(sequence.link_for(2)).toBe '[data-element="2"]'
--
GitLab