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

[7.7 multiple][2.0.181121-rc1] Integration tests #2172

Closed
sschiessl-bcp opened this Issue Nov 15, 2018 · 49 comments

Comments

Projects
None yet
8 participants
@sschiessl-bcp
Copy link
Contributor

commented Nov 15, 2018

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. Copy the below report, choose your OS and browser and add it as a comment
  2. Wait to be approved by the UI team
  3. Tick the checkboxes in your comment while doing the integration test

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 in github and add a link as comment in here. 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:
 - [ ] #422 Added Price Alerts for Exchange
    - Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
 - [ ] #1724 Private login feature for Cloud Login
    - Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
 - [ ] #1696 Add easier way to create a report for debugging
    - Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
 - [ ] #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
    - Test https://staging.bitshares.org/#/market/BTS_OPEN.ETH the "Depost ETH" buttons in the exchange. Are all hints showing properly? Are there two options to "Deposit ETH"?
 - [ ] #2110 'My open orders' contains mixed pairs orders in latest release

Participants

Accepted Issues

- @toi16 #2173 #2174 #2175 #2176 #2177 #2179 #2180 #2181 - @thehapax #2194 - @TechsUsInc #2202 #2203 #2204 - @bieglich #2215 #2214

Payout

@sschiessl-bcp sschiessl-bcp added this to the 181128 milestone Nov 16, 2018

@bieglich

This comment has been minimized.

Copy link

commented Nov 16, 2018

I'd be interested in doing Windows 10 (Chrome, Firefox and Edge) and Android (Chrome)

@thehapax

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Mac Version: 10.13.6

My Browser:

  • Chrome Version: 70.0.3538.77

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:

  • #422 Added Price Alerts for Exchange

    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert
      triggers and if it displays properly
      alerts do not trigger
  • #1724 Private login feature for Cloud Login

    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained . Username is retained.
  • #1696 Add easier way to create a report for debugging

    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips

  • #2110 'My open orders' contains mixed pairs orders in latest release can't reproduce this bug.

Chrome loads very slowly compared to Safari. Not sure why, same computer, same connection

@thehapax

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Mac Version: 10.13.6

My Browser:

  • Safari Version: 12.0.1

My notes in bold, below along with corresponding bug report.

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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained - closing wallet still remembers username but not password.
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release, can't reproduce this bug.

sometimes exchange display is delayed. especially on first visit
Safari loads faster than Chrome!

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 16, 2018

@thehapax go ahead on both

@bieglich please create a comment like shown above for each pair you want to test

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Windows 10, Version: 1803 (OS Build 17134.407)

