Skip to content

Commit 6c144e5

Browse files
committed
[FAB-2893] Update BDD to latest proto changes
Incoporated prior move of msp related protos and couple of case changes. Only left solo uncommented for now until the make can be modified to support kafka/zookeeper. Change-Id: Ice6a04e3cb488650f147f013f737dd3e1c4ae1ec Signed-off-by: jeffgarratt <[email protected]>
1 parent 397f5de commit 6c144e5

10 files changed

+61
-59
lines changed

.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ tags
2222
bddtests/coverage
2323
# bddtest Dockerfile
2424
bddtests/environments/kafka/Dockerfile
25+
bddtests/tmp
26+
bddtests/volumes
2527
*.cov
2628
# Makefile dummy artifacts
2729
.*-dummy

bddtests/common/policies_pb2.py

+19-19
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

bddtests/features/bootstrap.feature

+3-3
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ Feature: Bootstrap
279279
Examples: Orderer Options
280280
| ComposeFile | SystemUpWaitTime | ConsensusType | BroadcastWaitTime | orderer0 | orderer1 | orderer2 |Orderer Specific Info|
281281
| docker-compose-next-4.yml | 0 | solo | 2 | orderer0 | orderer0 | orderer0 | |
282-
| docker-compose-next-4.yml ./environments/orderer-1-kafka-1/docker-compose.yml orderer-3-kafka-1.yml | 5 | kafka | 5 | orderer0 | orderer1 | orderer2 | |
283-
| docker-compose-next-4.yml docker-compose-next-4-couchdb.yml | 5 | solo | 2 | orderer0 | orderer0 | orderer0 | |
284-
# | docker-compose-next-4.yml docker-compose-next-4-couchdb.yml ./environments/orderer-1-kafka-1/docker-compose.yml orderer-3-kafka-1.yml | 8 | kafka | 5 | orderer0 | orderer1 | orderer2 | |
282+
# | docker-compose-next-4.yml ./environments/orderer-1-kafka-1/docker-compose.yml orderer-3-kafka-1.yml | 5 | kafka | 5 | orderer0 | orderer1 | orderer2 | |
283+
# | docker-compose-next-4.yml docker-compose-next-4-couchdb.yml | 10 | solo | 2 | orderer0 | orderer0 | orderer0 | |
284+
# | docker-compose-next-4.yml docker-compose-next-4-couchdb.yml ./environments/orderer-1-kafka-1/docker-compose.yml orderer-3-kafka-1.yml | 10 | kafka | 5 | orderer0 | orderer1 | orderer2 | |
285285
# | docker-compose-next-4.yml ./environments/orderer-1-kafka-3/docker-compose.yml | 5 | kafka | 5 | orderer0 | orderer1 | orderer2 | |

bddtests/identities_pb2.py bddtests/msp/identities_pb2.py

+9-9
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
File renamed without changes.

bddtests/common/msp_principal_pb2.py bddtests/msp/msp_principal_pb2.py

+19-19
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

bddtests/identities_pb2_grpc.py bddtests/msp/msp_principal_pb2_grpc.py

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
# Generated by the gRPC Python protocol compiler plugin. DO NOT EDIT!
12
import grpc
23
from grpc.framework.common import cardinality
34
from grpc.framework.interfaces.face import utilities as face_utilities

bddtests/msp/mspconfig_pb2_grpc.py

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
# Generated by the gRPC Python protocol compiler plugin. DO NOT EDIT!
12
import grpc
23
from grpc.framework.common import cardinality
34
from grpc.framework.interfaces.face import utilities as face_utilities

bddtests/steps/bootstrap_util.py

+5-7
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,9 @@
3434
from common import configtx_pb2 as common_dot_configtx_pb2
3535
from common import configuration_pb2 as common_dot_configuration_pb2
3636
from common import policies_pb2 as common_dot_policies_pb2
37-
from common import msp_principal_pb2
38-
from msp import mspconfig_pb2
37+
from msp import mspconfig_pb2, msp_principal_pb2, identities_pb2
3938
from peer import configuration_pb2 as peer_dot_configuration_pb2
4039
from orderer import configuration_pb2 as orderer_dot_configuration_pb2
41-
import identities_pb2
4240
import orderer_util
4341

