Skip to content
This repository has been archived by the owner on Jun 10, 2022. It is now read-only.

Prevent unauthorized users from running certain tests #466

Merged
merged 4 commits into from
Nov 7, 2019

Conversation

stevenvergenz
Copy link
Contributor

No description provided.

@@ -66,4 +68,15 @@ export abstract class Test {
}
});
}

public checkPermission(user: User) {
if (this.modsOnly) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure all tests can run in the testbed.

Copy link
Contributor

@eanders-ms eanders-ms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, with the condition that all tests must be able to run in the testbed.

@stevenvergenz stevenvergenz force-pushed the stvergen/protected-tests branch from 2c2c8d7 to b677683 Compare November 7, 2019 17:54
@stevenvergenz stevenvergenz merged commit caa7c3a into red Nov 7, 2019
@stevenvergenz stevenvergenz deleted the stvergen/protected-tests branch November 7, 2019 17:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants