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

performance: added memory tests in packages #4228

Merged
merged 6 commits into from
Apr 2, 2024
Merged

Conversation

Rajdeepc
Copy link
Contributor

@Rajdeepc Rajdeepc commented Mar 28, 2024

Objective

The objective of this RFC is to propose a standardised approach for adding memory tests to each package in the component library using Chrome's Performance API, specifically the measureUserAgentSpecificMemory() method.

refs #3722

Background

Memory leaks in web applications can lead to performance issues and degraded user experience. By adding memory tests to each package in the component library, we can proactively identify and fix potential memory leaks, ensuring the overall stability and performance of the library.

Proposed Solution

Implementation of Memory Tests

  1. The memory tests will be implemented using the measureUserAgentSpecificMemory() method from the Chrome Performance API.
  2. The memory test for each package will consist of the following steps:
    - Create a test scenario that triggers the component's functionality.
    - Measure the memory usage before and after the test scenario.
    - Calculate the memory usage difference (DOM and JavaScript) to identify any memory leaks.

Test Structure

  1. Each memory test will be structured as a separate test case, following a standardised format.
  2. The test case will include the following components:
    - Setup: Prepare the environment for the test.
    - Execution: Trigger the component's functionality.
    - Measurement: Measure the memory usage before and after the test.
    - Assertion: Verify that the memory usage difference is within an acceptable range.

Test Helper Function

  1. A test helper function named usedHeapMB() will be used to measure the memory usage.
  2. The usedHeapMB() function will:
    - Call measureUserAgentSpecificMemory() to get the memory sample.
    - Parse the memory sample to extract the memory usage breakdown (DOM, JavaScript).
    - Return the memory usage breakdown in megabytes.

Integration with Testing Framework

  1. The memory tests will be integrated into the existing testing framework @open-wc/testing used for testing the SWC component library.
  2. The memory tests will be executed as part of the automated test suite to ensure they are run regularly and consistently.

For more details on this experiment please go through this RFC

Related Issue:

Building on top of #3721

How has this been tested?

  • Test case 1
    1. Go here
    2. Do this
  • Test case 2
    1. Go here
    2. Do this

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

Checklist

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • If my change required a change to the documentation, I have updated the documentation in this pull request.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have reviewed at the Accessibility Practices for this feature, see: Aria Practices

Best practices

This repository uses conventional commit syntax for each commit message; note that the GitHub UI does not use this by default so be cautious when accepting suggested changes. Avoid the "Update branch" button on the pull request and opt instead for rebasing your branch against main.

@Rajdeepc Rajdeepc linked an issue Mar 28, 2024 that may be closed by this pull request
1 task
Copy link

github-actions bot commented Mar 28, 2024

Lighthouse scores

Category Latest (report) Main (report) Branch (report)
Performance 0.97 0.98 0.97
Accessibility 1 1 1
Best Practices 1 1 1
SEO 1 0.92 0.92
PWA 1 1 1
What is this?

Lighthouse scores comparing the documentation site built from the PR ("Branch") to that of the production documentation site ("Latest") and the build currently on main ("Main"). Higher scores are better, but note that the SEO scores on Netlify URLs are artifically constrained to 0.92.

Transfer Size

Category Latest Main Branch
Total 228.332 kB 217.621 kB 217.487 kB 🏆
Scripts 60.441 kB 54.824 kB 54.596 kB 🏆
Stylesheet 35.305 kB 30.829 kB 🏆 30.933 kB
Document 5.874 kB 5.256 kB 5.246 kB 🏆
Third Party 126.712 kB 126.712 kB 126.712 kB

Request Count

Category Latest Main Branch
Total 43 43 43
Scripts 35 35 35
Stylesheet 5 5 5
Document 1 1 1
Third Party 2 2 2

@Rajdeepc Rajdeepc marked this pull request as draft March 28, 2024 11:41
Copy link

github-actions bot commented Mar 28, 2024

Tachometer results

Chrome

accordion permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 415 kB 131.77ms - 134.62ms - unsure 🔍
-2% - +2%
-2.63ms - +2.37ms
branch 406 kB 131.27ms - 135.37ms unsure 🔍
-2% - +2%
-2.37ms - +2.63ms
-

action-bar permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 483 kB 71.00ms - 74.10ms - unsure 🔍
-5% - +1%
-3.38ms - +1.11ms
branch 474 kB 72.06ms - 75.31ms unsure 🔍
-2% - +5%
-1.11ms - +3.38ms
-

action-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 513 kB 102.53ms - 105.03ms - slower ❌
1% - 4%
1.22ms - 4.16ms
branch 504 kB 100.31ms - 101.87ms faster ✔
1% - 4%
1.22ms - 4.16ms
-

action-group permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 535 kB 65.03ms - 66.22ms - slower ❌
0% - 3%
0.09ms - 1.94ms
branch 526 kB 63.90ms - 65.32ms faster ✔
0% - 3%
0.09ms - 1.94ms
-

action-menu permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 645 kB 163.03ms - 166.25ms - unsure 🔍
-2% - +1%
-3.31ms - +1.81ms
branch 637 kB 163.39ms - 167.38ms unsure 🔍
-1% - +2%
-1.81ms - +3.31ms
-

test-directive permalink

Version Bytes Avg Time vs remote vs branch
npm latest 602 kB 89.32ms - 90.50ms - unsure 🔍
-2% - +1%
-1.37ms - +0.59ms
branch 594 kB 89.52ms - 91.08ms unsure 🔍
-1% - +2%
-0.59ms - +1.37ms
-

test-lazy permalink

Version Bytes Avg Time vs remote vs branch
npm latest 602 kB 87.72ms - 89.16ms - slower ❌
0% - 2%
0.13ms - 1.92ms
branch 593 kB 86.88ms - 87.95ms faster ✔
0% - 2%
0.13ms - 1.92ms
-

test-open-close-directive permalink

Version Bytes Avg Time vs remote vs branch
npm latest 785 kB 1871.87ms - 1875.04ms - unsure 🔍
-0% - +0%
-2.10ms - +2.14ms
branch 777 kB 1872.04ms - 1874.84ms unsure 🔍
-0% - +0%
-2.14ms - +2.10ms
-

test-open-close permalink

Version Bytes Avg Time vs remote vs branch
npm latest 783 kB 1859.81ms - 1862.39ms - unsure 🔍
-0% - +0%
-0.96ms - +3.02ms
branch 775 kB 1858.55ms - 1861.59ms unsure 🔍
-0% - +0%
-3.02ms - +0.96ms
-

alert-dialog permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 455 kB 97.84ms - 98.80ms - unsure 🔍
-1% - +1%
-0.72ms - +0.73ms
branch 442 kB 97.77ms - 98.86ms unsure 🔍
-1% - +1%
-0.73ms - +0.72ms
-

asset permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 359 kB 12.20ms - 12.54ms - unsure 🔍
-0% - +3%
-0.04ms - +0.32ms
branch 350 kB 12.17ms - 12.29ms unsure 🔍
-3% - +0%
-0.32ms - +0.04ms
-

