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

feat: add ability to disable suggestions on people-picker #3048

Merged
merged 5 commits into from
Feb 26, 2024

Merge branch 'main' into feat/disable-suggestions

43ebacf
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

feat: add ability to disable suggestions on people-picker #3048

Merge branch 'main' into feat/disable-suggestions
43ebacf
Select commit
Loading
Failed to load commit list.
GitHub Actions / Test Results succeeded Feb 26, 2024 in 0s

45 passed, 0 failed and 0 skipped

Tests passed successfully

✅ junit.xml

45 tests were completed in 6s with 45 passed, 0 failed and 0 skipped.

Test suite Passed Failed Skipped Time
Chromium_playwright_/packages/mgt-components/src/components/mgt-file-list/mgt-file-upload/mgt-file-upload.tests.ts 2✅ 78ms
Chromium_playwright_/packages/mgt-components/src/components/mgt-person-card/getMgtPersonCardScopes.tests.ts 7✅ 5ms
Chromium_playwright_/packages/mgt-components/src/components/mgt-person/mgt-person.tests.ts 12✅ 5s
Chromium_playwright_/packages/mgt-components/src/components/mgt-theme-toggle/mgt-theme-toggle.tests.ts 6✅ 580ms
Chromium_playwright_/packages/mgt-components/src/components/mgt-todo/mgt-todo.tests.ts 1✅ 2ms
Chromium_playwright_/packages/mgt-components/src/components/sub-components/mgt-spinner/mgt-spinner.tests.ts 1✅ 39ms
Chromium_playwright_/packages/mgt-element/src/providers/IProvider.tests.ts 4✅ 9ms
Chromium_playwright_/packages/mgt-element/src/utils/equals.tests.ts 2✅ 7ms
Chromium_playwright_/packages/mgt-element/src/utils/EventDispatcher.tests.ts 2✅ 40ms
Chromium_playwright_/packages/mgt-element/src/utils/GraphHelpers.tests.ts 8✅ 21ms

✅ Chromium_playwright_/packages/mgt-components/src/components/mgt-file-list/mgt-file-upload/mgt-file-upload.tests.ts

mgt-file-upload - tests
  ✅ should render
  ✅ has required scopes

✅ Chromium_playwright_/packages/mgt-components/src/components/mgt-person-card/getMgtPersonCardScopes.tests.ts

getMgtPersonCardScopes() tests
  ✅ should have a minimal permission set
  ✅ should have not have Sites.Read.All if files is configured off
  ✅ should have not have Mail scopes if mail is configured off
  ✅ should have People.Read but not People.Read.All if showWorksWith is false
  ✅ should have not have User.Read.All if profile and organization are false
  ✅ should have not have Chat.ReadWrite if isSendMessageVisible is false
  ✅ should have not have Chat.ReadWrite if useContactApis is false

✅ Chromium_playwright_/packages/mgt-components/src/components/mgt-person/mgt-person.tests.ts

mgt-person - tests
  ✅ should render
  ✅ should pop up a flyout on click
  ✅ should render with initials when given name and surname are supplied
  ✅ should render with initials when given name and surname are null
  ✅ should render with first initial when only given name is supplied
  ✅ should render with first initial when only given name is populated and surname is an empty string
  ✅ should render with last initial when only surname is supplied
  ✅ should render with last initial when only surname is populated and given name is an empty string
  ✅ should render with one initial when only displayName of one word is supplied
  ✅ should render with two initial when only displayName of more than two words is supplied
mgt-person - localization
  ✅ should render with updated photo for text
  ✅ should render with updated email address text

✅ Chromium_playwright_/packages/mgt-components/src/components/mgt-theme-toggle/mgt-theme-toggle.tests.ts

mgt-theme-toggle - media behavior tests
  ✅ should render as checked whe color scheme is dark
  ✅ should emit darkmodechanged true on inital render when the color scheme is dark
  ✅ should render as unchecked when color scheme is light
mgt-theme-toggle - tests
  ✅ should render
  ✅ should emit darkmodechanged with the current 'checked' state on click
  ✅ should have a checked switch if mode is dark

✅ Chromium_playwright_/packages/mgt-components/src/components/mgt-todo/mgt-todo.tests.ts

mgt-todo - tests
  ✅ has required scopes

✅ Chromium_playwright_/packages/mgt-components/src/components/sub-components/mgt-spinner/mgt-spinner.tests.ts

mgt-spinner tests
  ✅ should render

✅ Chromium_playwright_/packages/mgt-element/src/providers/IProvider.tests.ts

IProvider.needsAdditionalScopes tests
  ✅ should provide an empty array when one scope is already present
  ✅ should provide an empty array when one scope is already present ignoring case of scopes in provider
  ✅ should provide an empty array when one scope is already present ignoring case of scopes in provider
  ✅ should provide an the first element in the passed array where there is no overlap

✅ Chromium_playwright_/packages/mgt-element/src/utils/equals.tests.ts

objectEquals
  ✅ should return true
  ✅ should return false

✅ Chromium_playwright_/packages/mgt-element/src/utils/EventDispatcher.tests.ts

EventDispatcher tests
  ✅ should add and remove event handlers
  ✅ should not throw when remove is called with an unregistered handler

✅ Chromium_playwright_/packages/mgt-element/src/utils/GraphHelpers.tests.ts

GraphHelpers - prepScopes
  ✅ should return an empty array when incremental consent is disabled
  ✅ should return an array of AuthenticationHandlerOptions when incremental consent is enabled with only the first scope in the list
GraphHelpers - chainMiddleware
  ✅ should return the first middleware when only one is passed
  ✅ should return undefined when the middleware array is empty
  ✅ should now throw when the middleware array is undefined
GraphHelpers - validateBaseUrl
  ✅ should return as a valid Url
  ✅ should return undefeined for invalid Url
  ✅ should return undefined for when supplied a %p which is not a well formed url