[FAB-7236] peer chaincode list installed requires chid

This CR removes the requirement of supplying the channel parameter when
listing installed chaincodes via the peer CLI, which is not a
channel-scoped operation.

Change-Id: I038a5814d18c0f0b2b50a5f79e263d4298a7db5a
Signed-off-by: Will Lahti <wtlahti@us.ibm.com>

Author: wlahti

Diff for: peer/chaincode/list.go

@@ -48,7 +48,7 @@ func listCmd(cf *ChaincodeCmdFactory) *cobra.Command {
 }
 
 func getChaincodes(cmd *cobra.Command, cf *ChaincodeCmdFactory) error {
-	if channelID == "" {
+	if getInstantiatedChaincodes && channelID == "" {
 		return errors.New("The required parameter 'channelID' is empty. Rerun the command with -C flag")
 	}
 	var err error

Diff for: peer/chaincode/list_test.go

@@ -59,11 +59,6 @@ func TestChaincodeListCmd(t *testing.T) {
 
 	args := []string{"--installed"}
 	installedChaincodesCmd.SetArgs(args)
-	err = installedChaincodesCmd.Execute()
-	assert.Error(t, err, "Run chaincode list cmd to get installed chaincodes should fail if invoked without -C flag")
-
-	args = []string{"--installed", "-C", "mychannel"}
-	installedChaincodesCmd.SetArgs(args)
 	if err := installedChaincodesCmd.Execute(); err != nil {
 		t.Errorf("Run chaincode list cmd to get installed chaincodes error:%v", err)
 	}