avatar permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 396 kB 24.41ms - 25.01ms - unsure 🔍
-1% - +2%
-0.24ms - +0.44ms
branch 387 kB 24.46ms - 24.76ms unsure 🔍
-2% - +1%
-0.44ms - +0.24ms
-

badge permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 397 kB 33.21ms - 34.06ms - unsure 🔍
-2% - +1%
-0.57ms - +0.39ms
branch 388 kB 33.49ms - 33.95ms unsure 🔍
-1% - +2%
-0.39ms - +0.57ms
-

button-group permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 442 kB 59.71ms - 60.91ms - slower ❌
1% - 4%
0.68ms - 2.27ms
branch 433 kB 58.32ms - 59.34ms faster ✔
1% - 4%
0.68ms - 2.27ms
-

button permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 449 kB 69.42ms - 70.84ms - slower ❌
0% - 3%
0.15ms - 2.09ms
branch 439 kB 68.35ms - 69.67ms faster ✔
0% - 3%
0.15ms - 2.09ms
-

card permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 501 kB 59.16ms - 59.86ms - unsure 🔍
-2% - +0%
-0.91ms - +0.16ms
branch 493 kB 59.49ms - 60.29ms unsure 🔍
-0% - +2%
-0.16ms - +0.91ms
-

coachmark permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 626 kB 114.69ms - 116.90ms - unsure 🔍
-1% - +2%
-0.98ms - +2.14ms
branch 617 kB 114.12ms - 116.31ms unsure 🔍
-2% - +1%
-2.14ms - +0.98ms
-

combobox permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 705 kB 37.06ms - 37.64ms - unsure 🔍
-1% - +1%
-0.34ms - +0.37ms
branch 697 kB 37.13ms - 37.54ms unsure 🔍
-1% - +1%
-0.37ms - +0.34ms
-

light-dom-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 705 kB 385.30ms - 391.82ms - faster ✔
0% - 2%
0.80ms - 9.82ms
branch 697 kB 390.75ms - 396.98ms slower ❌
0% - 3%
0.80ms - 9.82ms
-

dialog permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 507 kB 71.17ms - 72.12ms - faster ✔
0% - 2%
0.20ms - 1.66ms
branch 498 kB 72.03ms - 73.13ms slower ❌
0% - 2%
0.20ms - 1.66ms
-

grid permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 458 kB 36.43ms - 37.03ms - slower ❌
2% - 4%
0.72ms - 1.53ms
branch 431 kB 35.34ms - 35.87ms faster ✔
2% - 4%
0.72ms - 1.53ms
-

infield-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 401 kB 19.45ms - 19.84ms - slower ❌
1% - 3%
0.10ms - 0.57ms
branch 392 kB 19.18ms - 19.44ms faster ✔
1% - 3%
0.10ms - 0.57ms
-

menu permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 473 kB 207.08ms - 211.16ms - unsure 🔍
-2% - +1%
-4.38ms - +1.38ms
branch 466 kB 208.59ms - 212.65ms unsure 🔍
-1% - +2%
-1.38ms - +4.38ms
-

number-field permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 521 kB 92.50ms - 94.28ms - faster ✔
2% - 5%
1.99ms - 4.67ms
branch 512 kB 95.71ms - 97.73ms slower ❌
2% - 5%
1.99ms - 4.67ms
-

overlay permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 679 kB 464.19ms - 467.92ms - unsure 🔍
-1% - +0%
-4.44ms - +1.11ms
branch 671 kB 465.66ms - 469.78ms unsure 🔍
-0% - +1%
-1.11ms - +4.44ms
-

directive-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 782 kB 35.43ms - 35.88ms - unsure 🔍
-3% - +2%
-0.96ms - +0.61ms
branch 774 kB 35.08ms - 36.59ms unsure 🔍
-2% - +3%
-0.61ms - +0.96ms
-

element-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 756 kB 344.19ms - 346.81ms - faster ✔
0% - 2%
0.99ms - 5.42ms
branch 748 kB 346.91ms - 350.50ms slower ❌
0% - 2%
0.99ms - 5.42ms
-

lazy-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 557 kB 53.83ms - 54.74ms - unsure 🔍
-0% - +2%
-0.21ms - +1.13ms
branch 549 kB 53.32ms - 54.31ms unsure 🔍
-2% - +0%
-1.13ms - +0.21ms
-

picker-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 413 kB 37.75ms - 38.20ms - slower ❌
0% - 2%
0.09ms - 0.74ms
branch 404 kB 37.32ms - 37.80ms faster ✔
0% - 2%
0.09ms - 0.74ms
-

picker permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 511 kB 568.74ms - 577.75ms - unsure 🔍
-1% - +1%
-4.59ms - +8.52ms
branch 503 kB 566.52ms - 576.04ms unsure 🔍
-1% - +1%
-8.52ms - +4.59ms
-

popover permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 382 kB 21.33ms - 22.00ms - unsure 🔍
-1% - +3%
-0.13ms - +0.57ms
branch 373 kB 21.36ms - 21.53ms unsure 🔍
-3% - +1%
-0.57ms - +0.13ms
-

search permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 471 kB 57.12ms - 57.98ms - faster ✔
3% - 5%
1.96ms - 3.14ms
branch 462 kB 59.69ms - 60.50ms slower ❌
3% - 5%
1.96ms - 3.14ms
-

slider permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 477 kB 104.45ms - 106.80ms - faster ✔
1% - 4%
0.92ms - 4.27ms
branch 469 kB 107.03ms - 109.41ms slower ❌
1% - 4%
0.92ms - 4.27ms
-

split-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 718 kB 1857.53ms - 1860.91ms - unsure 🔍
-0% - +0%
-2.47ms - +2.31ms
branch 710 kB 1857.61ms - 1861.00ms unsure 🔍
-0% - +0%
-2.31ms - +2.47ms
-

tags permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 449 kB 20.49ms - 20.69ms - unsure 🔍
-0% - +1%
-0.10ms - +0.24ms
branch 439 kB 20.39ms - 20.65ms unsure 🔍
-1% - +0%
-0.24ms - +0.10ms
-

toast permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 430 kB 42.99ms - 43.59ms - slower ❌
0% - 2%
0.13ms - 0.95ms
branch 421 kB 42.46ms - 43.03ms faster ✔
0% - 2%
0.13ms - 0.95ms
-

tooltip permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 552 kB 45.16ms - 46.48ms - slower ❌
0% - 4%
0.16ms - 1.69ms
branch 543 kB 44.51ms - 45.28ms faster ✔
0% - 4%
0.16ms - 1.69ms
-

test-directive permalink

Version Bytes Avg Time vs remote vs branch
npm latest 631 kB 36.03ms - 36.54ms - unsure 🔍
-0% - +2%
-0.13ms - +0.70ms
branch 632 kB 35.68ms - 36.33ms unsure 🔍
-2% - +0%
-0.70ms - +0.13ms
-

test-element permalink

Version Bytes Avg Time vs remote vs branch
npm latest 650 kB 63.67ms - 65.09ms - unsure 🔍
-0% - +2%
-0.31ms - +1.44ms
branch 642 kB 63.31ms - 64.33ms unsure 🔍
-2% - +0%
-1.44ms - +0.31ms
-

test-lazy permalink

