[FAB-14324] remove mock-only nil check

The tests pass without it.

Change-Id: I909cc42105569ca942815e72e19379d11822a0ba
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Author: sykesm

core/peer/peer.go

@@ -131,25 +131,20 @@ func (c *chain) Apply(configtx *common.ConfigEnvelope) error {
 		return err
 	}
 
-	// If the chainSupport is being mocked, this field will be nil
-	if c.bundleSource != nil {
-		bundle, err := channelconfig.NewBundle(c.cs.ConfigtxValidator().ChainID(), configtx.Config)
-		if err != nil {
-			return err
-		}
-
-		channelconfig.LogSanityChecks(bundle)
-
-		err = c.bundleSource.ValidateNew(bundle)
-		if err != nil {
-			return err
-		}
-
-		capabilitiesSupportedOrPanic(bundle)
+	bundle, err := channelconfig.NewBundle(c.cs.ConfigtxValidator().ChainID(), configtx.Config)
+	if err != nil {
+		return err
+	}
 
-		c.bundleSource.Update(bundle)
+	channelconfig.LogSanityChecks(bundle)
+	err = c.bundleSource.ValidateNew(bundle)
+	if err != nil {
+		return err
 	}
 
+	capabilitiesSupportedOrPanic(bundle)
+
+	c.bundleSource.Update(bundle)
 	return nil
 }