MockAsset method getID() should have concrete implementation
Description
Activity
Show:
Stefan Seifert August 3, 2018 at 12:14 PM
its planned for next monday!
Richard Hand August 3, 2018 at 11:50 AM
Thanks Stefan for this. Do you know when a 2.2.16 release is likely?
Stefan Seifert July 30, 2018 at 1:20 PM
thanks - i've merged the PR with some modifications (we must not use JCR API in MockAsset because it should also be usable with resource resolver types not using JCR API).
Samyak Jain July 27, 2018 at 5:42 AM
Have made the required changes. Please check the link - https://github.com/wcm-io/wcm-io-testing/pull/8
Fixed
Details
Details
Assignee
Stefan SeifertReporter
Samyak JainComponents
Fix versions
Affects versions
Priority
MajorCreated July 19, 2018 at 11:15 AM
Updated August 6, 2018 at 10:15 AM
Resolved July 30, 2018 at 1:20 PM
Please see the getID() method at this
https://github.com/wcm-io/wcm-io-testing/blob/develop/aem-mock/src/main/java/io/wcm/testing/mock/aem/MockAsset.java#L273
It currently throws an UnsupportedOperationException but it would be beneficial if it would have a concrete implementation since the method is being used at various places when unit testing are code.
I suggest to have a simple implementation such as just returning the jcr:uuid of the asset.