Skip to content
This repository has been archived by the owner on May 31, 2021. It is now read-only.

web-angular: support running component tests in the browser #447

Closed

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Sep 15, 2017

Fixes #445

I've tested this and it seem to work. I.e., the framework reports that test pass, even if a "Bad state" exception is thrown (angulardart/angular#631).

<!DOCTYPE html>
<html lang="en">

<head>
Copy link
Contributor

Choose a reason for hiding this comment

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

This file isn't needed – unless you're doing something special.

pkg/test will generate it automatically...

...unless I'm missing something

Copy link
Contributor Author

Choose a reason for hiding this comment

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

W/o this file I can't get the angular_test-based tests to run (I'm not saying that it is impossible, just that I don't know how).

I haven't run tests in this manner very much. I based the assumption that such a file was needed by looking at @matanlurey's angular_test demo app. In particular see the test folder.

I'll let @matanlurey comment.

cc @kwalrath

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah...maybe it's angular-test specific.

...ah! Or it's pub-serve specific. That's it, I bet!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chalin chalin force-pushed the chalin-add-web-ng-app-test-0914 branch from ae16e8c to 4f1b4cd Compare October 8, 2017 11:39
@kevmoo
Copy link
Contributor

kevmoo commented Oct 8, 2017

@matanlurey ?

@chalin chalin force-pushed the chalin-add-web-ng-app-test-0914 branch from 4f1b4cd to 8f47657 Compare October 19, 2017 14:30
@kwalrath
Copy link
Contributor

What's the status of this PR, @chalin? Are we still waiting for feedback from @matanlurey? If so, I can bug him...

@matanlurey
Copy link
Contributor

We can't even run our own tests yet until @alorenzen and friends get it working.

So the answer is... TBD.

@chalin
Copy link
Contributor Author

chalin commented Feb 1, 2018

Closing this PR. We'll address this feature under Angular 5.

@chalin chalin closed this Feb 1, 2018
@chalin chalin deleted the chalin-add-web-ng-app-test-0914 branch February 1, 2018 11:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants