ui5-typescript-helloworld icon indicating copy to clipboard operation
ui5-typescript-helloworld copied to clipboard

feat(testing): add wdi5 e2e scope

Open vobu opened this issue 2 years ago • 5 comments

adds wdi5 for e2e tests:

  • in TS flavor
  • with minimal scope (config + test file)

vobu avatar Sep 01 '23 08:09 vobu

move to "draft" status, b/c should only be merge once wdi5 2.0 is out

vobu avatar Sep 01 '23 09:09 vobu

This "helloworld" repository started out with a different focus and indeed I have had also difficulties to convey this towards @petermuessig before. I think it works best like this:

Initially this repo was meant to describe the TypeScript setup as such. Which additional files are needed, what do they need to contain, to have TS code translated to JS in a UI5 app. Yes, there was also a minimal UI5 app contained, but only to have some code to compile, not to show how TS app content should look. Consequently, the documentation focused on a few steps that created those setup-relevant files like the tsconfig and the babelrc.

Now with the testing content added by Peter and more so with the wdi5 content, it turns more and more into a sample how all kinds of aspects should look inside a UI5 app written in TypeScript, the focus on how the setup works under the hood is gone.

And that's ok. We need such a sample, after all, plus it's getting less and less important to understand how the build pipeline works, with ui5-tooling-transpile wrapping the transpilation and making configuration unnecessary and also with Easy UI5 setting everything up for most users.

The only thing is that the once central part of this repo - https://github.com/SAP-samples/ui5-typescript-helloworld/blob/main/step-by-step.md - gets more and more anachronistic with this transition. This page was meant to get to successful TS transpilation from scratch, not anything beyond that. It still says "Note that the scope of this tutorial is the TypeScript setup of a project, not the application code itself.". So we'll need to see how to describe that part and whether this is the place where to also describe how all the parts of the app, incl. testing should be described.

TL;DR: yep, all is good, go ahead, but I'll need to check where and how to describe best how the transpilation process works under the hood.

akudev avatar Sep 04 '23 12:09 akudev

TL;DR: yep, all is good, go ahead

here we are :)

vobu avatar Sep 18 '23 09:09 vobu

Apart from few minor comments, the code itself is fine for me.

akudev avatar Oct 10 '23 10:10 akudev

CLA assistant check
All committers have signed the CLA.

cla-assistant[bot] avatar Jun 18 '24 04:06 cla-assistant[bot]