Version Bytes Avg Time vs remote vs branch
npm latest 628 kB 53.65ms - 54.64ms - unsure 🔍
-1% - +2%
-0.57ms - +1.15ms
branch 619 kB 53.15ms - 54.55ms unsure 🔍
-2% - +1%
-1.15ms - +0.57ms
-

truncated permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 523 kB 57.13ms - 58.03ms - unsure 🔍
-1% - +2%
-0.29ms - +1.01ms
branch 515 kB 56.75ms - 57.68ms unsure 🔍
-2% - +0%
-1.01ms - +0.29ms
-
Firefox

accordion permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 415 kB 251.86ms - 263.06ms - unsure 🔍
-2% - +4%
-6.02ms - +10.70ms
branch 406 kB 248.92ms - 261.32ms unsure 🔍
-4% - +2%
-10.70ms - +6.02ms
-

action-bar permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 483 kB 135.52ms - 141.04ms - unsure 🔍
-6% - +1%
-8.38ms - +0.98ms
branch 474 kB 138.20ms - 145.76ms unsure 🔍
-1% - +6%
-0.98ms - +8.38ms
-

action-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 513 kB 207.97ms - 216.31ms - unsure 🔍
-3% - +3%
-6.08ms - +6.32ms
branch 504 kB 207.43ms - 216.61ms unsure 🔍
-3% - +3%
-6.32ms - +6.08ms
-

action-group permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 535 kB 146.97ms - 152.55ms - unsure 🔍
-3% - +2%
-4.95ms - +3.71ms
branch 526 kB 147.07ms - 153.69ms unsure 🔍
-2% - +3%
-3.71ms - +4.95ms
-

action-menu permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 645 kB 298.90ms - 301.18ms - faster ✔
6% - 8%
20.42ms - 24.90ms
branch 637 kB 320.77ms - 324.63ms slower ❌
7% - 8%
20.42ms - 24.90ms
-

test-directive permalink

Version Bytes Avg Time vs remote vs branch
npm latest 602 kB 152.71ms - 155.05ms - unsure 🔍
-0% - +2%
-0.72ms - +2.36ms
branch 594 kB 152.05ms - 154.07ms unsure 🔍
-2% - +0%
-2.36ms - +0.72ms
-

test-lazy permalink

Version Bytes Avg Time vs remote vs branch
npm latest 602 kB 171.40ms - 175.48ms - slower ❌
4% - 8%
5.92ms - 12.84ms
branch 593 kB 161.26ms - 166.86ms faster ✔
3% - 7%
5.92ms - 12.84ms
-

test-open-close-directive permalink

Version Bytes Avg Time vs remote vs branch
npm latest 785 kB 1470.83ms - 1476.17ms - slower ❌
2% - 2%
28.70ms - 35.06ms
branch 777 kB 1439.89ms - 1443.35ms faster ✔
2% - 2%
28.70ms - 35.06ms
-

test-open-close permalink

Version Bytes Avg Time vs remote vs branch
npm latest 783 kB 1462.69ms - 1466.51ms - unsure 🔍
-0% - +0%
-5.85ms - +0.77ms
branch 775 kB 1464.43ms - 1469.85ms unsure 🔍
-0% - +0%
-0.77ms - +5.85ms
-

alert-dialog permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 455 kB 188.58ms - 195.10ms - unsure 🔍
-4% - +1%
-7.62ms - +2.34ms
branch 442 kB 190.71ms - 198.25ms unsure 🔍
-1% - +4%
-2.34ms - +7.62ms
-

asset permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 359 kB 26.70ms - 28.98ms - unsure 🔍
-5% - +7%
-1.29ms - +1.97ms
branch 350 kB 26.33ms - 28.67ms unsure 🔍
-7% - +5%
-1.97ms - +1.29ms
-

avatar permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 396 kB 50.13ms - 52.87ms - unsure 🔍
-5% - +3%
-2.69ms - +1.57ms
branch 387 kB 50.43ms - 53.69ms unsure 🔍
-3% - +5%
-1.57ms - +2.69ms
-

badge permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 397 kB 70.05ms - 75.31ms - unsure 🔍
-4% - +6%
-2.95ms - +4.31ms
branch 388 kB 69.49ms - 74.51ms unsure 🔍
-6% - +4%
-4.31ms - +2.95ms
-

button-group permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 442 kB 124.52ms - 133.04ms - unsure 🔍
-5% - +5%
-5.83ms - +5.83ms
branch 433 kB 124.79ms - 132.77ms unsure 🔍
-5% - +5%
-5.83ms - +5.83ms
-

button permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 449 kB 138.99ms - 147.17ms - unsure 🔍
-3% - +6%
-3.73ms - +7.85ms
branch 439 kB 136.92ms - 145.12ms unsure 🔍
-5% - +3%
-7.85ms - +3.73ms
-

card permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 501 kB 99.37ms - 105.19ms - faster ✔
1% - 9%
0.76ms - 10.40ms
branch 493 kB 104.02ms - 111.70ms slower ❌
1% - 10%
0.76ms - 10.40ms
-

coachmark permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 626 kB 359.39ms - 365.37ms - unsure 🔍
-2% - +1%
-7.57ms - +4.13ms
branch 617 kB 359.07ms - 369.13ms unsure 🔍
-1% - +2%
-4.13ms - +7.57ms
-

combobox permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 705 kB 62.55ms - 68.37ms - slower ❌
6% - 16%
3.39ms - 9.29ms
branch 697 kB 58.63ms - 59.61ms faster ✔
6% - 14%
3.39ms - 9.29ms
-

light-dom-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 705 kB 711.61ms - 733.27ms - slower ❌
3% - 7%
20.05ms - 47.27ms
branch 697 kB 680.53ms - 697.03ms faster ✔
3% - 6%
20.05ms - 47.27ms
-

dialog permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 507 kB 110.26ms - 117.74ms - unsure 🔍
-5% - +3%
-5.83ms - +3.63ms
branch 498 kB 112.21ms - 117.99ms unsure 🔍
-3% - +5%
-3.63ms - +5.83ms
-

grid permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 458 kB 77.68ms - 81.04ms - unsure 🔍
-1% - +6%
-0.92ms - +4.72ms
branch 431 kB 75.19ms - 79.73ms unsure 🔍
-6% - +1%
-4.72ms - +0.92ms
-

infield-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 401 kB 43.06ms - 45.38ms - unsure 🔍
-3% - +4%
-1.52ms - +1.80ms
branch 392 kB 42.90ms - 45.26ms unsure 🔍
-4% - +3%
-1.80ms - +1.52ms
-

menu permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 473 kB 415.99ms - 427.29ms - unsure 🔍
-2% - +3%
-7.16ms - +10.52ms
branch 466 kB 413.17ms - 426.75ms unsure 🔍
-2% - +2%
-10.52ms - +7.16ms
-

number-field permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 521 kB 191.47ms - 200.37ms - unsure 🔍
-5% - +1%
-10.54ms - +1.82ms
branch 512 kB 195.99ms - 204.57ms unsure 🔍
-1% - +5%
-1.82ms - +10.54ms
-