My Browser:

  • Chrome Version: 70.0.3538.102 (Official Build) (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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Windows 10, Version: 1803 (OS Build 17134.407)

My Browser:

  • Microsoft Edge 42.17134.1.0

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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Windows 10, Version: 1803 (OS Build 17134.407)

My Browser:

  • Firefox Version: 63.0.3 (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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@Bluffet

This comment has been minimized.

Copy link

commented Nov 16, 2018

Is this a joke? I can't get to the log in.

Mac OS 10.9.5 / Firefox

Chrome gets there.

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 16, 2018

Is this a joke? I can't get to the log in.

@Bluffet It is not a joke, it is exactly to find those incompatibilities and gather information, edge for example is probably not working as well. We are in the process of signing up at browserstack, which allows us to do more compatibility testing.

Please do participate and report. I suppose your report would end with "Wallet loads and connects" then...

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 16, 2018

@toi16 Please go ahead with all three

@bieglich

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Windows 10

My Browser:

  • Chrome

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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@bieglich

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Windows 10

My Browser:

  • Firefox

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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@bieglich

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Windows 10

My Browser:

  • Microsoft EdgeHTML 17.17134

Wallet does not load, error message is

SCRIPT1028: SCRIPT1028: Expected identifier, string or number

@bieglich

This comment has been minimized.

Copy link

commented Nov 16, 2018

My OS:

  • Android 6.0.1 SGP611

My Browser:

  • Chrome 70.0.3538.80

General

  • Wallet loads and connects
  • Login via cloud login and local wallet: Login only with Cloud
  • 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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1724 Private login feature for Cloud Login
    • Login while having "Remember Me" disabled. Close the wallet in some way and reopen it. No account information may be retained
  • #1696 Add easier way to create a report for debugging: Screenshot took 4 seconds to load
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 16, 2018

@bieglich please fill out the details of version, then go ahead with all four

@Bluffet

This comment has been minimized.

Copy link

commented Nov 16, 2018

image

Well, Mac OS 10.9.5 / firefox stopped at this point @sschiessl-bcp

Coincides with a user report this morning.

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2173

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2174

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2175

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2176

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Dummy Report Submitted for Win10/Chrome: #2177

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug #2176 updated with Firefox specific problem.

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2179

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2180

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

Bug Logged: #2181

@toi16

This comment has been minimized.

Copy link

commented Nov 16, 2018

@sschiessl-bcp
I have finished testing all three configurations as far as I can.
Please let me know if you would like anything else tested.
I can do Linux testing if required.

@TechsUsInc

This comment has been minimized.

Copy link
Contributor

commented Nov 17, 2018

My OS:

  • Windows OS, Version: 8.1

My Browser:

  • Chrome/Firefox/Opera

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:

  • #422 Added Price Alerts for Exchange
    • Put an alert above and below current market prive of e.g. BTS/CNY or BTS/USD. Wait until the alert triggers and if it displays properly
  • #1696 Add easier way to create a report for debugging
    • Click on "Report" in the lower right bottom. Does the Screenshot and log files present properly? Create a new github issue and do a dummy report, just to try the functionality. Link the link to new issue here
  • #2130 Multiple quick deposit buttons/ borrow buttons- as well as inactive tooltips
  • #2110 'My open orders' contains mixed pairs orders in latest release
@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 18, 2018

@TechsUsInc Please go ahead with Opera. For the other two browsers we have already testers. Could you also do something on mobile?

@TechsUsInc

This comment has been minimized.

Copy link
Contributor

commented Nov 19, 2018

I'm unable to do mobile ATM but working on it. I can also test Brave browser.

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 19, 2018

@thehapax @bieglich How is testing going?

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 19, 2018

I'm unable to do mobile ATM but working on it. I can also test Brave browser.

@TechsUsInc Please stick to Opera for now

@thehapax

This comment has been minimized.

Copy link

commented Nov 19, 2018

@thehapax @bieglich How is testing going?
@sschiessl-bcp I started working on the first part. give me a day, sorry things a little hectic here.

@thehapax

This comment has been minimized.

Copy link

commented Nov 21, 2018

Bug logged #2194

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 21, 2018

@thehapax you have two points in the test above unchecked, does this mean still to be done, or it didnt work?

@thehapax

This comment has been minimized.

Copy link

commented Nov 21, 2018

@thehapax you have two points in the test above unchecked, does this mean still to be done, or it didnt work?

I have now checked off the two items, as I tried them and one of them does not work. If they are not working the comments are now in bold.

I tried loading the staging site repeatedly yesterday and it wasn't coming up. I wasn't sure if it was due to my connection, the chrome browser or the node issues.

@TechsUsInc

This comment has been minimized.

Copy link
Contributor

commented Nov 23, 2018

Bug logged #2202

@TechsUsInc

This comment has been minimized.

Copy link
Contributor

commented Nov 23, 2018

bug logged #2203

@TechsUsInc

This comment has been minimized.

Copy link
Contributor

commented Nov 23, 2018

Bug logged #2204

@bieglich

This comment has been minimized.

Copy link

commented Nov 24, 2018

Multiple buttons still not present with the actual release on Chrome (Windows and Android)

@thehapax

This comment has been minimized.

Copy link

commented Nov 26, 2018

@thehapax you have two points in the test above unchecked, does this mean still to be done, or it didnt work?

@sschiessl-bcp I am now done

@bieglich

This comment has been minimized.

Copy link

commented Nov 28, 2018

Bug #2214 and #2215

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 28, 2018

This testing is completed. Please come back for the next release candidate! Any and all feedback is welcome (especially with respect to the estimates for testing).

@startailcoon @wmbutler
My suggestion for the title: "[7.7 multiple][2.0.181121-rc1] Integration tests". The multiple in the head indicates that the payout distribution is stated in the first post of the issue.

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Nov 28, 2018

@toi16 @thehapax @TechsUsInc @bieglich
Please state your BitShares Account name for payout.

@bieglich

This comment has been minimized.

Copy link

commented Nov 28, 2018

Just the same: bieglich

@toi16

This comment has been minimized.

Copy link

commented Nov 28, 2018

My Bitshares account name is: toi16

@TechsUsInc

This comment has been minimized.

Copy link
Contributor

commented Nov 28, 2018

@sschiessl-bcp
lil-bi.t-of-techs-us

@startailcoon

This comment has been minimized.

Copy link
Contributor

commented Nov 28, 2018

My suggestion for the title: "[7.7 multiple][2.0.181121-rc1] Integration tests"

Sounds good to me. 👍

@startailcoon startailcoon changed the title [2.0.181121-rc1] Integration tests [7.7 multiple][2.0.181121-rc1] Integration tests Nov 28, 2018

@wmbutler

This comment has been minimized.

Copy link
Contributor

commented Dec 4, 2018

Should I just split the payment between @toi16 @thehapax @TechsUsInc @bieglich equally?

@sschiessl-bcp

This comment has been minimized.

Copy link
Contributor Author

commented Dec 4, 2018

The first post contains the distribution in the end, sorry about marking that not clearly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.