Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Lesson
    • Labels:
      None
    • Rank:
      50092

      Description

      The 'add a question page' section name should be renamed to avoid conflict with the 'add a question page' button in Behat tests.

        Issue Links

          Activity

          Hide
          Eloy Lafuente (stronk7) added a comment -

          Just to confirm that this seems to prevent the alert=> exception problem in behat, so now mod_lesson tests are all run (although there are still 2 failed steps), both in the same scenario:

          • Forbidding lesson accesses until a specified date.

          Although that may be unrelated to this.

          Show
          Eloy Lafuente (stronk7) added a comment - Just to confirm that this seems to prevent the alert=> exception problem in behat, so now mod_lesson tests are all run (although there are still 2 failed steps), both in the same scenario: Forbidding lesson accesses until a specified date. Although that may be unrelated to this.
          Show
          David Monllaó added a comment - Yes, I commented in here: https://tracker.moodle.org/browse/MDL-38714?focusedCommentId=215907&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-215907 Rossie removed the additional step in MDL-38714 ( https://github.com/rwijaya/moodle/compare/moodle:master...MDL-38714_tmp#L3L58 ) when both this and MDL-38714 are integrated @mod_lesson should pass
          Hide
          David Monllaó added a comment -

          Looks good Rossie, I've ran the scenario and it passed, sending to integration

          Show
          David Monllaó added a comment - Looks good Rossie, I've ran the scenario and it passed, sending to integration
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Sorry, but I continue getting 2 errors with mod_lesson, grrr:

          01. Exception thrown by (//html/.//input[./@type = 'checkbox'][(((./@id = 'id_available_enabled' or ./@name = 'id_available_enabled') or ./@id = //label[contains(normalize-space(string(.)), 'id_available_enabled')]/@for) or ./@placeholder = 'id_available_enabled')] | .//label[contains(normalize-space(string(.)), 'id_available_enabled')]//.//input[./@type = 'checkbox'])[1]
              Element is not currently visible and so may not be interacted with
              Command duration or timeout: 9 milliseconds
              Build info: version: '2.32.0', revision: '6c40c18', time: '2013-04-09 17:22:56'
              System info: os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.8.3', java.version: '1.6.0_45'
              Session ID: 81132b9e-6325-f649-bf59-6862669d859b
              Driver info: org.openqa.selenium.firefox.FirefoxDriver
              Capabilities [{platform=MAC, acceptSslCerts=true, javascriptEnabled=true, browserName=firefox, rotatable=false, locationContextEnabled=true, version=20.0, cssSelectorsEnabled=true, databaseEnabled=true, handlesAlerts=true, browserConnectionEnabled=true, nativeEvents=false, webStorageEnabled=true, applicationCacheEnabled=true, takesScreenshot=true}]
              In step `And I click on "id_available_enabled" "checkbox"'.                          # behat_general::i_click_on()
              From scenario `Forbidding lesson accesses until a specified date'.                   # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature:24
              Of feature `A teacher can set available from and deadline dates to access a lesson'. # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature
          
          02. Exception thrown by (//html/.//input[./@type = 'checkbox'][(((./@id = 'id_deadline_enabled' or ./@name = 'id_deadline_enabled') or ./@id = //label[contains(normalize-space(string(.)), 'id_deadline_enabled')]/@for) or ./@placeholder = 'id_deadline_enabled')] | .//label[contains(normalize-space(string(.)), 'id_deadline_enabled')]//.//input[./@type = 'checkbox'])[1]
              Element is not currently visible and so may not be interacted with
              Command duration or timeout: 6 milliseconds
              Build info: version: '2.32.0', revision: '6c40c18', time: '2013-04-09 17:22:56'
              System info: os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.8.3', java.version: '1.6.0_45'
              Session ID: 81132b9e-6325-f649-bf59-6862669d859b
              Driver info: org.openqa.selenium.firefox.FirefoxDriver
              Capabilities [{platform=MAC, acceptSslCerts=true, javascriptEnabled=true, browserName=firefox, rotatable=false, locationContextEnabled=true, version=20.0, cssSelectorsEnabled=true, databaseEnabled=true, handlesAlerts=true, browserConnectionEnabled=true, nativeEvents=false, webStorageEnabled=true, applicationCacheEnabled=true, takesScreenshot=true}]
              In step `And I click on "id_deadline_enabled" "checkbox"'.                           # behat_general::i_click_on()
              From scenario `Forbidding lesson accesses until a specified date'.                   # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature:50
              Of feature `A teacher can set available from and deadline dates to access a lesson'. # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature
          
          5 escenarios (3 exitosos, 2 fallidos)
          139 pasos (113 exitosos, 24 omitidos, 2 fallidos)
          

          (note that before the patch I get the 5 scenarios failing)

          Also, as far as this introduces a small change in the visualization (new header) and it's not verified by behat... this should have some simple human testing instructions covering it.

          I'll keep this open (and NOT integrated), until some feedback lands... ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Sorry, but I continue getting 2 errors with mod_lesson, grrr: 01. Exception thrown by ( //html/.//input[./@type = 'checkbox'][(((./@id = 'id_available_enabled' or ./@name = 'id_available_enabled') or ./@id = //label[contains(normalize-space(string(.)), 'id_available_enabled')]/@ for ) or ./@placeholder = 'id_available_enabled')] | .//label[contains(normalize-space(string(.)), 'id_available_enabled')]//.//input[./@type = 'checkbox'])[1] Element is not currently visible and so may not be interacted with Command duration or timeout: 9 milliseconds Build info: version: '2.32.0', revision: '6c40c18', time: '2013-04-09 17:22:56' System info: os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.8.3', java.version: '1.6.0_45' Session ID: 81132b9e-6325-f649-bf59-6862669d859b Driver info: org.openqa.selenium.firefox.FirefoxDriver Capabilities [{platform=MAC, acceptSslCerts= true , javascriptEnabled= true , browserName=firefox, rotatable= false , locationContextEnabled= true , version=20.0, cssSelectorsEnabled= true , databaseEnabled= true , handlesAlerts= true , browserConnectionEnabled= true , nativeEvents= false , webStorageEnabled= true , applicationCacheEnabled= true , takesScreenshot= true }] In step `And I click on "id_available_enabled" "checkbox" '. # behat_general::i_click_on() From scenario `Forbidding lesson accesses until a specified date'. # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature:24 Of feature `A teacher can set available from and deadline dates to access a lesson'. # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature 02. Exception thrown by ( //html/.//input[./@type = 'checkbox'][(((./@id = 'id_deadline_enabled' or ./@name = 'id_deadline_enabled') or ./@id = //label[contains(normalize-space(string(.)), 'id_deadline_enabled')]/@ for ) or ./@placeholder = 'id_deadline_enabled')] | .//label[contains(normalize-space(string(.)), 'id_deadline_enabled')]//.//input[./@type = 'checkbox'])[1] Element is not currently visible and so may not be interacted with Command duration or timeout: 6 milliseconds Build info: version: '2.32.0', revision: '6c40c18', time: '2013-04-09 17:22:56' System info: os.name: 'Mac OS X', os.arch: 'x86_64', os.version: '10.8.3', java.version: '1.6.0_45' Session ID: 81132b9e-6325-f649-bf59-6862669d859b Driver info: org.openqa.selenium.firefox.FirefoxDriver Capabilities [{platform=MAC, acceptSslCerts= true , javascriptEnabled= true , browserName=firefox, rotatable= false , locationContextEnabled= true , version=20.0, cssSelectorsEnabled= true , databaseEnabled= true , handlesAlerts= true , browserConnectionEnabled= true , nativeEvents= false , webStorageEnabled= true , applicationCacheEnabled= true , takesScreenshot= true }] In step `And I click on "id_deadline_enabled" "checkbox" '. # behat_general::i_click_on() From scenario `Forbidding lesson accesses until a specified date'. # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature:50 Of feature `A teacher can set available from and deadline dates to access a lesson'. # /Users/stronk7/git_moodle/integration/mod/lesson/tests/behat/date_availability.feature 5 escenarios (3 exitosos, 2 fallidos) 139 pasos (113 exitosos, 24 omitidos, 2 fallidos) (note that before the patch I get the 5 scenarios failing) Also, as far as this introduces a small change in the visualization (new header) and it's not verified by behat... this should have some simple human testing instructions covering it. I'll keep this open (and NOT integrated), until some feedback lands... ciao
          Hide
          David Monllaó added a comment -

          I've confirmed that this last 2 failures are caused by MDL-39270 changes, adding Fred as a watcher. If you are changing a lesson form, please, run the behat tests before sending to integration, the same is applicable for all issues and not only related with forms, but forms is a critical part and even more with big changes like this.

          Adding a quick fix for this on top of this MDL-39437 patch: git pull git://github.com/dmonllao/moodle.git MDL-39437_master-fix

          Show
          David Monllaó added a comment - I've confirmed that this last 2 failures are caused by MDL-39270 changes, adding Fred as a watcher. If you are changing a lesson form, please, run the behat tests before sending to integration, the same is applicable for all issues and not only related with forms, but forms is a critical part and even more with big changes like this. Adding a quick fix for this on top of this MDL-39437 patch: git pull git://github.com/dmonllao/moodle.git MDL-39437 _master-fix
          Hide
          Rossiani Wijaya added a comment - - edited

          Thanks David for the patch.

          I pushed David's patch on top of mine.

          Running the test locally now. Will post the result soon.

          Show
          Rossiani Wijaya added a comment - - edited Thanks David for the patch. I pushed David's patch on top of mine. Running the test locally now. Will post the result soon.
          Hide
          Rossiani Wijaya added a comment -

          Behat test passed successfully.

          Show
          Rossiani Wijaya added a comment - Behat test passed successfully.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Passing, got all mod_lesson passing too and the new string was correctly displayed when adding a question. Thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Passing, got all mod_lesson passing too and the new string was correctly displayed when adding a question. Thanks!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Did you think this day was not going to arrive ever?

          Your patience has been rewarded, yay, sent upstream, thanks!

          Closing...ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: