Skip to content
This repository has been archived by the owner. It is now read-only.

fix(experiments): test against the latest tagged branch of experiments #4766

Closed
wants to merge 1 commit into from

Conversation

@vladikoff
Copy link
Contributor

@vladikoff vladikoff commented Feb 28, 2017

this change will probably break the tests mozilla/fxa-content-experiments@2bccc5c so we need to fallback

cc @shane-tomlinson

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Mar 1, 2017

Ah yes... Or update the tests.

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Mar 2, 2017

This is a strange one. Since we are still using the experiment infrastructure, the functional tests don't need to be updated. The functional tests by default force users through the "control" group, which is what the existing tests test for. It's just that in real life, nobody will be in the control group.

@vladikoff
Copy link
Contributor Author

@vladikoff vladikoff commented Mar 2, 2017

The functional tests by default force users through the "control" group, which is what the existing tests test for. It's just that in real life, nobody will be in the control group.

Oh great! ok then we don't have to worry about that 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants