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

CI / tests tracking issue #4866

Open
5 of 17 tasks
szymonlesisz opened this issue Mar 18, 2021 · 4 comments
Open
5 of 17 tasks

CI / tests tracking issue #4866

szymonlesisz opened this issue Mar 18, 2021 · 4 comments
Labels
ci Continuous Integration (CI) related code Code improvements connect Connect API related (ie. fee calculation)

Comments

@szymonlesisz
Copy link
Contributor

szymonlesisz commented Mar 18, 2021

  • github actions OS runners matrix (osx + windows)
  • collect coverage (jest, karma) from separated CI jobs
  • karma: add firefox workspace done using playwright
  • karma: implement in gitlab
  • karma: handle new window opened (popup mode) done in playwright
  • karma: move jasmine matcher (toMatchObject, expect.any in getFeatures fixtures) to karma plugin karma deprecated, moving to playwright
  • karma: serve from https (webusb case) karma depracted testing these things using playwright on sldev
  • karma: add more browser/popup behavior scenarios there is a popup test using playwright
  • karma: add webextension scenarios there is a basic test using playwright
  • karma: add webusb scenarios (mock?) not really possible
  • missing fixtures Connect tests: missing fixtures #4867
  • unit tests for method params, WIP PR: [WIP] tests: add methods params unit tests connect#782 depends on @trezor/connect: improve validateParams function #5297
  • unify tests with https://github.com/trezor/trezor-firmware/tree/master/common/tests/fixtures/cardano
  • move tx cache to trezor-common
  • move fixtures to trezor-common
  • refactor all signTransaction tests to refTx from cache OR mock blockbook to return from cache (or both?)
  • remove legacy tests src/__tests__
@tsusanka tsusanka transferred this issue from trezor/connect Feb 4, 2022
@tsusanka tsusanka added ci Continuous Integration (CI) related connect Connect API related (ie. fee calculation) labels Feb 4, 2022
@tsusanka
Copy link
Contributor

tsusanka commented Feb 4, 2022

@mroz22 @szymonlesisz I think more things are done already? Could you update the check boxes?

@tsusanka tsusanka added the code Code improvements label Feb 4, 2022
@hynek-jina hynek-jina added the LOW label Feb 11, 2022
@szymonlesisz
Copy link
Contributor Author

i think nothing was changed in that area

@hynek-jina hynek-jina removed the LOW label Jun 8, 2022
@mroz22
Copy link
Contributor

mroz22 commented May 24, 2023

checkboxes updated.

@mroz22
Copy link
Contributor

mroz22 commented May 15, 2024

  • collect coverage (jest, karma) from separated CI jobs

this worked for me to merge reports https://stackoverflow.com/questions/62560224/jest-how-to-merge-coverage-reports-from-different-jest-test-runs#answer-63008134

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration (CI) related code Code improvements connect Connect API related (ie. fee calculation)
Projects
Status: No status
Development

No branches or pull requests

4 participants