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

[2.0.190207-rc2] Integration tests #2436

Closed
sschiessl-bcp opened this issue Feb 1, 2019 · 13 comments
Closed

[2.0.190207-rc2] Integration tests #2436

sschiessl-bcp opened this issue Feb 1, 2019 · 13 comments

Comments

@sschiessl-bcp
Copy link
Contributor

sschiessl-bcp commented Feb 1, 2019

Disclaimer

The UI worker is currently underfunded, a new one is one the way but not posted yet. Please be aware that any testing might end up being volunteer work.

Release notes of the corresponding candidate can be found here:
https://github.com/bitshares/bitshares-ui/releases/tag/2.0.190207-rc2

Looking for Integration testers

This issue is to track integration tests of the mentioned release candidate. Targets are the most popular platforms

  • Mac
  • Windows 7 / 10
  • Linux
  • Android
  • iOS

and for each for the most popular browsers

  • Chrome
  • Firefox
  • Opera
  • Edge
  • Standard Android / iOS browser

Please indicate which of those you are able to test, expectancy is 0.5 hours per platform / browser. Reports are mandatory and must be done for every browser and platform. If you are interested in doing integration tests please do the following:

  1. Choose your OS(s) and browser(s) and indicate with a comment that you are interested
  2. Wait to be approved by the UI team, a new issue will be created by us for your testing
  3. Copy the below template in the new issue that was created for you, for each OS/browser pair
  4. Tick the checkboxes in your comment while doing the integration test for each OS/browser pair

If you find any problems in rendering, error messages or simply weird behavior in the functionality listed in this issue, please report them as a separate issue in github and add a link as comment in the new issue that was created for you. Every issue must include a way how to reproduce them, what the error (please get familiar with how to obtain browser console logs) is and what an expected solution is. Reported issues that are accepted by the UI team will be added as bonus 0.1 hours for your payout.

Reporting

Copy everything in the box in your comment as template for the report, only one browser / OS per report
My OS:
  - Mac/Windows/Linux/Android/iOS, Version: <version>

My Browser:
 - Chrome/Firefox/Opera/Edge/etc., Version: <version>

General 
 - [ ] Wallet loads and connects
 - [ ] Login via cloud login and local wallet
 - [ ] Transfer between accounts
 - [ ] Exchange opens, orders are visible and limit orders can be put
 - [ ] Margin creation and adjustment is possible
 - [ ] Voting is possible (committee, witness, proposals)

Specific components:
 - [ ] #1859 #2408 Spotlight: Barter the hell out of it
 - [ ] #2232 Read through the CDB description, ask if anything is unclear (important!)
 - [ ] Export your account history and check the details (go to Dashboard > Activity)
 - [ ] #2386 Play around with new login option and the focus behavior
 - [ ] #2151 Click around in exchange, buy/sell, personalize and check the browser console log for error, report if any
@TechsUsInc
Copy link
Contributor

My OS:
Windows, Version: 8.1 (Primary)
Windows, Version 10
Linux Ubuntu Version 16.04
Linux Ubuntu Version 18.04
Android 7.1.1 (Nougat)

My Browser:
Opera 56.0.3051.116 (Primary)
Google Version 70.0.3538.110 (Official Build) (64-bit)
Firefox Version 63.0.3(64-bit)
Brave Version 0.25.2 V8 6.9.427.23 OS Release 6.3.9600
Bitorrent Version 7.10.4.44847
General

  • Wallet loads and connects
  • Login via cloud login and local wallet
  • Transfer between accounts
  • Exchange opens, orders are visible and limit orders can be put
  • Margin creation and adjustment is possible
  • Voting is possible (committee, witness, proposals)

Specific components:

@sschiessl-bcp
Copy link
Contributor Author

sschiessl-bcp commented Feb 4, 2019

@TechsUsInc
Please go ahead with

  • Windows, Version 10, Brave
  • Linux Ubuntu Version 18.04, Brave
  • Linux Ubuntu Version 18.04, Version 10, Firefox
  • Android, any browser of your choosing

Your isse #2440

@xiaolujidan
Copy link

@sschiessl-bcp
I know little about it
I can use https://staging.bitshares.org Address of the test.
Or how to get started quickly.
Is there documentation of the testing process

