Merge pull request #1048 from repository-settings/org-rulesets

Author: travi

lib/plugins/rulesets.js

@@ -4,7 +4,7 @@ import Diffable from './diffable.js'
 
 export default class Rulesets extends Diffable {
   async find () {
-    const { data: rulesets } = await this.github.repos.getRepoRulesets(this.repo)
+    const { data: rulesets } = await this.github.repos.getRepoRulesets({ ...this.repo, includes_parents: false })
 
     const expandedRulesetsData = await Promise.all(
       rulesets.map(({ id }) => this.github.repos.getRepoRuleset({ ...this.repo, ruleset_id: id }))

test/integration/features/step_definitions/rulesets-steps.js

@@ -23,11 +23,19 @@ Given('no rulesets are defined for the repository', async function () {
 
 Given('a ruleset exists for the repository', async function () {
   const rulesetSubset = { name: rulesetName }
+  const existingRulesets = [{ id: rulesetId, ...rulesetSubset }]
 
   this.server.use(
-    http.get(`https://api.github.com/repos/${repository.owner.name}/${repository.name}/rulesets`, ({ request }) =>
-      HttpResponse.json([{ id: rulesetId, ...rulesetSubset }])
-    ),
+    http.get(`https://api.github.com/repos/${repository.owner.name}/${repository.name}/rulesets`, ({ request }) => {
+      const url = new URL(request.url)
+
+      if (url.searchParams.get('includes_parents') === 'false') return HttpResponse.json(existingRulesets)
+
+      return HttpResponse.json([
+        ...existingRulesets,
+        ...any.listOf(() => ({ id: any.integer(), ...any.simpleObject() }))
+      ])
+    }),
     http.get(
       `https://api.github.com/repos/${repository.owner.name}/${repository.name}/rulesets/${rulesetId}`,
       ({ request }) => HttpResponse.json({ id: rulesetId, ...rulesetSubset, rules: existingRules })

test/unit/lib/plugins/rulesets.test.js

@@ -44,7 +44,7 @@ describe('rulesets', () => {
       { id: removedRulesetId, name: any.word() }
     ]
     when(github.repos.getRepoRulesets)
-      .calledWith({ owner, repo })
+      .calledWith({ owner, repo, includes_parents: false })
       .mockResolvedValue({
         data: existingRulesets.map(
           ({ rules, conditions, bypass_actors: bypassActors, ...rulesetListProperties }) => rulesetListProperties