Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • C chouette-core
  • 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 36
    • Merge requests 36
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Administrator
  • chouette-core
  • Merge requests
  • !140

Merged
Created Dec 06, 2017 by Administrator@rootMaintainer

5197 compliance control set view spec

  • Overview 1
  • Commits 5
  • Changes 4

Created by: RobertDober

Le PR a 3 commits, le 1° ajoutes les specs, mais comme ça ne veut rien dire, je suis passé dans un Red-Green cycle dans le 2° commit où j'ai corrigé le spelling et simplifié le helper.

Le 3° commit ajoute une suggestion d'ajouter une id semantique pour

  • rendre les tests plus stables, car si on créé un 2° .select_toolbox les specs ne veulent plus rien dire (j'ai un commit à vous montrer si vous êtes sceptiques)
  • effet de bord sympathique, dans la dev console c'est plus simple d'identifier les élements également
Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: 5197-compliance-control-set-view-spec