test: add non regression tests for domain imports #1497
Conversation
Large diff due to formatting
nas-tabchiche
changed the title
nas-tabchiche
requested review from
ab-smith,
eric-intuitem and
Mohamed-Hacene
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. 💤 Files selected but had no reviewable changes (2)
You can disable this status message by setting the WalkthroughThis pull request makes targeted adjustments across the frontend codebase. The changes modify cache handling in the form update function of a Svelte component, add a testing attribute to another UI component, and update form data processing in a server route. In addition, the end-to-end test script has been reformatted with new environment variable exports, and a new functional test suite for domain import and demo data loading has been added. Changes
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (3)
frontend/tests/functional/domain-import.test.ts (3)
4-18: LGTM! Consider adding error handling.The modal dismissal logic is well-structured and includes a good defensive fallback. Consider adding try-catch blocks to handle potential failures gracefully.
async function dismissBlockingModals(page: Page) { + try { const modalBackdrop = page.getByTestId('modal-backdrop'); if (await modalBackdrop.isVisible()) { await modalBackdrop.press('Escape'); await expect(modalBackdrop).toBeHidden(); } const dummyElement = page.locator('#driver-dummy-element'); if (await dummyElement.isVisible()) { await page.locator('.driver-popover-close-btn').first().click(); await expect(dummyElement).toBeHidden(); } // Ensure no lingering modals remain. await page.locator('body').press('Escape'); + } catch (error) { + console.error('Failed to dismiss modals:', error); + throw error; + } }
28-34: Consider enhancing the row count extraction logic.The current implementation could be more robust:
- The regex pattern could be more specific to match the expected format
- Add validation for the extracted number
async function getRowCount(page: Page) { const rowCountText = await page.getByTestId('row-count').innerText(); - const match = rowCountText.match(/\d+$/); + const match = rowCountText.match(/Total:\s*(\d+)/i); + if (!match) { + console.warn(`Unexpected row count format: ${rowCountText}`); + return 0; + } - return match ? parseInt(match[0], 10) : 0; + const count = parseInt(match[1], 10); + return isNaN(count) ? 0 : count; }
36-100: LGTM! Consider adding more assertions.The test cases are well-structured with clear steps and proper verifications. Consider adding:
- Assertions for the domain name in the UI after import
- Verification of the demo data content
await test.step('Import sample domain', async () => { // ... existing code ... await expect(toast).toHaveText(/successfully imported/i); + // Verify domain name appears in the UI + await expect(page.getByText(domainName)).toBeVisible(); // ... existing code ... }); await test.step('Load demo data', async () => { // ... existing code ... await expect(toast).toHaveText(/successfully imported/i); + // Verify demo data content + await expect(page.getByText('Demo Organization')).toBeVisible(); // ... existing code ... });
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
frontend/src/lib/components/Forms/ModelForm.svelte(0 hunks)frontend/src/lib/components/ModelTable/RowCount.svelte(1 hunks)frontend/src/routes/(app)/(internal)/[model=urlmodel]/+page.server.ts(1 hunks)frontend/tests/e2e-tests.sh(4 hunks)frontend/tests/functional/domain-import.test.ts(1 hunks)
💤 Files with no reviewable changes (1)
- frontend/src/lib/components/Forms/ModelForm.svelte
✅ Files skipped from review due to trivial changes (1)
- frontend/src/lib/components/ModelTable/RowCount.svelte
🧰 Additional context used
🪛 Shellcheck (0.10.0)
frontend/tests/e2e-tests.sh
[error] 49-49: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 49-49: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 106-106: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 111-111: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 157-157: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[warning] 184-184: Declare and assign separately to avoid masking return values.
(SC2155)
[warning] 185-185: Declare and assign separately to avoid masking return values.
(SC2155)
[warning] 187-187: Use 'cd ... || exit' or 'cd ... || return' in case cd fails.
(SC2164)
[error] 220-220: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 225-225: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 238-238: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 240-240: Argument mixes string and array. Use * or separate argument.
(SC2145)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: functional-tests (3.12, chromium)
- GitHub Check: enterprise-functional-tests (3.12, chromium)
🔇 Additional comments (2)
frontend/tests/functional/domain-import.test.ts (1)
20-26: LGTM! Navigation helper is well-implemented.The function uses appropriate selectors and includes proper verification steps.
frontend/src/routes/(app)/(internal)/[model=urlmodel]/+page.server.ts (1)
75-83: LGTM! Form data handling is improved.The changes correctly handle form data extraction and file processing.
| pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" --project=chromium "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}" | ||
| else | ||
| pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}" | ||
| pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}" | ||
| fi |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Fix array argument handling in test commands.
Use proper array expansion to avoid argument mixing issues.
- pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" --project=chromium "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}"
+ pnpm playwright test ./tests/functional/"${TEST_PATHS[*]}" --project=chromium "${SCRIPT_LONG_ARGS[*]}" "${SCRIPT_SHORT_ARGS[*]}"
- pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}"
+ pnpm playwright test ./tests/functional/"${TEST_PATHS[*]}" "${SCRIPT_LONG_ARGS[*]}" "${SCRIPT_SHORT_ARGS[*]}"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" --project=chromium "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}" | |
| else | |
| pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}" | |
| pnpm playwright test ./tests/functional/"${TEST_PATHS[@]}" "${SCRIPT_LONG_ARGS[@]}" "${SCRIPT_SHORT_ARGS[@]}" | |
| fi | |
| pnpm playwright test ./tests/functional/"${TEST_PATHS[*]}" --project=chromium "${SCRIPT_LONG_ARGS[*]}" "${SCRIPT_SHORT_ARGS[*]}" | |
| else | |
| pnpm playwright test ./tests/functional/"${TEST_PATHS[*]}" "${SCRIPT_LONG_ARGS[*]}" "${SCRIPT_SHORT_ARGS[*]}" | |
| fi |
🧰 Tools
🪛 Shellcheck (0.10.0)
[error] 238-238: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 240-240: Argument mixes string and array. Use * or separate argument.
(SC2145)
| export CISO_ASSISTANT_VERSION=$(git describe --tags --always) | ||
| export CISO_ASSISTANT_BUILD=$(git rev-parse --short HEAD) |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Fix variable declaration and assignment.
Separate declaration and assignment to avoid masking return values.
-export CISO_ASSISTANT_VERSION=$(git describe --tags --always)
-export CISO_ASSISTANT_BUILD=$(git rev-parse --short HEAD)
+CISO_ASSISTANT_VERSION=""
+CISO_ASSISTANT_BUILD=""
+CISO_ASSISTANT_VERSION=$(git describe --tags --always)
+CISO_ASSISTANT_BUILD=$(git rev-parse --short HEAD)
+export CISO_ASSISTANT_VERSION
+export CISO_ASSISTANT_BUILD📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| export CISO_ASSISTANT_VERSION=$(git describe --tags --always) | |
| export CISO_ASSISTANT_BUILD=$(git rev-parse --short HEAD) | |
| CISO_ASSISTANT_VERSION="" | |
| CISO_ASSISTANT_BUILD="" | |
| CISO_ASSISTANT_VERSION=$(git describe --tags --always) | |
| CISO_ASSISTANT_BUILD=$(git rev-parse --short HEAD) | |
| export CISO_ASSISTANT_VERSION | |
| export CISO_ASSISTANT_BUILD |
🧰 Tools
🪛 Shellcheck (0.10.0)
[warning] 184-184: Declare and assign separately to avoid masking return values.
(SC2155)
[warning] 185-185: Declare and assign separately to avoid masking return values.
(SC2155)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (2)
frontend/tests/e2e-tests.sh (2)
229-235: Consider using git hash for frontend cache invalidation.Instead of using md5sum which can be slow for large codebases, consider using git hash:
-FRONTEND_HASH=$(find "$APP_DIR/frontend/src" -type f \( -name "*.ts" -o -name "*.svelte" \) -print0 | xargs -0 md5sum | md5sum) +FRONTEND_HASH=$(git hash-object $(find "$APP_DIR/frontend/src" -type f \( -name "*.ts" -o -name "*.svelte" \)))
171-183: Consider using a .env file for environment variables.The large number of environment variables could be better managed using a .env file.
Consider moving these environment variables to a
.env.testfile and sourcing it:# .env.test CISO_ASSISTANT_URL=http://localhost:4173 ALLOWED_HOSTS=localhost,127.0.0.1,0.0.0.0 ...Then in the script:
source "$APP_DIR/frontend/tests/.env.test"
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
frontend/tests/e2e-tests.sh(4 hunks)
🧰 Additional context used
🪛 Shellcheck (0.10.0)
frontend/tests/e2e-tests.sh
[error] 49-49: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 49-49: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 106-106: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 111-111: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 157-157: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[warning] 184-184: Declare and assign separately to avoid masking return values.
(SC2155)
[warning] 185-185: Declare and assign separately to avoid masking return values.
(SC2155)
[error] 220-220: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 225-225: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 238-238: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 240-240: Argument mixes string and array. Use * or separate argument.
(SC2145)
⏰ Context from checks skipped due to timeout of 90000ms (8)
- GitHub Check: startup-docker-compose-test
- GitHub Check: enterprise-startup-functional-test (3.12)
- GitHub Check: enterprise-startup-docker-compose-test
- GitHub Check: functional-tests (3.12, chromium)
- GitHub Check: enterprise-functional-tests (3.12, chromium)
- GitHub Check: startup-functional-test (3.12)
- GitHub Check: Analyze (python)
- GitHub Check: Analyze (javascript-typescript)
🔇 Additional comments (3)
frontend/tests/e2e-tests.sh (3)
15-47: LGTM! Argument handling is well-structured.The argument parsing logic is clear and handles all cases appropriately with proper validation.
184-185: Fix variable declaration and assignment to avoid masking return values.The current export with inline assignment can mask return values from the git commands.
🧰 Tools
🪛 Shellcheck (0.10.0)
[warning] 184-184: Declare and assign separately to avoid masking return values.
(SC2155)
[warning] 185-185: Declare and assign separately to avoid masking return values.
(SC2155)
237-241: Fix array argument handling in test commands.The current array expansion can lead to argument mixing issues.
🧰 Tools
🪛 Shellcheck (0.10.0)
[error] 238-238: Argument mixes string and array. Use * or separate argument.
(SC2145)
[error] 240-240: Argument mixes string and array. Use * or separate argument.
(SC2145)
| SCRIPT_SHORT_ARGS+=("$arg") | ||
| elif [[ $arg != -* ]]; then | ||
| TEST_PATHS+=("$arg") | ||
| fi | ||
| done | ||
|
|
||
| if [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -h " ]] || [[ " ${SCRIPT_LONG_ARGS[@]} " =~ " --help " ]]; then |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Fix array concatenation in conditional check.
The current array concatenation in the conditional statement can lead to unexpected behavior.
-if [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -h " ]] || [[ " ${SCRIPT_LONG_ARGS[@]} " =~ " --help " ]]; then
+if printf '%s\n' "${SCRIPT_SHORT_ARGS[@]}" | grep -q '^-h$' || printf '%s\n' "${SCRIPT_LONG_ARGS[@]}" | grep -q '^--help$'; then📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| if [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -h " ]] || [[ " ${SCRIPT_LONG_ARGS[@]} " =~ " --help " ]]; then | |
| if printf '%s\n' "${SCRIPT_SHORT_ARGS[@]}" | grep -q '^-h$' || printf '%s\n' "${SCRIPT_LONG_ARGS[@]}" | grep -q '^--help$'; then |
🧰 Tools
🪛 Shellcheck (0.10.0)
[error] 49-49: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 49-49: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
| for PORT in $MAILER_WEB_SERVER_PORT $MAILER_SMTP_SERVER_PORT; do | ||
| if python3 -c "import socket;exit(0 if 0 == socket.socket(socket.AF_INET, socket.SOCK_STREAM).connect_ex(('localhost',$PORT)) else 1)"; then | ||
| if [[ ! " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | ||
| echo "The port $PORT is already in use!" | ||
| echo "Please stop the running process using the port or change the mailer port and try again." | ||
| exit 1 | ||
| fi | ||
| elif [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | ||
| echo "No mailer service is running on port $PORT!" | ||
| echo "Please start a mailer service on port $PORT or change the mailer port using --mailer=PORT/PORT and try again." | ||
| echo "You can also use the isolated test mailer service by removing the -m option." | ||
| exit 1 | ||
| fi | ||
| if python3 -c "import socket;exit(0 if 0 == socket.socket(socket.AF_INET, socket.SOCK_STREAM).connect_ex(('localhost',$PORT)) else 1)"; then | ||
| if [[ ! " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | ||
| echo "The port $PORT is already in use!" | ||
| echo "Please stop the running process using the port or change the mailer port and try again." | ||
| exit 1 | ||
| fi | ||
| elif [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | ||
| echo "No mailer service is running on port $PORT!" | ||
| echo "Please start a mailer service on port $PORT or change the mailer port using --mailer=PORT/PORT and try again." | ||
| echo "You can also use the isolated test mailer service by removing the -m option." | ||
| exit 1 | ||
| fi | ||
| done |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Improve port checking logic.
The port checking logic has similar array concatenation issues in the conditionals.
- if [[ ! " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then
+ if ! printf '%s\n' "${SCRIPT_SHORT_ARGS[@]}" | grep -q '^-m$'; thenAlso, consider adding a timeout to the port check to avoid hanging:
- if python3 -c "import socket;exit(0 if 0 == socket.socket(socket.AF_INET, socket.SOCK_STREAM).connect_ex(('localhost',$PORT)) else 1)"; then
+ if python3 -c "
+import socket
+s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
+s.settimeout(1)
+exit(0 if 0 == s.connect_ex(('localhost',$PORT)) else 1)
+"; then📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| for PORT in $MAILER_WEB_SERVER_PORT $MAILER_SMTP_SERVER_PORT; do | |
| if python3 -c "import socket;exit(0 if 0 == socket.socket(socket.AF_INET, socket.SOCK_STREAM).connect_ex(('localhost',$PORT)) else 1)"; then | |
| if [[ ! " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | |
| echo "The port $PORT is already in use!" | |
| echo "Please stop the running process using the port or change the mailer port and try again." | |
| exit 1 | |
| fi | |
| elif [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | |
| echo "No mailer service is running on port $PORT!" | |
| echo "Please start a mailer service on port $PORT or change the mailer port using --mailer=PORT/PORT and try again." | |
| echo "You can also use the isolated test mailer service by removing the -m option." | |
| exit 1 | |
| fi | |
| if python3 -c "import socket;exit(0 if 0 == socket.socket(socket.AF_INET, socket.SOCK_STREAM).connect_ex(('localhost',$PORT)) else 1)"; then | |
| if [[ ! " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | |
| echo "The port $PORT is already in use!" | |
| echo "Please stop the running process using the port or change the mailer port and try again." | |
| exit 1 | |
| fi | |
| elif [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | |
| echo "No mailer service is running on port $PORT!" | |
| echo "Please start a mailer service on port $PORT or change the mailer port using --mailer=PORT/PORT and try again." | |
| echo "You can also use the isolated test mailer service by removing the -m option." | |
| exit 1 | |
| fi | |
| done | |
| for PORT in $MAILER_WEB_SERVER_PORT $MAILER_SMTP_SERVER_PORT; do | |
| if python3 -c " | |
| import socket | |
| s = socket.socket(socket.AF_INET, socket.SOCK_STREAM) | |
| s.settimeout(1) | |
| exit(0 if 0 == s.connect_ex(('localhost',$PORT)) else 1) | |
| "; then | |
| if ! printf '%s\n' "${SCRIPT_SHORT_ARGS[@]}" | grep -q '^-m$'; then | |
| echo "The port $PORT is already in use!" | |
| echo "Please stop the running process using the port or change the mailer port and try again." | |
| exit 1 | |
| fi | |
| elif [[ " ${SCRIPT_SHORT_ARGS[@]} " =~ " -m " ]]; then | |
| echo "No mailer service is running on port $PORT!" | |
| echo "Please start a mailer service on port $PORT or change the mailer port using --mailer=PORT/PORT and try again." | |
| echo "You can also use the isolated test mailer service by removing the -m option." | |
| exit 1 | |
| fi | |
| done |
🧰 Tools
🪛 Shellcheck (0.10.0)
[error] 106-106: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
[error] 111-111: Arrays implicitly concatenate in [[ ]]. Use a loop (or explicit * instead of @).
(SC2199)
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/workflows/functional-tests.yml (2)
97-98: Consider using dynamic version and centralizing configuration.The version is hardcoded while the build uses a git command. Consider:
- Using
$(git describe --tags --always)for version to match e2e-tests.sh- Moving these values to workflow-level env variables for better maintainability
- echo CISO_ASSISTANT_VERSION=v4.2.0 >> .env + echo CISO_ASSISTANT_VERSION=$(git describe --tags --always) >> .envOr better yet, move to workflow-level env:
env: BRANCH_NAME: ${{ github.head_ref || github.ref_name }} GITHUB_WORKFLOW: github_actions + CISO_ASSISTANT_VERSION: ${{ steps.version.outputs.version }} + CISO_ASSISTANT_BUILD: ${{ steps.build.outputs.build }} backend-directory: ./backend ... jobs: functional-tests: steps: + - id: version + run: echo "version=$(git describe --tags --always)" >> $GITHUB_OUTPUT + - id: build + run: echo "build=$(git rev-parse --short HEAD)" >> $GITHUB_OUTPUT
209-210: Remove duplicated configuration.This configuration is identical to the one in the
functional-testsjob. If you move the version and build to workflow-level env variables as suggested above, this duplication can be eliminated.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/functional-tests.yml(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (8)
- GitHub Check: startup-functional-test (3.12)
- GitHub Check: startup-docker-compose-test
- GitHub Check: functional-tests (3.12, chromium)
- GitHub Check: enterprise-startup-functional-test (3.12)
- GitHub Check: enterprise-functional-tests (3.12, chromium)
- GitHub Check: enterprise-startup-docker-compose-test
- GitHub Check: Analyze (python)
- GitHub Check: Analyze (javascript-typescript)
Mohamed-Hacene
deleted the
CA-864-add-non-regression-tests-for-domain-imports
branch
Summary by CodeRabbit
Refactor
New Features
Tests
.bakfiles and loading demo data.