Enhance Test

- Observer, with test case - block collector, with new test case - nit fixing reslove #149 Signed-off-by: SamYuan1990 <[email protected]>
Hyperledger-TWGC · Apr 10, 2021 · dd8e508 · dd8e508
1 parent 0d2922a
commit dd8e508
Show file tree

Hide file tree

Showing 7 changed files with 355 additions and 37 deletions.
diff --git a/e2e/mock/peer.go b/e2e/mock/peer.go
@@ -14,6 +14,7 @@ type Peer struct {
 	GrpcServer     *grpc.Server
 	BlkSize, txCnt uint64
 	TxC            chan struct{}
+	CtlCh          chan struct{}
 }
 
 func (p *Peer) ProcessProposal(context.Context, *peer.SignedProposal) (*peer.ProposalResponse, error) {
@@ -30,16 +31,25 @@ func (p *Peer) DeliverFiltered(srv peer.Deliver_DeliverFilteredServer) error {
 		panic("expect no recv error")
 	}
 	srv.Send(&peer.DeliverResponse{})
-
-	for range p.TxC {
-		p.txCnt++
-		if p.txCnt%p.BlkSize == 0 {
-			srv.Send(&peer.DeliverResponse{Type: &peer.DeliverResponse_FilteredBlock{
-				FilteredBlock: &peer.FilteredBlock{FilteredTransactions: make([]*peer.FilteredTransaction, p.BlkSize)}}})
+	txc := p.TxC
+	for {
+		select {
+		case <-txc:
+			p.txCnt++
+			if p.txCnt%p.BlkSize == 0 {
+				srv.Send(&peer.DeliverResponse{Type: &peer.DeliverResponse_FilteredBlock{
+					FilteredBlock: &peer.FilteredBlock{
+						Number:               p.txCnt / p.BlkSize,
+						FilteredTransactions: make([]*peer.FilteredTransaction, p.BlkSize)}}})
+			}
+		case <-p.CtlCh:
+			if txc != nil {
+				txc = nil
+			} else {
+				txc = p.TxC
+			}
 		}
 	}
-
-	return nil
 }
 
 func (p *Peer) DeliverWithPrivateData(peer.Deliver_DeliverWithPrivateDataServer) error {
@@ -64,15 +74,25 @@ func NewPeer(TxC chan struct{}, credentials credentials.TransportCredentials) (*
 	if err != nil {
 		return nil, err
 	}
+	CtlCh := make(chan struct{})
 	instance := &Peer{
 		Listener:   lis,
 		GrpcServer: grpc.NewServer(grpc.Creds(credentials)),
 		BlkSize:    10,
 		TxC:        TxC,
+		CtlCh:      CtlCh,
 	}
 
 	peer.RegisterEndorserServer(instance.GrpcServer, instance)
 	peer.RegisterDeliverServer(instance.GrpcServer, instance)
 
 	return instance, nil
 }
+
+func (p *Peer) Suspend() {
+	p.CtlCh <- struct{}{}
+}
+
+func (p *Peer) Resume() {
+	p.CtlCh <- struct{}{}
+}
diff --git a/e2e/mock/server.go b/e2e/mock/server.go
@@ -9,12 +9,17 @@ type Server struct {
 	orderer *Orderer
 }
 
+// this is the channel size for mock server, peer and orderer
+// when use or send tx to mock server/peer/orderer
+// try not over this size to avoid hang up or over size
+const MOCK_TX_SIZE = 1000
+
 func NewServer(peerN int, credentials credentials.TransportCredentials) (*Server, error) {
 	var txCs []chan struct{}
 	var peers []*Peer
 
 	for i := 0; i < peerN; i++ {
-		txC := make(chan struct{}, 1000)
+		txC := make(chan struct{}, MOCK_TX_SIZE)
 		peer, err := NewPeer(txC, credentials)
 		if err != nil {
 			return nil, err

diff --git a/pkg/infra/benchmark_test.go b/pkg/infra/benchmark_test.go
@@ -61,12 +61,22 @@ func benchmarkSyncCollector(concurrency int, b *testing.B) {
 	instance, _ := NewBlockCollector(concurrency, concurrency)
 	processed := make(chan struct{}, b.N)
 	defer close(processed)
+	now := time.Now()
+	finishCh := make(chan struct{})
 	b.ReportAllocs()
 	b.ResetTimer()
 	for i := 0; i < concurrency; i++ {
 		go func() {
 			for j := 0; j < b.N; j++ {
-				if instance.Commit(uint64(j)) {
+				ft := make([]*peer.FilteredTransaction, 1)
+				fb := &peer.FilteredBlock{
+					Number:               uint64(j),
+					FilteredTransactions: ft,
+				}
+				block := &peer.DeliverResponse_FilteredBlock{
+					FilteredBlock: fb,
+				}
+				if instance.Commit(block, finishCh, now) {
 					processed <- struct{}{}
 				}
 			}

diff --git a/pkg/infra/block_collector.go b/pkg/infra/block_collector.go
@@ -77,24 +77,27 @@ func (bc *BlockCollector) Start(
 //
 // Commit commits a block to collector. It returns true iff the number of peers on which
 // this block has been committed has satisfied thresholdP.
-func (bc *BlockCollector) Commit(block uint64) (committed bool) {
+func (bc *BlockCollector) Commit(block *peer.DeliverResponse_FilteredBlock, finishCh chan struct{}, now time.Time) (committed bool) {
 	bc.Lock()
 	defer bc.Unlock()
 
-	cnt := bc.registry[block] // cnt is default to 0 when key does not exist
+	cnt := bc.registry[block.FilteredBlock.Number] // cnt is default to 0 when key does not exist
 	cnt++
 
 	// newly committed block just hits threshold
 	if cnt == bc.thresholdP {
 		committed = true
+		duration := time.Since(now)
+		bc.totalTx += len(block.FilteredBlock.FilteredTransactions)
+		fmt.Printf("tx: %d, duration: %+v, tps: %f\n", bc.totalTx, duration, float64(bc.totalTx)/duration.Seconds())
 	}
 
 	if cnt == bc.totalP {
 		// committed on all peers, remove from registry
-		delete(bc.registry, block)
+		delete(bc.registry, block.FilteredBlock.Number)
 	} else {
 		// upsert back to registry
-		bc.registry[block] = cnt
+		bc.registry[block.FilteredBlock.Number] = cnt
 	}
 
 	return

diff --git a/pkg/infra/block_collector_test.go b/pkg/infra/block_collector_test.go
@@ -7,13 +7,14 @@ import (
 	"time"
 
 	"github.com/hyperledger/fabric-protos-go/peer"
-
 	. "github.com/onsi/ginkgo"
 	. "github.com/onsi/gomega"
 )
 
 var _ = Describe("BlockCollector", func() {
 
+	now := time.Now()
+
 	Context("Async Commit", func() {
 		It("should work with threshold 1 and observer 1", func() {
 			instance, err := infra.NewBlockCollector(1, 1)
@@ -134,38 +135,122 @@ var _ = Describe("BlockCollector", func() {
 			wg.Wait()
 			Eventually(done).Should(BeClosed())
 		})
+
+		It("Should supports threshold 3 and observer 5 as parallel committers", func() {
+			instance, err := infra.NewBlockCollector(3, 5)
+			Expect(err).NotTo(HaveOccurred())
+
+			block := make(chan *peer.FilteredBlock)
+			done := make(chan struct{})
+			go instance.Start(context.Background(), block, done, 10, time.Now(), false)
+
+			var wg sync.WaitGroup
+			wg.Add(3)
+			for i := 0; i < 3; i++ {
+				go func() {
+					defer wg.Done()
+					for j := 0; j < 10; j++ {
+						block <- &peer.FilteredBlock{
+							Number:               uint64(j),
+							FilteredTransactions: make([]*peer.FilteredTransaction, 1)}
+					}
+				}()
+			}
+			wg.Wait()
+			Eventually(done).Should(BeClosed())
+		})
+
+		It("Should supports threshold 5 and observer 5 as parallel committers", func() {
+			instance, err := infra.NewBlockCollector(5, 5)
+			Expect(err).NotTo(HaveOccurred())
+
+			block := make(chan *peer.FilteredBlock)
+			done := make(chan struct{})
+			go instance.Start(context.Background(), block, done, 10, time.Now(), false)
+
+			var wg sync.WaitGroup
+			wg.Add(5)
+			for i := 0; i < 5; i++ {
+				go func() {
+					defer wg.Done()
+					for j := 0; j < 10; j++ {
+						block <- &peer.FilteredBlock{
+							Number:               uint64(j),
+							FilteredTransactions: make([]*peer.FilteredTransaction, 1)}
+					}
+				}()
+			}
+			wg.Wait()
+			Eventually(done).Should(BeClosed())
+		})
 	})
 
 	Context("Sync Commit", func() {
 		It("should work with threshold 1 and observer 1", func() {
+			finishCh := make(chan struct{})
 			instance, err := infra.NewBlockCollector(1, 1)
 			Expect(err).NotTo(HaveOccurred())
-			Expect(instance.Commit(1)).To(BeTrue())
+			ft := make([]*peer.FilteredTransaction, 1)
+			fb := &peer.FilteredBlock{
+				Number:               uint64(1),
+				FilteredTransactions: ft,
+			}
+			block := &peer.DeliverResponse_FilteredBlock{
+				FilteredBlock: fb,
+			}
+			Expect(instance.Commit(block, finishCh, now)).To(BeTrue())
 		})
 
 		It("should work with threshold 1 and observer 2", func() {
+			finishCh := make(chan struct{})
 			instance, err := infra.NewBlockCollector(1, 2)
 			Expect(err).NotTo(HaveOccurred())
-			Expect(instance.Commit(1)).To(BeTrue())
-			Expect(instance.Commit(1)).To(BeFalse())
+			ft := make([]*peer.FilteredTransaction, 1)
+			fb := &peer.FilteredBlock{
+				Number:               uint64(1),
+				FilteredTransactions: ft,
+			}
+			block := &peer.DeliverResponse_FilteredBlock{
+				FilteredBlock: fb,
+			}
+			Expect(instance.Commit(block, finishCh, now)).To(BeTrue())
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
 		})
 
 		It("should work with threshold 4 and observer 4", func() {
+			finishCh := make(chan struct{})
 			instance, err := infra.NewBlockCollector(4, 4)
 			Expect(err).NotTo(HaveOccurred())
-			Expect(instance.Commit(1)).To(BeFalse())
-			Expect(instance.Commit(1)).To(BeFalse())
-			Expect(instance.Commit(1)).To(BeFalse())
-			Expect(instance.Commit(1)).To(BeTrue())
+			ft := make([]*peer.FilteredTransaction, 1)
+			fb := &peer.FilteredBlock{
+				Number:               uint64(1),
+				FilteredTransactions: ft,
+			}
+			block := &peer.DeliverResponse_FilteredBlock{
+				FilteredBlock: fb,
+			}
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
+			Expect(instance.Commit(block, finishCh, now)).To(BeTrue())
 		})
 
 		It("should work with threshold 2 and observer 4", func() {
+			finishCh := make(chan struct{})
 			instance, err := infra.NewBlockCollector(2, 4)
 			Expect(err).NotTo(HaveOccurred())
-			Expect(instance.Commit(1)).To(BeFalse())
-			Expect(instance.Commit(1)).To(BeTrue())
-			Expect(instance.Commit(1)).To(BeFalse())
-			Expect(instance.Commit(1)).To(BeFalse())
+			ft := make([]*peer.FilteredTransaction, 1)
+			fb := &peer.FilteredBlock{
+				Number:               uint64(1),
+				FilteredTransactions: ft,
+			}
+			block := &peer.DeliverResponse_FilteredBlock{
+				FilteredBlock: fb,
+			}
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
+			Expect(instance.Commit(block, finishCh, now)).To(BeTrue())
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
+			Expect(instance.Commit(block, finishCh, now)).To(BeFalse())
 		})
 
 		It("should return err when threshold is greater than total", func() {
@@ -174,21 +259,56 @@ var _ = Describe("BlockCollector", func() {
 			Expect(instance).Should(BeNil())
 		})
 
-		It("Should supports parallel committers", func() {
-			instance, _ := infra.NewBlockCollector(100, 100)
-			signal := make(chan struct{})
+		It("Should work with threshold 3 and observer 5 in parallel", func() {
+			instance, _ := infra.NewBlockCollector(3, 5)
+			finishCh := make(chan struct{})
 			var wg sync.WaitGroup
-			wg.Add(100)
-			for i := 0; i < 100; i++ {
+			wg.Add(3)
+			for i := 0; i < 3; i++ {
+				go func() {
+					defer wg.Done()
+					ft := make([]*peer.FilteredTransaction, 1)
+					fb := &peer.FilteredBlock{
+						Number:               uint64(1),
+						FilteredTransactions: ft,
+					}
+					block := &peer.DeliverResponse_FilteredBlock{
+						FilteredBlock: fb,
+					}
+					if instance.Commit(block, finishCh, now) {
+						close(finishCh)
+					}
+				}()
+			}
+			wg.Wait()
+			<-finishCh
+			Expect(finishCh).To(BeClosed())
+		})
+
+		It("Should work with threshold 5 and observer 5 in parallel", func() {
+			instance, _ := infra.NewBlockCollector(5, 5)
+			finishCh := make(chan struct{})
+			var wg sync.WaitGroup
+			wg.Add(5)
+			for i := 0; i < 5; i++ {
 				go func() {
 					defer wg.Done()
-					if instance.Commit(1) {
-						close(signal)
+					ft := make([]*peer.FilteredTransaction, 1)
+					fb := &peer.FilteredBlock{
+						Number:               uint64(1),
+						FilteredTransactions: ft,
+					}
+					block := &peer.DeliverResponse_FilteredBlock{
+						FilteredBlock: fb,
+					}
+					if instance.Commit(block, finishCh, now) {
+						close(finishCh)
 					}
 				}()
 			}
 			wg.Wait()
-			Expect(signal).To(BeClosed())
+			<-finishCh
+			Expect(finishCh).To(BeClosed())
 		})
 	})
 })
diff --git a/pkg/infra/observer.go b/pkg/infra/observer.go
@@ -62,7 +62,6 @@ func CreateObserver(ctx context.Context, channel string, node Node, crypto *Cryp
 
 func (o *Observer) Start(errorCh chan error, blockCh chan<- *peer.FilteredBlock, now time.Time) {
 	o.logger.Debugf("start observer for orderer %s", o.Address)
-	n := 0
 	for {
 		r, err := o.d.Recv()
 		if err != nil {
@@ -78,7 +77,5 @@ func (o *Observer) Start(errorCh chan error, blockCh chan<- *peer.FilteredBlock,
 		o.logger.Debugf("receivedTime %8.2fs\tBlock %6d\tTx %6d\t Address %s\n", time.Since(now).Seconds(), fb.FilteredBlock.Number, len(fb.FilteredBlock.FilteredTransactions), o.Address)
 
 		blockCh <- fb.FilteredBlock
-
-		n = n + len(fb.FilteredBlock.FilteredTransactions)
 	}
 }