Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • W wendelin
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • nexedi
  • wendelin
  • Merge requests
  • !162

Merged
Created Apr 08, 2024 by Paul Graydon@paul.graydonContributor

Extend Wendelin tutorial notebook script tests

  • Overview 3
  • Commits 1
  • Pipelines 1
  • Changes 2

Following the discussion at https://lab.nexedi.com/nexedi/wendelin/-/commit/111919f56f3785a2f01bb54b89e1273ef2de24b3, this change extends the Selenium tests in bt5/erp5_wendelin_tutorial_ui_test by testing that the graphs produced by the notebook scripts contain the correct data.

Since the data points themselves are drawn in a canvas, which is difficult to test in Selenium, we instead test for the presence of correct graph axis labels: since the data used comes from a fixed sample set and the scripts autoscale the axes, we can reasonably assume that the data is correctly plotted if the labels match.

While this isn't perfect, this does effectively break if the data points do not appear.

Edited Apr 08, 2024 by Paul Graydon
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: tutorial_ui_test_fix
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7