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

api-test: fix file-search tests #9693

Merged
merged 1 commit into from
Jul 4, 2021
Merged

Conversation

vince-fugnitto
Copy link
Member

What it does

Fixes: #9692

The commit updates the file-search API tests following the monaco upgrade which introduced breaking changes to the way we deal with QuickOpenItems.

How to test

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto [email protected]

The commit updates the `file-search` API tests following the monaco upgrade
which introduced breaking changes to the way we deal with `QuickOpenItems`.

Signed-off-by: vince-fugnitto <[email protected]>
@vince-fugnitto vince-fugnitto added file search issues related to the file search test issues related to unit and api tests labels Jul 2, 2021
@vince-fugnitto vince-fugnitto self-assigned this Jul 2, 2021
@vince-fugnitto
Copy link
Member Author

@EstherPerelman there are issues with the api-tests related to monaco functionality, please be sure to take a look at them and resolve the problems:

These failures were also present on your commit to master.

@RomanNikitenko
Copy link
Contributor

The problems related to file-search tests were fixed within the current PR, so I think we can merge it.

There are still some problems related to upgrade Monaco - I think it should be fixed within a separate PR

tests_failure

@vince-fugnitto vince-fugnitto merged commit 3bd43c9 into master Jul 4, 2021
@vince-fugnitto vince-fugnitto deleted the vf/file-search-api branch July 4, 2021 17:20
@github-actions github-actions bot added this to the 1.16.0 milestone Jul 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
file search issues related to the file search test issues related to unit and api tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test failure in examples/api-tests/src/file-search.spec.js
2 participants