overlay permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 763 kB 644.39ms - 650.69ms - slower ❌
2% - 3%
11.97ms - 20.11ms
branch 755 kB 628.91ms - 634.09ms faster ✔
2% - 3%
11.97ms - 20.11ms
-

directive-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 767 kB 65.12ms - 67.20ms - unsure 🔍
-1% - +2%
-0.85ms - +1.61ms
branch 759 kB 65.12ms - 66.44ms unsure 🔍
-2% - +1%
-1.61ms - +0.85ms
-

element-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 756 kB 612.45ms - 628.59ms - slower ❌
6% - 8%
32.65ms - 49.11ms
branch 748 kB 578.01ms - 581.27ms faster ✔
5% - 8%
32.65ms - 49.11ms
-

lazy-test permalink

Version Bytes Avg Time vs remote vs branch
npm latest 557 kB 112.70ms - 117.50ms - slower ❌
10% - 15%
10.30ms - 15.58ms
branch 549 kB 101.06ms - 103.26ms faster ✔
9% - 13%
10.30ms - 15.58ms
-

picker-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 413 kB 75.66ms - 80.94ms - unsure 🔍
-4% - +4%
-3.53ms - +3.33ms
branch 404 kB 76.22ms - 80.58ms unsure 🔍
-4% - +5%
-3.33ms - +3.53ms
-

picker permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 511 kB 1000.66ms - 1030.06ms - slower ❌
0% - 3%
1.16ms - 31.80ms
branch 503 kB 994.57ms - 1003.19ms faster ✔
0% - 3%
1.16ms - 31.80ms
-

popover permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 382 kB 39.36ms - 43.52ms - unsure 🔍
-11% - +3%
-4.85ms - +1.49ms
branch 373 kB 40.74ms - 45.50ms unsure 🔍
-4% - +12%
-1.49ms - +4.85ms
-

search permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 471 kB 93.63ms - 101.53ms - unsure 🔍
-9% - +0%
-9.61ms - +0.37ms
branch 462 kB 99.16ms - 105.24ms unsure 🔍
-1% - +10%
-0.37ms - +9.61ms
-

slider permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 477 kB 193.37ms - 198.59ms - unsure 🔍
-3% - +2%
-5.13ms - +3.05ms
branch 469 kB 193.88ms - 200.16ms unsure 🔍
-2% - +3%
-3.05ms - +5.13ms
-

split-button permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 718 kB 1428.90ms - 1432.42ms - unsure 🔍
-0% - +0%
-2.83ms - +2.47ms
branch 710 kB 1428.86ms - 1432.82ms unsure 🔍
-0% - +0%
-2.47ms - +2.83ms
-

tags permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 449 kB 39.46ms - 42.42ms - unsure 🔍
-5% - +6%
-1.89ms - +2.45ms
branch 439 kB 39.07ms - 42.25ms unsure 🔍
-6% - +5%
-2.45ms - +1.89ms
-

toast permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 430 kB 83.04ms - 88.68ms - unsure 🔍
-4% - +5%
-3.41ms - +4.41ms
branch 421 kB 82.65ms - 88.07ms unsure 🔍
-5% - +4%
-4.41ms - +3.41ms
-

tooltip permalink

test-basic

Version Bytes Avg Time vs remote vs branch
npm latest 650 kB 94.18ms - 98.94ms - slower ❌
1% - 14%
0.99ms - 12.25ms
branch 642 kB 84.84ms - 95.04ms faster ✔
1% - 13%
0.99ms - 12.25ms
-

test-directive permalink

Version Bytes Avg Time vs remote vs branch
npm latest 631 kB 66.90ms - 68.58ms - faster ✔
18% - 25%
15.14ms - 21.82ms
branch 622 kB 82.99ms - 89.45ms slower ❌
22% - 32%
15.14ms - 21.82ms
-

test-element permalink

Version Bytes Avg Time vs remote vs branch
npm latest 650 kB 129.93ms - 137.43ms - unsure 🔍
-3% - +3%
-4.65ms - +4.65ms
branch 642 kB 130.94ms - 136.42ms unsure 🔍
-3% - +3%
-4.65ms - +4.65ms
-

test-lazy permalink

Version Bytes Avg Time vs remote vs branch
npm latest 628 kB 107.42ms - 112.14ms - unsure 🔍
-5% - +1%
-5.48ms - +1.60ms
branch 619 kB 109.09ms - 114.35ms unsure 🔍
-1% - +5%
-1.60ms - +5.48ms
-

truncated permalink

basic-test

Version Bytes Avg Time vs remote vs branch
npm latest 523 kB 100.08ms - 107.76ms - unsure 🔍
-4% - +6%
-4.36ms - +5.88ms
branch 515 kB 99.77ms - 106.55ms unsure 🔍
-6% - +4%
-5.88ms - +4.36ms
-

@Westbrook
Copy link
Contributor

Great start here. Notice that the three files are now effectively the same. If there a path, similar to the Lit Dev Mode test that would allow for this boilerplate to be externalized?

@Westbrook
Copy link
Contributor

Should the structure here look to interact with the abstraction listed as To Dos in #1822?

@Rajdeepc Rajdeepc changed the title test: added memory tests in packages performance: added memory tests in packages Mar 29, 2024
@Rajdeepc
Copy link
Contributor Author

Rajdeepc commented Apr 1, 2024

Should the structure here look to interact with the abstraction listed as To Dos in #1822?

Yes!! I was was about to go into this next. I would like to take this up with the team to get this abstraction landed in the codebase so that we can formalize it for all the components. But the memory tests can be added in isolation for now with an empty object passed as a param for these components under To-do..

@Rajdeepc
Copy link
Contributor Author

Rajdeepc commented Apr 1, 2024

Great start here. Notice that the three files are now effectively the same. If there a path, similar to the Lit Dev Mode test that would allow for this boilerplate to be externalized?

Yes certainly, This has now been abstracted to a testing helper called testForMemoryLeaks

@Rajdeepc Rajdeepc self-assigned this Apr 1, 2024
@Rajdeepc Rajdeepc requested a review from Westbrook April 1, 2024 12:12
@Rajdeepc Rajdeepc marked this pull request as ready for review April 1, 2024 12:12
Copy link
Contributor

@Westbrook Westbrook left a comment

Choose a reason for hiding this comment

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

Please confirm that the helper is applicable to the grid and menu tests that were previously available.

@Rajdeepc
Copy link
Contributor Author

Rajdeepc commented Apr 2, 2024

Please confirm that the helper is applicable to the grid and menu tests that were previously available.

Grid and Menu has been refactored to use the testing helper

Copy link
Contributor

@Westbrook Westbrook left a comment

Choose a reason for hiding this comment

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

LGTM! Catching up to main and then we can merge.

@Westbrook Westbrook merged commit 50b2c22 into main Apr 2, 2024
48 checks passed
@Westbrook Westbrook deleted the memory/optimizations branch April 2, 2024 12:48
TarunAdobe pushed a commit that referenced this pull request Apr 26, 2024
* test: added memory tests in packages

* test: updated  memory tests in packages

* chore: asbtracted the memory profiler as a testing helper

* chore: added memory tests for components

* chore: update grid and menu to use testing helper

