🧹 improve user resource handling in Microsoft 365

[chris-rock]

This PR improves the microsoft.user resource.

Simpler init functions

User can easily created by their object id and principal name now:

cnspec> microsoft.user(userPrincipalName: "ben@example.onmicrosoft.com")
microsoft.user: microsoft.user id="a692f8ff-42e3-4156-8ac2-523379d9b664" displayName="Ben Rockwood" userPrincipalName="ben@example.onmicrosoft.com"

cnspec> microsoft.user(id: "a692f8ff-42e3-4156-8ac2-523379d9b664")
microsoft.user: microsoft.user id="a692f8ff-42e3-4156-8ac2-523379d9b664" displayName="Ben Rockwood" userPrincipalName="ben@example.onmicrosoft.com"

Retrieve owner for application registration

We added a new owners field to microsoft.application which allows you to see all owners of your application:

cnspec> microsoft.application(name: "chris-testing") { name owners }
microsoft.application: {
  name: "chris-testing"
  owners: [
    0: microsoft.user id="efd9e3d3-eb7a-46f6-a560-09a894e00fdd" displayName="Christoph Hartmann" userPrincipalName="chris@example.onmicrosoft.com"
  ]
}

Group members

For groups, I it is essential to see all members. This can easily done now via:

cnspec> microsoft.groups { name members }

[github-actions]

Test Results

   28 files  + 24    370 suites  +167   1m 22s ⏱️ -16s
3 097 tests +143  3 096 ✅ +146  1 💤 ±0  0 ❌  - 3 
3 097 runs  +144  3 096 ✅ +148  1 💤 ±0  0 ❌  - 4 

Results for commit 7548494. ± Comparison against base commit 8bab9f0.

This pull request removes 1 and adds 144 tests. Note that renamed tests count towards both.

TestMain

go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_default_playbook
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_blocks_and_errors
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_multiple_plays
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_roles
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_serial
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestPlaybookDecoding/load_playbook_with_vars
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestTaskDecoding
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestTaskDecoding/load_task_with_blocks
go.mondoo.com/cnquery/v11/providers/ansible/play ‑ TestTaskDecoding/load_task_with_vars
…

♻️ This comment has been updated with latest results.

[preslavgerchev]

nit: it doesnt seem we use this index to fetch users, can we remove it if thats the case?

[chris-rock]

Yes, lets remove it