summaryrefslogtreecommitdiffstats
path: root/testing/chromoting
Commit message (Collapse)AuthorAgeFilesLines
* Temporarily increase verbosity of Chromoting test-driver execution logs, to ↵anandc2015-12-181-2/+2
| | | | | | | | | | | | get more info on random Signaling errors. With this change, test log-size increases from ~32K to ~522K; we hope to revert to the previous less-verbose level soon. BUG=570840 Review URL: https://codereview.chromium.org/1534283002 Cr-Commit-Position: refs/heads/master@{#366190}
* Add parameter passing in host JID to Test-Driver commands.anandc2015-12-143-10/+39
| | | | | | | | | | Before launching a test, scan most recent host log on test bot for the JID used to register host. Pass that JID as a command-line parameter to the Choromoting test-driver tests. BUG=543880 Review URL: https://codereview.chromium.org/1516433006 Cr-Commit-Position: refs/heads/master@{#365052}
* Clean up user-profile dir when re-running test on host JID mismatch.anandc2015-10-152-7/+13
| | | | | | | | | | When the connect-to-remote host test was retried upon a host-JID mismatch, the re-attempt would always fail because of junk left in the user-profile dir being used by the test. This fixes that problem. BUG= Review URL: https://codereview.chromium.org/1395693010 Cr-Commit-Position: refs/heads/master@{#354347}
* Fix exception for Chromoting integration tests.anandc2015-10-123-30/+39
| | | | | | | | | | Also, ensure temporary host-logs get deleted even when a test fails. BUG= Review URL: https://codereview.chromium.org/1399553004 Cr-Commit-Position: refs/heads/master@{#353539}
* Obtain host-log file-names from stdout of host-start process.anandc2015-10-092-25/+128
| | | | | | | | | | | | Recently, the location where me2me host logs are written changed on the Swarming bots (root-cause not yet clear). To be more resilient to such changes, to get the host log file-name we'll parse the output of the process that starts the host. Also, in order to rule out replication delay as a cause of test failures, we'll check the JID used by a test and, if it is different from the JID logged by the host for that execution, sleep for 30-seconds and rerun the test if it failed. BUG= Review URL: https://codereview.chromium.org/1375373006 Cr-Commit-Position: refs/heads/master@{#353409}
* Add OWNERS file to testing/chromoting.anandc2015-10-091-0/+4
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1399003003 Cr-Commit-Position: refs/heads/master@{#353363}
* Removing Test and Staging environments from App Remoting Test Driver tool.joedow2015-09-221-1/+0
| | | | | | | | | | | Since these environments are no longer present, I am removing them from the test tool to prevent anyone from attempting to target them. BUG= Review URL: https://codereview.chromium.org/1357973002 Cr-Commit-Position: refs/heads/master@{#350256}
* Removing App Remoting build target from waterfall build file.joedow2015-09-221-1/+0
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1345413003 Cr-Commit-Position: refs/heads/master@{#350165}
* Fix chromoting_browser_tests when run isolated.maruel2015-09-181-1/+1
| | | | | | | | | | | File case is sensitive on all OSes. R=anandc@chromium.org BUG= Review URL: https://codereview.chromium.org/1349283003 Cr-Commit-Position: refs/heads/master@{#349709}
* Updating App Remoting Test Driver to target the dev environment.joedow2015-09-011-1/+1
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1323083003 Cr-Commit-Position: refs/heads/master@{#346732}
* Separating the two tests into different executions.tonychun2015-08-131-1/+2
| | | | | | | | | | By specifying the gtest_filter to run the two tests separately, we will be restart the host, which allows for independence between the two tests. If the host is not restarted before the 2nd test, the first connection numbers that it obtains is already a reconnect. BUG= Review URL: https://codereview.chromium.org/1295533002 Cr-Commit-Position: refs/heads/master@{#343290}
* Using chromoting_integration_tests.isolate for chromoting test driver.tonychun2015-08-1110-268/+451
| | | | | | | | | | | | | Added my own launcher and renamed chromoting_integration_tests.isolate to browser_integration_tests.isolate. I've added chromoting_test_driver_commands_linux.txt to make it easy to add tests and run them for different hosts. Note: These changes should land after the changes to internal have been made. Some of the files specified in the isolate require those files. BUG= Review URL: https://codereview.chromium.org/1272713006 Cr-Commit-Position: refs/heads/master@{#342762}
* Custom install me2me host, built as part of chromoting-integration-tests, on ↵anandc2015-08-083-15/+19
| | | | | | | | | | | | | | | | | | Linux Swarming bots. Instead of installing the me2me debian package, we'll temporarily place the required files in a custom location and run a subset of tests with it. Because the me2me host is launched from a custom location, the local-machine shows up as a remote-machine, so we'll now run connect-to-remote-host test. Most of the IT2Me tests fail because we have to update the path to the native-messaging-host in the manifest file. I tested with the manifest files thus modified but couldn't get the tests passing on the Swarming bots, so will investigate that separately (noted in tracking bug). This version should get at least some tests passing again on the Linux builder. This whole exercise is a temporary solution until we get a proper method in place to update the host. That work is tracked in http://crbug/461920 BUG=517695 Review URL: https://codereview.chromium.org/1279823004 Cr-Commit-Position: refs/heads/master@{#342516}
* Remove legacy CRD application types.sergeyu2015-08-042-3/+1
| | | | | | | | | 1. Removed CRD v1 app. 2. Removed support for native plugin from the webapp. Review URL: https://codereview.chromium.org/1251273002 Cr-Commit-Position: refs/heads/master@{#341794}
* Revert "Remove legacy CRD application types."Mitsuru Oshima2015-08-042-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | Reason for revert: TSan bot build failuire: KeyError: '/mnt/data/b/build/slave/Chromium_Linux_TSan_Builder/build/src/remoting/remoting.gyp:remoting_webapp#target' Error: Command '/usr/bin/python src/build/gyp_chromium' returned non-zero exit status 1 in /mnt/data/b/build/slave/Chromium_Linux_TSan_Builder/build > > Remove legacy CRD application types. > > 1. Removed CRD v1 app. > 2. Removed support for native plugin from the webapp. > > Review URL: https://codereview.chromium.org/1251273002 > > Cr-Commit-Position: refs/heads/master@{#341635} BUG=516539 TBR=sergeyu@chromium.org Review URL: https://codereview.chromium.org/1270693003 . Cr-Commit-Position: refs/heads/master@{#341684}
* Remove legacy CRD application types.sergeyu2015-08-032-3/+1
| | | | | | | | | 1. Removed CRD v1 app. 2. Removed support for native plugin from the webapp. Review URL: https://codereview.chromium.org/1251273002 Cr-Commit-Position: refs/heads/master@{#341635}
* Increasing logging level to help track down intermittent app remoting ↵joedow2015-07-271-1/+2
| | | | | | | | | | connection issue. BUG= Review URL: https://codereview.chromium.org/1254723011 Cr-Commit-Position: refs/heads/master@{#340545}
* Remove 'read_only': 1 from most .isolate files.maruel2015-07-201-1/+0
| | | | | | | | | | | | | | | These were mostly due to blind copy-paste. read_only:1 is the default so there's no point in specifying it. As per crbug.com/342913, only unit_tests has issue with running from read only files. R=jam@chromium.org BUG=116251 Review URL: https://codereview.chromium.org/1240303002 Cr-Commit-Position: refs/heads/master@{#339490}
* Perform download of files from GCS on only Linux.anandc2015-07-161-23/+27
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1238973008 Cr-Commit-Position: refs/heads/master@{#339159}
* Remove file expected to be downloaded from GCS from Chromoting isolate.anandc2015-07-161-1/+0
| | | | | | | | | | This is a temporary change until we figure out why the download action is not being performed for every build. BUG= Review URL: https://codereview.chromium.org/1234543004 Cr-Commit-Position: refs/heads/master@{#338999}
* Fix 'gsutil: not found' error; use gsutil.py instead.anandc2015-07-151-3/+2
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1239583005 Cr-Commit-Position: refs/heads/master@{#338912}
* Add functionality to download files from GCS, and include them in the ↵anandc2015-07-154-0/+85
| | | | | | | | | | | | Chromoting isolate. For now, this is just downloading 1 file and adding them to the isolate, and only on the Linux builder. Once that is confirmed working reliably on our waterfall, we can add more. BUG= Review URL: https://codereview.chromium.org/1231663003 Cr-Commit-Position: refs/heads/master@{#338816}
* Updating App Remoting Test Driver isolate params to help debugging.joedow2015-07-081-1/+3
| | | | | | | | | | | | | I'm adjusting the verbosity and adding the show-host-availability flag to make debugging easier if the test fails. I've also added the release-hosts flag so that each test run will occur against a fresh host. BUG= Review URL: https://codereview.chromium.org/1224893003 Cr-Commit-Position: refs/heads/master@{#337756}
* Updating Chromoting Waterfall tests to build App Remoting Test Driver for Linux.joedow2015-06-251-18/+32
| | | | | | | | | | | | | | | | | | | | | | The OS condition in the isolate file prevented tests from running however the target in our GYP file was still built. This caused a problem for mac and windows since I am not building the tool for those platforms. My change here is to use an 'all' pattern in the all.gyp file which will simplify bringing additional waterfall targets online in the future. Using this mechanism also allows us to move our platform conditional logic into our GYP file (keeps all.GYP cleaner). I have also added an OS condition for the app remoting test driver and an overridable variable for our multi machine example tests (so they do not run by default). BUG= Review URL: https://codereview.chromium.org/1212563002 Cr-Commit-Position: refs/heads/master@{#336290}
* Chromoting swarming file(s) maintenance.joedow2015-06-242-9/+21
| | | | | | | | | | | | | I re-ordered the targets in the gyp file so they are listed alphabetically and I've also updated the isolate command to use a python script wrapper that was suggested in the chromium swarming best practices guide. BUG= Review URL: https://codereview.chromium.org/1202783002 Cr-Commit-Position: refs/heads/master@{#335975}
* Initial CL for addding App Remoting Test Driver to the chromoting waterfall.joedow2015-06-203-0/+34
| | | | | | | | | | | | | | This change adds the first set of tests to the chromoting waterfall for the app remoting test driver. There was one other change needed to support this work. The refresh token store was not handling relative paths correctly which prevented me from passing in a token file from a sibling folder. BUG= Review URL: https://codereview.chromium.org/1192753002 Cr-Commit-Position: refs/heads/master@{#335409}
* Stop running browser-tests for v1 web-app.anandc2015-06-161-8/+4
| | | | | | | | | | Also updated IT2Me browser-tests to use v2 web-app. 2 of these tests fail, MANUAL_InvalidAccessCode and MANUAL_CancelShare, tracked in bug https://code.google.com/p/chromium/issues/detail?id=500687 BUG= Review URL: https://codereview.chromium.org/1187073004 Cr-Commit-Position: refs/heads/master@{#334701}
* Retry Chromoting browser-test execution if test times-out without launching ↵anandc2015-06-102-31/+65
| | | | | | | | | | browser. BUG=480025 Review URL: https://codereview.chromium.org/1172183003 Cr-Commit-Position: refs/heads/master@{#333646}
* Chromium changes to statically link ffmpeg.chcunningham2015-06-091-3/+0
| | | | | | | | | | | | | | | | | | | | Motivations: 1. Eliminates a large number of windows crashes hit while loading the DLL (see bugs) 2. Reducing the size of binary (about 400KB on linux, about 265KB on 32bit windows) due to dead-code elimination 3. Simplifying media library initialization (Remove bool-ness, it just works) Licensing: We've audited all the licenses for the files we actually use and have the green light from legal as long as we include the license text in the credits. BUG=435455, 429131, 441908 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Committed: https://crrev.com/bfef6d6ba82b172ac0e48c80d9ccf1a1ca0ea5c5 Cr-Commit-Position: refs/heads/master@{#332891} Review URL: https://codereview.chromium.org/1141703002 Cr-Commit-Position: refs/heads/master@{#333423}
* Revert "Chromium changes to statically link ffmpeg."chcunningham2015-06-051-0/+3
| | | | | | | | | | | | | | | | Reverting because of component test failures around isolate expectations. This change may not actually be directly responsible, but I'm out of time to investigate for the day. Will re-land it soon. BUG=496975 TBR=enne@chromium.org, gunsch@chromium.org, jochen@chromium.org, isherman@chromium.org, thomasvl@chromium.org, phajdan@chromium.org, grt@chromium.org, mmoss@chromium.org, sergeyu@chromium.org, sky@chromium.org, dalecurtis@chromium.org This reverts commit bfef6d6ba82b172ac0e48c80d9ccf1a1ca0ea5c5. NOPRESUBMIT=true NOTRY=true Review URL: https://codereview.chromium.org/1161183003 Cr-Commit-Position: refs/heads/master@{#332994}
* Chromium changes to statically link ffmpeg.chcunningham2015-06-041-3/+0
| | | | | | | | | | | | | | | | | Motivations: 1. Eliminates a large number of windows crashes hit while loading the DLL (see bugs) 2. Reducing the size of binary (about 400KB on linux, about 265KB on 32bit windows) due to dead-code elimination 3. Simplifying media library initialization (Remove bool-ness, it just works) Licensing: We've audited all the licenses for the files we actually use and have the green light from legal as long as we include the license text in the credits. BUG=435455, 429131, 441908 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1141703002 Cr-Commit-Position: refs/heads/master@{#332891}
* Enable verbose logging for Chromoting browser-tests, to understand more ↵anandc2015-05-261-18/+16
| | | | | | | | | | | | | | about failures where the browser itself seems to not launch (see bug for more info). Also update filters to take advantage of sharding where we can. The cases where this is applicable are not too many, because we use a specific user-data-dir where some required files must be placed. With this change, our test logs-size increases from ~450KB, to ~16MB. This should however be only a temporary change. Once we determine whether or not the additional information tells us something actionable, we'll go back to the non-verbose logging. BUG=480025 Review URL: https://codereview.chromium.org/1153973002 Cr-Commit-Position: refs/heads/master@{#331311}
* Temporarily disable AUTH test on Mac.anandc2015-05-091-2/+17
| | | | | | | | BUG=485328 Review URL: https://codereview.chromium.org/1131923006 Cr-Commit-Position: refs/heads/master@{#329044}
* Implement browser test for Me2Me retry logic when a host's JID is stale.kelvinp2015-05-071-0/+1
| | | | | | | | BUG=478331 Review URL: https://codereview.chromium.org/1126543007 Cr-Commit-Position: refs/heads/master@{#328682}
* Fix Chromoting browser-tests failure on Windows builders.anandc2015-05-071-0/+7
| | | | | | | | | | Include kasko.dll if Kasko, a memory-violation reporting tool, is enabled. BUG= Review URL: https://codereview.chromium.org/1115343003 Cr-Commit-Position: refs/heads/master@{#328677}
* Add more logging to browser-tests.anandc2015-05-052-17/+31
| | | | | | | | BUG=480025 Review URL: https://codereview.chromium.org/1120743004 Cr-Commit-Position: refs/heads/master@{#328322}
* A side-effect of this CL to summarise failing tests: ↵anandc2015-04-211-4/+3
| | | | | | | | | | https://codereview.chromium.org/1060613003/, was to lose information on output of failing tests. This CL restores that by recording the output of tests that thrown an exception during execution. BUG= Review URL: https://codereview.chromium.org/1096143004 Cr-Commit-Position: refs/heads/master@{#326028}
* Summarise failing tests towards the end of test execution logs.anandc2015-04-131-10/+47
| | | | | | | | | | Also, stop-start me2me host at the end of each test, to ensure all tests start from close to same state. BUG= Review URL: https://codereview.chromium.org/1060613003 Cr-Commit-Position: refs/heads/master@{#324852}
* Updating dependencies file to use the new location of the internal app ↵joedow2015-04-071-1/+1
| | | | | | | | | | remoting GYP files. BUG= Review URL: https://codereview.chromium.org/1067003002 Cr-Commit-Position: refs/heads/master@{#324096}
* Start running script that validates app-streaming JSON configs.anandc2015-04-042-1/+3
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1059893002 Cr-Commit-Position: refs/heads/master@{#323868}
* Factoring out common items into a gypi file.mmeade2015-04-032-32/+27
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1030393002 Cr-Commit-Position: refs/heads/master@{#323748}
* Stop host and clean-up user-profile dir even on test failures.anandc2015-04-011-21/+23
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1045163002 Cr-Commit-Position: refs/heads/master@{#323336}
* Specify whole directory as a dependency, rather than individual files.anandc2015-03-261-11/+1
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1030013004 Cr-Commit-Position: refs/heads/master@{#322308}
* Add pnacl dependencies to get browser-tests passing again.anandc2015-03-251-0/+13
| | | | | | | | BUG=470158 Review URL: https://codereview.chromium.org/1031213002 Cr-Commit-Position: refs/heads/master@{#322227}
* Include host-logs in browser-test results even if all tests pass.anandc2015-03-231-8/+10
| | | | | | | | | | This should help with diff-ing the contents of the host-logs when tests fail. BUG= Review URL: https://codereview.chromium.org/1016273004 Cr-Commit-Position: refs/heads/master@{#321901}
* Build app_streaming apps only on Linux.anandc2015-03-211-1/+1
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/1026953002 Cr-Commit-Position: refs/heads/master@{#321665}
* Build app-streaming web-apps when building the Chromoting integration tests ↵anandc2015-03-201-0/+1
| | | | | | | | | | target. BUG= Review URL: https://codereview.chromium.org/1007993003 Cr-Commit-Position: refs/heads/master@{#321569}
* Adding an example multi-machine test for Chromoting. This example starts a ↵mmeade2015-03-174-1/+219
| | | | | | | | | | test controller machine which then starts a dedicated task machine to run the browser_tests_launcher on. The output is then streamed back to the test controller to be logged and the output (pass or fail) is mirrored on the test controller. This is a simple example of how to write and run multi-machine tests using the Legion framework. BUG=467620 Review URL: https://codereview.chromium.org/1012883002 Cr-Commit-Position: refs/heads/master@{#320945}
* Implements It2MeBrowserTest CancelShare and VerifyAccessCodeNotReusablekelvinp2015-03-171-0/+2
| | | | | | | | | | | | | | | | | This CL implements two P0 browser test cases - It2MeBrowserTest.CancelShare - It2MeBrowserTest.VerifyAccessCodeNotReusable This only behavioral change in this CL is that we will always return INVALID_ACCESS_CODE for It2Me Connections failed with HOST_IS_OFFLINE. HOST_IS_OFFLINE suggests that the host portion of the access code is invalid and we should hide this implementation detail from the user. BUG=466849,466850 Review URL: https://codereview.chromium.org/1007543003 Cr-Commit-Position: refs/heads/master@{#320835}
* Include LIBGLESV2.DLL in Chromoting browser-tests isolate.anandc2015-03-111-0/+1
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/996043002 Cr-Commit-Position: refs/heads/master@{#320037}
generated by cgit v1.1 at 2024-06-09 23:13:32 +0000