0
Commit Graph

13 Commits

Author SHA1 Message Date
Christian Flach
9ed633ccbc iwa: Move Isolated Web App docs to /docs
Bug: None
Change-Id: I8563fb97c12c9c1e3c1033228631fc2eb0cbe224
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4047288
Commit-Queue: Christian Flach <cmfcmf@chromium.org>
Reviewed-by: Phillis Tang <phillis@chromium.org>
Auto-Submit: Christian Flach <cmfcmf@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1075065}
2022-11-23 09:28:40 +00:00
Dibyajyoti Pal
5bb546af5c [dPWA] Introduce manifest update documentation
This CL adds documentation for the manifest updaate process.

Bug: b/256847062
Change-Id: I4c602cf98d1b446a7a7b237e6697780915bdde6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4021345
Reviewed-by: Phillis Tang <phillis@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Dibyajyoti Pal <dibyapal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1073614}
2022-11-18 23:05:09 +00:00
Daniel Murphy
f296b367b2 [dPWA] Documentation pass
Change-Id: Ib37e5fd25f8c7e6014013cc30285ab3dd5f83a66
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4011202
Reviewed-by: Dibyajyoti Pal <dibyapal@chromium.org>
Reviewed-by: Phillis Tang <phillis@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1073071}
2022-11-17 22:06:16 +00:00
Daniel d'Andrada
73ace7386b Replace "*WebAppIntegration_*" with "WebAppIntegration*"
The former gtest_filter doesn't not yield any tests.

Change-Id: Ib2564dcc3e7d33715f668d8dccc7ef01501e90e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3891015
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Auto-Submit: Daniel d'Andrada <dandrader@google.com>
Cr-Commit-Position: refs/heads/main@{#1045863}
2022-09-12 16:23:47 +00:00
Daniel Murphy
87bb89cb94 [dPWA Testing] Refactor & simplification pass on tests
This change:
* Changes the site names to be more specific. For example,
  SiteA -> Standalone.
* Updates some test actions to take advantage of the new wildcard
  feature.
* Updates other tests to use new actions, deleting unnecessary ones.
* Other misc cleanup.
* Shortening names now that the test names are too long.

Bug: b/242063423
Change-Id: Iaf80d30ca6ea0653cca382d9d4f91ed58f9cd07c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3828313
Reviewed-by: Phillis Tang <phillis@chromium.org>
Reviewed-by: Dana Fried <dfried@chromium.org>
Reviewed-by: Dibyajyoti Pal <dibyapal@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1036815}
2022-08-18 22:20:46 +00:00
Daniel Murphy
6ad32d6995 [dPWA Testing] Adding parameter wildcards to test generation framework
This change allows critical user journey to specify a wildcard like
"EnumType::All", which will cause the script to expand that single test
into multiple tests for every combination of argument expansion.

This allows us to continue to simplify our test list and implementation.

Bug: b/240571429
Change-Id: I4c45cea08048ab817d246c4208ac428c797e8ff6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3827079
Commit-Queue: Dibyajyoti Pal <dibyapal@chromium.org>
Reviewed-by: Dibyajyoti Pal <dibyapal@chromium.org>
Auto-Submit: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1035189}
2022-08-15 20:40:27 +00:00
Daniel Murphy
0ffd42c8c2 [dPWA Testing] Added why-is-this-test-failing.md doc
Draft message to update all tests with:
// Test contents are generated by script. Please do not modify!
// See `docs/webapps/why-is-this-test-failing.md` or
// `docs/webaps/integration-testing-framework` for more info.
// Sheriffs: Disabling this test is supported.

Bug: 1342424
Change-Id: I9c09822c66a4e704a7b789bb60580e756d189c85
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3750044
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Glen Robertson <glenrob@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1027444}
2022-07-22 22:12:25 +00:00
Zhaoyang Li
5d3713d60d Create mac12-arm64-rel and mac12-arm64-rel-tests
Added the 2 builders with the same configs as mac 11 version, except for
specified machine versions. mac12-arm64-rel will replace the mac 11
equivalent in CQ in a follow up CL.

Bug: 1334005
Change-Id: I8de42744410652a57685edd2c60d01afd26cf770
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3699392
Reviewed-by: Sven Zheng <svenzheng@chromium.org>
Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1013557}
2022-06-13 18:01:17 +00:00
Daniel Murphy
fd7822a50d [dPWA Testing] Moving to markdown table for CUJs, actions, etc.
This change completely removes the dependency on the old spreadsheet
for the dPWA product critical user journeys, and puts them into
markdown files.

Bug: 1314822
Change-Id: I4555d78a0e167bfe70512203ea1370269c3f48c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3574811
Auto-Submit: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Phillis Tang <phillis@chromium.org>
Commit-Queue: Phillis Tang <phillis@chromium.org>
Cr-Commit-Position: refs/heads/main@{#991115}
2022-04-11 18:39:29 +00:00
Daniel Murphy
81a05f586a [dPWA Testing] Clarify step in test generation.
Change-Id: Ieb46f26a0c3825a319ff4c926d18bcebe4a7a9a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3549820
Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Auto-Submit: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/main@{#989963}
2022-04-07 16:22:01 +00:00
Daniel Murphy
d8a8ce9a8d [dPWA Testing] Upgrade test script to support multiple arguments
This patch updates the dPWA testing framework script to allow for
multiple arguments in the CUJ actions.

Behavior changes:
* Actions not specify a list of argument (or enumeration) types.
* These types are defined in a new sheet titled "Enums" with values.
* Parameterized actions have a new format to specify the output action,
  where it uses bash argument syntax ($1 for the first arg, $2 for the
  second, etc) to forward arguments of the parameterized action to the
  output actions.

This change does not affect the tests themselves or coverage, and
should be a no-op for test generation.

Future changes:
* After this change has landed, the current cpp method call of
  MethodName("Argument") can change to be MethodName(ArgType::kArg),
  providing a much better coding interface to work with.

Bug: 1310428
Change-Id: I2d6271e08ad2d19b2d4c2dd6ac87df9fe4c8b02d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3553829
Reviewed-by: Phillis Tang <phillis@chromium.org>
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/main@{#986982}
2022-03-30 14:51:36 +00:00
Phillis Tang
55bb5e7063 DPWA: update doc about when to add integration tests
Change-Id: I340f02d852325a7cf3705c0f64f39822f0bdd50a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3457685
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Phillis Tang <phillis@chromium.org>
Cr-Commit-Position: refs/heads/main@{#970159}
2022-02-11 21:59:58 +00:00
Daniel Murphy
27a6bd024d [dPWA Testing] Moved docs and added test addition how-to.
This change updates a lot of documentation, adds a how-to guide for
adding new integration tests, and starts the documentation move to
docs/webapps.

Bug: 1264155
Change-Id: I44fbd3e79f4153c91b977170671d37d503d868fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3256208
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Reviewed-by: Phillis Tang <phillis@chromium.org>
Cr-Commit-Position: refs/heads/main@{#937979}
2021-11-03 18:58:46 +00:00