Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Add tests for 'WorkspaceFolderUpdater' #1008

Merged
merged 1 commit into from
Mar 3, 2021

Conversation

RomanNikitenko
Copy link
Member

@RomanNikitenko RomanNikitenko commented Mar 1, 2021

Signed-off-by: Roman Nikitenko [email protected]

What does this PR do?

Add tests for 'WorkspaceFolderUpdater'

Screenshot/screencast of this PR

What issues does this PR fix or reference?

It's related to multi-root workspaces support logic eclipse-che/che#17212

How to test this PR?

The tests should be happy.

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Happy Path Channel

HAPPY_PATH_CHANNEL=stable

@codecov
Copy link

codecov bot commented Mar 1, 2021

Codecov Report

Merging #1008 (2504b74) into master (9a76d18) will increase coverage by 1.58%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1008      +/-   ##
==========================================
+ Coverage   63.13%   64.72%   +1.58%     
==========================================
  Files          68       68              
  Lines        2466     2466              
  Branches      391      391              
==========================================
+ Hits         1557     1596      +39     
+ Misses        898      860      -38     
+ Partials       11       10       -1     
Impacted Files Coverage Δ
...s/workspace-plugin/src/workspace-folder-updater.ts 100.00% <100.00%> (+100.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a76d18...2504b74. Read the comment docs.

@che-bot
Copy link
Contributor

che-bot commented Mar 1, 2021

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia quay.io/crw_pr/che-theia:1008
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1008

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@RomanNikitenko RomanNikitenko force-pushed the test-workspace-folder-updater branch from 7edc264 to 2504b74 Compare March 3, 2021 08:02
@che-bot
Copy link
Contributor

che-bot commented Mar 3, 2021

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia quay.io/crw_pr/che-theia:1008
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1008

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@RomanNikitenko RomanNikitenko marked this pull request as ready for review March 3, 2021 10:19
Copy link
Member

@azatsarynnyy azatsarynnyy left a comment

Choose a reason for hiding this comment

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

👍

@RomanNikitenko RomanNikitenko merged commit bbf4dc6 into master Mar 3, 2021
@RomanNikitenko RomanNikitenko deleted the test-workspace-folder-updater branch March 3, 2021 12:32
@che-bot che-bot added this to the 7.28 milestone Mar 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants