Skip to content

introduce new detector (Tuscan, Tuscan-Intra, Tuscan-Inter)#63

Open
ChopinLi-cp wants to merge 1 commit into
UT-SE-Research:masterfrom
lcpChopin:master
Open

introduce new detector (Tuscan, Tuscan-Intra, Tuscan-Inter)#63
ChopinLi-cp wants to merge 1 commit into
UT-SE-Research:masterfrom
lcpChopin:master

Conversation

@ChopinLi-cp

Copy link
Copy Markdown
Contributor

Overview

This pull request modifies the current Tuscan Detector and introduces two new Detector Tuscan-Intra and Tuscan-Inter.

Modifications

  1. For all three kinds of new detectors, by default they will generate all orders and at the same time run all the generated orders. The way that you can disable running all orders is to add -Ddt.randomize.rounds=0 in the command line when running iDFlakies. (If you use Tuscan-related detector, the default num of rounds is Integer.MAX_INT and we define it as initRoundNum at idflakies-maven-plugin/src/main/java/edu/illinois/cs/dt/tools/plugin/DetectorMojo.java). No matter how many rounds you want to run, Tuscan-related runner always generates all orders.
  2. In idflakies-core/src/main/java/edu/illinois/cs/dt/tools/utility/PathManager.java, idflakies-legacy/src/main/java/edu/illinois/cs/dt/tools/detection/LegacyDetectorPathManager.java and idflakies-maven-plugin/src/main/java/edu/illinois/cs/dt/tools/detection/MavenDetectorPathManager.java, we introduce two locations to output the number of orders and corresponding orders for Tuscan-related detector.
  3. Modified idflakies-core/src/main/java/edu/illinois/cs/dt/tools/detection/TestShuffler.java to include the helper methods for these three Tuscan-related detectors.
  4. Introduced some writing helper methods in idflakies-core/src/main/java/edu/illinois/cs/dt/tools/detection/detectors/ExecutingDetector.java.
  5. Changed the matrix of Tuscan Square when the size is 3 or 5 in idflakies-core/src/main/java/edu/illinois/cs/dt/tools/utility/Tuscan.java to make sure the length of test orders generated is the same and all tests appear at the beginning and end of the matrix at least once.

@ChopinLi-cp
ChopinLi-cp marked this pull request as ready for review September 3, 2024 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant