che icon indicating copy to clipboard operation
che copied to clipboard

Implement CheCode Editor POC test

Open ScrewTSW opened this issue 3 years ago • 2 comments

What does this PR do?

  • Prunes the existing codebase
  • Integrates vscode-extension-tester into the codebase
  • Adds a POC implementation for workspace readiness check in JavaSpringBoot devfile

Screenshot/screencast of this PR

What issues does this PR fix or reference?

https://issues.redhat.com/browse/CRW-3107

How to test this PR?

  • Set environment variables:
    • TS_SELENIUM_BASE_URL
    • TS_SELENIUM_USERNAME
    • TS_SELENIUM_OCP_USERNAME
    • TS_SELENIUM_PASSWORD
    • TS_SELENIUM_OCP_PASSWORD
    • TS_OCP_LOGIN_PROVIDER_TITLE
    • TS_SELENIUM_VALUE_OPENSHIFT_OAUTH="true"
  • Run initDefaultValues.sh
  • Execute tests npm run test-java-springboot

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.

ScrewTSW avatar Sep 05 '22 08:09 ScrewTSW

@ScrewTSW : could you, please, write about how to run the test, and give an example of test run?

dmytro-ndp avatar Sep 05 '22 09:09 dmytro-ndp

@ScrewTSW : commented code could be removed having commit history.

It could also be useful to have list of tests which needs to be rewritten to be run in VS Code Editor, e.g. tests/e2e/tests/e2e/GitPublishBranch.spec.ts

dmytro-ndp avatar Sep 20 '22 10:09 dmytro-ndp

@ScrewTSW: is there a draft of e2e/tests/e2e_happy_path/vscode/HappyPath.spec.ts ready to run?

dmytro-ndp avatar Nov 15 '22 19:11 dmytro-ndp

@ScrewTSW: is there a draft of e2e/tests/e2e_happy_path/vscode/HappyPath.spec.ts ready to run?

No, not yet. This is just POC refactor, so that it's ready for additional implementations

ScrewTSW avatar Nov 15 '22 19:11 ScrewTSW

I see.

Actually, what has already been done looks commonly good. Waiting on final result.

dmytro-ndp avatar Nov 15 '22 20:11 dmytro-ndp

wkspc abbreviation in filenames tests/e2e/mocha-wkspc-creation-and-ls-theia.json looks strange

If it's shot name for workspace, then there is wide spread abbreviation existed: ws.

dmytro-ndp avatar Nov 22 '22 13:11 dmytro-ndp