* chore: added mem test for card, button and button group

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>
Rajdeepc added a commit that referenced this pull request May 14, 2024
* refactor(theme): adds new system attribute and deprecates theme attribute

* chore(theme): made changes on theme attribute

* chore: reverted THEME Key to support migrations

* chore: theme registrated hard break

* refactor(theme): adds new system attribute and deprecates theme attribute

* chore(theme): made changes on theme attribute

* chore: reverted THEME Key to support migrations

* chore: theme registrated hard break

* chore(theme): updated theme property to system

* chore(theme): updated theme to system in sp-theme in layout

* chore(theme): updated config for VRT

* chore(theme): updated config for VRT from theme to system

* chore: update theme property to system

* fix: added spectrum system variant

* chore: updated theme for deprecation against system in theme files

* chore: updated config to support system

* chore: updated config yml

* chore: updated config yml

* chore: updated config yml

* chore: undo system changes in stories

* chore: updated runner utils

* chore: review to run with theme

* chore(theme): updated command-line arguments to ue system instea do f theme

* chore(theme): updated command-line arguments to ue system instea do f theme

* chore(theme): updated and cleaned up reduntant theme definitions with system

* chore: added theme fallback to the storydecotrator render

* chore: added system flag in the create review site job in CI

* chore: added system flag in the create review site job in CI

* chore: updated config to include system

* chore: removed classic from system variant

* chore: removed classic from system variant

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added theme  in parameters in config

* chore: reverted to theme in currentStyles

* chore: update styles for both system/theme changes

* chore: fixed updating theme when system value is not there

* chore: added t theme coverage tests

* chore: updated doc on prperty theme

* chore: updated golden image cache

* chore: updated golden image cache

* chore: check added to update system for theme only attribute during deprecation

* chore: rebased to systems pr

* feat: added s2 theme to storyDecorator

* chore: added s2 to documentation site

* chore: added spectrum-two to the visual testing pipeline

* chore: added s2 in web-test-util

* chore: added s2 theme to vrts

* chore: testing if vrts run

* chore: added spectrum-two-tokens-config in package.json

* chore: removed extra console log and updated tokens

* chore: trying s2 as a theme in vrt instead of spectrum-two

* chore: updated config

* chore: testing a hacky fix for s2 vrts

* chore: updated button and fieldlable to s2

* chore: added core.ts to import theme helpers inside spectrum-two

* chore: spectrum-two core export

* chore: testing if updating the global vars fixes the build

* chore: revert back css updates for button and field-label

