ComponentBinariesModelRuleExtractorTest.groovy

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Improve fully qualified representation of nested model types

Use `.` instead of `$` to separate the enclosing type name from the

nested type name.

    • -1
    • +3
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 24 more files in changeset.
Fix test

    • -6
    • +4
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 1 more file in changeset.
Fix platform-base tests

    • -4
    • +5
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 1 more file in changeset.
Changed a number of usages of `ComponentSpec` to use more specific types.

    • -7
    • +7
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 8 more files in changeset.
Fixed the `@ComponentBinaries` annotation so that it works for any `ComponentSpec`, not just those in `components`.

Changed the test plugins to use this to define the binaries for test suites.

    • -10
    • +5
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 9 more files in changeset.
Use contextualized `RuleInvoker` to run `@ComponentBinaries` rule methods.

+review REVIEW-5756

    • -3
    • +0
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 1 more file in changeset.
Reworked the parameters to `ExtractedModelRule.apply()` so that per-application context, such as explicit bindings, can be provided.

+review REVIEW-5756

    • -2
    • +5
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 12 more files in changeset.
Moved some classes from o.g.model.internal.core to o.g.model.internal.inspect.

Also moved a class that was only used by a single test to live in that test class.

+review REVIEW-5756

    • -0
    • +1
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 27 more files in changeset.
Collect several more `RuleSource` validation problems rather than failing on first problem.

In particular, problems with `@ComponentBinaries` and `@BinaryTasks` are collected.

+review REVIEW-5739

    • -14
    • +16
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 7 more files in changeset.
More improvements to validation of a RuleSource type.

- Fixed NPE when a rule method has no parameters.

- Collect more kinds of validation failures, rather than failing on first of these kinds of failures.

- Validate that the first parameter of a `@Rules` metod is assignable to `RuleSource`

+review REVIEW-5739

    • -2
    • +2
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 19 more files in changeset.
Add type parameter to ModelAction

AbstractModelAction already had a type parameter, so ModelAction should

have one too.

+review REVIEW-5738

    • -1
    • +1
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 36 more files in changeset.
Moved test interfaces back to their tests themselves

Since IBM JDK 6 is not throwing errors about these anymore, might as

well move them back to the tests they belong to.

+review REVIEW-5658

    • -4
    • +9
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 21 more files in changeset.
Model actions don't get access automatically to node view

+review REVIEW-5629

    • -1
    • +1
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 29 more files in changeset.
Fixed tests

+review REVIEW-5620

    • -8
    • +16
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 4 more files in changeset.
Extract out inlined interfaces to fix issues with IBM jdk

    • -8
    • +4
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 12 more files in changeset.
Use ComponentSpecContainer in a couple of places.

    • -2
    • +2
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 3 more files in changeset.
Introduce ModelMap extends CollectionBuilder and allow to use them interchangeably in rules

+review REVIEW-5491

    • -13
    • +13
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 117 more files in changeset.
Make components.«component».binaries.«binary» addressable/visible in rule space.

+review REVIEW-5472

    • -2
    • +2
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 62 more files in changeset.
Revert "Make components.«component».binaries.«binary» addressable/visible in rule space"

This reverts commit d10b1f3da00c8e7cbae000974f615c73e8b7325e.

    • -2
    • +2
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 59 more files in changeset.
Make components.«component».binaries.«binary» addressable/visible in rule space

+review REVIEW-5472

    • -2
    • +2
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 59 more files in changeset.
Make @ComponentBinaries and @BinaryTasks model reuse “aware”.

Use less closed over state, preferring rule inputs.

+review REVIEW-5382

    • -1
    • +2
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 5 more files in changeset.
Test fixes after recent API change.

+review REVIEW-5353

    • -11
    • +6
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 4 more files in changeset.
Collection mutation rule specifies input taking mutation rule for particular model element

+review REVIEW-5353

    • -2
    • +3
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 68 more files in changeset.
Rules are extracted from plugins once and cached globally

+review REVIEW-5335

    • -8
    • +12
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 27 more files in changeset.
Rename rule definition handlers to rule extractors

+review REVIEW-5335

    • -0
    • +133
    ./ComponentBinariesModelRuleExtractorTest.groovy
  1. … 52 more files in changeset.