Add release notes for 2.4.1 #183
Merged
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
Signed-off-by: Subhobrata Dey <[email protected]>
eirsep
approved these changes
Dec 13, 2022
getsaurabh02
approved these changes
Dec 13, 2022
opensearch-trigger-bot bot
pushed a commit
that referenced
this pull request
Dec 13, 2022
Signed-off-by: Subhobrata Dey <[email protected]> (cherry picked from commit b7e5469)
opensearch-trigger-bot bot
pushed a commit
that referenced
this pull request
Dec 13, 2022
Signed-off-by: Subhobrata Dey <[email protected]> (cherry picked from commit b7e5469)
sbcd90
pushed a commit
that referenced
this pull request
Dec 13, 2022
Signed-off-by: Subhobrata Dey <[email protected]>
sbcd90
pushed a commit
that referenced
this pull request
Dec 13, 2022
Signed-off-by: Subhobrata Dey <[email protected]>
riysaxen-amzn
pushed a commit
to riysaxen-amzn/security-analytics
that referenced
this pull request
Feb 20, 2024
…ect#183) * Implement cypress tests for Findings Signed-off-by: Chris Hesterman <[email protected]> * Progress with findings_spec Signed-off-by: Chris Hesterman <[email protected]> * Add tests for Finding details flyout Signed-off-by: Chris Hesterman <[email protected]> * Add comments. Keeping manual detector creation. Signed-off-by: Chris Hesterman <[email protected]> * Correct imports in detectors_spec.js Signed-off-by: Chris Hesterman <[email protected]> * Small refactor of findings_spec Signed-off-by: Chris Hesterman <[email protected]> * Remove leftover console.log Signed-off-by: Chris Hesterman <[email protected]> * Implement requested changes, findings_spec, fixtures/constants Signed-off-by: Chris Hesterman <[email protected]> * Make requested changes, 4_findings.spec, rename files Signed-off-by: Chris Hesterman <[email protected]> * Incorporate use of cypress url() for better testing Signed-off-by: Chris Hesterman <[email protected]> * Specify OSD 2.4.0 in cypress-workflow Signed-off-by: Chris Hesterman <[email protected]> * Change intermittently failing selector, detectors_spec Signed-off-by: Chris Hesterman <[email protected]> * Remove all specs but deteectors_spec for fix Signed-off-by: Chris Hesterman <[email protected]> * Resolve issues and brittleness in detectors_spec Signed-off-by: Chris Hesterman <[email protected]> * Resolve issues with detectors_spec, Make more durable Signed-off-by: Chris Hesterman <[email protected]> * Small changes to findings_spec Signed-off-by: Chris Hesterman <[email protected]> * Add timeout after create detector button clicked Signed-off-by: Chris Hesterman <[email protected]> * Add back hard wait instead of timeout. Signed-off-by: Chris Hesterman <[email protected]> * Add timeout before disable all rules Signed-off-by: Chris Hesterman <[email protected]> * Current progress, findings_spec refactor Signed-off-by: Chris Hesterman <[email protected]> * Add wait before detector name change, testing result Signed-off-by: Chris Hesterman <[email protected]> * Working solidly locally... again Signed-off-by: Chris Hesterman <[email protected]> * Refactor creation phase, detectors_spec Signed-off-by: Chris Hesterman <[email protected]> * Clean up unused code Signed-off-by: Chris Hesterman <[email protected]> * Working though issues findings_spec Signed-off-by: Chris Hesterman <[email protected]> * Working through icon button not being clicked Signed-off-by: Chris Hesterman <[email protected]> * Remove as many timeout/waits as possible while still pass Signed-off-by: Chris Hesterman <[email protected]> * Refactor findings.spec Signed-off-by: Chris Hesterman <[email protected]> * Update helpers.js to work with new mappings elements Signed-off-by: Chris Hesterman <[email protected]> * Change assertion to 'include', line 27 findings.spec Signed-off-by: Chris Hesterman <[email protected]> * Add url assertion after detector created to fix issue. Signed-off-by: Chris Hesterman <[email protected]> * Add check to confirm url before deleting test detector Signed-off-by: Chris Hesterman <[email protected]> * Changes in response to PR comments Signed-off-by: Chris Hesterman <[email protected]> * Make use of added commands to set up test env, findings.spec Signed-off-by: Chris Hesterman <[email protected]> Delete now obsolete 'helpers.js' in cypress/support Signed-off-by: Chris Hesterman <[email protected]> * Add timeout before clicking finding details icon Signed-off-by: Chris Hesterman <[email protected]> * Use existing commands to set up findings spec detector Signed-off-by: Chris Hesterman <[email protected]> Fix detectors spec after above change affected it. Signed-off-by: Chris Hesterman <[email protected]> * Make selection of finding id button more specific, findings.spec Signed-off-by: Chris Hesterman <[email protected]> * Remove commented code Signed-off-by: Chris Hesterman <[email protected]> * Fix details icon selector to account for multiple findings Signed-off-by: Chris Hesterman <[email protected]> * Resolve cy.visit causing two detectors to be created Signed-off-by: Chris Hesterman <[email protected]> trim/remove waits/timeouts as much as possible Signed-off-by: Chris Hesterman <[email protected]> * Change parameter name back to documentJSON, commands.js Signed-off-by: Chris Hesterman <[email protected]> * Update findings spec, sample_detector variables/names Signed-off-by: Chris Hesterman <[email protected]> * Change to have detector created after mappings, findings spec Signed-off-by: Chris Hesterman <[email protected]> * Address PR comments. Signed-off-by: Chris Hesterman <[email protected]> Signed-off-by: Chris Hesterman <[email protected]>
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.
Signed-off-by: Subhobrata Dey [email protected]
Description
Add release notes for 2.4.1
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.