New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor: change the way config listeners in testng #3

Merged

rajatt95 merged 1 commit into rajatt95:master from lucas-nguyen-17:TestngListener

Oct 19, 2021

Contributor

lucas-nguyen-17 commented Oct 19, 2021

Setting listeners in testng file has 2 disadvatages:

Error like NullPointerException at com.spotify.oauth2.reports.ExtentLogger.info(ExtentLogger.java:39) #1
It requires people run by testng.xml file --> kind of annoying when you need to change this file to run only one test

This pull requests changes the setting from by file to by code:

people can run directly from IDE for each or many tests
if we create more testng.xml files or add more listeners, just one place to change.


          refactor: change the way config listeners in testng

d4787c1

rajatt95 merged commit 042acad into rajatt95:master

lucas-nguyen-17 deleted the TestngListener branch

October 28, 2021 10:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment