Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Done changes to fix bug, Unable to report for greater than 200 classes#63

Merged
awillia2 merged 3 commits intomasterfrom
rduvvarapu_apexunit_dev
Mar 27, 2017
Merged

Done changes to fix bug, Unable to report for greater than 200 classes#63
awillia2 merged 3 commits intomasterfrom
rduvvarapu_apexunit_dev

Conversation

@rduvvarapu
Copy link
Contributor

Creates Batches of test cases, each batch contains 200 test cases.
Based on number batches, it loops through and executes batch by batch.
Consolidates all batches results and returns back

Copy link
Contributor

@awillia2 awillia2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Long end = System.currentTimeMillis();
LOG.debug("Total Time taken by ApexUnit tool in secs: " + (end - start) / 1000);
LOG.info("Total test methods executed: " + TestStatusPollerAndResultHandler.totalTestMethodsExecuted);
//LOG.info("Total test methods executed: " + TestStatusPollerAndResultHandler.totalTestMethodsExecuted);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rduvvarapu would you remove this commented line since it's now in the for loo?

private static final int BATCH_SIZE = 200;

public ApexReportBean[] testExecutionFlow() {
/*public ApexReportBean[] testExecutionFlow() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rduvvarapu would you remove the commented testExecutionFlow method since it's implemented below?

@awillia2 awillia2 merged commit 5238733 into master Mar 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants