Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Make all tests runnable together #218

Merged
merged 6 commits into from
Jan 18, 2017
Merged

Make all tests runnable together #218

merged 6 commits into from
Jan 18, 2017

Conversation

kjin
Copy link
Contributor

@kjin kjin commented Jan 13, 2017

Moved remaining standalone tests to the root test folder.

The simplest way to go about this was to introduce an undocumented, debug-only flag force_, that allows both the debug agent and V8 debug API singletons to be re-assigned. I'm open to doing this a different way, however, like a stop function.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 13, 2017
@@ -59,7 +59,7 @@ var formatInterval = function(msg, interval) {

var singleton;
module.exports.create = function(logger_, config_, jsFiles_, sourcemapper_) {
if (singleton) {
if (singleton && !config_.force_) {

This comment was marked as spam.

This comment was marked as spam.

@kjin kjin changed the title Make all tests non-interference Make all tests runnable together Jan 14, 2017
@kjin
Copy link
Contributor Author

kjin commented Jan 18, 2017

@ofrobots @matthewloring PTAL

Copy link
Contributor

@matthewloring matthewloring left a comment

Choose a reason for hiding this comment

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

LGTM once merge conflicts are addressed.

@kjin kjin merged commit a00ce2a into googleapis:master Jan 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants