-
Notifications
You must be signed in to change notification settings - Fork 0
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
Playwright Initial Setup #3
Open
panoramix360
wants to merge
44
commits into
main
Choose a base branch
from
feature/playwright-setup
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 10 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
a726963
feat: first playwright config
panoramix360 ebf53dc
feat: adding utils files
panoramix360 e433aa0
feat: change the export way
panoramix360 dc16baa
adds config to the RunContainer function
panoramix360 08db487
feat: configuring the node-fetch package
panoramix360 fb1149a
feat: externalizing PluginConfig
panoramix360 90336a0
feat: adding external plugin configuration
panoramix360 eefcd4c
feat: adding an example-test-folder
panoramix360 63bba59
chore: removes the console log
panoramix360 d91e7f7
feat: moving the constructor to the top and changing the name of the …
panoramix360 730844d
feat: enhancing the readfile for loop
panoramix360 814cdef
feat: configuring eslint and formatting
panoramix360 12c2372
chore: removes unecessary imports
panoramix360 4905419
feat: removing the `no-process-env` and `no-undefined` to not complai…
panoramix360 a1afbb6
feat: creating the concept of the MattermostPlugin
panoramix360 9f05ee9
fix: eslint problem on the promises of the plugins to install
panoramix360 c4d5ab0
refactor to make it simpler to use an external plugin
panoramix360 c29ee4d
chore: fix babel old version
panoramix360 8fea238
chore: setting the nvm version
panoramix360 d3814c9
chore: actions on save
panoramix360 1ad4c91
chore: removing eslint n
panoramix360 49fa826
chore: fixing babel es lint parser
panoramix360 0a94b3f
newline in the end of file
panoramix360 e4a5a52
chore: renaming
panoramix360 833594b
chore: rename pluginid
panoramix360 d89e03a
feat: changing how the client types the configs for the plugin
panoramix360 f47d8d6
feat: adding demo plugin as a constant
panoramix360 7dc7fb2
chore: changing the default node version and the settings
panoramix360 359360a
feat: increasing the timeout and creating a dialog test
panoramix360 97aedfe
feat: adding the env
panoramix360 a5ba3aa
feat: adding a force boolean on the install commands
panoramix360 743d259
feat: testing github actions
panoramix360 4fb61e2
feat: change workflow action
panoramix360 5c66bb1
ci: moving the playwright workflow
panoramix360 ace8f8f
ci: test another command
panoramix360 19c8eb3
ci: adding a working dir
panoramix360 1867c0d
ci: playwright report path
panoramix360 462be25
chore: public var
panoramix360 e4e4aea
feat: changing node version to equal to nvmrc
panoramix360 f073874
chore: renaming folder
panoramix360 b5a874b
feat: adding the dist files
panoramix360 2369d27
feat: updates package json
panoramix360 40010ef
fix: issue when it's a local binary, the path was not being filled
panoramix360 a214fe3
feat: update MattermostContainer to set user ID after login and clean…
panoramix360 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
20.11 | ||
16.20 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,5 @@ | ||
{ | ||
"editor.codeActionsOnSave": { | ||
"source.fixAll.eslint": "always" | ||
}, | ||
"eslint.validate": ["javascript"], | ||
"files.insertFinalNewline": true, | ||
"files.trimFinalNewlines": true, | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if it makes sense to have this workflow be in a centralized GH action for other plugin projects to use. The one thing is that the way we set up our shared actions, the caller cannot pass any arbitrary environment variables. I think that works fine since pretty much all configuration is done in the actual test now due to using
testcontainers
🎉 🎉Made a ticket for creating a centralized action/workflow to run playwright tests #6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That makes sense! let's work on that in the new ticket!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keeping this conversation unresolved to surface the discussion