Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B buck
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 201
    • Issues 201
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 22
    • Merge requests 22
  • 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
  • Meta
  • buck
  • Merge requests
  • !1428

[TestNG] Support Guice injection in test classes

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/zhan-xiong/zhan.xiong/testng_guice_injector_constructor into master Jul 19, 2017
  • Overview 17
  • Commits 12
  • Pipelines 0
  • Changes 14

Created by: zhan-xiong

Formerly, the TestNG runner would skip over test classes that were initialized using guice/javax injection, since they might not have no-arg constructor. This PR makes it so that these classes are now correctly detected and run as test classes. This PR also adds simple tests for both this change and for the TestNG runner in general.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/zhan-xiong/zhan.xiong/testng_guice_injector_constructor