Skip to content

Conversation

@abarth
Copy link
Contributor

@abarth abarth commented Nov 23, 2015

This patch still requires a locally built engine. I'll remove the need for a
locally built engine in a future patch.

Fixes #278

Copy link
Contributor

Choose a reason for hiding this comment

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

add "instead of the current directory" for clarity.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@Hixie
Copy link
Contributor

Hixie commented Nov 23, 2015

LGTM but it looks like it doesn't actually work (travis is red because of failure to run tests).

This patch still requires a locally built engine. I'll remove the need for a
locally built engine in a future patch.

Fixes flutter#278
abarth added a commit that referenced this pull request Nov 23, 2015
Make it possible to run tests outside the Flutter repo
@abarth abarth merged commit 587b5bc into flutter:master Nov 23, 2015
@abarth abarth deleted the test_outside_repo branch November 23, 2015 21:51
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Running an individual unit test with flutter test needs an absolute path

2 participants