cancel
Showing results for 
Search instead for 
Did you mean: 

SAP Community Code Challenge - Testing UI5 Apps with wdi5

nicogeburek
Employee
Employee
We have a new code challenge for you as part of the Community Spotlight for the month of June. This time it's about testing UI5 apps with wdi5. Check out the corresponding blog post explaining all the details of the challenge, how to take part, and how to submit your entry.

This thread is the place to share your experience with the challenge and what kind of app you are building. Feel free to ask any questions (technical or non-technical) and leave your comments. Also feel free to post a link to your pull request as your challenge entry.

We are very much looking forward to your ideas and contributions! 😃
26 REPLIES 26

marianzeis
Member

My Entry with a minimal app to pass tests for week1, nothing fancy 🙂

https://github.com/SAP-samples/sap-community-code-challenge-ui5/pull/7

nicogeburek
Employee
Employee

I also submitted my entry for week1, who can guess what kind of app I am planning to build? Any golf fans out here? ️🏌🏼‍

https://github.com/SAP-samples/sap-community-code-challenge-ui5/pull/8

enric101
Member

My pull request is ready after passing all the tests, all in green!

https://github.com/SAP-samples/sap-community-code-challenge-ui5/pull/9

Thanks!

N-Spahija
Member

Here is my Pull Request for my very simple ui5 app. 😃

https://github.com/SAP-samples/sap-community-code-challenge-ui5/pull/17

Pull request updated for week2 code challenge

iklopkov
Employee
Employee

Hi colleagues! Thanks for the great challenge 👍

Please, find my entry here: https://github.com/SAP-samples/sap-community-code-challenge-ui5/pull/16

Hi,

We can use the same pull request right?

Is there a way to reduce the output of npm run ci-tests? Loglevel in config did not seem to do much,

As a wish for the next week I would prefer some test cases which are more Fiori Floor Plan oriented. In week 2 the additional Button to navigation is not really best practice if we start with a list 🙂

Hi there!

Sorry for the late reply.

Yes, you can use the same PR. 
There are two places in the wdio.conf.js where you have to set the logLevel (one global config and one wdi5 specific), make sure you set them both to 'silent'.
Thanks a lot for your suggestions and feedback. You are right, the app we are building here doesn't follow best practices. It was more important to us that it was 'easy to start with' and that people would get familiar with wdi5 in the first place, and I think that was achieved 😉 

Pull request updated for week2 

Updated for week 3... not sure where this thing is going. I hope to see some back & forth navigation testing for duplicate IDs 😉

nair
Employee
Employee

Interesting idea with the todo list, let's see if you'll tick all the boxes 😉

Week 2 submitted

 

Week 3 Summited

ArnaudBuchholz
Employee
Employee

I wouldn't use control.firePress() to simulate a user click. Some controls expect parameters in the firePress internal UI5 method. Why not using .click instead and let UI5 fill these parameters ?

I agree.

Hi @ArnaudBuchholz, good idea, thanks for the suggestion. There is also a press() method provided by wdi5 that calls click() under the hood so you can use it on combination with wdi5 selectors (see wdi5 documentation). I will keep this in mind for future projects 🤙