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

Src route was incorrect and breaks the tests #174

Closed
wants to merge 1 commit into from

Conversation

stefanopassaro
Copy link

No description provided.

Copy link

cla-assistant bot commented Dec 29, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

cla-assistant bot commented Dec 29, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@flovogt flovogt self-assigned this Dec 30, 2024
@flovogt
Copy link

flovogt commented Dec 30, 2024

Hi @stefanopassaro, thanks a lot for creating this PR. Can you explain more detailed in which scenario the tests are breaking on your side, please? I am not able to reproduce the failure. For me the tests run successfully locally and in the demokit.

@stefanopassaro
Copy link
Author

stefanopassaro commented Dec 30, 2024 via email

@flovogt
Copy link

flovogt commented Dec 30, 2024

The code stored in the openui5 repo differs from the one here in the documentation. The reason is that the code in openui5 has to work when developing openui5 code locally and in the UI5 Demo Kit itself, where the UI5 Runtime is located on a different location compared to the scenario where you implement an application standalone. The UI5 Demo Kit rewrites these URLs when the application is downloaded.

If we have missed any scenario / application to work locally or in the UI5 Demo Kit, feel free to report it with a step-by-step guide.

Happy new year and all the best.

@stefanopassaro
Copy link
Author

stefanopassaro commented Dec 30, 2024 via email

@flovogt
Copy link

flovogt commented Dec 30, 2024

Thats strange.

  1. Open https://sdk.openui5.org/entity/sap.m.tutorial.walkthrough/sample/sap.m.tutorial.walkthrough.27
  2. Click on the "Download" button in the upper right corner
  3. Unzip the project
  4. Execute "npm install"
  5. Execute "npm start", your browser will open "http://localhost:8080/test/mockServer.html"
  6. Open "http://localhost:8080/test/testsuite.qunit.html"
  7. Click on "unit/unitTests"
  8. A new browser tab will be opened and the test is executed successfully

The code is exactly the same as written in the tutorial. I can only image that you serve locally the UI5 runtime on a different URL than the application.

@stefanopassaro
Copy link
Author

stefanopassaro commented Jan 1, 2025 via email

@flovogt
Copy link

flovogt commented Jan 2, 2025

The code in the tutorial exactly uses src="../resources/sap/ui/test/starter/runTest.js" which is the right one to use.

@flovogt flovogt closed this Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants