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

test/cases: add test case for rand.intn #5207

Merged
merged 1 commit into from Oct 5, 2022

Conversation

srenatus
Copy link
Contributor

@srenatus srenatus commented Oct 5, 2022

It looks like we didn't have any before. While it's inherently random, we still know that for the same inputs, within a single evaluation run, we should only get one output. So let's assert that.

It looks like we didn't have any before. While it's inherently random, we still
know that for the same inputs, within a single evaluation run, we should only
get one output. So let's assert that.

Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
@srenatus srenatus marked this pull request as ready for review October 5, 2022 10:07
@anderseknert anderseknert merged commit 20982c3 into open-policy-agent:main Oct 5, 2022
@srenatus srenatus deleted the sr/tests/rand.intn branch October 5, 2022 10:54
byronic pushed a commit to byronic/opa that referenced this pull request Oct 17, 2022
It looks like we didn't have any before. While it's inherently random, we still
know that for the same inputs, within a single evaluation run, we should only
get one output. So let's assert that.

Signed-off-by: Stephan Renatus <stephan.renatus@gmail.com>
Signed-off-by: Byron Lagrone <byron.lagrone@seqster.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants