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

Add tests related to backward compatibility for single-root workspaces #1075

Conversation

RomanNikitenko
Copy link
Member

@RomanNikitenko RomanNikitenko commented Apr 13, 2021

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

What does this PR do?

Add tests related to backward compatibility for single-root workspaces

Screenshot/screencast of this PR

What issues does this PR fix or reference?

related to eclipse-che/che#19551

How to test this PR?

the tests should pass successfully

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=next

@codecov
Copy link

codecov bot commented Apr 13, 2021

Codecov Report

Merging #1075 (79a2329) into master (eeb4526) will increase coverage by 1.88%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1075      +/-   ##
==========================================
+ Coverage   29.45%   31.33%   +1.88%     
==========================================
  Files         277      277              
  Lines        9336     9362      +26     
  Branches     1380     1392      +12     
==========================================
+ Hits         2750     2934     +184     
+ Misses       6487     6331     -156     
+ Partials       99       97       -2     
Impacted Files Coverage Δ
...theia-about/src/browser/about-che-theia-dialog.tsx 0.00% <ø> (ø)
...ovisioner/src/node/git-configuration-controller.ts 0.00% <0.00%> (ø)
...che-theia-remote-api/src/common/devfile-service.ts 0.00% <ø> (ø)
...er/src/node/che-server-certificate-service-impl.ts 0.00% <0.00%> (ø)
...-impl-k8s/src/node/k8s-certificate-service-impl.ts 0.00% <0.00%> (ø)
...ces/src/node/che-theia-preferences-synchronizer.ts 0.00% <0.00%> (ø)
...e-theia-workspace/src/node/che-workspace-server.ts 0.00% <0.00%> (ø)
generator/src/init-sources.ts 98.17% <ø> (ø)
...ainers-plugin/src/containers-tree-data-provider.ts 0.00% <0.00%> (ø)
plugins/task-plugin/src/task/che-task-runner.ts 0.00% <ø> (ø)
... and 13 more

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 566529c...79a2329. Read the comment docs.

@che-bot

This comment has been minimized.

@RomanNikitenko
Copy link
Member Author

[crw-ci-test]

@che-bot
Copy link
Contributor

che-bot commented Apr 13, 2021

❌ E2E Happy path tests failed ❗

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:1075
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1075

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
Copy link
Member Author

[crw-ci-test]

@RomanNikitenko
Copy link
Member Author

[crw-ci-test --rebuild]

@RomanNikitenko
Copy link
Member Author

RomanNikitenko commented Apr 13, 2021

I guess I can merge the PR - there are successful jobs:

  • Happy Path / happy-path (alpine, next) (pull_request)
  • Happy Path / happy-path (ubi8, next) (pull_request)

or should I wait for crw-ci-e2e-happy-path-tests
I'm not able to restart the last one due to eclipse-che/che#19571

@benoitf @azatsarynnyy ?

@dmytro-ndp
Copy link
Contributor

[crw-ci-test --rebuild]

@benoitf
Copy link
Contributor

benoitf commented Apr 13, 2021

@RomanNikitenko yes, one job to rule them all :-) if one happy path test is ok, you win !

@RomanNikitenko RomanNikitenko merged commit 1333e65 into eclipse-che:master Apr 13, 2021
@RomanNikitenko RomanNikitenko deleted the unit-tests-workspace-projects-manager branch April 13, 2021 12:10
@che-bot che-bot added this to the 7.29 milestone Apr 13, 2021
@che-bot
Copy link
Contributor

che-bot commented Apr 13, 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:1075
che-theia-endpoint-runtime-binary quay.io/crw_pr/che-theia-endpoint-runtime-binary:1075

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.

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