diff --git a/.github/workflows/run_unit_test.sh b/.github/workflows/run_unit_test.sh index 050c074f..63fbb99f 100755 --- a/.github/workflows/run_unit_test.sh +++ b/.github/workflows/run_unit_test.sh @@ -6,6 +6,7 @@ set -e start_dir=$(pwd) temp_dir=$(mktemp -d) cd $temp_dir +echo "git clone -b $(cat /tmp/git_head_ref) https://github.com/$(cat /tmp/git_repository).git" git clone -b $(cat /tmp/git_head_ref) "https://github.com/$(cat /tmp/git_repository).git" cd ipld-eth-server diff --git a/pkg/graphql/client.go b/pkg/graphql/client.go index 096b7a06..f7650e29 100644 --- a/pkg/graphql/client.go +++ b/pkg/graphql/client.go @@ -4,6 +4,7 @@ import ( "context" "encoding/json" "fmt" + "strings" "github.com/ethereum/go-ethereum/common" "github.com/ethereum/go-ethereum/common/hexutil" @@ -91,10 +92,16 @@ func NewClient(endpoint string) *Client { return &Client{client: client} } -func (c *Client) GetLogs(ctx context.Context, hash common.Hash, address *common.Address) ([]LogResponse, error) { +func (c *Client) GetLogs(ctx context.Context, hash common.Hash, addresses []common.Address) ([]LogResponse, error) { params := fmt.Sprintf(`blockHash: "%s"`, hash.String()) - if address != nil { - params += fmt.Sprintf(`, contract: "%s"`, address.String()) + + if addresses != nil { + addressStrings := make([]string, len(addresses)) + for i, address := range addresses { + addressStrings[i] = fmt.Sprintf(`"%s"`, address.String()) + } + + params += fmt.Sprintf(`, addresses: [%s]`, strings.Join(addressStrings, ",")) } getLogsQuery := fmt.Sprintf(`query{ diff --git a/pkg/graphql/graphql_test.go b/pkg/graphql/graphql_test.go index aae047ce..258d6afe 100644 --- a/pkg/graphql/graphql_test.go +++ b/pkg/graphql/graphql_test.go @@ -174,7 +174,7 @@ var _ = Describe("GraphQL", func() { Describe("eth_getLogs", func() { It("Retrieves logs that matches the provided blockHash and contract address", func() { - logs, err := client.GetLogs(ctx, blockHash, &contractAddress) + logs, err := client.GetLogs(ctx, blockHash, []common.Address{contractAddress}) Expect(err).ToNot(HaveOccurred()) expectedLogs := []graphql.LogResponse{ @@ -191,7 +191,7 @@ var _ = Describe("GraphQL", func() { }) It("Retrieves logs for the failed receipt status that matches the provided blockHash and another contract address", func() { - logs, err := client.GetLogs(ctx, blockHash, &test_helpers.AnotherAddress2) + logs, err := client.GetLogs(ctx, blockHash, []common.Address{test_helpers.AnotherAddress2}) Expect(err).ToNot(HaveOccurred()) expectedLogs := []graphql.LogResponse{ @@ -207,6 +207,30 @@ var _ = Describe("GraphQL", func() { Expect(logs).To(Equal(expectedLogs)) }) + It("Retrieves logs that matches the provided blockHash and multiple contract addresses", func() { + logs, err := client.GetLogs(ctx, blockHash, []common.Address{contractAddress, test_helpers.AnotherAddress2}) + Expect(err).ToNot(HaveOccurred()) + + expectedLogs := []graphql.LogResponse{ + { + Topics: test_helpers.MockLog1.Topics, + Data: hexutil.Bytes(test_helpers.MockLog1.Data), + Transaction: graphql.TransactionResponse{Hash: test_helpers.MockTransactions[0].Hash()}, + ReceiptCID: test_helpers.Rct1CID.String(), + Status: int32(test_helpers.MockReceipts[0].Status), + }, + { + Topics: test_helpers.MockLog6.Topics, + Data: hexutil.Bytes(test_helpers.MockLog6.Data), + Transaction: graphql.TransactionResponse{Hash: test_helpers.MockTransactions[3].Hash()}, + ReceiptCID: test_helpers.Rct4CID.String(), + Status: int32(test_helpers.MockReceipts[3].Status), + }, + } + + Expect(logs).To(Equal(expectedLogs)) + }) + It("Retrieves all the logs for the receipt that matches the provided blockHash and nil contract address", func() { logs, err := client.GetLogs(ctx, blockHash, nil) Expect(err).ToNot(HaveOccurred()) @@ -214,7 +238,7 @@ var _ = Describe("GraphQL", func() { }) It("Retrieves logs with random hash", func() { - logs, err := client.GetLogs(ctx, randomHash, &contractAddress) + logs, err := client.GetLogs(ctx, randomHash, []common.Address{contractAddress}) Expect(err).ToNot(HaveOccurred()) Expect(len(logs)).To(Equal(0)) })