Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Merge requests
  • !13853

Major unit tests cleanup

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/hnrch02/unit-tests-cleanup into master Jun 18, 2014
  • Overview 0
  • Commits 16
  • Pipelines 0
  • Changes 13

Created by: hnrch02

This is a pretty big PR attempting to clean up our unit tests. I reworked a few, but mostly performed style/consistency updates.

I created separate commits for the cleanup of each file, I don't know but that may help during the review. Will ofc squash if it is decided that this should be merged.

Also, I'll need to spend some time on the scrollspy tests, my shallow inspection concluded that they might not work as intended. The scrollspy test will work once #13500 is merged.

/cc @fat @cvrebert @XhmikosR

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/hnrch02/unit-tests-cleanup