Skip to content

feat(mock-data): allow throwing uncaught exception, errors, warning #14105

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

anshbansal
Copy link
Collaborator

This allows for better testing various scenarios in Ingestion UI

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Jul 16, 2025
Copy link

codecov bot commented Jul 16, 2025

Codecov Report

Attention: Patch coverage is 73.68421% with 5 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ub/ingestion/source/mock_data/datahub_mock_data.py 73.68% 5 Missing ⚠️

❌ Your patch check has failed because the patch coverage (73.68%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

📢 Thoughts on this report? Let us know!

Copy link

alwaysmeticulous bot commented Jul 16, 2025

✅ Meticulous spotted 0 visual differences across 1453 screens tested: view results.

Meticulous evaluated ~9 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit 7ce5504. This comment will update as new commits are pushed.

@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Jul 16, 2025
Copy link

codecov bot commented Jul 16, 2025

Bundle Report

Changes will decrease total bundle size by 311 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 22.16MB -311 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js -311 bytes 18.53MB -0.0%

@anshbansal
Copy link
Collaborator Author

anshbansal commented Jul 16, 2025

🤖 PR Nanny Status - 🚨 Needs Attention

Last updated: 2025-07-16 10:13:10 UTC / 2025-07-16 15:43:10 IST

🚨 Needs Attention (Manual Review Required):

  • build (frontend, UTC)

    Click for AI Analysis

    Summary: The frontend Gradle build encountered multiple test failures and build tool failures. Test failures included authentication and configuration mismatches, CSS errors and warnings, prop type warnings, and timeouts in rendering components. Build tool failures included issues with outdated tools and mismatching runtime versions. The build also failed with an exit code 1 during the yarnTest task.

    Evidence from Logs: View full log

    Authentication failed for user datahub: Login Failure: all modules ignored
    ANTLR Tool version 4.5 used for code generation does not match the current runtime version 4.9.3
    ANTLR Runtime version 4.5 used for parser compilation does not match the current runtime version 4.9.3
    CSS errors and warnings encountered during the build
    Failed prop type warnings in tests
    Test failure in EntityProfile switching tab content due to a test timeout
    Prop type warnings related to undefined values
    React state update warning on an unmounted component
    Each child in a list should have a unique 'key' prop warning
    Test timeout errors
    Issues with yarnTest task execution
    
  • build (frontend, America/New_York)

    Click for AI Analysis

    Summary: The CI/CD build is failing due to multiple test failures, linting errors, and warnings. Test failures occurred in AuthenticationManagerTest, ApplicationTest, OidcConfigurationTest, and EntityProfile tests. Additionally, prop type errors were flagged, and there were warnings related to URL parsing and React component handling.

    Evidence from Logs: View full log

    Test failures in AuthenticationManagerTest, ApplicationTest, and OidcConfigurationTest.
    Test 'switches tab content' in EntityProfile.test.tsx timed out after 5000ms.
    Prop type errors and warnings related to React component handling.
    URL parsing warning: Error fetching capability summary: TypeError: Failed to parse URL from /assets/ingestion/capability_summary.json
    
  • Run Smoke Tests (cypress, Batch 0/11)

    Click for AI Analysis

    Summary: Multiple test failures occurred during the CI/CD build. The Cypress tests failed due to exceeding the current billing plan limits, with specific failures in the 'glossaryV2/v2_glossary_navigation.js' test case. Additionally, the 'glossary sidebar navigation' test failed in three attempts. In the integration_test.py file, the 'test_run_cypress' function failed, leading to a build failure with exit code 1.

    Evidence from Logs: View full log

    There was a test failure during the execution of Cypress tests. The test results exceeded the limit of the current billing plan for Cypress, resulting in a failure to continue testing.
    There was a test failure in the glossaryV2/v2_glossary_navigation.js test case.
    There were test failures in the batch. The test 'glossary sidebar navigation' failed in three attempts.
    There is a test failure in the 'test_run_cypress' function of the integration_test.py file, resulting in a build failure.
    One test failed during the execution with exit code 1.
    

Some checks require manual intervention. Please review the escalated jobs above.


🚀 Missing build failures? Never miss one again - Get PR Nanny

@anshbansal
Copy link
Collaborator Author

anshbansal commented Jul 16, 2025

🤖 PR Nanny Status - 🚨 Needs Attention

Last updated: 2025-07-16 20:21:32 UTC / 2025-07-16 13:21:32 PDT

🚨 Needs Attention (Manual Review Required):

  • Run Smoke Tests (cypress, Batch 0/11)

✅ Recovered:

  • build (frontend, UTC) (recovered after 0 retries)
  • build (frontend, America/New_York) (recovered after 0 retries)
  • metadata-ingestion (3.11, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testIntegrationBatch2) (recovered after 1 retries)

Some checks require manual intervention. Please review the escalated jobs above.


⚡ Sick of manual retries? Get smart automation - Get PR Nanny

@anshbansal
Copy link
Collaborator Author

This has nothing to do with frontend so ignoring frontend.

@anshbansal
Copy link
Collaborator Author

anshbansal commented Jul 16, 2025

🤖 PR Nanny Status - 🚨 Needs Attention

Last updated: 2025-07-17 02:34:14 UTC / 2025-07-16 19:34:14 PDT

🚨 Needs Attention (Manual Review Required):

  • Run Smoke Tests (cypress, Batch 0/11)

✅ Recovered:

  • build (frontend, UTC) (recovered after 0 retries)
  • build (frontend, America/New_York) (recovered after 0 retries)
  • metadata-ingestion (3.11, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testIntegrationBatch2) (recovered after 1 retries)

Some checks require manual intervention. Please review the escalated jobs above.


🚀 Missing build failures? Never miss one again - Get PR Nanny

@anshbansal
Copy link
Collaborator Author

anshbansal commented Jul 17, 2025

🤖 PR Nanny Status - 🚨 Needs Attention

Last updated: 2025-07-17 05:06:22 UTC / 2025-07-16 22:06:22 PDT

🚨 Needs Attention (Manual Review Required):

  • Run Smoke Tests (cypress, Batch 0/11)

✅ Recovered:

  • build (frontend, UTC) (recovered after 0 retries)
  • build (frontend, America/New_York) (recovered after 0 retries)
  • metadata-ingestion (3.11, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testIntegrationBatch2) (recovered after 1 retries)

Some checks require manual intervention. Please review the escalated jobs above.


💬 Generic PR comments? Comment like a pro - Get PR Nanny

@anshbansal
Copy link
Collaborator Author

anshbansal commented Jul 17, 2025

🤖 PR Nanny Status - 🚨 Needs Attention

Last updated: 2025-07-17 05:18:30 UTC / 2025-07-16 22:18:30 PDT

🚨 Needs Attention (Manual Review Required):

  • Run Smoke Tests (cypress, Batch 0/11)

✅ Recovered:

  • build (frontend, UTC) (recovered after 0 retries)
  • build (frontend, America/New_York) (recovered after 0 retries)
  • metadata-ingestion (3.11, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testIntegrationBatch2) (recovered after 1 retries)

Some checks require manual intervention. Please review the escalated jobs above.


🚀 Missing build failures? Never miss one again - Get PR Nanny

@anshbansal
Copy link
Collaborator Author

anshbansal commented Jul 17, 2025

🤖 PR Nanny Status - 🚨 Needs Attention

Last updated: 2025-07-17 06:57:06 UTC / 2025-07-16 23:57:06 PDT

🚨 Needs Attention (Manual Review Required):

  • Run Smoke Tests (cypress, Batch 0/11)

✅ Recovered:

  • build (frontend, UTC) (recovered after 0 retries)
  • build (frontend, America/New_York) (recovered after 0 retries)
  • metadata-ingestion (3.11, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testQuick) (recovered after 0 retries)
  • metadata-ingestion (3.9, testIntegrationBatch2) (recovered after 1 retries)

Some checks require manual intervention. Please review the escalated jobs above.


🔥 Constantly refreshing GitHub? Get real-time updates - Get PR Nanny

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ingestion PR or Issue related to the ingestion of metadata pending-submitter-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants