Add support for EnterpriseOwner

[jackmtpt]

Resolves #3210

---

Before the change?

• Adds support for setting EnterpriseOwner in rulesets

After the change?

• The EnterpriseOwner value can be set as a bypass_actor

Pull request checklist

• Schema migrations have been created if needed (example)
• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

• Yes
• No

---

[github-actions]

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

[jackmtpt]

I don't have a local golang dev environment so I'll have to wait for the CI build before I can test this locally.

[deiga]

Thanks for the contribution! I left a few comments

[deiga]

Thanks for the changes! Have you been able to run the tests locally?

[jackmtpt]

Thanks for the changes! Have you been able to run the tests locally?

Sadly not; setting up a golang dev environment in my company has proven to be a giant pain...

[deiga]

Ran tests locally and they pass on Enterprise. But I think you need to split the tests with EnterpriseOwner into separate tests with skipUnlessEnterprise

[jackmtpt]

That should be done now. There was an existing test create_branch_ruleset_with_enterprise_features that I've changed to be gated behind the enterprise setting and put the EnterpriseOwner stuff in that. I've then copied that test to the org rulesets file as well.

I'm still pretty new to golang but I have managed to get the tests passing locally.

[deiga]

LGTM!

[deiga]

Could you refactor this to use ConfigStateChecks instead of Check?

[deiga]

Could you create a new test for this, instead of changing this existing test and where it can run?