@sschiessl-bcp
Copy link
Contributor Author

sschiessl-bcp commented Feb 12, 2019

The only documentation that exists right now is the header of this issue.

What kind of introduction would you like, what questions do you have?

The idea of integration test is: Go to staging.bitshares.org and use it and follow the bullet points that I have laid out above

@xiaolujidan
Copy link

My OS:
Windows, Version 10

My Browser:
Google Version 69.0.3493.3 (official) (32-bit)
Firefox Version 61.0.2 (64-bit)
General

  • Wallet loads and connects
  • Login via cloud login and local wallet
  • Transfer between accounts
  • Exchange opens, orders are visible and limit orders can be put
  • Margin creation and adjustment is possible
  • Voting is possible (committee, witness, proposals)

Specific components:

@xiaolujidan
Copy link

I create accounts directly through the main network.
image

What should I do next

@xiaolujidan
Copy link

The desktop version error
image

@sschiessl-bcp
Copy link
Contributor Author

sschiessl-bcp commented Feb 13, 2019

@xiaolujidan

Please open a new issue and describe exactly what you did to get to that point

@sschiessl-bcp sschiessl-bcp changed the title [2.0.190207-rc2] Integration tests [2.0.190207-rc3] Integration tests Feb 13, 2019
@sschiessl-bcp sschiessl-bcp changed the title [2.0.190207-rc3] Integration tests [2.0.190207-rc2] Integration tests Feb 13, 2019
@startailcoon
Copy link
Contributor

startailcoon commented Feb 14, 2019

My OS:

  • Windows 10, Version: 1803, 17134.523

My Browser:

  • Firefox., Version: 65.0 (64-bit)

General

  • Wallet loads and connects (see additional issues below)
  • Login via cloud login and local wallet (see additional issues below)
  • Transfer between accounts
  • Exchange opens, orders are visible and limit orders can be put and canceled
  • Margin creation and adjustment is possible
  • Voting is possible (committee, witness, proposals), adding and removing Proxy possible

Specific components:

Addition Reports

Startup

  • Failed required checks
Warning: Failed value type: The value `isRequired` is marked as required in `Select`, but its value is `undefined`. checkPropTypes.js:19
Warning: Failed defaultValue type: The defaultValue `isRequired` is marked as required in `Select`, but its value is `undefined`. checkPropTypes.js:19 
  • Unreachable code on AccountNameInputStyleGuide.jsx:181
  • Unreachable code on Barter.jsx:840
  • Unreachable code on Barter.jsx:904

Login

  • Trying to login without a username will highlight password field as error regardless
  • Login cloud with Rember me false, will not do anything. Username still filled in next time.

Exchange

  • Introduction still does not scrollthe page if highlighted item is outside the scope of the window. A blank background is visible with the highlighted area and text.

Showcase: Barter

  • Unable to click amount on asset select to use all
  • Perhaps a note that Escrow Memo can't be read by Peer 2?
  • Possible to set Peer 1 + Escrow as the same account.
  • Possible to set Peer 2 + Escrow as the same account.

CDB description

  • Confusing when Previous is after Next in button order.
  • Penalty shoulldn't be defined specifically, as it's defined by MSSR.
    • Ex: ... reduce your debt, with a 10% penalty to ensure the...`
  • Note that TCR can't be below MCR
  • Missing parantese on "CDP"
  • Note that there is a maximum settlement amount per maintenance period
  • Explain maintenance period briefly

@sschiessl-bcp
Copy link
Contributor Author

  • Unreachable code on AccountNameInputStyleGuide.jsx:181
  • Unreachable code on Barter.jsx:840
  • Unreachable code on Barter.jsx:904

Can you please specify (screenshot or permalink) what lines those are?

@sschiessl-bcp
Copy link
Contributor Author

  • Trying to login without a username will highlight password field as error regardless

image

Can you please specify how to reproduce?

@sschiessl-bcp
Copy link
Contributor Author

Closed, release is published

@sschiessl-bcp
Copy link
Contributor Author

Thanks to everyone for participating. Payouts should have been received now.

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

No branches or pull requests

4 participants