fabric
fabric copied to clipboard
asset management test case should be moved out of examples/... path
asset management test case will not be run as part of the running unit test case per doc directions if its under examples/... folder. It should be moved to chaincode/ folder.
I think it should stay with the asset management chaincode. I'll try to figure out how to make the UTXO chaincode test case not break. That's the only reason we have the documentation to exclude the examples/ folder.
The entire .../asset_management/* is a test for ACL with chaincode and is standalone. Why would we not run it as part of unit tests ?
I think I misunderstood @srderson comment... if we can arrange it so we don't exclude examples/ from unit tests, then, the asset_management test case (rightly) belongs where it is.
personally, I think that 'examples' is an awful name for chaincode that we use for tests. We should have a test/artifacts tree for that and examples should be exemplar code that is well documented and exhibits best practices. Sometimes, such as with chaincode that exhibits nondeterminism for purposes of ensuring that such chaincode is flagged, etc we will have bad practices and we don't want users emulating this sort of code.