litmuschaos / litmus-e2e

Test the Chaos!!! An e2e (end-to-end) validation pipeline for the LitmusChaos infrastructure
Apache License 2.0
15 stars 27 forks source link

Added myHub Api tests #363

Closed Aman-Codes closed 2 years ago

Aman-Codes commented 2 years ago

Proposed changes

Summarize your changes here to communicate with the reviewers and make sure to put the link of the related issue

How has this been tested:

Types of changes

What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Documentation

Code Review

Special notes for your reviewer:

Out of 23 tests 17 tests passed and 6 tests failed. The six failing tests are:

  1. Fetching the experiment manifest from the hub of a project with no access [ Should not be possible ]
  2. Fetching all the pre-defined workflows of a project with no access [ Should not be possible ]
  3. Fetching the pre-defined experiment manifest from a hub of a project with no access [ Should not be possible ]
  4. Deleting the hub of a project with no access [ Should not be possible ]
  5. Deleting the hub of a project with viewer access [ Should not be possible ]
  6. Syncing the hub of a project with no access [ Should not be possible ]

These tests will pass after the appropriate fix are merged (Tracked on #362) cc @Jonsy13

image

Aman-Codes commented 2 years ago

@Jonsy13 Changes added in last commit: