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

[Testing Confirmation] Confirm current testing requirements #622

Open
stephen-crawford opened this issue Feb 20, 2023 · 1 comment
Open
Labels
enhancement New feature or request

Comments

@stephen-crawford
Copy link

stephen-crawford commented Feb 20, 2023

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
  • Link checker #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 stephen-crawford added enhancement New feature or request untriaged labels Feb 20, 2023
    @stephen-crawford stephen-crawford changed the title [FEATURE] [Testing Confirmation] Confirm current testing requirements Feb 20, 2023
    @AWSHurneyt
    Copy link
    Contributor

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

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    enhancement New feature or request
    Projects
    None yet
    Development

    No branches or pull requests

    2 participants