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

User not notified after successful edit/update 'Branding' details #1506

Closed
saralavanip opened this issue Sep 8, 2016 · 6 comments
Closed

Comments

@saralavanip
Copy link
Contributor

saralavanip commented Sep 8, 2016

Steps

  1. Visit https://nightly2.apinf.io
  2. Login as admin
  3. Go to Branding section
  4. Update/edit Branding details (eg. site title or site slogan)
  5. Click on 'Update' button

Findings

  • First click on 'Update' button doesn't respond though the details are updated. It remains highlighted until next click.
  • User not notified about successful update.

Browser

  1. Mozilla Firefox 48.0
    (Mozilla Firefox for Ubuntu Canonical – 1.0 )
  2. Google Chrome 52.0.2743.116

Operating System

  1. Ubuntu 16.0 LTS

Screenshot

edit_update_branding

@saralavanip
Copy link
Contributor Author

@bajiat , verified on https://nightly.apinf.io

Findings:

  • Update/Edit 'Branding' details is successful but user not notified.

@philippeluickx
Copy link
Contributor

@mauriciovieira If you are going to write unit tests for this, check out https://www.meteortesting.com/

@mauriciovieira
Copy link
Contributor

After strugling with the recommended meteor way of doing unit tests. I decided to try the way Rocket.Chat does tests, using chimp in a end2end way.

Related: xolvio/chimp#433

@mauriciovieira
Copy link
Contributor

I believe that we can test server methods using a setup similar to the recommended on meteor guides, such as https://github.com/gagarinjs/meteor-mocha

@mauriciovieira
Copy link
Contributor

@philippeluickx this link points to a book that's being written at least since February and never gets released http://quality.xolv.io/. Nevertheless, the test runner that is being used by Rocket.Chat, chimp.js, was made by this xolvio. :-) Thanks for the tip.

@mauriciovieira
Copy link
Contributor

closed by #2617

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants