fixed import order, and removed unused crypto/rand imports

Author: mkhandker19

cmd/ceremony/crl_test.go

@@ -4,7 +4,6 @@ import (
 	"crypto"
 	"crypto/ecdsa"
 	"crypto/elliptic"
-	"crypto/rand"
 	"crypto/x509"
 	"crypto/x509/pkix"
 	"encoding/asn1"
@@ -48,7 +47,7 @@ func TestGenerateCRLTimeBounds(t *testing.T) {
 // we need to wrap it as we pass a purposefully broken io.Reader to Sign in order
 // to verify that go isn't using it as a source of randomness (we expect this
 // randomness to come from the HSM). If we directly call Sign on the crypto.Signer
-// it would fail, so we wrap it so that we can use a shim nil in the Sign
+// it would fail, so we wrap it so that we can use a shim rand.Reader in the Sign
 // call.
 type wrappedSigner struct{ k crypto.Signer }
 

core/util.go

@@ -1,10 +1,10 @@
 package core
 
 import (
-	"crypto/rand"
 	"context"
 	"crypto"
 	"crypto/ecdsa"
+	"crypto/rand"
 	"crypto/rsa"
 	"crypto/sha256"
 	"crypto/x509"

csr/csr_test.go

@@ -2,7 +2,6 @@ package csr
 
 import (
 	"context"
-	"crypto/rand"
 	"crypto/rsa"
 	"crypto/x509"
 	"crypto/x509/pkix"

goodkey/good_key_test.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"crypto/ecdsa"
 	"crypto/elliptic"
-	"crypto/rand"
 	"crypto/rsa"
 	"fmt"
 	"math/big"

goodkey/sagoodkey/good_key_test.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"crypto/ecdsa"
 	"crypto/elliptic"
-	"crypto/rand"
 	"testing"
 
 	"google.golang.org/grpc"