Test that leaf bundles contain expected data (#238)

* return the dir of the log

* add TODO

* verify that entries in the log are correct

# Conflicts:
#	internal/scti/handlers_test.go

# Conflicts:
#	internal/scti/handlers_test.go

* add a function to parse Entries

* display a more useful message

# Conflicts:
#	internal/scti/handlers_test.go

* Address comments

Author: phbnf

internal/scti/handlers.go

@@ -392,6 +392,8 @@ func deadlineTime(opts *HandlerOptions) time.Time {
 
 // verifyAddChain is used by add-chain and add-pre-chain. It does the checks that the supplied
 // cert is of the correct type and chains to a trusted root.
+// TODO(phbnf): add tests
+// TODO(phbnf): move to chain_validation.go
 func verifyAddChain(log *log, req rfc6962.AddChainRequest, expectingPrecert bool) ([]*x509.Certificate, error) {
 	// We already checked that the chain is not empty so can move on to verification
 	validPath, err := validateChain(req.Chain, log.chainValidationOpts)
@@ -451,7 +453,8 @@ func marshalAndWriteAddChainResponse(sct *rfc6962.SignedCertificateTimestamp, w
 
 // entryFromChain generates an Entry from a chain and timestamp.
 // copied from certificate-transparency-go/serialization.go
-// TODO(phboneff): move in a different file maybe?
+// TODO(phboneff): move to ct.go
+// TODO(phboneff): add tests
 func entryFromChain(chain []*x509.Certificate, isPrecert bool, timestamp uint64) (*ctonly.Entry, error) {
 	leaf := ctonly.Entry{
 		IsPrecert: isPrecert,