notifications icon indicating copy to clipboard operation
notifications copied to clipboard

[Testing Confirmation] Confirm current testing requirements

Open stephen-crawford opened this issue 2 years ago • 1 comments
trafficstars

As part of the discussion around implementing an organization-wide testing policy, I am visiting each repo to see what tests they currently perform. I am conducting this work on GitHub so that it is easy to reference.

Looking at the Notifications repository, it appears there is

Repository Unit Tests Integration Tests Backwards Compatibility Tests Additional Tests Link
Notifications
  • - [ ]
  • - [x]
  • - [ ]
  • Link checker https://github.com/opensearch-project/notifications/issues/622

    I don't see any requirements for testing. If there are any specific requirements could you respond to this issue to let me know?

    From looking at the workflow file, it seems like the security-notifications-test-workflow.yml test is an integration test? Is this correct?

    If there are any tests I missed or anything you think all repositories in OpenSearch should have for testing please respond to this issue with details.

    stephen-crawford avatar Feb 20 '23 19:02 stephen-crawford

    I believe we do have unit tests as well for this plugin? https://github.com/opensearch-project/notifications/tree/main/notifications/notifications/src/test

    AWSHurneyt avatar Feb 25 '23 01:02 AWSHurneyt