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

feat: check audit logs access via jimm #1852

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

huwshimi
Copy link
Contributor

Done

  • Refactor tabs into a component.
  • Check permissions for audit logs via JIMM.

QA

  • Connect to JIMM.
  • Go to a model and check that there is a Logs tab.
  • Click on the logs tab and check that there are two sub-tabs: "Audit logs" and "Action logs" and that you can view the "Audit logs" tab.
  • Connect to a local (non-jimm) controller.
  • Go to a model and check that there is an "Action logs" tab and no sub-tabs.

Details

https://warthogs.atlassian.net/browse/WD-19056

@webteam-app
Copy link

import ModelTabs from "./ModelTabs";
import { Label } from "./types";

describe("ModelTabs", () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These tests have been moved from the EntityDetails tests, with a few improvements.

Copy link

github-actions bot commented Feb 13, 2025

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 98.01% (🎯 95%) 15115 / 15421
🟢 Statements 98.01% (🎯 95%) 15115 / 15421
🟢 Functions 98.19% (🎯 95%) 598 / 609
🟢 Branches 92.03% (🎯 90%) 2958 / 3214
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/pages/EntityDetails/EntityDetails.tsx 100% 97.36% 100% 100%
src/pages/EntityDetails/Model/Logs/Logs.tsx 100% 100% 100% 100%
src/pages/EntityDetails/Model/ModelTabs/ModelTabs.tsx 100% 100% 100% 100%
Generated in workflow #118 for commit 36bf2bf by the Vitest Coverage Report Action

@huwshimi huwshimi marked this pull request as ready for review February 17, 2025 23:32
@huwshimi huwshimi force-pushed the model-audit-logs-rebac branch from 6a4b632 to 36bf2bf Compare February 19, 2025 05:03
Copy link
Contributor

@Ninfa-Jeon Ninfa-Jeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@huwshimi huwshimi merged commit 8412417 into canonical:main Feb 19, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants