Skip to content

Commit c33a055

Browse files
committed
Merge fixes
1 parent 43c564b commit c33a055

3 files changed

Lines changed: 10 additions & 10 deletions

File tree

meta/src/meta/templates/parser.jl.template

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ end
345345

346346
function parse_fragment(input::String)
347347
lexer = Lexer(input)
348-
parser = ParserState(lexer.tokens)
348+
parser = ParserState(lexer.tokens, input)
349349
result = parse_fragment(parser)
350350
# Check for unconsumed tokens (except EOF)
351351
if parser.pos <= length(parser.tokens)
@@ -354,7 +354,7 @@ function parse_fragment(input::String)
354354
throw(ParseError("Unexpected token at end of input: $remaining_token"))
355355
end
356356
end
357-
return result
357+
return result, parser.provenance
358358
end
359359

360360
function parse(input::String)

sdks/go/test/parser_test.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ func TestBasicParsing(t *testing.T) {
3535
// TestParseTransaction tests the ParseTransaction entry point.
3636
func TestParseTransaction(t *testing.T) {
3737
input := `(transaction (epoch (writes) (reads)))`
38-
result, err := lqp.ParseTransaction(input)
38+
result, _, err := lqp.ParseTransaction(input)
3939
if err != nil {
4040
t.Fatalf("Failed to parse transaction: %v", err)
4141
}
@@ -50,7 +50,7 @@ func TestParseTransaction(t *testing.T) {
5050
// TestParseFragment tests the ParseFragment entry point.
5151
func TestParseFragment(t *testing.T) {
5252
input := `(fragment :test_frag (def :my_rel ([x::INT] (relatom :my_rel x))))`
53-
result, err := lqp.ParseFragment(input)
53+
result, _, err := lqp.ParseFragment(input)
5454
if err != nil {
5555
t.Fatalf("Failed to parse fragment: %v", err)
5656
}
@@ -62,11 +62,11 @@ func TestParseFragment(t *testing.T) {
6262
// TestParseDelegatesToParseTransaction verifies Parse and ParseTransaction return equal results.
6363
func TestParseDelegatesToParseTransaction(t *testing.T) {
6464
input := `(transaction (epoch (writes) (reads)))`
65-
r1, err := lqp.Parse(input)
65+
r1, _, err := lqp.Parse(input)
6666
if err != nil {
6767
t.Fatalf("Parse failed: %v", err)
6868
}
69-
r2, err := lqp.ParseTransaction(input)
69+
r2, _, err := lqp.ParseTransaction(input)
7070
if err != nil {
7171
t.Fatalf("ParseTransaction failed: %v", err)
7272
}
@@ -77,15 +77,15 @@ func TestParseDelegatesToParseTransaction(t *testing.T) {
7777

7878
// TestParseFragmentRejectsTransaction verifies ParseFragment rejects transaction input.
7979
func TestParseFragmentRejectsTransaction(t *testing.T) {
80-
_, err := lqp.ParseFragment(`(transaction (epoch (writes) (reads)))`)
80+
_, _, err := lqp.ParseFragment(`(transaction (epoch (writes) (reads)))`)
8181
if err == nil {
8282
t.Error("ParseFragment should reject transaction input")
8383
}
8484
}
8585

8686
// TestParseTransactionRejectsFragment verifies ParseTransaction rejects fragment input.
8787
func TestParseTransactionRejectsFragment(t *testing.T) {
88-
_, err := lqp.ParseTransaction(`(fragment :f (def :r ([x::INT] (relatom :r x))))`)
88+
_, _, err := lqp.ParseTransaction(`(fragment :f (def :r ([x::INT] (relatom :r x)))`)
8989
if err == nil {
9090
t.Error("ParseTransaction should reject fragment input")
9191
}

sdks/python/tests/test_parser.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,13 @@ def test_parse_lqp(snapshot: Snapshot, input_file):
2828

2929

3030
def test_parse_transaction():
31-
result = parse_transaction(_SIMPLE_TXN)
31+
result, _provenance = parse_transaction(_SIMPLE_TXN)
3232
assert isinstance(result, transactions_pb2.Transaction)
3333
assert len(result.epochs) == 1
3434

3535

3636
def test_parse_fragment():
37-
result = parse_fragment(_SIMPLE_FRAGMENT)
37+
result, _provenance = parse_fragment(_SIMPLE_FRAGMENT)
3838
assert isinstance(result, fragments_pb2.Fragment)
3939

4040

0 commit comments

Comments
 (0)