4442
from contexthelper import ContextHelper
@@ -256,7 +254,7 @@ def isInNetwork(self, network):
256254
return False
257255

258256
def getMspPrincipalAsRole(self, mspRoleTypeAsString):
259-
mspRole = msp_principal_pb2.MSPRole(msp_identifier=self.name, Role=msp_principal_pb2.MSPRole.MSPRoleType.Value(mspRoleTypeAsString))
257+
mspRole = msp_principal_pb2.MSPRole(msp_identifier=self.name, role=msp_principal_pb2.MSPRole.MSPRoleType.Value(mspRoleTypeAsString))
260258
mspPrincipal = msp_principal_pb2.MSPPrincipal(
261259
principal_classification=msp_principal_pb2.MSPPrincipal.Classification.Value('ROLE'),
262260
principal=mspRole.SerializeToString())
@@ -378,7 +376,7 @@ def NOutOf(cls, n, policies):
378376
'NOutOf creates a policy which requires N out of the slice of policies to evaluate to true'
379377
return common_dot_policies_pb2.SignaturePolicy(
380378
n_out_of=common_dot_policies_pb2.SignaturePolicy.NOutOf(
381-
N=n,
379+
n=n,
382380
policies=policies,
383381
),
384382
)
@@ -427,7 +425,7 @@ def getNonce(cls):
427425

428426
@classmethod
429427
def addSignatureToSignedConfigItem(cls, configUpdateEnvelope, (entity, cert)):
430-
serializedIdentity = identities_pb2.SerializedIdentity(Mspid=entity.name, IdBytes=crypto.dump_certificate(crypto.FILETYPE_PEM, cert))
428+
serializedIdentity = identities_pb2.SerializedIdentity(mspid=entity.name, id_bytes=crypto.dump_certificate(crypto.FILETYPE_PEM, cert))
431429
sigHeader = common_dot_common_pb2.SignatureHeader(creator=serializedIdentity.SerializeToString(),
432430
nonce=BootstrapHelper.getNonce())
433431
sigHeaderBytes = sigHeader.SerializeToString()
@@ -775,7 +773,7 @@ def createEnvelopeForMsg(directory, nodeAdminTuple, chainId, msg, typeAsString):
775773
org = directory.getOrganization(nodeAdminTuple.organization)
776774
user = directory.getUser(nodeAdminTuple.user)
777775
cert = directory.findCertForNodeAdminTuple(nodeAdminTuple)
778-
serializedIdentity = identities_pb2.SerializedIdentity(Mspid=org.name, IdBytes=crypto.dump_certificate(crypto.FILETYPE_PEM, cert))
776+
serializedIdentity = identities_pb2.SerializedIdentity(mspid=org.name, id_bytes=crypto.dump_certificate(crypto.FILETYPE_PEM, cert))
779777
serializedCreatorCertChain = serializedIdentity.SerializeToString()
780778
nonce = None
781779
payloadSignatureHeader = common_dot_common_pb2.SignatureHeader(

bddtests/steps/endorser_util.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
from peer import transaction_pb2
2424
from peer import proposal_pb2
2525
from peer import peer_pb2_grpc
26-
import identities_pb2
26+
from msp import identities_pb2
2727

2828
from common import common_pb2 as common_dot_common_pb2
2929

@@ -62,7 +62,7 @@ def createInvokeProposalForBDD(context, ccSpec, chainID, signersCert, Mspid, typ
6262

6363
bootstrapHelper = ContextHelper.GetHelper(context=context).getBootrapHelper(chainId=chainID)
6464

65-
serializedIdentity = identities_pb2.SerializedIdentity(Mspid=Mspid, IdBytes=crypto.dump_certificate(crypto.FILETYPE_PEM, signersCert))
65+
serializedIdentity = identities_pb2.SerializedIdentity(mspid=Mspid, id_bytes=crypto.dump_certificate(crypto.FILETYPE_PEM, signersCert))
6666

6767
nonce = bootstrap_util.BootstrapHelper.getNonce()
6868

0 commit comments

Comments
 (0)