Skip to content

test(dart/transform): Update e2e tests to run in a separate file - #5935

Closed
kegluneq wants to merge 1 commit into
angular:masterfrom
kegluneq:dart_test
Closed

kegluneq wants to merge 1 commit into
angular:masterfrom
kegluneq:dart_test

Conversation

@kegluneq

Copy link
Copy Markdown

A conflict between package:guinness and package:test is causing our
end-to-end Dart transformer tests to be skipped.

Run these end-to-end tests in a separate file to fix this.

Closes #5934

/cc @kevmoo

@kegluneq kegluneq added type: bug/fix action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 16, 2015
@kegluneq

Copy link
Copy Markdown
Author

Either this did not work or we have been running these tests all along:
(from this CI run)

Chrome 39.0.2171 (Linux 0.0.0): Executed 2412 of 2412 SUCCESS (23.502 secs / 22.341 secs)

(from https://travis-ci.org/angular/angular/jobs/97037419)

Chrome 39.0.2171 (Linux 0.0.0): Executed 2412 of 2412 SUCCESS (25.59 secs / 24.241 secs)

@kegluneq
kegluneq force-pushed the dart_test branch 2 times, most recently from 893d95d to 8ecf34d Compare December 30, 2015 18:04
@kegluneq

Copy link
Copy Markdown
Author

These tests are not currently being skipped, but once we upgrade package:code_transformers, package:analyzer, and package:observe to their latest versions (which incorporate package:test), they will be.

Updated so that these tests are run in separate processes, which prevents our package:test unit tests from being skipped.

@kegluneq

kegluneq commented Jan 4, 2016

Copy link
Copy Markdown
Author

I'm particularly curious if the additions of

Html5LibDomAdapter.makeCurrent();
testSetup();

to multi_client_server_message_bus.server.spec.dart and single_client_server_message_bus.server.spec.dart are undesirable for any reason.

@kegluneq kegluneq assigned vsavkin and unassigned yjbanov Jan 4, 2016
@kegluneq kegluneq added refactoring Issue that involves refactoring or code-cleanup and removed type: bug/fix labels Jan 4, 2016
@vsavkin vsavkin added pr_state: LGTM and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 5, 2016
@vsavkin vsavkin assigned kegluneq and unassigned vsavkin Jan 5, 2016
@kegluneq

kegluneq commented Jan 8, 2016

Copy link
Copy Markdown
Author

This is waiting on #5931 since they touch the same files.

Widen version dependencies for `package:angular`, `package:code_transformers`,
and `package:observe`.

`package:guinness` uses `package:unittest` while
the newest versions of `package:code_transformers` use `package:test`.
This causes our end-to-end Dart transformer tests (which use testing
code in `package:code_transformers`) to be skipped.

To fix this:
- Move e2e tests to run in a separate file
- Run `gulp test.server.dart` tests serially

Closes angular#5922
@kegluneq kegluneq added action: merge The PR is ready for merge by the caretaker and removed state: blocked labels Jan 13, 2016
@mary-poppins

Copy link
Copy Markdown

Merging PR #5935 on behalf of @jeffbcross to branch presubmit-jeffbcross-pr-5935.

@jeffbcross

Copy link
Copy Markdown
Contributor

@kegluneq this is failing when I try to merge to master, even after restarting. Mind taking a look?

https://travis-ci.org/angular/angular/builds/102234160

@jeffbcross

Copy link
Copy Markdown
Contributor

@kegluneq sorry, wrong PR

@kegluneq
kegluneq deleted the dart_test branch January 14, 2016 21:50
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes refactoring Issue that involves refactoring or code-cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Some Dart transformer tests are *probably* being skipped

6 participants