Skip to content

Mocha Test Explorer#3024

Closed
garrytrinder wants to merge 1 commit intopnp:mainfrom
garrytrinder:mocha-test-explorer
Closed

Mocha Test Explorer#3024
garrytrinder wants to merge 1 commit intopnp:mainfrom
garrytrinder:mocha-test-explorer

Conversation

@garrytrinder
Copy link
Copy Markdown
Member

@garrytrinder garrytrinder commented Feb 6, 2022

Closes #2817

Easiest way to test this is to create a new Codespaces/Remote Container from my branch as I have included the required extensions into the devcontainer.json file so they will be auto installed.

However you can manually install the Test Explorer UI, Mocha Test Explorer and GitLens extensions, you will also need to do an npm install to pull in the source-map-support package.

Run npm watch and open the Tests explorer, if no tests appear, use the refresh option after you have run npm watch. If you add a new test you will need to use refresh again to make it appear, from there you can run and debug the test whilst npm watch is running in the background.

Change log

  • Adds mocha options file
  • Adds source-map-support package
  • Adds Mocha Test Explorer UI extension
  • Adds Test Explorer UI extension
  • Adds GitLens extension
  • Fixes duplicate test name in Auth
  • Fixes duplicate test names in msalCachePlugin
  • Fixes duplicate test names in Cli
  • Fixes duplicate test names in app-role-delete
  • Fixes duplicate test name in approleassignment-add
  • Fixes duplicate test names in o365group-list
  • Fixes duplicate test names in user-get
  • Fixes duplicate test names in user-hibp
  • Fixes duplicate test name in completion-pwsh-setup
  • Fixes duplicate test names in solution-reference-add
  • Fixes duplicate test name in project-upgrade
  • Fixes duplicate test names in app-add
  • Fixes duplicate test names in app-get
  • Fixes duplicate test names in hubsite-data-get
  • Fixes duplicate test names in list-add
  • Fixes duplicate test names in list-set
  • Fixes duplicate test names in list-view-remove
  • Fixes duplicate test names in listitem-list
  • Fixes duplicate test names in orgassetlibrary-remove
  • Fixes duplicate test names in page-template-list
  • Fixes duplicate test names in propertybag-get
  • Fixes duplicate test names in site-add
  • Fixes duplicate test names in site-commsite-enable
  • Fixes duplicate test names in site-inplacerecordsmanagement
  • Fixes duplicate test names in tenant-settings-set
  • Fixes duplicate test names in web-set
  • Fixes duplicate test names in conversationmember-add
  • Fixes duplicate test names in conversationmember-list
  • Fixes duplicate test names in team-clone
  • Fixes duplicate test names in task-set
  • Fixes duplicate test names in message-add
  • Fixes duplicate test names in message-like-set
  • Fixes duplicate test names in user-get
  • Fixes duplicate test names in user-list
  • Fixes duplicate test names in yammer-search
  • Fixes duplicate test names in Utils

@waldekmastykarz waldekmastykarz force-pushed the main branch 2 times, most recently from 2d81d43 to c75d5fc Compare February 7, 2022 11:48
Copy link
Copy Markdown
Member

@waldekmastykarz waldekmastykarz left a comment

Choose a reason for hiding this comment

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

Hey @garrytrinder, it seems like you pulled in some additional commits. Could you please double check it before we proceed?

@waldekmastykarz waldekmastykarz marked this pull request as draft February 9, 2022 12:44
@garrytrinder garrytrinder marked this pull request as ready for review February 13, 2022 10:57
@garrytrinder
Copy link
Copy Markdown
Member Author

Hey @garrytrinder, it seems like you pulled in some additional commits. Could you please double check it before we proceed?

Good catch, not sure what has happened there, but should be all fixed now 👍

@waldekmastykarz
Copy link
Copy Markdown
Member

Brilliant! We'll go through it asap

Copy link
Copy Markdown
Member

@waldekmastykarz waldekmastykarz left a comment

Choose a reason for hiding this comment

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

Works like a charm. Awesome spring cleaning 😄

@waldekmastykarz
Copy link
Copy Markdown
Member

Merged manually. Thank you! 👏

@garrytrinder garrytrinder deleted the mocha-test-explorer branch July 10, 2022 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mocha Test Explorer

2 participants