This is an automated email from the ASF dual-hosted git repository. aledsage pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/brooklyn-server.git.
from 061057c This closes #1024 new 890ef56 Bump assertj from 2.2.0 to 3.11.1 new f08ed2c [core] add assertj as test dependency new 2c0f0c2 [core] Refactor rebind test for clarity new 5d5a2dd [core] rename excessively long private constant new 442a194 [common] add assertj as test dependency new 8f11da5 [common] Add unit tests for Reflections#findMappedName{AndLog,Maybe}() new 75f41da [common] Add test for mapping classes from one bundle to another new c4234de [common] fix typo in test new 2be6cd4 Extract nested tests for findMappedName into top level class new 170b351 fix typo new a0d2d84 Add assertj as test dependency new 8e91fbe Coverage tests for existing CatalogUpgrades.getBundleUpgradedIfNecessary() new 0ec0418 Extract newMockBundle() utility methods to BundleTestUtil new 441213c whenUpgradingAWrappedBundleThenUpdatedName() new 8419337 Add missing apache license headers new 748bfa0 Load all jars to find upgrades for wrapped bundles new 6127e03 Remove stopwatches new ef42313 Extract scanManagedBundles() and scanAllBundles() new 7328d9f Extract RegisteredTypesSupplier to outer class new 64c869c Extract CatalogUpgradeScanner new 0fc25b2 Extract overloaded typeSupplier() helpers new b6500ed The presence of an OsgiManager is not the concern of CatalogUpgradeScanner new ab67707 Extract static methods as a constructor parameters new cdb298f Limit violation of the law of demeter new 943f3b8 Make BundleUtilTest public new 9fe502a Add unit tests for CatalogUpgradeSCanner new 09e65df Remove previous attempt at failing test new aabab44 Add missing header new 2e025d1 Remove duplicated version for assertj new 5c22839 Changes requested on PR new da8bf93 This closes #1010 The 31 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: core/pom.xml | 5 + .../catalog/internal/CatalogInitialization.java | 62 ++-- .../catalog/internal/CatalogUpgradeScanner.java | 110 +++++++ .../catalog/internal/RegisteredTypesSupplier.java | 25 +- .../persist/DeserializingClassRenamesProvider.java | 5 +- .../brooklyn/core/typereg/BundleUpgradeParser.java | 2 +- .../internal/CatalogUpgradeScannerTest.java | 322 +++++++++++++++++++++ .../RebindWithDeserializingClassRenamesTest.java | 152 +++++++--- .../brooklyn/core/typereg/BundleTestUtil.java | 57 ++++ .../core/typereg/BundleUpgradeParserTest.java | 27 +- ...ogUpgradesGetBundleUpgradedIfNecessaryTest.java | 246 ++++++++++++++++ parent/pom.xml | 41 +-- pom.xml | 6 +- utils/common/pom.xml | 6 + .../javalang/ReflectionsFindMappedNameTest.java | 141 +++++++++ 15 files changed, 1053 insertions(+), 154 deletions(-) create mode 100644 core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogUpgradeScanner.java copy utils/common/dependencies/osgi/com-example-entities/src/main/java/com/example/brooklyn/test/osgi/PrefixedDummyExternalConfigSupplier.java => core/src/main/java/org/apache/brooklyn/core/catalog/internal/RegisteredTypesSupplier.java (58%) create mode 100644 core/src/test/java/org/apache/brooklyn/core/catalog/internal/CatalogUpgradeScannerTest.java create mode 100644 core/src/test/java/org/apache/brooklyn/core/typereg/BundleTestUtil.java create mode 100644 core/src/test/java/org/apache/brooklyn/core/typereg/CatalogUpgradesGetBundleUpgradedIfNecessaryTest.java create mode 100644 utils/common/src/test/java/org/apache/brooklyn/util/javalang/ReflectionsFindMappedNameTest.java