NativeBinariesIntegrationTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Rename @FailsWithInstantExecution to @ToBeFixedForInstantExecution

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -10
    • +10
    ./NativeBinariesIntegrationTest.groovy
  1. … 872 more files in changeset.
Annotate integ tests failing with instant execution in :platformNative

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -0
    • +10
    ./NativeBinariesIntegrationTest.groovy
  1. … 24 more files in changeset.
Annotate integ tests failing with instant execution in :platformNative

Signed-off-by: Paul Merlin <paul@gradle.com>

    • -0
    • +10
    ./NativeBinariesIntegrationTest.groovy
  1. … 24 more files in changeset.
Tweak the output produced by `TreeFormatter`.

    • -9
    • +16
    ./NativeBinariesIntegrationTest.groovy
  1. … 36 more files in changeset.
Removed `@LeaksFileHandles` from ':platform-native'

    • -3
    • +0
    ./NativeBinariesIntegrationTest.groovy
  1. … 17 more files in changeset.
Cleaned out a bunch of TODOs

    • -1
    • +0
    ./NativeBinariesIntegrationTest.groovy
  1. … 30 more files in changeset.
Changed the `ModelMap` 'cannot add element of unknown type' error message to list only those known types for which it is actually possible to use to create an element.

In particular, filters those types that don't satisfy all the type constraints for the map.

This change allows a more general hierarchy to be handled by particular registry, while providing somewhat useful error messages when a more specific hierarchy is required.

    • -1
    • +3
    ./NativeBinariesIntegrationTest.groovy
  1. … 23 more files in changeset.
Changed the naming scheme for output directories for native components, to simplify the paths and reduce their length. Also split up the 'binaries' directory into 'exe' and 'libs'.

    • -5
    • +5
    ./NativeBinariesIntegrationTest.groovy
  1. … 58 more files in changeset.
Always used generated view types for `ComponentSpec`, `BinarySpec` and `LanguageSourceSet` model elements, regardless of whether the public type is `@Managed` or not.

This continues to make the behaviour between the two types of elements consistent, and in particular, the model DSL. It also means that the internals of a non-`@Managed` element are no longer completely exposed.

+review REVIEW-5708

    • -5
    • +5
    ./NativeBinariesIntegrationTest.groovy
  1. … 7 more files in changeset.
Fix error message in integration test

    • -2
    • +2
    ./NativeBinariesIntegrationTest.groovy
More test coverage for `BinarySpec.sources`

- Displayed correctly in 'model' report

- Reasonable error message when attempting to create unknown type

    • -2
    • +12
    ./NativeBinariesIntegrationTest.groovy
  1. … 3 more files in changeset.
Fix integration tests for changes to `binary.sources` visibility in model

    • -10
    • +15
    ./NativeBinariesIntegrationTest.groovy
  1. … 2 more files in changeset.
Enabled nested DSL rules for `components`, `binaries` and `testSuites`.

Each closure passed to a delegate method in these blocks will be treated as a nested rule. This means that components, binaries and test suites can be configured using one or more of their siblings as input.

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
  1. … 5 more files in changeset.
Changed the descriptor for DSL rules so that it more closely matches the expression used in the code to reference the subject.

This change also detangles the subject path from the DSL rule descriptor, so that they can be different.

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
  1. … 25 more files in changeset.
Integration test coverage for component.sources displayed in model report

Issue: gradle/langos#29 +review REVIEW

    • -0
    • +48
    ./NativeBinariesIntegrationTest.groovy
  1. … 3 more files in changeset.
Use a real `ModelMap` for `ComponentSpec.sources`

This commit removes the backing `FunctionalSourceSet` instance from `BaseComponentSpec`,

instead using a real `ModelMap` implementation backed by a `ModelNode`. This removes

a lot of complexity from the implementation of `BaseComponentSpec`, as well as

making the behaviour of `ComponentSpec.sources` consistent with other `ModelMap`

properties.

Each `LanguageSourceSet` requires a 'parentName' property to be set, which is used

for providing a project-scoped unique name as well as source default locations.

For an element in `ComponentSpec.sources`, the 'parentName' is equal to the owning

component name. To make this possible, `LanguageSourceSetNodeIntializer` is now

aware of this relationship, and will set the 'parentName' appropriately. This is

somewhat consistent with the way the `owner` component is set when constructing

a `BaseBinarySpec` instance.

Notes:

- Elements of `ComponentSpec.sources` are now configured on demand.

- Deleted tests that verified sources sets couldn't be removed: these are

no longer relevant with a `ModelMap` implementation.

Issue: gradle/langos#40 +review REVIEW-5681

    • -2
    • +3
    ./NativeBinariesIntegrationTest.groovy
  1. … 8 more files in changeset.
Report supported types when failing to create a LanguageSourceSet

+review REVIEW-5669

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
  1. … 3 more files in changeset.
Changed the `assemble` task to use binary displayName in error message when no binaries are buildable.

Binary name is no longer unique, and display name provides the context in a more human-consumable format.

    • -5
    • +5
    ./NativeBinariesIntegrationTest.groovy
  1. … 3 more files in changeset.
Changed the JVM and native component plugins to use names for binaries that reflect the role they play.

Now that more than one component may have a binary with a given name, use:

- `jar` as the name for a Jar binary that belongs to a JVM library

- `executable` as the name for an executable that belongs to a native application

- `sharedLibrary` and `staticLibrary` as the name for the binaries of a native library.

These can later turn into static properties of the appropriate `ComponentSpec` types.

There is a breaking change in this commit: the lifecycle task name and output directory name of the binaries of components with multiple variants now include the component name at the start of the name, rather than somewhere in the middle.

    • -2
    • +2
    ./NativeBinariesIntegrationTest.groovy
  1. … 26 more files in changeset.
Always transform model rules in the DSL.

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
  1. … 6 more files in changeset.
Changed a test case to use `binaries` model element rather than `binaries` extension.

    • -11
    • +12
    ./NativeBinariesIntegrationTest.groovy
Fixed broken int tests following refactor.

    • -12
    • +15
    ./NativeBinariesIntegrationTest.groovy
  1. … 2 more files in changeset.
Changed a bunch of int tests to use `binaries` from model land rather than legacy land.

    • -14
    • +13
    ./NativeBinariesIntegrationTest.groovy
  1. … 11 more files in changeset.
Fix test after model rule descriptor change.

+review REVIEW-5561

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
Expose ComponentSpec.source() as a node backed model map, instead of a domain object collection wrapped model map.

+review REVIEW-5498

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
  1. … 39 more files in changeset.
Mark some more tests as leaking file handles.

+review REVIEW-5504

    • -0
    • +2
    ./NativeBinariesIntegrationTest.groovy
  1. … 13 more files in changeset.
Fix test after change to rule descriptor format for model map init actions.

    • -1
    • +1
    ./NativeBinariesIntegrationTest.groovy
Changed `NativePlatforms` so that it its methods are not static.

    • -4
    • +5
    ./NativeBinariesIntegrationTest.groovy
  1. … 13 more files in changeset.
Move limit of Exception causes into MultipleBuildOperationFailures

+review REVIEW-5406

    • -3
    • +5
    ./NativeBinariesIntegrationTest.groovy
  1. … 10 more files in changeset.
Add information about what is being compiled/linked to error message

+review REVIEW-5406

    • -3
    • +6
    ./NativeBinariesIntegrationTest.groovy
  1. … 18 more files in changeset.