fix(tessera): updated tessera version error in quorum-all-in-one #2411 #2549
Conversation
ruzell22
requested review from
petermetz,
takeutak,
izuru0,
jagpreetsinghsasan,
VRamakrishna and
sandeepnRES
as code owners
|
docker run --priveleged --rm qaio successfully ran and built
|
There was a problem hiding this comment.
@ruzell22 Please also run one of the tests against the locally built image (by modifying the test case to use it) and then post the exact path of that test case here. Once we have that, we know for sure that it doesn't just work as in the image does boot without crashing, but also that the test cases are still passing with the new image changes.
Other than that, it looks good to me already. Thank you!
|
@petermetz I have tested the following with launch.json JEST test: • deploy-contract-via-web-service.test.ts (cactus/packages/cactus-test-plugin-ledger-connector-quorum/src/test/typescript/integration/plugin-ledger-connector-quorum/deploy-contract/deploy-contract-via-web-service.test.ts) • v21.4.1-deploy-contract-from-json.test.ts (cactus/packages/cactus-plugin-ledger-connector-quorum/src/test/typescript/integration/plugin-ledger-connector-quorum/deploy-contract/v21.4.1-deploy-contract-from-json.test.ts) • v21.4.1-invoke-contract.test.ts (cactus/packages/cactus-plugin-ledger-connector-quorum/src/test/typescript/integration/plugin-ledger-connector-quorum/deploy-contract/v21.4.1-invoke-contract.test.ts) |
…rledger#2411 fixes: hyperledger#2411 related to: hyperledger#2240 and hyperledger#2274 Signed-off-by: ruzell22 <ruzell.vince.aquino@accenture.com>
petermetz
force-pushed
the
updatetesseraqaio2411
branch
from
b615ce4
to
5be685d
Compare
fixes: #2411
related to: #2240 and #2274
Tessera version was updated and some flags were deprecated. It has been
fixed in this commit. Also, the quorum and tessera compatibility with
quorum-all-in-one. The container and image has been built and ran
successfully.