Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests with json model_file are broken #228

Closed
KlausVii opened this issue Jan 16, 2024 · 2 comments · Fixed by #229
Closed

Tests with json model_file are broken #228

KlausVii opened this issue Jan 16, 2024 · 2 comments · Fixed by #229
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@KlausVii
Copy link

name: test-1
model_file: ./authorizationModel.json
tuples:
...

Running a test like this fails with

Error: error running tests due to failed to transform due to 2 errors occurred:
       * syntax error at line=1, column=0: extraneous input '{' expecting {'#', 'model', WHITESPACE, NEWLINE}
       * syntax error at line=2, column=2: mismatched input '"schema_version"' expecting {'#', 'model'}

Looks like the format is not being captured and the model is being parsed as the native fga format. This is a regression as this test used to pass for us.

I'm pretty it was this PR that broke it: #215

@rhamzeh
Copy link
Member

rhamzeh commented Jan 16, 2024

Thanks for noticing and reporting this issue @KlausVii!

We'll be fixing this in the next few days. You can use v0.2.1 for now until we release a fix

@rhamzeh rhamzeh added bug Something isn't working good first issue Good for newcomers labels Jan 16, 2024
@rhamzeh
Copy link
Member

rhamzeh commented Jan 16, 2024

Thanks for the report @KlausVii - we just rolled out a fix in v0.2.4, it may take a few minutes for this release to be available for installation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants