bazelbuild / starlark

Starlark Language
Apache License 2.0
2.44k stars 163 forks source link

Fix tests #259

Closed stepancheg closed 1 year ago

stepancheg commented 1 year ago

Seems two things changes:

stepancheg commented 1 year ago

Something is not right. This PR fixes the test on my machine. There must be something different, but I don't understand what and where.

stepancheg commented 1 year ago

Figured out, the problem was with rust, not with go (confused by go in test name). Starlark-rust no longer allows d[1, 2] = 3.

Also, I figured we don't actually check error messages in Rust, because error message in Rust includes code snippet, added a todo.