BuildInvocationsBuilderTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Remove unnecessary @Unroll annotations from "ide"

  1. … 9 more files in changeset.
Remove unnecessary @Unroll annotations from "ide"

  1. … 9 more files in changeset.
Remove unnecessary @Unroll annotations from "ide"

  1. … 9 more files in changeset.
Remove unnecessary @Unroll annotations from "ide"

  1. … 9 more files in changeset.
Remove unnecessary @Unroll annotations from "ide"

  1. … 9 more files in changeset.
Simplify remaining @Unroll descriptions

  1. … 4 more files in changeset.
Simplify remaining @Unroll descriptions

  1. … 4 more files in changeset.
Simplify remaining @Unroll descriptions

  1. … 4 more files in changeset.
Simplify remaining @Unroll descriptions

  1. … 4 more files in changeset.
Simplify remaining @Unroll descriptions

  1. … 4 more files in changeset.
Force AbstractTestDirectoryProvider to use Class (#12431)

Closes https://github.com/gradle/gradle-private/issues/2988

This PR adds `className` to `AbstractTestDirectoryProvider` so there'll be no more `unknown-test-class`.

  1. … 407 more files in changeset.
Fix tests

  1. … 372 more files in changeset.
Always build BuildInvocations for current project

The BuildInvocations model was built for the root project

when calling the single-argument `getModels()` method. It was

the only model behaving in this unintuitive way.

    • -14
    • +0
    ./BuildInvocationsBuilderTest.groovy
  1. … 7 more files in changeset.
Remove deprecated methods on TestUtil (#672)

In order to use project builder correctly without having

leaking files on windows it is necessary to initialize

the test fixture for NativeServices and clean up

the test directory after building.

AbstractProjectBuilderSpec provides a nice base class

for Groovy tests.

I removed the deprecated methods since using them leads

to files lying around. Migrating all the usages to the "new"

way ensures it is used correctly.

    • -4
    • +14
    ./BuildInvocationsBuilderTest.groovy
  1. … 97 more files in changeset.
Ignore test that breaks when reverting Tooling API changes

Improve test names

+review REVIEW-5289

Keep null descriptions as null values rather than conerting them to empty strings

+review REVIEW-5289

  1. … 1 more file in changeset.
Revert change related to visibility logic (null vs. empty string)

+review REVIEW-5289

  1. … 1 more file in changeset.
Add more coverage for task description precedence

+review REVIEW-5289

    • -19
    • +19
    ./BuildInvocationsBuilderTest.groovy
Consider empty group string as non-visible task

+review REVIEW-5289

  1. … 1 more file in changeset.
Implement proper logic of selectors' description

+review REVIEW-5289

  1. … 1 more file in changeset.
Add another sibling

    • -15
    • +28
    ./BuildInvocationsBuilderTest.groovy
Remove commented code

    • -18
    • +0
    ./BuildInvocationsBuilderTest.groovy
Fully refactor the test

+review REVIEW-5289

    • -53
    • +83
    ./BuildInvocationsBuilderTest.groovy
Clean up test setup

+review REVIEW-5289

    • -9
    • +10
    ./BuildInvocationsBuilderTest.groovy
Clarify test setup

+review REVIEW-5289

Tune task selector description.

+review REVIEW-5289

  1. … 1 more file in changeset.
Copy task description into task selector.

    • -2
    • +27
    ./BuildInvocationsBuilderTest.groovy
  1. … 1 more file in changeset.
REVIEW-5028 Rename Launchable.isVisible() -> isPublic().

  1. … 7 more files in changeset.
Expose task visibility in Tooling API.

    • -6
    • +11
    ./BuildInvocationsBuilderTest.groovy
  1. … 6 more files in changeset.