Skip to content

Commit

Permalink
test(connector-corda): fix flow-database-access-v4-8 test case
Browse files Browse the repository at this point in the history
1. The test case was broken due to a number of different issues related
to the AIO image build an also the connector image build, but on top of
those problems it also had misconfiguration issues where the port number
wasn't set to what it should be for the RPC connection that the connector
container uses to establish communications with the AIO ledger image.

In order to properly verify that this test case is working, a few other
pull requests have to be merged first and container images from those
sources published as well.

In addition to the pull request dependencies we also depend on a permission
issue being resolved in the larger GitHub organization itself as well:
hyperledger/governance#299

Depends on hyperledger-cacti#3386
Depends on hyperledger-cacti#3387

Signed-off-by: Peter Somogyvari <[email protected]>
  • Loading branch information
petermetz committed Jul 8, 2024
1 parent 997ee75 commit f833fa4
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,9 @@ export class CordaConnectorContainer {
[`${this.apiPort}/tcp`]: {}, // REST API HTTP port
[`9001/tcp`]: {}, // SupervisorD Web UI
},
PublishAllPorts: true,
HostConfig: {
PublishAllPorts: true,
},
Env: this.envVars,
},
{},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,9 @@ export class CordaTestLedger implements ITestLedger {
[`${this.rpcPortC}/tcp`]: {}, // corda PartyC RPC
"22/tcp": {}, // ssh server
},
PublishAllPorts: true,
HostConfig: {
PublishAllPorts: true,
},
// TODO: this can be removed once the new docker image is published and
// specified as the default one to be used by the tests.
// Healthcheck: {
Expand Down

0 comments on commit f833fa4

Please sign in to comment.