Skip to content

UI unit test: expected values#7792

Merged
weizhouapache merged 1 commit intoapache:4.18from
shapeblue:uiUnitTestAutogenview
Jul 31, 2023
Merged

UI unit test: expected values#7792
weizhouapache merged 1 commit intoapache:4.18from
shapeblue:uiUnitTestAutogenview

Conversation

@DaanHoogland
Copy link
Copy Markdown
Contributor

Description

This PR fixes a failure in UI unit tests

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

npm run test;unit

@DaanHoogland DaanHoogland changed the base branch from main to 4.18 July 31, 2023 09:08
@DaanHoogland DaanHoogland marked this pull request as ready for review July 31, 2023 09:09
@DaanHoogland DaanHoogland mentioned this pull request Jul 31, 2023
12 tasks
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 31, 2023

Codecov Report

Merging #7792 (80b28cd) into 4.18 (a2eb103) will increase coverage by 0.41%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               4.18    #7792      +/-   ##
============================================
+ Coverage     13.02%   13.43%   +0.41%     
- Complexity     9030     9394     +364     
============================================
  Files          2720     2747      +27     
  Lines        257001   258844    +1843     
  Branches      40083    40313     +230     
============================================
+ Hits          33463    34779    +1316     
- Misses       219337   219685     +348     
- Partials       4201     4380     +179     

see 104 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Copy Markdown
Contributor

@slavkap slavkap left a comment

Choose a reason for hiding this comment

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

code LGTM

Copy link
Copy Markdown
Member

@weizhouapache weizhouapache left a comment

Choose a reason for hiding this comment

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

manually tested ok

not sure if there is regression

@weizhouapache weizhouapache merged commit fd13184 into apache:4.18 Jul 31, 2023
Copy link
Copy Markdown
Member

@harikrishna-patnala harikrishna-patnala left a comment

Choose a reason for hiding this comment

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

LGTM, build passed

@DaanHoogland DaanHoogland deleted the uiUnitTestAutogenview branch July 31, 2023 11:54
DaanHoogland added a commit that referenced this pull request Aug 3, 2023
* 4.18:
  UI: allow new keys for VM details (#7793)
  Refactoring StorPool's smoke tests (#7392)
  UI: decode userdata in EditVM dialog (#7796)
  packaging: unalias cp before package upgrade (#7722)
  make NoopDbUpgrade do a systemvm template check (#7564)
  UI unit test: fix expected values (#7792)
@DaanHoogland DaanHoogland added this to the 4.18.1.0 milestone Sep 29, 2023
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.

5 participants