-
Notifications
You must be signed in to change notification settings - Fork 6
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
Major refactor #42
Merged
Merged
Major refactor #42
Conversation
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
Collaborator
edro15
commented
Jan 25, 2024
•
edited
Loading
edited
- Added high level tests for different Splunk versions
- Fixed bug with Github action when packaging app
- Major repo refactoring
edro15
added a commit
that referenced
this pull request
Sep 5, 2024
* Major refactor (#42) * Updated .gitignore * Major refactor * Fixed tests pipeline * Updated pipelines * Fix for appinspect pipeline * Fix for appinspect pipeline * Updated tests pipeline * Updated tests pipeline * Updates for troubleshooting * Updates for troubleshooting 2 * Launching Splunk via docker compose in the pipeline * Fixed env issue * Removed troubleshooting * Adding troubleshooting for tests * Changed url to fix pipeline * Restored service w/ fixes * Fixes for pipeline * Fix options in pipeline * Restore version * Fixes for pipeline * Fixes for pipeline 2 * Fixes for pipeline 3 * Massive changes for pipeline * Added specific container image * Adding curl to container * Reverting pipeline * Troubleshooting * Troubleshooting * Fixed ip issue * Fixing tests * Fixing tests * Fixing tests * Removing package-lock for testing * Testing gh action for cypress * Testing gh action for cypress * Testing gh action for cypress * Edited SPL safeguards settings * npm versions * Fix for npm versions * Fix for npm versions * Testing with different browser * Testing with no gpu * Testing with chrome * Testing with chrome * Fixed bug with commands * Updated cypress config * Edited waiting times in tests * Fixes for tests in pipeline * Updated upload artifact version in pipeline * Fixes for tests pipeline * Fixes * Tests fixes * Fixed reports * Fixed tests * Testing with Electron * Testing with Electron * Testing with Electron * Testing with Electron * Fixes for junit report * Executing tests w/ chrome * Electron * Added permissions to fix test-reporter * Testing with Chrome * Testing with Chrome * Testing with Chrome. Latest node and actions * Testing with Chrome. Latest cypress * Testing with electron, added timeout * Replaced splunklogout command in tests * More fixes * More fixes * More fixes * More fixes * More fixes * More fixes * Tests fixes and more * Tests fixes * Tests fixes * Run Electron w. video * Run Electron w. video * Videos done. Overall cleanup * Fixed junit report * Enhanced test graph_analysis * Changed runner * Reconfigured self-hosted runner * Updated self-hosted runner * Changed self-hosted runner * ping splunk version * removed mem management for chrome * added debug * Updated tests for graph_analysis page * Fixed graph_analysis tests * Fixed graph_analysis tests * Fixed graph_analysis tests * Installing older version of python-for-scientific app * Added 3 retries at failures * Fixed graph_analysis tests * Changed to en-US * Fixed tests * Testing CodeBuild * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * Troubleshooting action * increased timeout in pipeline * removed tests retries * added py4scientific app for testing * introduced splunk matrix in automated tests * introduced splunk matrix in automated tests * modified matrix in test pipeline * extended matrix in test pipeline * added strategy flag * edited docker-compose to support different platform * edited tests to run in splunk 9.2+ * edited package-lock for npm upgrade * edited test to get splunk 9.2+ compatibility * edited pipeline to test in 2 specific splunk versions * fixes for testing in pipeline * fixes for testing in pipeline * fixes for testing in pipeline * reverted tests for analysis tab * skipping tests for analysis tab till finalised * edited test pipeline * updated README --------- Co-authored-by: Erica Pescio <edro15@users.noreply.github.com> * docs:edited contributing file --------- Co-authored-by: Erica Pescio <edro15@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.