Skip to content
Snippets Groups Projects

Let tests check blob viewer content

Merged Mark Lapierre requested to merge ml-update-test-using-blob-viewer into master
All threads resolved!

What does this MR do and why?

Enables the QA tests to check the blob viewer content when the highlight_js is enabled or disabled

Closes #362194 (closed) and #362193 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Mark Lapierre

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mark Lapierre added 1 commit

    added 1 commit

    • d18e947b - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Mark Lapierre resolved all threads

    resolved all threads

  • Allure report

    allure-report-publisher generated test report!

    review-qa-blocking: :exclamation: test report for d18e947b

    expand test summary
    +-------------------------------------------------------------------+
    |                          suites summary                           |
    +----------------------+--------+--------+---------+-------+--------+
    |                      | passed | failed | skipped | flaky | result |
    +----------------------+--------+--------+---------+-------+--------+
    | Manage               | 31     | 0      | 2       | 32    | ❗     |
    | Create               | 24     | 0      | 2       | 22    | ❗     |
    | Plan                 | 41     | 0      | 1       | 41    | ❗     |
    | Verify               | 12     | 0      | 1       | 12    | ❗     |
    | Version sanity check | 0      | 0      | 1       | 0     | ➖     |
    | Configure            | 0      | 0      | 1       | 0     | ➖     |
    | Protect              | 2      | 0      | 0       | 2     | ❗     |
    | Package              | 0      | 0      | 1       | 0     | ➖     |
    +----------------------+--------+--------+---------+-------+--------+
    | Total                | 110    | 0      | 9       | 109   | ❗     |
    +----------------------+--------+--------+---------+-------+--------+

    package-and-qa: :x: test report for d18e947b

    expand test summary
    +-------------------------------------------------------------------+
    |                          suites summary                           |
    +----------------------+--------+--------+---------+-------+--------+
    |                      | passed | failed | skipped | flaky | result |
    +----------------------+--------+--------+---------+-------+--------+
    | Create               | 606    | 0      | 28      | 25    | ❗     |
    | Configure            | 1      | 0      | 12      | 0     | ✅     |
    | Fulfillment          | 9      | 0      | 40      | 0     | ✅     |
    | Package              | 140    | 9      | 27      | 46    | ❌     |
    | Manage               | 380    | 0      | 24      | 1     | ❗     |
    | Plan                 | 219    | 0      | 1       | 9     | ❗     |
    | Protect              | 8      | 0      | 0       | 0     | ✅     |
    | Verify               | 132    | 0      | 32      | 17    | ❗     |
    | Release              | 24     | 0      | 0       | 0     | ✅     |
    | Secure               | 60     | 0      | 24      | 25    | ❗     |
    | Non-devops           | 9      | 0      | 0       | 0     | ✅     |
    | Enablement:Search    | 9      | 0      | 1       | 0     | ✅     |
    | Version sanity check | 0      | 0      | 4       | 0     | ➖     |
    +----------------------+--------+--------+---------+-------+--------+
    | Total                | 1597   | 9      | 193     | 123   | ❌     |
    +----------------------+--------+--------+---------+-------+--------+

    review-qa-all: :x: test report for d18e947b

    expand test summary
    +----------------------------------------------------------+
    |                      suites summary                      |
    +-------------+--------+--------+---------+-------+--------+
    |             | passed | failed | skipped | flaky | result |
    +-------------+--------+--------+---------+-------+--------+
    | Manage      | 34     | 19     | 7       | 38    | ❌     |
    | Verify      | 20     | 2      | 7       | 24    | ❌     |
    | Create      | 103    | 22     | 6       | 111   | ❌     |
    | Secure      | 15     | 0      | 6       | 20    | ❗     |
    | Release     | 3      | 3      | 0       | 6     | ❌     |
    | Fulfillment | 0      | 1      | 12      | 1     | ❌     |
    | Plan        | 9      | 2      | 0       | 10    | ❌     |
    | Configure   | 0      | 0      | 2       | 0     | ➖     |
    | Non-devops  | 1      | 0      | 1       | 1     | ❗     |
    | Package     | 0      | 0      | 2       | 0     | ➖     |
    +-------------+--------+--------+---------+-------+--------+
    | Total       | 185    | 49     | 43      | 211   | ❌     |
    +-------------+--------+--------+---------+-------+--------+
  • requested review from @john.mcdonnell

  • John McDonnell approved this merge request

    approved this merge request

  • Mark Lapierre requested review from @acunskis and removed review request for @john.mcdonnell

    requested review from @acunskis and removed review request for @john.mcdonnell

  • Andrejs Cunskis approved this merge request

    approved this merge request

  • Andrejs Cunskis resolved all threads

    resolved all threads

  • Andrejs Cunskis enabled an automatic merge when the pipeline for c0193a83 succeeds

    enabled an automatic merge when the pipeline for c0193a83 succeeds

  • Andrejs Cunskis mentioned in commit 7c81616f

    mentioned in commit 7c81616f

  • added workflowstaging label and removed workflowcanary label

  • Anastasia McDonald mentioned in merge request !87423 (merged)

    mentioned in merge request !87423 (merged)

  • Please register or sign in to reply
    Loading