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

Integrate E2E testing #172

Open
larkox opened this issue Oct 5, 2020 · 6 comments
Open

Integrate E2E testing #172

larkox opened this issue Oct 5, 2020 · 6 comments
Labels
Difficulty/2:Medium Medium ticket Hacktoberfest Help Wanted Community help wanted Tech/ReactJS Type/Enhancement New feature or improvement of existing feature Up For Grabs Ready for help from the community. Removed when someone volunteers

Comments

@larkox
Copy link
Contributor

larkox commented Oct 5, 2020

Following a similar design as in the Incident Response Plugin integrate Cypress E2E testing for this plugin.

At least one simple test case has to be added to make sure the system works.

@larkox larkox added Help Wanted Community help wanted Up For Grabs Ready for help from the community. Removed when someone volunteers Tech/ReactJS Hacktoberfest Difficulty/2:Medium Medium ticket Type/Enhancement New feature or improvement of existing feature labels Oct 5, 2020
@patidar-jaishree
Copy link

@larkox Hi, I am new to open source. I would like to work on this issue.

@larkox
Copy link
Contributor Author

larkox commented Oct 13, 2020

All yours 🎉 Thank you very much!

@larkox larkox removed the Up For Grabs Ready for help from the community. Removed when someone volunteers label Oct 13, 2020
@patidar-jaishree
Copy link

@larkox I am facing issue while doing zoom plugin developer setup on my local - Can you point me to a relevant document or provide a way to configure zoom plugin for local

set following flags in config.json for mattermost-server repo

"EnableTesting": true,
"EnableUploads": true,
"EnableLocalMode": true,
"LocalModeSocketLocation": "/var/tmp/mattermost_local.socket",

For deploying plugin using below command

export MM_SERVICESETTINGS_SITEURL=http://localhost:8065
export MM_ADMIN_USERNAME=<USERNAME>
export MM_ADMIN_PASSWORD=<PASSWORD>
export MM_PLUGINSETTINGS_ENABLEUPLOADS=true
make deploy

Getting following error:
ERRO[2020-10-19T17:43:32.236332+05:30] Failed to sync plugin from file store bundle=plugins/zoom.tar.gz caller="app/plugin.go:285" error="installExtractedPlugin: Unable to restart plugin on upgrade., please configure APIKey"

@larkox
Copy link
Contributor Author

larkox commented Oct 20, 2020

The plugin should be already installed. You just need to configure the plugin. This error in particular is just about filling the APIKey.

@patidar-jaishree
Copy link

@larkox Sorry I don't have permission to generate token from my zoom account. Hence I am un-assigning myself.

@patidar-jaishree patidar-jaishree removed their assignment Oct 22, 2020
@larkox
Copy link
Contributor Author

larkox commented Oct 22, 2020

@patidar-jaishree You can create a free developer account at zoom at https://marketplace.zoom.us/ , and create a testing app there.
I will get the issue back to up for grabs, but if you are able to create the developer account and try it, feel free to ping me here and I will assign you one more time 😄

@larkox larkox added the Up For Grabs Ready for help from the community. Removed when someone volunteers label Oct 22, 2020
@DHaussermann DHaussermann removed the Up For Grabs Ready for help from the community. Removed when someone volunteers label Feb 11, 2022
@hanzei hanzei added the Up For Grabs Ready for help from the community. Removed when someone volunteers label May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty/2:Medium Medium ticket Hacktoberfest Help Wanted Community help wanted Tech/ReactJS Type/Enhancement New feature or improvement of existing feature Up For Grabs Ready for help from the community. Removed when someone volunteers
Projects
None yet
Development

No branches or pull requests

7 participants