Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 398
  • Merge Requests 53
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !666

Open
Opened 1 week ago by Olivia Madrid@omadrid
Report abuse

Pro settings e2e

Tests that require changes from Emi to pass:

  • it should update the title and headline (browser tab title currently reads 'Home | Minds')
  • it should allow the user to set light theme for posts (activities not always showing up on feed)
  • it should allow the user to set dark theme for posts (ditto)

Closes #2273

Edited 1 week ago by Olivia Madrid
Request to merge pro-settings-e2e-2273 into master
The source branch is 1 commit behind the target branch
Open in Web IDE
Pipeline #100185173 passed with warnings for b06cbc0b on pro-settings-e2e-2273
              Requires 3 more approvals from Devs, Deployers, and QA.
              Marcelo Rivera
              Marcelo Rivera
              Juan Manuel Solaro
              Juan Manuel Solaro
              Emiliano Balbuena
              Emiliano Balbuena
              Mark Harding
              Mark Harding
              Rami Albatal
              Rami Albatal
              Ready to be merged automatically. Ask someone with write access to this repository to merge this request

              Closes #2273

              • Discussion 12
              • Commits 7
              • Pipelines 4
              • Changes 5
              • Olivia Madrid @omadrid added Platform::Browser Priority::2 - Normal Product::Pro Squad::Blue Type::E2E scoped labels 1 week ago

                added scoped labels

              • Olivia Madrid @omadrid added 1 commit 1 week ago

                added 1 commit

                • 0a140e7f - (e2e): WIP pro settings tests

                Compare with previous version

              • Emiliano Balbuena @edgebal mentioned in issue engine#1165 (closed) 1 week ago

                mentioned in issue engine#1165 (closed)

              • Olivia Madrid @omadrid added 11 commits 1 week ago

                added 11 commits

                • 0a140e7f...559f9503 - 7 commits from branch master
                • 787abfb4 - Merge branch 'master' of gitlab.com:minds/front into pro-settings-e2e-2273
                • c9f80d64 - (e2e): test cleanup
                • 4f7db1f1 - Merge branch 'master' of gitlab.com:minds/front into pro-settings-e2e-2273
                • fd5beff2 - (e2e): fix footer contains error, skip post theme tests for now

                Compare with previous version

                Toggle commit list
              • Olivia Madrid @omadrid changed the description 1 week ago

                changed the description

              • Olivia Madrid @omadrid unmarked as a Work In Progress 1 week ago

                unmarked as a Work In Progress

              • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 6 days ago

                approved this merge request

              • Mark Harding @markeharding added MR::Requires Changes scoped label 1 day ago

                added scoped label

              • Mark Harding
                Mark Harding @markeharding · 1 day ago
                Owner

                @omadrid getting merge conflicts here

              • Guy Thouret
                Guy Thouret @gthouret · 16 hours ago
                Developer

                @omadrid I'm unable to test and give approval as credentials for minds_pro_cypress_tests are required and the password is set to empty string in the config.

                Edited by Guy Thouret 16 hours ago
              • Olivia Madrid @omadrid added 23 commits 7 hours ago

                added 23 commits

                • fd5beff2...6ad11c19 - 22 commits from branch master
                • b06cbc0b - Merge branch 'master' of gitlab.com:minds/front into pro-settings-e2e-2273

                Compare with previous version

              • Olivia Madrid
                Olivia Madrid @omadrid · 7 hours ago
                Developer

                @gthouret Ah, yes. To run the tests I've been entering:

                cypress/e2e.sh -h http://localhost:8080 -pu minds_pro_cypress_tests -pp p@ssw0rd -p p@ssw0rd -u minds_pro_cypress_tests

                (e.g. If you've created test user minds_pro_cypress_tests with password p@ssw0rd)

                p.s. I'm not sure how it's supposed to work for regular testing outside of localhost, but that's what I do to get the tests to run. Maybe @benhayward.ben has the official answer?

              • Guy Thouret
                Guy Thouret @gthouret · 3 hours ago
                Developer

                Only took an hour to get my stack up and running :face_palm_tone1:

                I'm getting the 'invalid password' error when trying to create a user with that password!!!

              • Guy Thouret
                Guy Thouret @gthouret · 3 hours ago
                Developer

                Screenshot_2019-12-03_at_11.28.50

              • Guy Thouret
                Guy Thouret @gthouret · 3 hours ago
                Developer

                p@ssw0rd has 8 characters, includes upper case, no spaces and even has a special character :thinking:

              • Guy Thouret
                Guy Thouret @gthouret · 3 hours ago
                Developer

                Pretty sure this has been widely reported by users so I'll dig in and see what's up.

              • Guy Thouret
                Guy Thouret @gthouret · 2 hours ago
                Developer

                It actually doesn't have upper case. Why do we enforce that, that's silly.

              • Guy Thouret
                Guy Thouret @gthouret · 2 hours ago
                Developer

                Yep that was it.

              • Guy Thouret
                Guy Thouret @gthouret · 2 hours ago
                Developer

                User needs to be pro for this to work...

                UPDATE entities SET value=9999999999 WHERE column1='pro_expires' AND key='<user_guid>'
              • Guy Thouret
                Guy Thouret @gthouret · 2 hours ago
                Developer

                Also, e2e tests run about 100x faster if you enable aot for the build. I should really enable aot by default on our builds.

              • Guy Thouret
                Guy Thouret @gthouret · 2 hours ago
                Developer

                Completed in 97.87s - failed across the board.

                Does this require a specific backend? I'm running against latest master backend.

              Please register or sign in to reply
              0 Assignees
              None
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              6
              Labels
              MR::Requires Changes Platform::Browser Priority::2 - Normal Product::Pro Squad::Blue Type::E2E
              Lock merge request
              Unlocked
              11
              11 participants
              user avatar
              Marcelo Rivera
              user avatar
              Juan Manuel Solaro
              user avatar
              Emiliano Balbuena
              user avatar
              Mark Harding
              user avatar
              Rami Albatal
              user avatar
              Xander Miller
              user avatar
              Ben Hayward
              Reference: minds/front!666