ZeroPhone / ZPUI

Official ZeroPhone UI framework, based on pyLCI
http://zpui.rtfd.org/
Apache License 2.0
79 stars 19 forks source link

[Discussion] Automated test results/informations #104

Open j340m3 opened 5 years ago

j340m3 commented 5 years ago

I don't find the README.md very informative. Since there is a travis.yml I suppose you're already using travis-ci. Why not publish the results?

  1. I would like to have a badge showing the build status of the main branch or a link to the build results on travis.
  2. We could also use coveralls or similar tools to display the code coverage.
j340m3 commented 5 years ago

Also check out cookiecutter. There are some ways to get all the automations you may need by running a small script and inserting this repo into it.

CRImier commented 5 years ago

I see. It's just always been weird to me - the "100%: passing" labels the like, I suppose, people want to know there are tests and they are passing? =D Code coverage would be interesting, of course - but I'm sure it's pretty low, most of the apps are not covered by tests and at the moment it's not easy to do so. For now, I only add tests when I find a bug or add code in UI elements, so I wager the coverage isn't that high.

So, we now have Codecov integration - for now, in the devel branch. Will look into labels now - however, I also think this raises a bigger question, is our README good enough? Is there something we could add? Say, screenshots, contribution intro for beginners?

CRImier commented 5 years ago

Added badges in devel

CRImier commented 5 years ago

Important part - we're also using tests for UI "OTA" updates, our updater uses tests as a marker that it's safe to update the UI and rolls back the updates if tests fail =)

j340m3 commented 5 years ago

I see. It's just always been weird to me - the "100%: passing" labels the like, I suppose, people want to know there are tests and they are passing? =D Code coverage would be interesting, of course - but I'm sure it's pretty low, most of the apps are not covered by tests and at the moment it's not easy to do so. For now, I only add tests when I find a bug or add code in UI elements, so I wager the coverage isn't that high.

So, we now have Codecov integration - for now, in the devel branch. Will look into labels now - however, I also think this raises a bigger question, is our README good enough? Is there something we could add? Say, screenshots, contribution intro for beginners?

Important part - we're also using tests for UI "OTA" updates, our updater uses tests as a marker that it's safe to update the UI and rolls back the updates if tests fail =)

Then tests are even more important and describe also the robustness of the OTA process.

Added badges in devel

Nice. Now I have easy access to the build logs. Like it!

CRImier commented 5 years ago