Open behnazh-w opened 3 weeks ago
I wonder which test cases do you have in mind for checking the JSON outputs. What I think would work best is to convert the rest of integration test cases (leftover from #764 ). After that, we can go through all of them and pick specific test cases that we wan to check for JSON outputs, then I can update those test cases correspondingly.
I wonder which test cases do you have in mind for checking the JSON outputs. What I think would work best is to convert the rest of integration test cases (leftover from #764 ). After that, we can go through all of them and pick specific test cases that we wan to check for JSON outputs, then I can update those test cases correspondingly.
Yes, sounds good. After converting the rest of the test cases, checking the JSON output for one of the test cases should be enough.
With this PR merged, we have started using Datalog policies in the e2e tests to check the
analyze
command results. But we no longer check the JSON outputs. We need to add a test for the JSON outputs to catch regressions.