* chore(styles): reapply some poorly cached build output (#4232)

* performance: added memory tests in packages (#4228)

* test: added memory tests in packages

* test: updated  memory tests in packages

* chore: asbtracted the memory profiler as a testing helper

* chore: added memory tests for components

* chore: update grid and menu to use testing helper

* chore: added mem test for card, button and button group

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>

* fix(coachmark): add "step-count" slot for custom/internationalized pagination content (#4215)

* chore(coachmark): added separator property

* chore(coachmark): updated tests

* chore(coachmark): updated golden image cache

* fix(coachmark): localized the step count

* chore(coachmark): updated golden image cache

* chore(coachmark): update readme and coachmark to default to english

* chore(coachmark): removed separator from the props

* chore(coachmark): updated to check for 1 of 4 stepcount context

* chore(coachmark): updated golden image cache

* chore(coachmark): added locallixation story for step count

* chore(coachmark): updated golden image cache

* chore(coachmark): update readme to include slot for docs site

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>

* chore: bump the commitlint-ecosystem group with 2 updates (#4219)

Bumps the commitlint-ecosystem group with 2 updates: [@commitlint/cli](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/cli) and [@commitlint/config-conventional](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/config-conventional).

Updates `@commitlint/cli` from 19.0.1 to 19.2.1
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/cli/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.2.1/@commitlint/cli)

Updates `@commitlint/config-conventional` from 19.0.0 to 19.1.0
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/config-conventional/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.1.0/@commitlint/config-conventional)

---
updated-dependencies:
- dependency-name: "@commitlint/cli"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
- dependency-name: "@commitlint/config-conventional"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* test: update benchmarking to configure "end" as "updateComplete" or "paint"

* fix(shared): ensure the "updateComplete" in Focusable is stable

* chore: release new versions #publish

* chore: bump webpack-dev-server from 5.0.2 to 5.0.4

Bumps [webpack-dev-server](https://github.com/webpack/webpack-dev-server) from 5.0.2 to 5.0.4.
- [Release notes](https://github.com/webpack/webpack-dev-server/releases)
- [Changelog](https://github.com/webpack/webpack-dev-server/blob/master/CHANGELOG.md)
- [Commits](webpack/webpack-dev-server@v5.0.2...v5.0.4)

---
updated-dependencies:
- dependency-name: webpack-dev-server
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump markdown-it from 14.0.0 to 14.1.0

Bumps [markdown-it](https://github.com/markdown-it/markdown-it) from 14.0.0 to 14.1.0.
- [Changelog](https://github.com/markdown-it/markdown-it/blob/master/CHANGELOG.md)
- [Commits](markdown-it/markdown-it@14.0.0...14.1.0)

---
updated-dependencies:
- dependency-name: markdown-it
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump fs-extra from 11.1.1 to 11.2.0

Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 11.1.1 to 11.2.0.
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@11.1.1...11.2.0)

---
updated-dependencies:
- dependency-name: fs-extra
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(picker): update event bind timing

* chore: release new versions #publish

* fix(color-area): providing x and y attributes renders color handle correctly (#4240)

* fix(color-area): providing x and y attributes renders color handle correctly

* fix(color-area): minor fix

* chore(menu): remove deprecated CSS file (#4249)

Co-authored-by: rmanole <rmanole@adobe.com>

* fix(combobox): correct package.json listings

* chore: bump webpack from 5.90.3 to 5.91.0

Bumps [webpack](https://github.com/webpack/webpack) from 5.90.3 to 5.91.0.
- [Release notes](https://github.com/webpack/webpack/releases)
- [Commits](webpack/webpack@v5.90.3...v5.91.0)

---
updated-dependencies:
- dependency-name: webpack
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump prettier-plugin-package from 1.3.0 to 1.4.0

Bumps [prettier-plugin-package](https://github.com/shellscape/prettier-plugin-package) from 1.3.0 to 1.4.0.
- [Release notes](https://github.com/shellscape/prettier-plugin-package/releases)
- [Commits](shellscape/prettier-plugin-package@v1.3.0...v1.4.0)

---
updated-dependencies:
- dependency-name: prettier-plugin-package
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump @types/react from 18.2.15 to 18.2.75

Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.2.15 to 18.2.75.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react)

---
updated-dependencies:
- dependency-name: "@types/react"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump protobufjs from 7.2.4 to 7.2.6

Bumps [protobufjs](https://github.com/protobufjs/protobuf.js) from 7.2.4 to 7.2.6.
- [Release notes](https://github.com/protobufjs/protobuf.js/releases)
- [Changelog](https://github.com/protobufjs/protobuf.js/blob/master/CHANGELOG.md)
- [Commits](protobufjs/protobuf.js@protobufjs-v7.2.4...protobufjs-v7.2.6)

---
updated-dependencies:
- dependency-name: protobufjs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump @11ty/eleventy in the eleventy-ecosystem group

Bumps the eleventy-ecosystem group with 1 update: [@11ty/eleventy](https://github.com/11ty/eleventy).


Updates `@11ty/eleventy` from 3.0.0-alpha.5 to 3.0.0-alpha.6
- [Release notes](https://github.com/11ty/eleventy/releases)
- [Changelog](https://github.com/11ty/eleventy/blob/main/docs/release-instructions.md)
- [Commits](https://github.com/11ty/eleventy/commits)

---
updated-dependencies:
- dependency-name: "@11ty/eleventy"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: eleventy-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump tar-stream from 3.1.6 to 3.1.7

Bumps [tar-stream](https://github.com/mafintosh/tar-stream) from 3.1.6 to 3.1.7.
- [Commits](mafintosh/tar-stream@v3.1.6...v3.1.7)

---
updated-dependencies:
- dependency-name: tar-stream
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: updated golden image hash

* chore: updated golden image hash

* chore: added sideeffectful registration of theme and removed lightest and darkest theme e examples

* chore: Configure Renovate (#4248)

* Add renovate.json

* chore: update renovate config

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Westbrook Johnson <wesjohns@adobe.com>

* chore: update dependency rollup to v4.16.2 (#4288)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: bump @11ty/eleventy in the eleventy-ecosystem group (#4291)

Bumps the eleventy-ecosystem group with 1 update: [@11ty/eleventy](https://github.com/11ty/eleventy).


Updates `@11ty/eleventy` from 3.0.0-alpha.6 to 3.0.0-alpha.7
- [Release notes](https://github.com/11ty/eleventy/releases)
- [Changelog](https://github.com/11ty/eleventy/blob/main/docs/release-instructions.md)
- [Commits](https://github.com/11ty/eleventy/commits/v3.0.0-alpha.7)

---
updated-dependencies:
- dependency-name: "@11ty/eleventy"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: eleventy-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: bump the commitlint-ecosystem group with 2 updates (#4290)

Bumps the commitlint-ecosystem group with 2 updates: [@commitlint/cli](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/cli) and [@commitlint/config-conventional](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/config-conventional).


Updates `@commitlint/cli` from 19.2.1 to 19.3.0
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/cli/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.3.0/@commitlint/cli)

Updates `@commitlint/config-conventional` from 19.1.0 to 19.2.2
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/config-conventional/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.2.2/@commitlint/config-conventional)

---
updated-dependencies:
- dependency-name: "@commitlint/cli"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
- dependency-name: "@commitlint/config-conventional"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: update chromedriver (#4293)

* chore: use latest playwright (#4295)

* chore: use latest playwright

* ci: update golden images cache

* chore: updated golden image hash

* chore: fixed merge conflicts

* chore: fixed merge conflicts

* chore: use system instead of theme in s2

* chore: externalised process spectrum script and update golden image hash

* chore: normalized spectrum-two var naming usage

* chore: updated golden image hash

* chore: testing something

* chore: freshly processed tokens after cleaning everything

* chore: fixed the generate-tokens script

* chore: updated golden image hash

* chore: updated golden image hash

* chore: normalized the classic theme to spectrum

* chore: updating baseline for spectrum

* chore: updated golden image hash

* chore: updated golden image hash

* chore: removed lightest and darkest theme for spectrum-two

* chore: fixed the ci config

* chore: updated golden image hash

* chore: fixed broken sidenav for vrts site

* chore: added devmode warning for S2 theme

* chore: show devmode warning on using s2 theme only in devmode

* chore: added devmode tests for code coverage

* chore: updated devmode tests for theme

* chore: update tests for theme

* chore: updated golden image cache

* chore: updated golden image cache

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Josh Johnson <jnjosh@adobe.com>
Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>
Co-authored-by: Westbrook Johnson <wesjohns@adobe.com>
Co-authored-by: Rajdeep Chandra <rajrock38@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Westbrook Johnson <westbrook.johnson@gmail.com>
Co-authored-by: Piyush Vashisht <piyush17303@iiitd.ac.in>
Co-authored-by: Roxana Burduja <13311865+Rocss@users.noreply.github.com>
Co-authored-by: rmanole <rmanole@adobe.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Rajdeepc added a commit that referenced this pull request Oct 16, 2024
* chore(theme): made changes on theme attribute

* chore: reverted THEME Key to support migrations

* chore: theme registrated hard break

* refactor(theme): adds new system attribute and deprecates theme attribute

* chore(theme): made changes on theme attribute

* chore: reverted THEME Key to support migrations

* chore: theme registrated hard break

* chore(theme): updated theme property to system

* chore(theme): updated theme to system in sp-theme in layout

* chore(theme): updated config for VRT

* chore(theme): updated config for VRT from theme to system

* chore: update theme property to system

* fix: added spectrum system variant

* chore: updated theme for deprecation against system in theme files

* chore: updated config to support system

* chore: updated config yml

* chore: updated config yml

* chore: updated config yml

* chore: undo system changes in stories

* chore: updated runner utils

* chore: review to run with theme

* chore(theme): updated command-line arguments to ue system instea do f theme

* chore(theme): updated command-line arguments to ue system instea do f theme

* chore(theme): updated and cleaned up reduntant theme definitions with system

* chore: added theme fallback to the storydecotrator render

* chore: added system flag in the create review site job in CI

* chore: added system flag in the create review site job in CI

* chore: updated config to include system

* chore: removed classic from system variant

* chore: removed classic from system variant

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added system in parameters in config

* chore: added theme  in parameters in config

* chore: reverted to theme in currentStyles

* chore: update styles for both system/theme changes

* chore: fixed updating theme when system value is not there

* chore: added t theme coverage tests

* chore: updated doc on prperty theme

* chore: updated golden image cache

* chore: updated golden image cache

* chore: check added to update system for theme only attribute during deprecation

* chore: rebased to systems pr

* feat: added s2 theme to storyDecorator

* chore: added s2 to documentation site

* chore: added spectrum-two to the visual testing pipeline

* chore: added s2 in web-test-util

* chore: added s2 theme to vrts

* chore: testing if vrts run

* chore: added spectrum-two-tokens-config in package.json

* chore: removed extra console log and updated tokens

* chore: trying s2 as a theme in vrt instead of spectrum-two

* chore: updated config

* chore: testing a hacky fix for s2 vrts

* chore: updated button and fieldlable to s2

* chore: added core.ts to import theme helpers inside spectrum-two

* chore: spectrum-two core export

* chore: testing if updating the global vars fixes the build

* chore: revert back css updates for button and field-label

* chore(styles): reapply some poorly cached build output (#4232)

* performance: added memory tests in packages (#4228)

* test: added memory tests in packages

* test: updated  memory tests in packages

* chore: asbtracted the memory profiler as a testing helper

* chore: added memory tests for components

* chore: update grid and menu to use testing helper

* chore: added mem test for card, button and button group

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>

* fix(coachmark): add "step-count" slot for custom/internationalized pagination content (#4215)

* chore(coachmark): added separator property

* chore(coachmark): updated tests

* chore(coachmark): updated golden image cache

* fix(coachmark): localized the step count

* chore(coachmark): updated golden image cache

* chore(coachmark): update readme and coachmark to default to english

* chore(coachmark): removed separator from the props

* chore(coachmark): updated to check for 1 of 4 stepcount context

* chore(coachmark): updated golden image cache

* chore(coachmark): added locallixation story for step count

* chore(coachmark): updated golden image cache

* chore(coachmark): update readme to include slot for docs site

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>

* chore: bump the commitlint-ecosystem group with 2 updates (#4219)

Bumps the commitlint-ecosystem group with 2 updates: [@commitlint/cli](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/cli) and [@commitlint/config-conventional](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/config-conventional).

Updates `@commitlint/cli` from 19.0.1 to 19.2.1
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/cli/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.2.1/@commitlint/cli)

Updates `@commitlint/config-conventional` from 19.0.0 to 19.1.0
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/config-conventional/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.1.0/@commitlint/config-conventional)

---
updated-dependencies:
- dependency-name: "@commitlint/cli"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
- dependency-name: "@commitlint/config-conventional"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* test: update benchmarking to configure "end" as "updateComplete" or "paint"

* fix(shared): ensure the "updateComplete" in Focusable is stable

* chore: release new versions #publish

* chore: bump webpack-dev-server from 5.0.2 to 5.0.4

Bumps [webpack-dev-server](https://github.com/webpack/webpack-dev-server) from 5.0.2 to 5.0.4.
- [Release notes](https://github.com/webpack/webpack-dev-server/releases)
- [Changelog](https://github.com/webpack/webpack-dev-server/blob/master/CHANGELOG.md)
- [Commits](webpack/webpack-dev-server@v5.0.2...v5.0.4)

---
updated-dependencies:
- dependency-name: webpack-dev-server
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump markdown-it from 14.0.0 to 14.1.0

Bumps [markdown-it](https://github.com/markdown-it/markdown-it) from 14.0.0 to 14.1.0.
- [Changelog](https://github.com/markdown-it/markdown-it/blob/master/CHANGELOG.md)
- [Commits](markdown-it/markdown-it@14.0.0...14.1.0)

---
updated-dependencies:
- dependency-name: markdown-it
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump fs-extra from 11.1.1 to 11.2.0

Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 11.1.1 to 11.2.0.
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@11.1.1...11.2.0)

---
updated-dependencies:
- dependency-name: fs-extra
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore(picker): update event bind timing

* chore: release new versions #publish

* fix(color-area): providing x and y attributes renders color handle correctly (#4240)

* fix(color-area): providing x and y attributes renders color handle correctly

* fix(color-area): minor fix

* chore(menu): remove deprecated CSS file (#4249)

Co-authored-by: rmanole <rmanole@adobe.com>

* fix(combobox): correct package.json listings

* chore: bump webpack from 5.90.3 to 5.91.0

Bumps [webpack](https://github.com/webpack/webpack) from 5.90.3 to 5.91.0.
- [Release notes](https://github.com/webpack/webpack/releases)
- [Commits](webpack/webpack@v5.90.3...v5.91.0)

---
updated-dependencies:
- dependency-name: webpack
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump prettier-plugin-package from 1.3.0 to 1.4.0

Bumps [prettier-plugin-package](https://github.com/shellscape/prettier-plugin-package) from 1.3.0 to 1.4.0.
- [Release notes](https://github.com/shellscape/prettier-plugin-package/releases)
- [Commits](shellscape/prettier-plugin-package@v1.3.0...v1.4.0)

---
updated-dependencies:
- dependency-name: prettier-plugin-package
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump @types/react from 18.2.15 to 18.2.75

Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.2.15 to 18.2.75.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react)

---
updated-dependencies:
- dependency-name: "@types/react"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump protobufjs from 7.2.4 to 7.2.6

Bumps [protobufjs](https://github.com/protobufjs/protobuf.js) from 7.2.4 to 7.2.6.
- [Release notes](https://github.com/protobufjs/protobuf.js/releases)
- [Changelog](https://github.com/protobufjs/protobuf.js/blob/master/CHANGELOG.md)
- [Commits](protobufjs/protobuf.js@protobufjs-v7.2.4...protobufjs-v7.2.6)

---
updated-dependencies:
- dependency-name: protobufjs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump @11ty/eleventy in the eleventy-ecosystem group

Bumps the eleventy-ecosystem group with 1 update: [@11ty/eleventy](https://github.com/11ty/eleventy).


Updates `@11ty/eleventy` from 3.0.0-alpha.5 to 3.0.0-alpha.6
- [Release notes](https://github.com/11ty/eleventy/releases)
- [Changelog](https://github.com/11ty/eleventy/blob/main/docs/release-instructions.md)
- [Commits](https://github.com/11ty/eleventy/commits)

---
updated-dependencies:
- dependency-name: "@11ty/eleventy"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: eleventy-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: bump tar-stream from 3.1.6 to 3.1.7

Bumps [tar-stream](https://github.com/mafintosh/tar-stream) from 3.1.6 to 3.1.7.
- [Commits](mafintosh/tar-stream@v3.1.6...v3.1.7)

---
updated-dependencies:
- dependency-name: tar-stream
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* chore: updated golden image hash

* chore: updated golden image hash

* chore: added sideeffectful registration of theme and removed lightest and darkest theme e examples

* chore: Configure Renovate (#4248)

* Add renovate.json

* chore: update renovate config

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Westbrook Johnson <wesjohns@adobe.com>

* chore: update dependency rollup to v4.16.2 (#4288)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: bump @11ty/eleventy in the eleventy-ecosystem group (#4291)

Bumps the eleventy-ecosystem group with 1 update: [@11ty/eleventy](https://github.com/11ty/eleventy).


Updates `@11ty/eleventy` from 3.0.0-alpha.6 to 3.0.0-alpha.7
- [Release notes](https://github.com/11ty/eleventy/releases)
- [Changelog](https://github.com/11ty/eleventy/blob/main/docs/release-instructions.md)
- [Commits](https://github.com/11ty/eleventy/commits/v3.0.0-alpha.7)

---
updated-dependencies:
- dependency-name: "@11ty/eleventy"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: eleventy-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: bump the commitlint-ecosystem group with 2 updates (#4290)

Bumps the commitlint-ecosystem group with 2 updates: [@commitlint/cli](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/cli) and [@commitlint/config-conventional](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/config-conventional).


Updates `@commitlint/cli` from 19.2.1 to 19.3.0
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/cli/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.3.0/@commitlint/cli)

Updates `@commitlint/config-conventional` from 19.1.0 to 19.2.2
- [Release notes](https://github.com/conventional-changelog/commitlint/releases)
- [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/config-conventional/CHANGELOG.md)
- [Commits](https://github.com/conventional-changelog/commitlint/commits/v19.2.2/@commitlint/config-conventional)

---
updated-dependencies:
- dependency-name: "@commitlint/cli"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
- dependency-name: "@commitlint/config-conventional"
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: commitlint-ecosystem
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: update chromedriver (#4293)

* chore: use latest playwright (#4295)

* chore: use latest playwright

* ci: update golden images cache

* chore: updated golden image hash

* chore: fixed merge conflicts

* chore: fixed merge conflicts

* chore: use system instead of theme in s2

* chore: externalised process spectrum script and update golden image hash

* chore: normalized spectrum-two var naming usage

* chore: updated golden image hash

* chore: testing something

* chore: freshly processed tokens after cleaning everything

* chore: fixed the generate-tokens script

* chore: updated golden image hash

* chore: updated golden image hash

* chore: normalized the classic theme to spectrum

* chore: updating baseline for spectrum

* chore: updated golden image hash

* chore: updated golden image hash

* chore: removed lightest and darkest theme for spectrum-two

* chore: fixed the ci config

* chore: updated golden image hash

* chore: fixed broken sidenav for vrts site

* chore: added devmode warning for S2 theme

* chore: show devmode warning on using s2 theme only in devmode

* chore: added devmode tests for code coverage

* chore: updated devmode tests for theme

* chore: updated tokens package

* chore: updated the process-spectrum task to process the legacy overrides

* chore: reverted for testing bridge tokens remapping

* chore: removed action-button updates

* chore: updated golden image hash

* chore: fixed global vars

* chore: rename legacy to system overrides

* chore: testing foundation tokens for components

* chore: added foundation version

* chore: s2 foundation release

* chore: updated s2 foundations version

* chore: corrected ui-icons foundations version

* chore: added bridges for all components

* chore: removed overrides from accordion

* chore: fresh yarn after merging main

* chore: updated foundations to 0.6

* chore: updated wrapper to take bridge tokens from foundations

* chore: fixed dialog css for foundations release

* chore: updated to foundations 8

* chore: updated scripts and foundations to .9

* chore: downgraded lightning css

* chore: setup errorrecovery to warning in lightning css

* chore: updated process-spectrum

* chore: upgrade typography to foundations

* chore: updated CSS for spectrum-overrides

* chore: updated to index-base.css

* chore: added missing system-overrides imports

* chore: added missing system-overrides imports

* chore: updated css processing script

* chore: fixed broken overrides import

* chore: removed extra override imports

* chore: included icon overrides in component CSS

* chore: updated vars to process s2 foundations

* chore: updated to foundations 10

* chore: added coach-indicator to generate tokens

* chore: updated alert banner overrides

* chore: removed console log

* chore: fixed alert-banner css version and updated dialog css

* chore: updated css fix for action button

* chore: updated to foundations 10 anf added alert banner overrides

* chore: updated to foundations 10

* chore: updated to foundations 11

* chore: upgraded typography to foundations .12

* chore: updated tokens to foundations.12

* chore: updated to foundations 13

* chore: updated to foundations.13 for checkbox and meter

* chore: updated tokens to foundations.14

* chore: updated action-button to foundations.14 and accordion to foundations.13

* chore: updated to latest .15 css release

* chore: updated theme.css to support spectrum typography

* chore: added errorRecovery true to avoid lightningcss failures

* chore: updated action-bar to foundations 13

* chore: updated radio to foundations 12

* chore: updated action-bar css based on latest package

* chore: fixed process-spectrum script

* chore: updated dialog css

* chore: added taggroup for global overrides

* chore: added chevron icon overrides to tabs CSS

* chore: updated combo-box to foundations 13

* chore: fresh yarn

* chore: updated textfield to foundations 12, and drop-zone  and combo-box to 13

* fix: ensured asset in coach-mark didn't interfere with image-wrapper and image sizing

* chore: updated thumbnail, toast and underlay components to foundations.12

* chore: updated css

* chore: updated to foundations 15 for action-button and 13 for combobox

* chore: fixed toast css dependency version

* chore: removed console log

* chore: updated to latest tokens version

* chore: updated tokens to foundations 17

* chore: included typography imports for all systems

* chore: set spectrum as default for storybook

* chore: updated color-wheel css processing

* chore: updated sp-theme background color to background-base-color for storybook design review

* chore: added a getter function to switch to new background color for spectrum-two in storybook

* chore: combo-box quiet variant config update

* chore: updated css to support mods

* chore: overrides import mappings specificty change

* chore: removed picker changes

* chore: updated to latest foundations

* fix: component-overrides.css should be listed after spectrum-component.css

* chore: updated foundations

* chore: updated foundations

* chore: updated tokens-v2

* chore: cascading pickers size to its icon

* chore: updated modal, button mapping css

* chore: updated actionbutton version to latest

* chore: fixed action-button config

* chore: action button edge to visual custom css

* fix(picker): picker CSS does not need to set '--spectrum-icon-size'

* chore: updated spectrum-css versions to latest release

* chore: updated foundations

* chore: updated foundations

* chore: added breadcrumb component generate tokens preprocessing

* chore: added opacitycheckerboard to preprocessing script

* chore: fixed icon sizing and accordion item border

* chore: fixed menu checkmark styles

* chore: fix menu spectrum-config

* fix: breadcrumbs icon overrides css

* fix: action-menu icon size inherit

* chore: s2 radii (#4800)

* chore: updated corner radii tokens of components

* chore: updated to latest foundations

* chore: updated to latest css version

* chore: s2 icons (#4747)

* chore: testing s2 workflow icons delivery

* chore: updated gitignore

* chore: added build-s2

* chore: added s2 ui-icons

* chore: testing the switch for icons

* chore: consolidated the buildscript for workflow icons

* chore: fixed the default icon condition

* feat: add system context to icon base

* chore: updated redundant vars

* chore: process s2 icons-ui

* chore: updated default icon

* fix: update default icon size

* chore: update workflow and ui icons

* chore: updated yarn lock

* chore: updated node version (#4802)

* chore: updated node version

* chore: updated readme

---------

Co-authored-by: Ruben Carvalho <rcarvalho@adobe.com>
Co-authored-by: Rajdeep Chandra <rajdeepchandra@rajdeeps-mbp-2.macromedia.com>

* chore: added disclaimer to icons-s2

---------

Co-authored-by: Rajdeep Chandra <rajdeepchandra@rajdeeps-mbp-2.macromedia.com>
Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>
Co-authored-by: TarunAdobe <ttomar@adobe.com>
Co-authored-by: Ruben Carvalho <rcarvalho@adobe.com>

* fix(search): fixed icon sizing issue

* chore: set m as default sizing for search

* chore: fixed wrong css property in colorwheel

* chore: fixed ui & workflow icon missing quotation marks

* chore: test menu item hover fix

* chore: updated tokens version and added typography definitions

* fix(number-field): add missing stepper conversion config and variable definition (#4816)

* chore(number-field): define stepper-width variable for host

* chore(number-field): add stepper-size-m conversion config

* chore: inherit icon size for button and splitbutton

* fix: menu hover css properties in tokens (#4821)

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>

* fix: add icon-search for s1 and s2 (#4827)

Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Rajdeep Chandra <rajdeepchandra@Rajdeeps-MacBook-Pro-2.local>
Co-authored-by: Rajdeep Chandra <rajrock38@gmail.com>
Co-authored-by: Rajdeep Chandra <rajdeepchandra@rajdeeps-mbp-2.macromedia.com>
Co-authored-by: Ruben Carvalho <rcarvalho@adobe.com>
Co-authored-by: Rúben Carvalho <rubcar@sapo.pt>
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.

[testing] Abstract and apply memory usage testing across the repo
2 participants